* [PATCH v3 0/3] pinctrl: pinctrl-g6: Add the 2nd sgpio
@ 2021-05-25 5:53 Steven Lee
2021-05-25 5:53 ` [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2 Steven Lee
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Steven Lee @ 2021-05-25 5:53 UTC (permalink / raw)
To: Andrew Jeffery, Linus Walleij, Rob Herring, Joel Stanley,
moderated list:ASPEED PINCTRL DRIVERS,
moderated list:ASPEED PINCTRL DRIVERS,
open list:ASPEED PINCTRL DRIVERS,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
moderated list:ARM/ASPEED MACHINE SUPPORT, open list
Cc: billy_tsai, ryan_chen, steven_lee, Hongweiz
AST2600 has 2 SGPIO master interfaces one with 128 pins and another one
has 80 pins, it also supports 2 SGPIO slave interfaces.
However, there is only the first sgpio master/slave interface defined in
dtsi and pinctrl driver.
The patch series adds the second SGPIO master and slave interfaces
in dt-bindings, dtsi and pinctrl driver.
Changes from v2:
* Fix the typo of the commit message of the 3rd patch.
Changes from v1:
* Correct the function priority of multi-function pin.
* Change the name of arguments of PIN_DECL_4.
Please help to review.
Thanks,
Steven
Steven Lee (3):
dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2
ARM: dts: aspeed-g6: Add pinctrl settings
pinctrl: pinctrl-aspeed-g6: Add sgpio pinctrl settings
.../pinctrl/aspeed,ast2600-pinctrl.yaml | 10 ++++----
arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi | 10 ++++++++
drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 24 +++++++++++++++----
drivers/pinctrl/aspeed/pinmux-aspeed.h | 9 +++++++
4 files changed, 44 insertions(+), 9 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2
2021-05-25 5:53 [PATCH v3 0/3] pinctrl: pinctrl-g6: Add the 2nd sgpio Steven Lee
@ 2021-05-25 5:53 ` Steven Lee
2021-05-27 23:45 ` Linus Walleij
2021-05-25 5:53 ` [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings Steven Lee
2021-05-25 5:53 ` [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio " Steven Lee
2 siblings, 1 reply; 9+ messages in thread
From: Steven Lee @ 2021-05-25 5:53 UTC (permalink / raw)
To: Andrew Jeffery, Linus Walleij, Rob Herring, Joel Stanley,
moderated list:ASPEED PINCTRL DRIVERS,
moderated list:ASPEED PINCTRL DRIVERS,
open list:ASPEED PINCTRL DRIVERS,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
moderated list:ARM/ASPEED MACHINE SUPPORT, open list
Cc: billy_tsai, ryan_chen, steven_lee, Hongweiz
AST2600 has 2 SGPIO master interfaces one with 128 pins and another one
has 80 pins. It also supports 2 SGPIO slave interfaces.
In the current bindings, there are only SGPM1 and SGPS1 defined in enum,
SGPM2 and SGPS2 should also be added in the bindings.
Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
---
.../bindings/pinctrl/aspeed,ast2600-pinctrl.yaml | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml
index ad91c0bc54da..ad2866c99738 100644
--- a/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml
@@ -46,8 +46,8 @@ patternProperties:
PWM9, RGMII1, RGMII2, RGMII3, RGMII4, RMII1, RMII2, RMII3, RMII4,
RXD1, RXD2, RXD3, RXD4, SALT1, SALT10, SALT11, SALT12, SALT13, SALT14,
SALT15, SALT16, SALT2, SALT3, SALT4, SALT5, SALT6, SALT7, SALT8,
- SALT9, SD1, SD2, SGPM1, SGPS1, SIOONCTRL, SIOPBI, SIOPBO, SIOPWREQ,
- SIOPWRGD, SIOS3, SIOS5, SIOSCI, SPI1, SPI1ABR, SPI1CS1, SPI1WP, SPI2,
+ SALT9, SD1, SD2, SGPM1, SGPM2, SGPS1, SGPS2, SIOONCTRL, SIOPBI, SIOPBO,
+ SIOPWREQ, SIOPWRGD, SIOS3, SIOS5, SIOSCI, SPI1, SPI1ABR, SPI1CS1, SPI1WP, SPI2,
SPI2CS1, SPI2CS2, TACH0, TACH1, TACH10, TACH11, TACH12, TACH13, TACH14,
TACH15, TACH2, TACH3, TACH4, TACH5, TACH6, TACH7, TACH8, TACH9, THRU0,
THRU1, THRU2, THRU3, TXD1, TXD2, TXD3, TXD4, UART10, UART11, UART12,
@@ -74,9 +74,9 @@ patternProperties:
RXD1, RXD2, RXD3, RXD4, SALT1, SALT10G0, SALT10G1, SALT11G0, SALT11G1,
SALT12G0, SALT12G1, SALT13G0, SALT13G1, SALT14G0, SALT14G1, SALT15G0,
SALT15G1, SALT16G0, SALT16G1, SALT2, SALT3, SALT4, SALT5, SALT6,
- SALT7, SALT8, SALT9G0, SALT9G1, SD1, SD2, SD3, SGPM1, SGPS1, SIOONCTRL,
- SIOPBI, SIOPBO, SIOPWREQ, SIOPWRGD, SIOS3, SIOS5, SIOSCI, SPI1, SPI1ABR,
- SPI1CS1, SPI1WP, SPI2, SPI2CS1, SPI2CS2, TACH0, TACH1, TACH10, TACH11,
+ SALT7, SALT8, SALT9G0, SALT9G1, SD1, SD2, SD3, SGPM1, SGPM2, SGPS1, SGPS2,
+ SIOONCTRL, SIOPBI, SIOPBO, SIOPWREQ, SIOPWRGD, SIOS3, SIOS5, SIOSCI, SPI1,
+ SPI1ABR, SPI1CS1, SPI1WP, SPI2, SPI2CS1, SPI2CS2, TACH0, TACH1, TACH10, TACH11,
TACH12, TACH13, TACH14, TACH15, TACH2, TACH3, TACH4, TACH5, TACH6,
TACH7, TACH8, TACH9, THRU0, THRU1, THRU2, THRU3, TXD1, TXD2, TXD3,
TXD4, UART10, UART11, UART12G0, UART12G1, UART13G0, UART13G1, UART6,
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings
2021-05-25 5:53 [PATCH v3 0/3] pinctrl: pinctrl-g6: Add the 2nd sgpio Steven Lee
2021-05-25 5:53 ` [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2 Steven Lee
@ 2021-05-25 5:53 ` Steven Lee
2021-05-25 7:29 ` Joel Stanley
2021-05-27 23:47 ` Linus Walleij
2021-05-25 5:53 ` [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio " Steven Lee
2 siblings, 2 replies; 9+ messages in thread
From: Steven Lee @ 2021-05-25 5:53 UTC (permalink / raw)
To: Andrew Jeffery, Linus Walleij, Rob Herring, Joel Stanley,
moderated list:ASPEED PINCTRL DRIVERS,
moderated list:ASPEED PINCTRL DRIVERS,
open list:ASPEED PINCTRL DRIVERS,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
moderated list:ARM/ASPEED MACHINE SUPPORT, open list
Cc: billy_tsai, ryan_chen, steven_lee, Hongweiz
AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
Currently, only SGPIO master 1 and SGPIO slve 1 in the pinctrl dtsi.
SGPIO master 2 and slave 2 should be added in pinctrl dtsi as well.
Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
---
arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
index 7028e21bdd98..7e90d713f5e5 100644
--- a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
+++ b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
@@ -862,11 +862,21 @@
groups = "SGPM1";
};
+ pinctrl_sgpm2_default: sgpm2_default {
+ function = "SGPM2";
+ groups = "SGPM2";
+ };
+
pinctrl_sgps1_default: sgps1_default {
function = "SGPS1";
groups = "SGPS1";
};
+ pinctrl_sgps2_default: sgps2_default {
+ function = "SGPS2";
+ groups = "SGPS2";
+ };
+
pinctrl_sioonctrl_default: sioonctrl_default {
function = "SIOONCTRL";
groups = "SIOONCTRL";
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio pinctrl settings
2021-05-25 5:53 [PATCH v3 0/3] pinctrl: pinctrl-g6: Add the 2nd sgpio Steven Lee
2021-05-25 5:53 ` [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2 Steven Lee
2021-05-25 5:53 ` [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings Steven Lee
@ 2021-05-25 5:53 ` Steven Lee
2021-05-27 23:46 ` Linus Walleij
2 siblings, 1 reply; 9+ messages in thread
From: Steven Lee @ 2021-05-25 5:53 UTC (permalink / raw)
To: Andrew Jeffery, Linus Walleij, Rob Herring, Joel Stanley,
moderated list:ASPEED PINCTRL DRIVERS,
moderated list:ASPEED PINCTRL DRIVERS,
open list:ASPEED PINCTRL DRIVERS,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
moderated list:ARM/ASPEED MACHINE SUPPORT, open list
Cc: billy_tsai, ryan_chen, steven_lee, Hongweiz
AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
Current pinctrl driver only define the first sgpio master and slave
interfaces.
The second SGPIO master and slave interfaces should be added in
pinctrl driver as well.
Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
---
drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 24 ++++++++++++++++++----
drivers/pinctrl/aspeed/pinmux-aspeed.h | 9 ++++++++
2 files changed, 29 insertions(+), 4 deletions(-)
diff --git a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
index 5c1a109842a7..36688793b3a0 100644
--- a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
+++ b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
@@ -46,8 +46,10 @@
#define SCU620 0x620 /* Disable GPIO Internal Pull-Down #4 */
#define SCU634 0x634 /* Disable GPIO Internal Pull-Down #5 */
#define SCU638 0x638 /* Disable GPIO Internal Pull-Down #6 */
+#define SCU690 0x690 /* Multi-function Pin Control #24 */
#define SCU694 0x694 /* Multi-function Pin Control #25 */
#define SCU69C 0x69C /* Multi-function Pin Control #27 */
+#define SCU6D0 0x6D0 /* Multi-function Pin Control #29 */
#define SCUC20 0xC20 /* PCIE configuration Setting Control */
#define ASPEED_G6_NR_PINS 256
@@ -81,13 +83,17 @@ FUNC_GROUP_DECL(I2C12, L26, K24);
#define K26 4
SIG_EXPR_LIST_DECL_SESG(K26, MACLINK1, MACLINK1, SIG_DESC_SET(SCU410, 4));
SIG_EXPR_LIST_DECL_SESG(K26, SCL13, I2C13, SIG_DESC_SET(SCU4B0, 4));
-PIN_DECL_2(K26, GPIOA4, MACLINK1, SCL13);
+SIG_EXPR_LIST_DECL_SESG(K26, SGPS2CK, SGPS2, SIG_DESC_SET(SCU690, 4));
+SIG_EXPR_LIST_DECL_SESG(K26, SGPM2CLK, SGPM2, SIG_DESC_SET(SCU6D0, 4));
+PIN_DECL_4(K26, GPIOA4, MACLINK1, SCL13, SGPS2CK, SGPM2CLK);
FUNC_GROUP_DECL(MACLINK1, K26);
#define L24 5
SIG_EXPR_LIST_DECL_SESG(L24, MACLINK2, MACLINK2, SIG_DESC_SET(SCU410, 5));
SIG_EXPR_LIST_DECL_SESG(L24, SDA13, I2C13, SIG_DESC_SET(SCU4B0, 5));
-PIN_DECL_2(L24, GPIOA5, MACLINK2, SDA13);
+SIG_EXPR_LIST_DECL_SESG(L24, SGPS2LD, SGPS2, SIG_DESC_SET(SCU690, 5));
+SIG_EXPR_LIST_DECL_SESG(L24, SGPM2LD, SGPM2, SIG_DESC_SET(SCU6D0, 5));
+PIN_DECL_4(L24, GPIOA5, MACLINK2, SDA13, SGPS2LD, SGPM2LD);
FUNC_GROUP_DECL(MACLINK2, L24);
FUNC_GROUP_DECL(I2C13, K26, L24);
@@ -95,16 +101,22 @@ FUNC_GROUP_DECL(I2C13, K26, L24);
#define L23 6
SIG_EXPR_LIST_DECL_SESG(L23, MACLINK3, MACLINK3, SIG_DESC_SET(SCU410, 6));
SIG_EXPR_LIST_DECL_SESG(L23, SCL14, I2C14, SIG_DESC_SET(SCU4B0, 6));
-PIN_DECL_2(L23, GPIOA6, MACLINK3, SCL14);
+SIG_EXPR_LIST_DECL_SESG(L23, SGPS2O, SGPS2, SIG_DESC_SET(SCU690, 6));
+SIG_EXPR_LIST_DECL_SESG(L23, SGPM2O, SGPM2, SIG_DESC_SET(SCU6D0, 6));
+PIN_DECL_4(L23, GPIOA6, MACLINK3, SCL14, SGPS2O, SGPM2O);
FUNC_GROUP_DECL(MACLINK3, L23);
#define K25 7
SIG_EXPR_LIST_DECL_SESG(K25, MACLINK4, MACLINK4, SIG_DESC_SET(SCU410, 7));
SIG_EXPR_LIST_DECL_SESG(K25, SDA14, I2C14, SIG_DESC_SET(SCU4B0, 7));
-PIN_DECL_2(K25, GPIOA7, MACLINK4, SDA14);
+SIG_EXPR_LIST_DECL_SESG(K25, SGPS2I, SGPS2, SIG_DESC_SET(SCU690, 7));
+SIG_EXPR_LIST_DECL_SESG(K25, SGPM2I, SGPM2, SIG_DESC_SET(SCU6D0, 7));
+PIN_DECL_4(K25, GPIOA7, MACLINK4, SDA14, SGPS2I, SGPM2I);
FUNC_GROUP_DECL(MACLINK4, K25);
FUNC_GROUP_DECL(I2C14, L23, K25);
+FUNC_GROUP_DECL(SGPM2, K26, L24, L23, K25);
+FUNC_GROUP_DECL(SGPS2, K26, L24, L23, K25);
#define J26 8
SIG_EXPR_LIST_DECL_SESG(J26, SALT1, SALT1, SIG_DESC_SET(SCU410, 8));
@@ -2060,7 +2072,9 @@ static const struct aspeed_pin_group aspeed_g6_groups[] = {
ASPEED_PINCTRL_GROUP(EMMCG4),
ASPEED_PINCTRL_GROUP(EMMCG8),
ASPEED_PINCTRL_GROUP(SGPM1),
+ ASPEED_PINCTRL_GROUP(SGPM2),
ASPEED_PINCTRL_GROUP(SGPS1),
+ ASPEED_PINCTRL_GROUP(SGPS2),
ASPEED_PINCTRL_GROUP(SIOONCTRL),
ASPEED_PINCTRL_GROUP(SIOPBI),
ASPEED_PINCTRL_GROUP(SIOPBO),
@@ -2276,7 +2290,9 @@ static const struct aspeed_pin_function aspeed_g6_functions[] = {
ASPEED_PINCTRL_FUNC(SD1),
ASPEED_PINCTRL_FUNC(SD2),
ASPEED_PINCTRL_FUNC(SGPM1),
+ ASPEED_PINCTRL_FUNC(SGPM2),
ASPEED_PINCTRL_FUNC(SGPS1),
+ ASPEED_PINCTRL_FUNC(SGPS2),
ASPEED_PINCTRL_FUNC(SIOONCTRL),
ASPEED_PINCTRL_FUNC(SIOPBI),
ASPEED_PINCTRL_FUNC(SIOPBO),
diff --git a/drivers/pinctrl/aspeed/pinmux-aspeed.h b/drivers/pinctrl/aspeed/pinmux-aspeed.h
index dba5875ff276..b69ba6b360a2 100644
--- a/drivers/pinctrl/aspeed/pinmux-aspeed.h
+++ b/drivers/pinctrl/aspeed/pinmux-aspeed.h
@@ -730,6 +730,15 @@ struct aspeed_pin_desc {
SIG_EXPR_LIST_PTR(pin, low), \
SIG_EXPR_LIST_PTR(pin, other))
+#define PIN_DECL_4(pin, other, prio1, prio2, prio3, prio4) \
+ SIG_EXPR_LIST_DECL_SESG(pin, other, other); \
+ PIN_DECL_(pin, \
+ SIG_EXPR_LIST_PTR(pin, prio1), \
+ SIG_EXPR_LIST_PTR(pin, prio2), \
+ SIG_EXPR_LIST_PTR(pin, prio3), \
+ SIG_EXPR_LIST_PTR(pin, prio4), \
+ SIG_EXPR_LIST_PTR(pin, other))
+
#define GROUP_SYM(group) group_pins_ ## group
#define GROUP_DECL(group, ...) \
static const int GROUP_SYM(group)[] = { __VA_ARGS__ }
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings
2021-05-25 5:53 ` [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings Steven Lee
@ 2021-05-25 7:29 ` Joel Stanley
2021-05-27 23:47 ` Linus Walleij
1 sibling, 0 replies; 9+ messages in thread
From: Joel Stanley @ 2021-05-25 7:29 UTC (permalink / raw)
To: Steven Lee
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Ryan Chen, moderated list:ASPEED PINCTRL DRIVERS, Andrew Jeffery,
Linus Walleij, open list, Hongwei Zhang, Billy Tsai,
open list:ASPEED PINCTRL DRIVERS, Rob Herring,
moderated list:ASPEED PINCTRL DRIVERS,
moderated list:ARM/ASPEED MACHINE SUPPORT
On Tue, 25 May 2021 at 05:53, Steven Lee <steven_lee@aspeedtech.com> wrote:
>
> AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
> Currently, only SGPIO master 1 and SGPIO slve 1 in the pinctrl dtsi.
> SGPIO master 2 and slave 2 should be added in pinctrl dtsi as well.
>
> Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Acked-by: Joel Stanley <joel@jms.id.au>
Linus, feel free to take this entire series through your tree.
> ---
> arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
> index 7028e21bdd98..7e90d713f5e5 100644
> --- a/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/aspeed-g6-pinctrl.dtsi
> @@ -862,11 +862,21 @@
> groups = "SGPM1";
> };
>
> + pinctrl_sgpm2_default: sgpm2_default {
> + function = "SGPM2";
> + groups = "SGPM2";
> + };
> +
> pinctrl_sgps1_default: sgps1_default {
> function = "SGPS1";
> groups = "SGPS1";
> };
>
> + pinctrl_sgps2_default: sgps2_default {
> + function = "SGPS2";
> + groups = "SGPS2";
> + };
> +
> pinctrl_sioonctrl_default: sioonctrl_default {
> function = "SIOONCTRL";
> groups = "SIOONCTRL";
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2
2021-05-25 5:53 ` [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2 Steven Lee
@ 2021-05-27 23:45 ` Linus Walleij
0 siblings, 0 replies; 9+ messages in thread
From: Linus Walleij @ 2021-05-27 23:45 UTC (permalink / raw)
To: Steven Lee
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Ryan Chen, moderated list:ASPEED PINCTRL DRIVERS, Andrew Jeffery,
moderated list:ASPEED PINCTRL DRIVERS, open list, Billy Tsai,
open list:ASPEED PINCTRL DRIVERS, Rob Herring, Hongwei Zhang,
moderated list:ARM/ASPEED MACHINE SUPPORT
On Tue, May 25, 2021 at 7:53 AM Steven Lee <steven_lee@aspeedtech.com> wrote:
> AST2600 has 2 SGPIO master interfaces one with 128 pins and another one
> has 80 pins. It also supports 2 SGPIO slave interfaces.
> In the current bindings, there are only SGPM1 and SGPS1 defined in enum,
> SGPM2 and SGPS2 should also be added in the bindings.
>
> Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Patch applied to the pinctrl tree.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio pinctrl settings
2021-05-25 5:53 ` [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio " Steven Lee
@ 2021-05-27 23:46 ` Linus Walleij
0 siblings, 0 replies; 9+ messages in thread
From: Linus Walleij @ 2021-05-27 23:46 UTC (permalink / raw)
To: Steven Lee
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Ryan Chen, moderated list:ASPEED PINCTRL DRIVERS, Andrew Jeffery,
moderated list:ASPEED PINCTRL DRIVERS, open list, Billy Tsai,
open list:ASPEED PINCTRL DRIVERS, Rob Herring, Hongwei Zhang,
moderated list:ARM/ASPEED MACHINE SUPPORT
On Tue, May 25, 2021 at 7:53 AM Steven Lee <steven_lee@aspeedtech.com> wrote:
> AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
> Current pinctrl driver only define the first sgpio master and slave
> interfaces.
> The second SGPIO master and slave interfaces should be added in
> pinctrl driver as well.
>
> Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Patch applied to the pinctrl tree.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings
2021-05-25 5:53 ` [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings Steven Lee
2021-05-25 7:29 ` Joel Stanley
@ 2021-05-27 23:47 ` Linus Walleij
2021-06-03 4:12 ` Joel Stanley
1 sibling, 1 reply; 9+ messages in thread
From: Linus Walleij @ 2021-05-27 23:47 UTC (permalink / raw)
To: Steven Lee
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Ryan Chen, moderated list:ASPEED PINCTRL DRIVERS, Andrew Jeffery,
moderated list:ASPEED PINCTRL DRIVERS, open list, Billy Tsai,
open list:ASPEED PINCTRL DRIVERS, Rob Herring, Hongwei Zhang,
moderated list:ARM/ASPEED MACHINE SUPPORT
On Tue, May 25, 2021 at 7:53 AM Steven Lee <steven_lee@aspeedtech.com> wrote:
> AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
> Currently, only SGPIO master 1 and SGPIO slve 1 in the pinctrl dtsi.
> SGPIO master 2 and slave 2 should be added in pinctrl dtsi as well.
>
> Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Please funnel this patch through the Aspeed/ARM SoC tree.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings
2021-05-27 23:47 ` Linus Walleij
@ 2021-06-03 4:12 ` Joel Stanley
0 siblings, 0 replies; 9+ messages in thread
From: Joel Stanley @ 2021-06-03 4:12 UTC (permalink / raw)
To: Linus Walleij
Cc: open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Ryan Chen, moderated list:ASPEED PINCTRL DRIVERS, Andrew Jeffery,
moderated list:ASPEED PINCTRL DRIVERS, Steven Lee, open list,
Billy Tsai, open list:ASPEED PINCTRL DRIVERS, Rob Herring,
Hongwei Zhang, moderated list:ARM/ASPEED MACHINE SUPPORT
On Thu, 27 May 2021 at 23:47, Linus Walleij <linus.walleij@linaro.org> wrote:
>
> On Tue, May 25, 2021 at 7:53 AM Steven Lee <steven_lee@aspeedtech.com> wrote:
>
> > AST2600 supports 2 SGPIO master interfaces and 2 SGPIO slave interfaces.
> > Currently, only SGPIO master 1 and SGPIO slve 1 in the pinctrl dtsi.
> > SGPIO master 2 and slave 2 should be added in pinctrl dtsi as well.
> >
> > Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> > Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
>
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
>
> Please funnel this patch through the Aspeed/ARM SoC tree.
Applied, thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-06-03 4:13 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-25 5:53 [PATCH v3 0/3] pinctrl: pinctrl-g6: Add the 2nd sgpio Steven Lee
2021-05-25 5:53 ` [PATCH v3 1/3] dt-bindings: pinctrl: Update enum for adding SGPM2 and SGPS2 Steven Lee
2021-05-27 23:45 ` Linus Walleij
2021-05-25 5:53 ` [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings Steven Lee
2021-05-25 7:29 ` Joel Stanley
2021-05-27 23:47 ` Linus Walleij
2021-06-03 4:12 ` Joel Stanley
2021-05-25 5:53 ` [PATCH v3 3/3] pinctrl: pinctrl-aspeed-g6: Add sgpio " Steven Lee
2021-05-27 23:46 ` Linus Walleij
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).