Re: [PATCH v3 1/3] extcon: axp288: Constify the axp288_pwr_up_down_info array
On 2018년 02월 13일 04:46, Hans de Goede wrote: > Make the axp288_pwr_up_down_info array const char * const, this leads > to the following section size changes: > > .text 0x674 -> 0x664 > .data 0x148 -> 0x0f0 > .rodata 0x0b4 -> 0x114 > > Signed-off-by: Hans de Goede> --- > drivers/extcon/extcon-axp288.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/extcon/extcon-axp288.c b/drivers/extcon/extcon-axp288.c > index 0a44d43802fe..c8f7b6435679 100644 > --- a/drivers/extcon/extcon-axp288.c > +++ b/drivers/extcon/extcon-axp288.c > @@ -106,7 +106,7 @@ struct axp288_extcon_info { > }; > > /* Power up/down reason string array */ > -static char *axp288_pwr_up_down_info[] = { > +static const char * const axp288_pwr_up_down_info[] = { > "Last wake caused by user pressing the power button", > "Last wake caused by a charger insertion", > "Last wake caused by a battery insertion", > @@ -124,7 +124,7 @@ static char *axp288_pwr_up_down_info[] = { > */ > static void axp288_extcon_log_rsi(struct axp288_extcon_info *info) > { > - char **rsi; > + const char * const *rsi; > unsigned int val, i, clear_mask = 0; > int ret; > > Applied it. -- Best Regards, Chanwoo Choi Samsung Electronics
Re: [PATCH v3 1/3] extcon: axp288: Constify the axp288_pwr_up_down_info array
On 2018년 02월 13일 04:46, Hans de Goede wrote: > Make the axp288_pwr_up_down_info array const char * const, this leads > to the following section size changes: > > .text 0x674 -> 0x664 > .data 0x148 -> 0x0f0 > .rodata 0x0b4 -> 0x114 > > Signed-off-by: Hans de Goede > --- > drivers/extcon/extcon-axp288.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/extcon/extcon-axp288.c b/drivers/extcon/extcon-axp288.c > index 0a44d43802fe..c8f7b6435679 100644 > --- a/drivers/extcon/extcon-axp288.c > +++ b/drivers/extcon/extcon-axp288.c > @@ -106,7 +106,7 @@ struct axp288_extcon_info { > }; > > /* Power up/down reason string array */ > -static char *axp288_pwr_up_down_info[] = { > +static const char * const axp288_pwr_up_down_info[] = { > "Last wake caused by user pressing the power button", > "Last wake caused by a charger insertion", > "Last wake caused by a battery insertion", > @@ -124,7 +124,7 @@ static char *axp288_pwr_up_down_info[] = { > */ > static void axp288_extcon_log_rsi(struct axp288_extcon_info *info) > { > - char **rsi; > + const char * const *rsi; > unsigned int val, i, clear_mask = 0; > int ret; > > Applied it. -- Best Regards, Chanwoo Choi Samsung Electronics
[PATCH v3 1/3] extcon: axp288: Constify the axp288_pwr_up_down_info array
Make the axp288_pwr_up_down_info array const char * const, this leads to the following section size changes: .text 0x674 -> 0x664 .data 0x148 -> 0x0f0 .rodata 0x0b4 -> 0x114 Signed-off-by: Hans de Goede--- drivers/extcon/extcon-axp288.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/extcon/extcon-axp288.c b/drivers/extcon/extcon-axp288.c index 0a44d43802fe..c8f7b6435679 100644 --- a/drivers/extcon/extcon-axp288.c +++ b/drivers/extcon/extcon-axp288.c @@ -106,7 +106,7 @@ struct axp288_extcon_info { }; /* Power up/down reason string array */ -static char *axp288_pwr_up_down_info[] = { +static const char * const axp288_pwr_up_down_info[] = { "Last wake caused by user pressing the power button", "Last wake caused by a charger insertion", "Last wake caused by a battery insertion", @@ -124,7 +124,7 @@ static char *axp288_pwr_up_down_info[] = { */ static void axp288_extcon_log_rsi(struct axp288_extcon_info *info) { - char **rsi; + const char * const *rsi; unsigned int val, i, clear_mask = 0; int ret; -- 2.14.3
[PATCH v3 1/3] extcon: axp288: Constify the axp288_pwr_up_down_info array
Make the axp288_pwr_up_down_info array const char * const, this leads to the following section size changes: .text 0x674 -> 0x664 .data 0x148 -> 0x0f0 .rodata 0x0b4 -> 0x114 Signed-off-by: Hans de Goede --- drivers/extcon/extcon-axp288.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/extcon/extcon-axp288.c b/drivers/extcon/extcon-axp288.c index 0a44d43802fe..c8f7b6435679 100644 --- a/drivers/extcon/extcon-axp288.c +++ b/drivers/extcon/extcon-axp288.c @@ -106,7 +106,7 @@ struct axp288_extcon_info { }; /* Power up/down reason string array */ -static char *axp288_pwr_up_down_info[] = { +static const char * const axp288_pwr_up_down_info[] = { "Last wake caused by user pressing the power button", "Last wake caused by a charger insertion", "Last wake caused by a battery insertion", @@ -124,7 +124,7 @@ static char *axp288_pwr_up_down_info[] = { */ static void axp288_extcon_log_rsi(struct axp288_extcon_info *info) { - char **rsi; + const char * const *rsi; unsigned int val, i, clear_mask = 0; int ret; -- 2.14.3