linux-kernel - Re: [PATCH] mfd: ti-lmu: constify mfd

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: <e10bdc74-8138-055e-c829-7befbc7161b4@ti.com>
Date:   Fri, 7 Sep 2018 12:20:21 -0500
From:   Dan Murphy <dmurphy@...com>
To:     Pavel Machek <pavel@....cz>,
        kernel list <linux-kernel@...r.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
        <linux-omap@...r.kernel.org>, <tony@...mide.com>, <sre@...nel.org>,
        <nekit1000@...il.com>, <mpartap@....net>, <merlijn@...zup.org>,
        <lee.jones@...aro.org>
Subject: Re: [PATCH] mfd: ti-lmu: constify mfd_cell tables

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@...labora.co.uk>
>>
>> mfd: ti-lmu: constify mfd_cell tables
>>     
>> Add const attribute to all mfd_cell structures.
>>     
>> Signed-off-by: Sebastian Reichel <sebastian.reichel@...labora.co.uk>
>> Signed-off-by: Pavel Machek <pavel@....cz>
> 
> 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