sh: improve pinmux support for single direction pins

This patch improves the support for gpio pins that are hard wired
to either input or output and lack control register association.
A special force enum id is used to allow use without control
register but still mark the gpio pin as input or output.

Signed-off-by: Magnus Damm <damm@igel.co.jp>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>

diff --git a/arch/sh/include/asm/gpio.h b/arch/sh/include/asm/gpio.h
index 02de5b609ba5327c8588e91e0fbb9ca9f29f280a..74c31a678a525057d3f6bbcfad856ed8db72eb84 100644 (file)
--- a/arch/sh/include/asm/gpio.h
+++ b/arch/sh/include/asm/gpio.h
@@ -64,6 +64,7 @@ struct pinmux_data_reg {
 struct pinmux_range {
        pinmux_enum_t begin;
        pinmux_enum_t end;
+       pinmux_enum_t force;
 };
 
 struct pinmux_info {

diff --git a/arch/sh/kernel/cpu/sh2a/pinmux-sh7203.c b/arch/sh/kernel/cpu/sh2a/pinmux-sh7203.c
index 6fbc2aa108317f5653ab9059e1d5c3fa3c2ced39..c465af7283fc89793ad1da0f76cb756e1c5b5237 100644 (file)
--- a/arch/sh/kernel/cpu/sh2a/pinmux-sh7203.c
+++ b/arch/sh/kernel/cpu/sh2a/pinmux-sh7203.c
@@ -46,11 +46,10 @@ enum {
        PINMUX_DATA_END,
 
        PINMUX_INPUT_BEGIN,
+       FORCE_IN,
        PA7_IN, PA6_IN, PA5_IN, PA4_IN,
        PA3_IN, PA2_IN, PA1_IN, PA0_IN,
        PB11_IN, PB10_IN, PB9_IN, PB8_IN,
-       PB7_IN, PB6_IN, PB5_IN, PB4_IN,
-       PB3_IN, PB2_IN, PB1_IN, PB0_IN,
        PC14_IN, PC13_IN, PC12_IN,
        PC11_IN, PC10_IN, PC9_IN, PC8_IN,
        PC7_IN, PC6_IN, PC5_IN, PC4_IN,
@@ -74,7 +73,7 @@ enum {
        PINMUX_INPUT_END,
 
        PINMUX_OUTPUT_BEGIN,
-       PB12_OUT,
+       FORCE_OUT,
        PB11_OUT, PB10_OUT, PB9_OUT, PB8_OUT,
        PC14_OUT, PC13_OUT, PC12_OUT,
        PC11_OUT, PC10_OUT, PC9_OUT, PC8_OUT,
@@ -285,7 +284,7 @@ static pinmux_enum_t pinmux_data[] = {
        PINMUX_DATA(PA0_DATA, PA0_IN),
 
        /* PB */
-       PINMUX_DATA(PB12_DATA, PB12MD_00, PB12_OUT),
+       PINMUX_DATA(PB12_DATA, PB12MD_00, FORCE_OUT),
        PINMUX_DATA(WDTOVF_MARK, PB12MD_01),
        PINMUX_DATA(IRQOUT_MARK, PB12MD_10, PB12IRQ_00),
        PINMUX_DATA(REFOUT_MARK, PB12MD_10, PB12IRQ_01),
@@ -306,42 +305,42 @@ static pinmux_enum_t pinmux_data[] = {
        PINMUX_DATA(CRX0_MARK, PB8MD_01),
        PINMUX_DATA(CRX0_CRX1_MARK, PB8MD_10),
 
-       PINMUX_DATA(PB7_DATA, PB7MD_00, PB7_IN),
+       PINMUX_DATA(PB7_DATA, PB7MD_00, FORCE_IN),
        PINMUX_DATA(SDA3_MARK, PB7MD_01),
        PINMUX_DATA(PINT7_PB_MARK, PB7MD_10),
        PINMUX_DATA(IRQ7_PB_MARK, PB7MD_11),
 
-       PINMUX_DATA(PB6_DATA, PB6MD_00, PB6_IN),
+       PINMUX_DATA(PB6_DATA, PB6MD_00, FORCE_IN),
        PINMUX_DATA(SCL3_MARK, PB6MD_01),
        PINMUX_DATA(PINT6_PB_MARK, PB6MD_10),
        PINMUX_DATA(IRQ6_PB_MARK, PB6MD_11),
 
-       PINMUX_DATA(PB5_DATA, PB5MD_00, PB5_IN),
+       PINMUX_DATA(PB5_DATA, PB5MD_00, FORCE_IN),
        PINMUX_DATA(SDA2_MARK, PB6MD_01),
        PINMUX_DATA(PINT5_PB_MARK, PB6MD_10),
        PINMUX_DATA(IRQ5_PB_MARK, PB6MD_11),
 
-       PINMUX_DATA(PB4_DATA, PB4MD_00, PB4_IN),
+       PINMUX_DATA(PB4_DATA, PB4MD_00, FORCE_IN),
        PINMUX_DATA(SCL2_MARK, PB4MD_01),
        PINMUX_DATA(PINT4_PB_MARK, PB4MD_10),
        PINMUX_DATA(IRQ4_PB_MARK, PB4MD_11),
 
-       PINMUX_DATA(PB3_DATA, PB3MD_00, PB3_IN),
+       PINMUX_DATA(PB3_DATA, PB3MD_00, FORCE_IN),
        PINMUX_DATA(SDA1_MARK, PB3MD_01),
        PINMUX_DATA(PINT3_PB_MARK, PB3MD_10),
        PINMUX_DATA(IRQ3_PB_MARK, PB3MD_11),
 
-       PINMUX_DATA(PB2_DATA, PB2MD_00, PB2_IN),
+       PINMUX_DATA(PB2_DATA, PB2MD_00, FORCE_IN),
        PINMUX_DATA(SCL1_MARK, PB2MD_01),
        PINMUX_DATA(PINT2_PB_MARK, PB2MD_10),
        PINMUX_DATA(IRQ2_PB_MARK, PB2MD_11),
 
-       PINMUX_DATA(PB1_DATA, PB1MD_00, PB1_IN),
+       PINMUX_DATA(PB1_DATA, PB1MD_00, FORCE_IN),
        PINMUX_DATA(SDA0_MARK, PB1MD_01),
        PINMUX_DATA(PINT1_PB_MARK, PB1MD_10),
        PINMUX_DATA(IRQ1_PB_MARK, PB1MD_11),
 
-       PINMUX_DATA(PB0_DATA, PB0MD_00, PB0_IN),
+       PINMUX_DATA(PB0_DATA, PB0MD_00, FORCE_IN),
        PINMUX_DATA(SCL0_MARK, PB0MD_01),
        PINMUX_DATA(PINT0_PB_MARK, PB0MD_10),
        PINMUX_DATA(IRQ0_PB_MARK, PB0MD_11),
@@ -1083,14 +1082,14 @@ static struct pinmux_cfg_reg pinmux_config_regs[] = {
                PB10_IN, PB10_OUT,
                PB9_IN, PB9_OUT,
                PB8_IN, PB8_OUT,
-               PB7_IN, 0,
-               PB6_IN, 0,
-               PB5_IN, 0,
-               PB4_IN, 0,
-               PB3_IN, 0,
-               PB2_IN, 0,
-               PB1_IN, 0,
-               PB0_IN, 0 }
+               0, 0,
+               0, 0,
+               0, 0,
+               0, 0,
+               0, 0,
+               0, 0,
+               0, 0,
+               0, 0 }
        },
        { PINMUX_CFG_REG("PBCRL4", 0xfffe3890, 16, 4) {
                0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
@@ -1575,8 +1574,8 @@ static struct pinmux_info sh7203_pinmux_info = {
        .name = "sh7203_pfc",
        .reserved_id = PINMUX_RESERVED,
        .data = { PINMUX_DATA_BEGIN, PINMUX_DATA_END },
-       .input = { PINMUX_INPUT_BEGIN, PINMUX_INPUT_END },
-       .output = { PINMUX_OUTPUT_BEGIN, PINMUX_OUTPUT_END },
+       .input = { PINMUX_INPUT_BEGIN, PINMUX_INPUT_END, FORCE_IN },
+       .output = { PINMUX_OUTPUT_BEGIN, PINMUX_OUTPUT_END, FORCE_OUT },
        .mark = { PINMUX_MARK_BEGIN, PINMUX_MARK_END },
        .function = { PINMUX_FUNCTION_BEGIN, PINMUX_FUNCTION_END },
 

diff --git a/arch/sh/kernel/gpio.c b/arch/sh/kernel/gpio.c
index 9ac0b8e6c217111edb387eaead63306a606087fa..d371653610348313dc4b6481764b9b0ea4a08988 100644 (file)
--- a/arch/sh/kernel/gpio.c
+++ b/arch/sh/kernel/gpio.c
@@ -267,9 +267,13 @@ int pinmux_config_gpio(struct pinmux_info *gpioc, unsigned gpio,
                        break;
 
                in_range = enum_in_range(enum_id, &gpioc->function);
-               if (!in_range && range)
+               if (!in_range && range) {
                        in_range = enum_in_range(enum_id, range);
 
+                       if (in_range && enum_id == range->force)
+                               continue;
+               }
+
                if (!in_range)
                        continue;