On Wed, Jan 26, 2022 at 12:02:04PM +0100, Hans Verkuil wrote:
> The commit that sets the direction directly without calling
> pinctrl_gpio_direction(), forgot to add chip->base to the offset when
> calling sunxi_pmx_gpio_set_direction().
>
> This caused failures for various Allwinner boards which have two
> GPIO blocks.
>
> Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx>
> Reported-by: 5kft <5kft@xxxxxxxx>
> Suggested-by: 5kft <5kft@xxxxxxxx>
> Reported-by: Corentin Labbe <clabbe.montjoie@xxxxxxxxx>
> Fixes: 8df89a7cbc63 (pinctrl-sunxi: don't call pinctrl_gpio_direction())
> Tested-by: Corentin Labbe <clabbe.montjoie@xxxxxxxxx>
> Tested-by: Jernej Skrabec <jernej.skrabec@xxxxxxxxx>
> Acked-by: Jernej Skrabec <jernej.skrabec@xxxxxxxxx>
Tested-by: Guenter Roeck <linux@xxxxxxxxxxxx>
Guenter
> ---
> Corentin, can you please test this patch to verify that this fixes your
> issue on the orangepiPC?
> ---
>
> diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> index 80d6750c74a6..061323eab8b1 100644
> --- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> +++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> @@ -837,7 +837,8 @@ static int sunxi_pinctrl_gpio_direction_input(struct gpio_chip *chip,
> {
> struct sunxi_pinctrl *pctl = gpiochip_get_data(chip);
>
> - return sunxi_pmx_gpio_set_direction(pctl->pctl_dev, NULL, offset, true);
> + return sunxi_pmx_gpio_set_direction(pctl->pctl_dev, NULL,
> + chip->base + offset, true);
> }
>
> static int sunxi_pinctrl_gpio_get(struct gpio_chip *chip, unsigned offset)
> @@ -890,7 +891,8 @@ static int sunxi_pinctrl_gpio_direction_output(struct gpio_chip *chip,
> struct sunxi_pinctrl *pctl = gpiochip_get_data(chip);
>
> sunxi_pinctrl_gpio_set(chip, offset, value);
> - return sunxi_pmx_gpio_set_direction(pctl->pctl_dev, NULL, offset, false);
> + return sunxi_pmx_gpio_set_direction(pctl->pctl_dev, NULL,
> + chip->base + offset, false);
> }
>
> static int sunxi_pinctrl_gpio_of_xlate(struct gpio_chip *gc,
