Re: [PATCH] mfd: max77620: Constify resources tables

From: Lee Jones
Date: Wed Jun 08 2016 - 11:39:33 EST


On Sun, 05 Jun 2016, Axel Lin wrote:

> These tables are never modified, so declare them as const.
>
> Signed-off-by: Axel Lin <axel.lin@xxxxxxxxxx>
> ---
> drivers/mfd/max77620.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)

Applied, thanks.

> diff --git a/drivers/mfd/max77620.c b/drivers/mfd/max77620.c
> index 199d261..e831422 100644
> --- a/drivers/mfd/max77620.c
> +++ b/drivers/mfd/max77620.c
> @@ -37,19 +37,19 @@
> #include <linux/regmap.h>
> #include <linux/slab.h>
>
> -static struct resource gpio_resources[] = {
> +static const struct resource gpio_resources[] = {
> DEFINE_RES_IRQ(MAX77620_IRQ_TOP_GPIO),
> };
>
> -static struct resource power_resources[] = {
> +static const struct resource power_resources[] = {
> DEFINE_RES_IRQ(MAX77620_IRQ_LBT_MBATLOW),
> };
>
> -static struct resource rtc_resources[] = {
> +static const struct resource rtc_resources[] = {
> DEFINE_RES_IRQ(MAX77620_IRQ_TOP_RTC),
> };
>
> -static struct resource thermal_resources[] = {
> +static const struct resource thermal_resources[] = {
> DEFINE_RES_IRQ(MAX77620_IRQ_LBT_TJALRM1),
> DEFINE_RES_IRQ(MAX77620_IRQ_LBT_TJALRM2),
> };

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog