Re: [PATCH] leds: lp8860: Constify reg_default tables

[Jacek Anaszewski <j.anaszewski@samsung.com>]

Hi Axel,

Thanks for the patch.

On 07/08/2015 04:43 AM, Axel Lin wrote:
> These reg_default tables are not modified after initialized, so make them
> const.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
>   drivers/leds/leds-lp8860.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/leds-lp8860.c b/drivers/leds/leds-lp8860.c
> index 8c2b7fb..79f0843 100644
> --- a/drivers/leds/leds-lp8860.c
> +++ b/drivers/leds/leds-lp8860.c
> @@ -302,7 +302,7 @@ out:
>   	return ret;
>   }
>
> -static struct reg_default lp8860_reg_defs[] = {
> +static const struct reg_default lp8860_reg_defs[] = {
>   	{ LP8860_DISP_CL1_BRT_MSB, 0x00},
>   	{ LP8860_DISP_CL1_BRT_LSB, 0x00},
>   	{ LP8860_DISP_CL1_CURR_MSB, 0x00},
> @@ -332,7 +332,7 @@ static const struct regmap_config lp8860_regmap_config = {
>   	.cache_type = REGCACHE_NONE,
>   };
>
> -static struct reg_default lp8860_eeprom_defs[] = {
> +static const struct reg_default lp8860_eeprom_defs[] = {
>   	{ LP8860_EEPROM_REG_0, 0x00 },
>   	{ LP8860_EEPROM_REG_1, 0x00 },
>   	{ LP8860_EEPROM_REG_2, 0x00 },
>

Acked-by: Jacek Anaszewski <j.anaszewski@samsung.com>

-- 
Best Regards,
Jacek Anaszewski