linux-kernel - Re: [PATCH 1/2] mfd: max5970: Rename driver and remove wildcard

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230510134631.GD8963@google.com>
Date:   Wed, 10 May 2023 14:46:31 +0100
From:   Lee Jones <lee@...nel.org>
To:     Naresh Solanki <naresh.solanki@...ements.com>
Cc:     Liam Girdwood <lgirdwood@...il.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] mfd: max5970: Rename driver and remove wildcard

On Thu, 04 May 2023, Naresh Solanki wrote:

> Hi Lee,
> 
> I hope this email finds you well. I am writing to inquire about the status
> of my patch.
> 
> The purpose of this patch set was to rename the driver to remove the
> wildcard and make it specific to avoid confusion with different similar
> devices. I want to emphasize that this patch set does not include any
> functional changes.
> 
> I would appreciate it if you could kindly provide an update on the status.
> Please let me know if there is anything else you need from my end to help
> move things forward.
> 
> Thank you for your time and attention.

Please refrain from top-posting.

If you think your patch has fallen through the gaps (it hasn't) please
submit a [RESEND].

The merge-window has been open for the last 2 weeks.  I'm just ramping
up again on my backlog.

I'll get to you soon.

> On 27-04-2023 05:00 pm, Naresh Solanki wrote:
> > The previous version of this driver included wildcards in file names
> > and descriptions. This patch renames the driver to only support MAX5970
> > and MAX5978, which are the only chips that the driver actually supports.
> > 
> > Signed-off-by: Naresh Solanki <Naresh.Solanki@...ements.com>
> > ---
> >   drivers/mfd/Kconfig                        |  4 ++--
> >   drivers/mfd/simple-mfd-i2c.c               | 18 +++++++++---------
> >   include/linux/mfd/{max597x.h => max5970.h} | 16 ++++++++--------
> >   3 files changed, 19 insertions(+), 19 deletions(-)
> >   rename include/linux/mfd/{max597x.h => max5970.h} (92%)
> > 
> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> > index e90463c4441c..71231388e03c 100644
> > --- a/drivers/mfd/Kconfig
> > +++ b/drivers/mfd/Kconfig
> > @@ -266,8 +266,8 @@ config MFD_MADERA_SPI
> >   	  Support for the Cirrus Logic Madera platform audio SoC
> >   	  core functionality controlled via SPI.
> > -config MFD_MAX597X
> > -	tristate "Maxim 597x power switch and monitor"
> > +config MFD_MAX5970
> > +	tristate "Maxim 5970/5978 power switch and monitor"
> >   	depends on (I2C && OF)
> >   	select MFD_SIMPLE_MFD_I2C
> >   	help
> > diff --git a/drivers/mfd/simple-mfd-i2c.c b/drivers/mfd/simple-mfd-i2c.c
> > index 20782b4dd172..1f1c007560d8 100644
> > --- a/drivers/mfd/simple-mfd-i2c.c
> > +++ b/drivers/mfd/simple-mfd-i2c.c
> > @@ -72,22 +72,22 @@ static const struct simple_mfd_data silergy_sy7636a = {
> >   	.mfd_cell_size = ARRAY_SIZE(sy7636a_cells),
> >   };
> > -static const struct mfd_cell max597x_cells[] = {
> > -	{ .name = "max597x-regulator", },
> > -	{ .name = "max597x-iio", },
> > -	{ .name = "max597x-led", },
> > +static const struct mfd_cell max5970_cells[] = {
> > +	{ .name = "max5970-regulator", },
> > +	{ .name = "max5970-iio", },
> > +	{ .name = "max5970-led", },
> >   };
> > -static const struct simple_mfd_data maxim_max597x = {
> > -	.mfd_cell = max597x_cells,
> > -	.mfd_cell_size = ARRAY_SIZE(max597x_cells),
> > +static const struct simple_mfd_data maxim_max5970 = {
> > +	.mfd_cell = max5970_cells,
> > +	.mfd_cell_size = ARRAY_SIZE(max5970_cells),
> >   };
> >   static const struct of_device_id simple_mfd_i2c_of_match[] = {
> >   	{ .compatible = "kontron,sl28cpld" },
> >   	{ .compatible = "silergy,sy7636a", .data = &silergy_sy7636a},
> > -	{ .compatible = "maxim,max5970", .data = &maxim_max597x},
> > -	{ .compatible = "maxim,max5978", .data = &maxim_max597x},
> > +	{ .compatible = "maxim,max5970", .data = &maxim_max5970},
> > +	{ .compatible = "maxim,max5978", .data = &maxim_max5970},
> >   	{}
> >   };
> >   MODULE_DEVICE_TABLE(of, simple_mfd_i2c_of_match);
> > diff --git a/include/linux/mfd/max597x.h b/include/linux/mfd/max5970.h
> > similarity index 92%
> > rename from include/linux/mfd/max597x.h
> > rename to include/linux/mfd/max5970.h
> > index a850b2e02e6a..762a7d40c843 100644
> > --- a/include/linux/mfd/max597x.h
> > +++ b/include/linux/mfd/max5970.h
> > @@ -7,25 +7,25 @@
> >    * Author: Patrick Rudolph <patrick.rudolph@...ements.com>
> >    */
> > -#ifndef _MFD_MAX597X_H
> > -#define _MFD_MAX597X_H
> > +#ifndef _MFD_MAX5970_H
> > +#define _MFD_MAX5970_H
> >   #include <linux/regmap.h>
> >   #define MAX5970_NUM_SWITCHES 2
> >   #define MAX5978_NUM_SWITCHES 1
> > -#define MAX597X_NUM_LEDS     4
> > +#define MAX5970_NUM_LEDS     4
> > -struct max597x_data {
> > +struct max5970_data {
> >   	int num_switches;
> >   	u32 irng[MAX5970_NUM_SWITCHES];
> >   	u32 mon_rng[MAX5970_NUM_SWITCHES];
> >   	u32 shunt_micro_ohms[MAX5970_NUM_SWITCHES];
> >   };
> > -enum max597x_chip_type {
> > -	MAX597x_TYPE_MAX5978 = 1,
> > -	MAX597x_TYPE_MAX5970,
> > +enum max5970_chip_type {
> > +	TYPE_MAX5978 = 1,
> > +	TYPE_MAX5970,
> >   };
> >   #define MAX5970_REG_CURRENT_L(ch)		(0x01 + (ch) * 4)
> > @@ -93,4 +93,4 @@ enum max597x_chip_type {
> >   #define MAX_REGISTERS			0x49
> >   #define ADC_MASK			0x3FF
> > -#endif				/* _MFD_MAX597X_H */
> > +#endif				/* _MFD_MAX5970_H */
> > 
> > base-commit: b4c288cfd2f84c44994330c408e14645d45dee5b

-- 
Lee Jones [李琼斯]