Re: [PATCH] mfd: ti-lmu: constify mfd_cell tables

[Dan Murphy]

Pavel

On 09/07/2018 04:39 AM, Pavel Machek wrote:
> On Wed 2018-08-29 11:31:08, Pavel Machek wrote:
>> From: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx>
>>
>> mfd: ti-lmu: constify mfd_cell tables
>>     
>> Add const attribute to all mfd_cell structures.
>>     
>> Signed-off-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxxxx>
>> Signed-off-by: Pavel Machek <pavel@xxxxxx>
> 
> Lee, I guess this is for you to apply. Any news there?
> 
> There are more patches ready,

As I stated in another email thread.  I don't see the need for this level of LMU framework.

Here is the reference thread

https://lore.kernel.org/patchwork/patch/982550/

> 
> https://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git/log/?h=droid4-pending-v4.19
> 
> and it would be good to get them in. (Alternatively, you can just
> cherry-pick them from droid4-pending-v4.19).
> 
> Thanks,
> 							Pavel
> 							
> 
>> diff --git a/drivers/mfd/ti-lmu.c b/drivers/mfd/ti-lmu.c
>> index cfb411c..990437e 100644
>> --- a/drivers/mfd/ti-lmu.c
>> +++ b/drivers/mfd/ti-lmu.c
>> @@ -25,7 +25,7 @@
>>  #include <linux/slab.h>
>>  
>>  struct ti_lmu_data {
>> -	struct mfd_cell *cells;
>> +	const struct mfd_cell *cells;
>>  	int num_cells;
>>  	unsigned int max_register;
>>  };
>> @@ -63,7 +63,7 @@ static void ti_lmu_disable_hw(struct ti_lmu *lmu)
>>  		gpio_set_value(lmu->en_gpio, 0);
>>  }
>>  
>> -static struct mfd_cell lm3532_devices[] = {
>> +static const struct mfd_cell lm3532_devices[] = {
>>  	{
>>  		.name          = "ti-lmu-backlight",
>>  		.id            = LM3532,
>> @@ -78,7 +78,7 @@ static struct mfd_cell lm3532_devices[] = {
>>  	.of_compatible = "ti,lm363x-regulator",	\
>>  }						\
>>  
>> -static struct mfd_cell lm3631_devices[] = {
>> +static const struct mfd_cell lm3631_devices[] = {
>>  	LM363X_REGULATOR(LM3631_BOOST),
>>  	LM363X_REGULATOR(LM3631_LDO_CONT),
>>  	LM363X_REGULATOR(LM3631_LDO_OREF),
>> @@ -91,7 +91,7 @@ static struct mfd_cell lm3631_devices[] = {
>>  	},
>>  };
>>  
>> -static struct mfd_cell lm3632_devices[] = {
>> +static const struct mfd_cell lm3632_devices[] = {
>>  	LM363X_REGULATOR(LM3632_BOOST),
>>  	LM363X_REGULATOR(LM3632_LDO_POS),
>>  	LM363X_REGULATOR(LM3632_LDO_NEG),
>> @@ -102,7 +102,7 @@ static struct mfd_cell lm3632_devices[] = {
>>  	},
>>  };
>>  
>> -static struct mfd_cell lm3633_devices[] = {
>> +static const struct mfd_cell lm3633_devices[] = {
>>  	{
>>  		.name          = "ti-lmu-backlight",
>>  		.id            = LM3633,
>> @@ -120,7 +120,7 @@ static struct mfd_cell lm3633_devices[] = {
>>  	},
>>  };
>>  
>> -static struct mfd_cell lm3695_devices[] = {
>> +static const struct mfd_cell lm3695_devices[] = {
>>  	{
>>  		.name          = "ti-lmu-backlight",
>>  		.id            = LM3695,
>> @@ -128,7 +128,7 @@ static struct mfd_cell lm3695_devices[] = {
>>  	},
>>  };
>>  
>> -static struct mfd_cell lm3697_devices[] = {
>> +static const struct mfd_cell lm3697_devices[] = {
>>  	{
>>  		.name          = "ti-lmu-backlight",
>>  		.id            = LM3697,
>>
> 
> 
> 


-- 
------------------
Dan Murphy