Messages in this thread | | | From | Prashant Malani <> | Date | Fri, 20 Dec 2019 11:35:47 -0800 | Subject | Re: [PATCH 2/2] mfd: cros_ec: Add usbpd-notify to usbpd_charger |
| |
On Fri, Dec 20, 2019 at 2:52 AM Lee Jones <lee.jones@linaro.org> wrote: > > On Fri, 20 Dec 2019, Enric Balletbo i Serra wrote: > > > Hi Prashant, > > > > This should be [PATCH v3 2/2]. All the patches in the series should have the > > same version otherwise makes difficult to follow. > > > > Thanks, > > Enric > > > > On 19/12/19 21:13, Prashant Malani wrote: > > > Add the cros-usbpd-notify driver as a cell for the cros_usbpd_charger > > > subdevice on non-ACPI platforms. > > > > > > This driver allows other cros-ec devices to receive PD event > > > notifications from the Chrome OS Embedded Controller (EC) via a > > > notification chain. > > > > > > Change-Id: I4c062d261fa1a504b43b0a0c0a98a661829593b9 > > > Signed-off-by: Prashant Malani <pmalani@chromium.org> > > > --- > > > drivers/mfd/cros_ec_dev.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > diff --git a/drivers/mfd/cros_ec_dev.c b/drivers/mfd/cros_ec_dev.c > > > index c4b977a5dd966..1dde480f35b93 100644 > > > --- a/drivers/mfd/cros_ec_dev.c > > > +++ b/drivers/mfd/cros_ec_dev.c > > > @@ -85,6 +85,9 @@ static const struct mfd_cell cros_ec_sensorhub_cells[] = { > > > static const struct mfd_cell cros_usbpd_charger_cells[] = { > > > { .name = "cros-usbpd-charger", }, > > > { .name = "cros-usbpd-logger", }, > > > +#ifndef CONFIG_ACPI > > > + { .name = "cros-usbpd-notify", }, > > > +#endif > > We don't want #iferry all over our c-files. If you *have* to rely on > Kconfig configurations, split this out into a separate cell and use > IS_ENABLED(). Done. Thanks. > > -- > Lee Jones [李琼斯] > Linaro Services Technical Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog
| |