linux-kernel - Re: [PATCH v2 4/8] pinctrl: sunxi: support moved power configuration registers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAGb2v67V7PAOC8MLVzE5SwwvpogN_WLNhQnRVD2pikGVDHtJjA@mail.gmail.com>
Date: Mon, 17 Feb 2025 23:48:13 +0800
From: Chen-Yu Tsai <wens@...e.org>
To: Andre Przywara <andre.przywara@....com>
Cc: Linus Walleij <linus.walleij@...aro.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Jernej Skrabec <jernej.skrabec@...il.com>, Samuel Holland <samuel@...lland.org>, 
	linux-gpio@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 4/8] pinctrl: sunxi: support moved power configuration registers

On Fri, Feb 14, 2025 at 8:40 AM Andre Przywara <andre.przywara@....com> wrote:
>
> The Allwinner pincontroller IP features some registers to control the
> withstand voltage of each pin group. So far those registers were always
> located at the same offset, but the A523 SoC has moved them (probably to
> accommodate all eleven pin banks).
>
> Add a flag to note this feature, and use that to program the registers
> either at offset 0x340 or 0x380. So far no pincontroller driver uses
> this flag, but we need it for the upcoming A523 support.
>
> Signed-off-by: Andre Przywara <andre.przywara@....com>
> ---
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c | 15 +++++++++++----
>  drivers/pinctrl/sunxi/pinctrl-sunxi.h |  7 +++++--
>  2 files changed, 16 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> index 83a031ceb29f2..a1057122272bd 100644
> --- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> +++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
> @@ -736,9 +736,9 @@ static int sunxi_pinctrl_set_io_bias_cfg(struct sunxi_pinctrl *pctl,
>                 val = uV > 1800000 && uV <= 2500000 ? BIT(bank) : 0;
>
>                 raw_spin_lock_irqsave(&pctl->lock, flags);
> -               reg = readl(pctl->membase + PIO_POW_MOD_CTL_REG);
> +               reg = readl(pctl->membase + pctl->pow_mod_sel_offset);
>                 reg &= ~BIT(bank);
> -               writel(reg | val, pctl->membase + PIO_POW_MOD_CTL_REG);
> +               writel(reg | val, pctl->membase + pctl->pow_mod_sel_offset);
>                 raw_spin_unlock_irqrestore(&pctl->lock, flags);
>
>                 fallthrough;
> @@ -746,9 +746,12 @@ static int sunxi_pinctrl_set_io_bias_cfg(struct sunxi_pinctrl *pctl,
>                 val = uV <= 1800000 ? 1 : 0;
>
>                 raw_spin_lock_irqsave(&pctl->lock, flags);
> -               reg = readl(pctl->membase + PIO_POW_MOD_SEL_REG);
> +               reg = readl(pctl->membase + pctl->pow_mod_sel_offset +
> +                           PIO_POW_MOD_SEL_OFS);
>                 reg &= ~(1 << bank);
> -               writel(reg | val << bank, pctl->membase + PIO_POW_MOD_SEL_REG);
> +               writel(reg | val << bank,
> +                      pctl->membase + pctl->pow_mod_sel_offset +
> +                      PIO_POW_MOD_SEL_OFS);
>                 raw_spin_unlock_irqrestore(&pctl->lock, flags);
>                 return 0;
>         default:
> @@ -1520,6 +1523,10 @@ int sunxi_pinctrl_init_with_flags(struct platform_device *pdev,
>                 pctl->pull_regs_offset = PULL_REGS_OFFSET;
>                 pctl->dlevel_field_width = DLEVEL_FIELD_WIDTH;
>         }
> +       if (flags & SUNXI_PINCTRL_ELEVEN_BANKS)
> +               pctl->pow_mod_sel_offset = PIO_11B_POW_MOD_SEL_REG;
> +       else
> +               pctl->pow_mod_sel_offset = PIO_POW_MOD_SEL_REG;
>
>         pctl->irq_array = devm_kcalloc(&pdev->dev,
>                                        IRQ_PER_BANK * pctl->desc->irq_banks,
> diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.h b/drivers/pinctrl/sunxi/pinctrl-sunxi.h
> index 6cf721876d89d..a93385e456a57 100644
> --- a/drivers/pinctrl/sunxi/pinctrl-sunxi.h
> +++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.h
> @@ -87,9 +87,11 @@
>  #define SUNXI_PINCTRL_VARIANT_MASK     GENMASK(7, 0)
>  #define SUNXI_PINCTRL_NEW_REG_LAYOUT   BIT(8)
>  #define SUNXI_PINCTRL_PORTF_SWITCH     BIT(9)
> +#define SUNXI_PINCTRL_ELEVEN_BANKS     BIT(10)
>
> -#define PIO_POW_MOD_SEL_REG    0x340
> -#define PIO_POW_MOD_CTL_REG    0x344
> +#define PIO_POW_MOD_SEL_REG            0x340
> +#define PIO_11B_POW_MOD_SEL_REG                0x380
> +#define PIO_POW_MOD_SEL_OFS            0x004

Shouldn't this be PIO_POW_MOD_CTL_OFS instead?

ChenYu

>
>  #define PIO_BANK_K_OFFSET              0x500
>
> @@ -173,6 +175,7 @@ struct sunxi_pinctrl {
>         u32                             bank_mem_size;
>         u32                             pull_regs_offset;
>         u32                             dlevel_field_width;
> +       u32                             pow_mod_sel_offset;
>  };
>
>  #define SUNXI_PIN(_pin, ...)                                   \
> --
> 2.46.3
>