Message ID | 20250415-dev-adp5589-fw-v2-4-3a799c3ed812@analog.com |
---|---|
State | New |
Headers | show |
Series | mfd: adp5585: support keymap events and drop legacy Input driver | expand |
On Tue, 15 Apr 2025, Nuno Sá via B4 Relay wrote: > From: Nuno Sá <nuno.sa@analog.com> > > Use the helper macro. No functional change intended... > > Signed-off-by: Nuno Sá <nuno.sa@analog.com> > --- > drivers/mfd/adp5585.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/mfd/adp5585.c b/drivers/mfd/adp5585.c > index f17b5f2474cac6a403556694066f438288264a49..fafe3ad93ea196e1eb8e79fecba58f36f12167eb 100644 > --- a/drivers/mfd/adp5585.c > +++ b/drivers/mfd/adp5585.c > @@ -4,6 +4,7 @@ > * > * Copyright 2022 NXP > * Copyright 2024 Ideas on Board Oy > + * Copyright 2025 Analog Devices Inc. > */ > > #include <linux/array_size.h> > @@ -18,8 +19,10 @@ > #include <linux/types.h> > > static const struct mfd_cell adp5585_devs[] = { > - { .name = "adp5585-gpio", }, > - { .name = "adp5585-pwm", }, > + MFD_CELL_NAME("adp5585-keys"), This looks like a functional change to me? > + MFD_CELL_NAME("adp5585-gpio"), > + MFD_CELL_NAME("adp5585-pwm"), > + Superfluous blank line. > }; > > static const struct regmap_range adp5585_volatile_ranges[] = { > > -- > 2.49.0 > >
diff --git a/drivers/mfd/adp5585.c b/drivers/mfd/adp5585.c index f17b5f2474cac6a403556694066f438288264a49..fafe3ad93ea196e1eb8e79fecba58f36f12167eb 100644 --- a/drivers/mfd/adp5585.c +++ b/drivers/mfd/adp5585.c @@ -4,6 +4,7 @@ * * Copyright 2022 NXP * Copyright 2024 Ideas on Board Oy + * Copyright 2025 Analog Devices Inc. */ #include <linux/array_size.h> @@ -18,8 +19,10 @@ #include <linux/types.h> static const struct mfd_cell adp5585_devs[] = { - { .name = "adp5585-gpio", }, - { .name = "adp5585-pwm", }, + MFD_CELL_NAME("adp5585-keys"), + MFD_CELL_NAME("adp5585-gpio"), + MFD_CELL_NAME("adp5585-pwm"), + }; static const struct regmap_range adp5585_volatile_ranges[] = {