pinctrl: sunxi: move bank K register offset

The Allwinner pincontroller register layout used to allow for at least
11 banks per controller, any more banks would reside at a second
controller instance.
When the per-bank register map size was increased with the D1, it turned
out that the last bank (port K) of those maximum 11 banks actually would
not fit anymore in the 512 bytes reserved for the pincontroller registers.
On new SoCs Allwinner thus moved the last bank beyond the existing
registers, at offset 0x500.

So far SoCs never used more than 9 banks per controller, but the new
Allwinner A523 actually uses all 11 banks. Since that SoC also uses the
extended layout, its PortK needs to be programmed at offset 0x500.

Factor out the bank offset calculation into a new function, and handle
the case for the last bank separately. Since none of the older SoCs ever
used PortK, we can ignore this case, and just always use offset 0x500
for the last bank.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Chen-Yu Tsai <wens@csie.org>
Link: https://lore.kernel.org/20250306235827.4895-4-andre.przywara@arm.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
index ae281a3c2ed34f84f7e68671132a8d6255685691..83a031ceb29f212d7c391a70c3e07fb6648aee3e 100644 (file)
--- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
+++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
@@ -58,13 +58,29 @@ static struct irq_chip sunxi_pinctrl_level_irq_chip;
  * The following functions calculate the register and the bit offset to access.
  * They take a pin number which is relative to the start of the current device.
  */
+
+/*
+ * When using the extended register layout, Bank K does not fit into the
+ * space used for the other banks. Instead it lives at offset 0x500.
+ */
+static u32 sunxi_bank_offset(const struct sunxi_pinctrl *pctl, u32 pin)
+{
+       u32 offset = 0;
+
+       if (pin >= PK_BASE) {
+               pin -= PK_BASE;
+               offset = PIO_BANK_K_OFFSET;
+       }
+
+       return offset + (pin / PINS_PER_BANK) * pctl->bank_mem_size;
+}
+
 static void sunxi_mux_reg(const struct sunxi_pinctrl *pctl,
                          u32 pin, u32 *reg, u32 *shift, u32 *mask)
 {
-       u32 bank   = pin / PINS_PER_BANK;
        u32 offset = pin % PINS_PER_BANK * MUX_FIELD_WIDTH;
 
-       *reg   = bank * pctl->bank_mem_size + MUX_REGS_OFFSET +
+       *reg   = sunxi_bank_offset(pctl, pin) + MUX_REGS_OFFSET +
                 offset / BITS_PER_TYPE(u32) * sizeof(u32);
        *shift = offset % BITS_PER_TYPE(u32);
        *mask  = (BIT(MUX_FIELD_WIDTH) - 1) << *shift;
@@ -73,10 +89,9 @@ static void sunxi_mux_reg(const struct sunxi_pinctrl *pctl,
 static void sunxi_data_reg(const struct sunxi_pinctrl *pctl,
                           u32 pin, u32 *reg, u32 *shift, u32 *mask)
 {
-       u32 bank   = pin / PINS_PER_BANK;
        u32 offset = pin % PINS_PER_BANK * DATA_FIELD_WIDTH;
 
-       *reg   = bank * pctl->bank_mem_size + DATA_REGS_OFFSET +
+       *reg   = sunxi_bank_offset(pctl, pin) + DATA_REGS_OFFSET +
                 offset / BITS_PER_TYPE(u32) * sizeof(u32);
        *shift = offset % BITS_PER_TYPE(u32);
        *mask  = (BIT(DATA_FIELD_WIDTH) - 1) << *shift;
@@ -85,10 +100,9 @@ static void sunxi_data_reg(const struct sunxi_pinctrl *pctl,
 static void sunxi_dlevel_reg(const struct sunxi_pinctrl *pctl,
                             u32 pin, u32 *reg, u32 *shift, u32 *mask)
 {
-       u32 bank   = pin / PINS_PER_BANK;
        u32 offset = pin % PINS_PER_BANK * pctl->dlevel_field_width;
 
-       *reg   = bank * pctl->bank_mem_size + DLEVEL_REGS_OFFSET +
+       *reg   = sunxi_bank_offset(pctl, pin) + DLEVEL_REGS_OFFSET +
                 offset / BITS_PER_TYPE(u32) * sizeof(u32);
        *shift = offset % BITS_PER_TYPE(u32);
        *mask  = (BIT(pctl->dlevel_field_width) - 1) << *shift;
@@ -97,10 +111,9 @@ static void sunxi_dlevel_reg(const struct sunxi_pinctrl *pctl,
 static void sunxi_pull_reg(const struct sunxi_pinctrl *pctl,
                           u32 pin, u32 *reg, u32 *shift, u32 *mask)
 {
-       u32 bank   = pin / PINS_PER_BANK;
        u32 offset = pin % PINS_PER_BANK * PULL_FIELD_WIDTH;
 
-       *reg   = bank * pctl->bank_mem_size + pctl->pull_regs_offset +
+       *reg   = sunxi_bank_offset(pctl, pin) + pctl->pull_regs_offset +
                 offset / BITS_PER_TYPE(u32) * sizeof(u32);
        *shift = offset % BITS_PER_TYPE(u32);
        *mask  = (BIT(PULL_FIELD_WIDTH) - 1) << *shift;

diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.h b/drivers/pinctrl/sunxi/pinctrl-sunxi.h
index fbbf070a87542aee0bba6d7dac0cfbf851c2861b..6cf721876d89db48a679d3b78fdee2759b87a25f 100644 (file)
--- a/drivers/pinctrl/sunxi/pinctrl-sunxi.h
+++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.h
@@ -25,6 +25,8 @@
 #define PG_BASE        192
 #define PH_BASE        224
 #define PI_BASE        256
+#define PJ_BASE        288
+#define PK_BASE        320
 #define PL_BASE        352
 #define PM_BASE        384
 #define PN_BASE        416
@@ -89,6 +91,8 @@
 #define PIO_POW_MOD_SEL_REG    0x340
 #define PIO_POW_MOD_CTL_REG    0x344
 
+#define PIO_BANK_K_OFFSET              0x500
+
 enum sunxi_desc_bias_voltage {
        BIAS_VOLTAGE_NONE,
        /*