Quoting Linus Walleij (2019-08-19 01:49:04) > diff --git a/drivers/pinctrl/qcom/pinctrl-msm.c b/drivers/pinctrl/qcom/pinctrl-msm.c > index 7f35c196bb3e..a5d8f75da4a7 100644 > --- a/drivers/pinctrl/qcom/pinctrl-msm.c > +++ b/drivers/pinctrl/qcom/pinctrl-msm.c > @@ -593,24 +593,25 @@ static void msm_gpio_dbg_show(struct seq_file *s, struct gpio_chip *chip) > #define msm_gpio_dbg_show NULL > #endif > > -static int msm_gpio_init_valid_mask(struct gpio_chip *chip) > +static int msm_gpio_init_valid_mask(struct gpio_chip *gc, Please leave the name as 'chip' like the other ops in this file. The other uses of 'gc' in gpiolib are fine though. Reviewed-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>