* [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T
@ 2024-03-13 9:28 Billy Tsai
2024-03-13 11:08 ` Paul Menzel
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Billy Tsai @ 2024-03-13 9:28 UTC (permalink / raw)
To: andrew, linus.walleij, joel, johnny_huang, linux-aspeed, openbmc,
linux-gpio, linux-arm-kernel, linux-kernel, BMC-SW, Ricky_CX_Wu
The register offset to disable the internal pull-down of GPIOR~T is 0x630
instead of 0x620, as specified in the Ast2600 datasheet v15
The datasheet can download from the official Aspeed website.
Fixes: 15711ba6ff19 ("pinctrl: aspeed-g6: Add AST2600 pinconf support")
Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
---
drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 34 +++++++++++-----------
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
index d376fa7114d1..029efe16f8cc 100644
--- a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
+++ b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
@@ -43,7 +43,7 @@
#define SCU614 0x614 /* Disable GPIO Internal Pull-Down #1 */
#define SCU618 0x618 /* Disable GPIO Internal Pull-Down #2 */
#define SCU61C 0x61c /* Disable GPIO Internal Pull-Down #3 */
-#define SCU620 0x620 /* Disable GPIO Internal Pull-Down #4 */
+#define SCU630 0x630 /* 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 */
@@ -2495,38 +2495,38 @@ static struct aspeed_pin_config aspeed_g6_configs[] = {
ASPEED_PULL_DOWN_PINCONF(D14, SCU61C, 0),
/* GPIOS7 */
- ASPEED_PULL_DOWN_PINCONF(T24, SCU620, 23),
+ ASPEED_PULL_DOWN_PINCONF(T24, SCU630, 23),
/* GPIOS6 */
- ASPEED_PULL_DOWN_PINCONF(P23, SCU620, 22),
+ ASPEED_PULL_DOWN_PINCONF(P23, SCU630, 22),
/* GPIOS5 */
- ASPEED_PULL_DOWN_PINCONF(P24, SCU620, 21),
+ ASPEED_PULL_DOWN_PINCONF(P24, SCU630, 21),
/* GPIOS4 */
- ASPEED_PULL_DOWN_PINCONF(R26, SCU620, 20),
+ ASPEED_PULL_DOWN_PINCONF(R26, SCU630, 20),
/* GPIOS3*/
- ASPEED_PULL_DOWN_PINCONF(R24, SCU620, 19),
+ ASPEED_PULL_DOWN_PINCONF(R24, SCU630, 19),
/* GPIOS2 */
- ASPEED_PULL_DOWN_PINCONF(T26, SCU620, 18),
+ ASPEED_PULL_DOWN_PINCONF(T26, SCU630, 18),
/* GPIOS1 */
- ASPEED_PULL_DOWN_PINCONF(T25, SCU620, 17),
+ ASPEED_PULL_DOWN_PINCONF(T25, SCU630, 17),
/* GPIOS0 */
- ASPEED_PULL_DOWN_PINCONF(R23, SCU620, 16),
+ ASPEED_PULL_DOWN_PINCONF(R23, SCU630, 16),
/* GPIOR7 */
- ASPEED_PULL_DOWN_PINCONF(U26, SCU620, 15),
+ ASPEED_PULL_DOWN_PINCONF(U26, SCU630, 15),
/* GPIOR6 */
- ASPEED_PULL_DOWN_PINCONF(W26, SCU620, 14),
+ ASPEED_PULL_DOWN_PINCONF(W26, SCU630, 14),
/* GPIOR5 */
- ASPEED_PULL_DOWN_PINCONF(T23, SCU620, 13),
+ ASPEED_PULL_DOWN_PINCONF(T23, SCU630, 13),
/* GPIOR4 */
- ASPEED_PULL_DOWN_PINCONF(U25, SCU620, 12),
+ ASPEED_PULL_DOWN_PINCONF(U25, SCU630, 12),
/* GPIOR3*/
- ASPEED_PULL_DOWN_PINCONF(V26, SCU620, 11),
+ ASPEED_PULL_DOWN_PINCONF(V26, SCU630, 11),
/* GPIOR2 */
- ASPEED_PULL_DOWN_PINCONF(V24, SCU620, 10),
+ ASPEED_PULL_DOWN_PINCONF(V24, SCU630, 10),
/* GPIOR1 */
- ASPEED_PULL_DOWN_PINCONF(U24, SCU620, 9),
+ ASPEED_PULL_DOWN_PINCONF(U24, SCU630, 9),
/* GPIOR0 */
- ASPEED_PULL_DOWN_PINCONF(V25, SCU620, 8),
+ ASPEED_PULL_DOWN_PINCONF(V25, SCU630, 8),
/* GPIOX7 */
ASPEED_PULL_DOWN_PINCONF(AB10, SCU634, 31),
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T
2024-03-13 9:28 [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T Billy Tsai
@ 2024-03-13 11:08 ` Paul Menzel
2024-03-13 22:53 ` Andrew Jeffery
2024-03-28 9:15 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Paul Menzel @ 2024-03-13 11:08 UTC (permalink / raw)
To: Billy Tsai
Cc: BMC-SW, johnny_huang, linux-aspeed, linus.walleij, Ricky_CX_Wu,
linux-kernel, linux-gpio, joel, openbmc, linux-arm-kernel
Dear Billy,
Thank you very much for sending version 2.
Am 13.03.24 um 10:28 schrieb Billy Tsai:
> The register offset to disable the internal pull-down of GPIOR~T is 0x630
> instead of 0x620, as specified in the Ast2600 datasheet v15
> The datasheet can download from the official Aspeed website.
(No need for another iteration for this, but “can be downloaded”.)
> Fixes: 15711ba6ff19 ("pinctrl: aspeed-g6: Add AST2600 pinconf support")
> Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
> ---
> drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 34 +++++++++++-----------
> 1 file changed, 17 insertions(+), 17 deletions(-)
[…]
Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>
Kind regards,
Paul
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T
2024-03-13 9:28 [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T Billy Tsai
2024-03-13 11:08 ` Paul Menzel
@ 2024-03-13 22:53 ` Andrew Jeffery
2024-03-28 9:15 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Andrew Jeffery @ 2024-03-13 22:53 UTC (permalink / raw)
To: Billy Tsai, linus.walleij, joel, johnny_huang, linux-aspeed,
openbmc, linux-gpio, linux-arm-kernel, linux-kernel, BMC-SW,
Ricky_CX_Wu
On Wed, 2024-03-13 at 17:28 +0800, Billy Tsai wrote:
> The register offset to disable the internal pull-down of GPIOR~T is 0x630
> instead of 0x620, as specified in the Ast2600 datasheet v15
> The datasheet can download from the official Aspeed website.
... if you're a customer with an account enabled to access it :)
>
> Fixes: 15711ba6ff19 ("pinctrl: aspeed-g6: Add AST2600 pinconf support")
> Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
Reviewed-by: Andrew Jeffery <andrew@codeconstruct.com.au>
Not sure what the history is exactly. Could have been a typo or a
change in the SCU register layout given it was likely written against
A0 (A3 is mass-production).
Andrew
> ---
> drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 34 +++++++++++-----------
> 1 file changed, 17 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
> index d376fa7114d1..029efe16f8cc 100644
> --- a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
> +++ b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c
> @@ -43,7 +43,7 @@
> #define SCU614 0x614 /* Disable GPIO Internal Pull-Down #1 */
> #define SCU618 0x618 /* Disable GPIO Internal Pull-Down #2 */
> #define SCU61C 0x61c /* Disable GPIO Internal Pull-Down #3 */
> -#define SCU620 0x620 /* Disable GPIO Internal Pull-Down #4 */
> +#define SCU630 0x630 /* 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 */
> @@ -2495,38 +2495,38 @@ static struct aspeed_pin_config aspeed_g6_configs[] = {
> ASPEED_PULL_DOWN_PINCONF(D14, SCU61C, 0),
>
> /* GPIOS7 */
> - ASPEED_PULL_DOWN_PINCONF(T24, SCU620, 23),
> + ASPEED_PULL_DOWN_PINCONF(T24, SCU630, 23),
> /* GPIOS6 */
> - ASPEED_PULL_DOWN_PINCONF(P23, SCU620, 22),
> + ASPEED_PULL_DOWN_PINCONF(P23, SCU630, 22),
> /* GPIOS5 */
> - ASPEED_PULL_DOWN_PINCONF(P24, SCU620, 21),
> + ASPEED_PULL_DOWN_PINCONF(P24, SCU630, 21),
> /* GPIOS4 */
> - ASPEED_PULL_DOWN_PINCONF(R26, SCU620, 20),
> + ASPEED_PULL_DOWN_PINCONF(R26, SCU630, 20),
> /* GPIOS3*/
> - ASPEED_PULL_DOWN_PINCONF(R24, SCU620, 19),
> + ASPEED_PULL_DOWN_PINCONF(R24, SCU630, 19),
> /* GPIOS2 */
> - ASPEED_PULL_DOWN_PINCONF(T26, SCU620, 18),
> + ASPEED_PULL_DOWN_PINCONF(T26, SCU630, 18),
> /* GPIOS1 */
> - ASPEED_PULL_DOWN_PINCONF(T25, SCU620, 17),
> + ASPEED_PULL_DOWN_PINCONF(T25, SCU630, 17),
> /* GPIOS0 */
> - ASPEED_PULL_DOWN_PINCONF(R23, SCU620, 16),
> + ASPEED_PULL_DOWN_PINCONF(R23, SCU630, 16),
>
> /* GPIOR7 */
> - ASPEED_PULL_DOWN_PINCONF(U26, SCU620, 15),
> + ASPEED_PULL_DOWN_PINCONF(U26, SCU630, 15),
> /* GPIOR6 */
> - ASPEED_PULL_DOWN_PINCONF(W26, SCU620, 14),
> + ASPEED_PULL_DOWN_PINCONF(W26, SCU630, 14),
> /* GPIOR5 */
> - ASPEED_PULL_DOWN_PINCONF(T23, SCU620, 13),
> + ASPEED_PULL_DOWN_PINCONF(T23, SCU630, 13),
> /* GPIOR4 */
> - ASPEED_PULL_DOWN_PINCONF(U25, SCU620, 12),
> + ASPEED_PULL_DOWN_PINCONF(U25, SCU630, 12),
> /* GPIOR3*/
> - ASPEED_PULL_DOWN_PINCONF(V26, SCU620, 11),
> + ASPEED_PULL_DOWN_PINCONF(V26, SCU630, 11),
> /* GPIOR2 */
> - ASPEED_PULL_DOWN_PINCONF(V24, SCU620, 10),
> + ASPEED_PULL_DOWN_PINCONF(V24, SCU630, 10),
> /* GPIOR1 */
> - ASPEED_PULL_DOWN_PINCONF(U24, SCU620, 9),
> + ASPEED_PULL_DOWN_PINCONF(U24, SCU630, 9),
> /* GPIOR0 */
> - ASPEED_PULL_DOWN_PINCONF(V25, SCU620, 8),
> + ASPEED_PULL_DOWN_PINCONF(V25, SCU630, 8),
>
> /* GPIOX7 */
> ASPEED_PULL_DOWN_PINCONF(AB10, SCU634, 31),
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T
2024-03-13 9:28 [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T Billy Tsai
2024-03-13 11:08 ` Paul Menzel
2024-03-13 22:53 ` Andrew Jeffery
@ 2024-03-28 9:15 ` Linus Walleij
2 siblings, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2024-03-28 9:15 UTC (permalink / raw)
To: Billy Tsai
Cc: BMC-SW, johnny_huang, linux-aspeed, openbmc, Ricky_CX_Wu,
linux-kernel, linux-gpio, joel, Delphine CC Chiu,
linux-arm-kernel
On Wed, Mar 13, 2024 at 10:28 AM Billy Tsai <billy_tsai@aspeedtech.com> wrote:
> The register offset to disable the internal pull-down of GPIOR~T is 0x630
> instead of 0x620, as specified in the Ast2600 datasheet v15
> The datasheet can download from the official Aspeed website.
>
> Fixes: 15711ba6ff19 ("pinctrl: aspeed-g6: Add AST2600 pinconf support")
> Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com>
Patch applied.
I recorded Delphine's similar patch as "reported-by".
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-03-28 9:16 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-13 9:28 [PATCH v2] pinctrl: pinctrl-aspeed-g6: Fix register offset for pinconf of GPIOR-T Billy Tsai
2024-03-13 11:08 ` Paul Menzel
2024-03-13 22:53 ` Andrew Jeffery
2024-03-28 9:15 ` 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).