2021-05-25 05:55:48

by Steven Lee

Subject: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings

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 <[email protected]>
Reviewed-by: Andrew Jeffery <[email protected]>
---
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

2021-05-25 08:24:31

by Joel Stanley

Subject: Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings

On Tue, 25 May 2021 at 05:53, Steven Lee <[email protected]> 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 <[email protected]>
> Reviewed-by: Andrew Jeffery <[email protected]>

Acked-by: Joel Stanley <[email protected]>

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
>

2021-05-28 00:25:43

by Linus Walleij

Subject: Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings

On Tue, May 25, 2021 at 7:53 AM Steven Lee <[email protected]> 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 <[email protected]>
> Reviewed-by: Andrew Jeffery <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

Please funnel this patch through the Aspeed/ARM SoC tree.

Yours,
Linus Walleij

2021-06-03 04:14:55

by Joel Stanley

Subject: Re: [PATCH v3 2/3] ARM: dts: aspeed-g6: Add pinctrl settings

On Thu, 27 May 2021 at 23:47, Linus Walleij <[email protected]> wrote:
>
> On Tue, May 25, 2021 at 7:53 AM Steven Lee <[email protected]> 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 <[email protected]>
> > Reviewed-by: Andrew Jeffery <[email protected]>
>
> Reviewed-by: Linus Walleij <[email protected]>
>
> Please funnel this patch through the Aspeed/ARM SoC tree.

Applied, thanks.