diff mbox series

pinctrl: ep93xx: Fix raster pins typo

Message ID 20240913045829.11904-1-nikita.shubin@maquefel.me (mailing list archive)
State Accepted
Commit 7427c5b34fbe191451a48c69c392c85b648caa4f
Headers show
Series pinctrl: ep93xx: Fix raster pins typo | expand

Commit Message

Nikita Shubin Sept. 13, 2024, 4:58 a.m. UTC
Fix raster_on_sdram0_pins used in PMX_GROUP() instead of
raster_on_sdram3_pins as it should be.

Fixes: d1661439f5a3 ("pinctrl: add a Cirrus ep93xx SoC pin controller")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202409130426.1OzNAC9M-lkp@intel.com/
Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me>
---
 drivers/pinctrl/pinctrl-ep93xx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Alexander Sverdlin Sept. 13, 2024, 6:16 a.m. UTC | #1
Hi Nikita!

On Fri, 2024-09-13 at 07:58 +0300, Nikita Shubin wrote:
> Fix raster_on_sdram0_pins used in PMX_GROUP() instead of
> raster_on_sdram3_pins as it should be.
> 
> Fixes: d1661439f5a3 ("pinctrl: add a Cirrus ep93xx SoC pin controller")
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202409130426.1OzNAC9M-lkp@intel.com/
> Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me>

Acked-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>

> ---
>  drivers/pinctrl/pinctrl-ep93xx.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/pinctrl/pinctrl-ep93xx.c b/drivers/pinctrl/pinctrl-ep93xx.c
> index 458cb0e99c65..abafbbb8fd6a 100644
> --- a/drivers/pinctrl/pinctrl-ep93xx.c
> +++ b/drivers/pinctrl/pinctrl-ep93xx.c
> @@ -714,7 +714,7 @@ static const struct ep93xx_pin_group ep9307_pin_groups[] = {
>  	PMX_GROUP("i2s_on_ac97", ac97_ep9301_pins, EP93XX_SYSCON_DEVCFG_I2SONAC97,
>  						EP93XX_SYSCON_DEVCFG_I2SONAC97),
>  	PMX_GROUP("rasteronsdram0grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3, 0),
> -	PMX_GROUP("rasteronsdram3grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
> +	PMX_GROUP("rasteronsdram3grp", raster_on_sdram3_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
>  							EP93XX_SYSCON_DEVCFG_RASONP3),
>  	PMX_GROUP("gpio2agrp", gpio2a_9307_pins, EP93XX_SYSCON_DEVCFG_GONK,
>  						EP93XX_SYSCON_DEVCFG_GONK),
> @@ -1159,7 +1159,7 @@ static const struct ep93xx_pin_group ep9312_pin_groups[] = {
>  	PMX_GROUP("i2s_on_ac97", ac97_ep9312_pins, EP93XX_SYSCON_DEVCFG_I2SONAC97,
>  						EP93XX_SYSCON_DEVCFG_I2SONAC97),
>  	PMX_GROUP("rasteronsdram0grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3, 0),
> -	PMX_GROUP("rasteronsdram3grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
> +	PMX_GROUP("rasteronsdram3grp", raster_on_sdram3_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
>  							EP93XX_SYSCON_DEVCFG_RASONP3),
>  	PMX_GROUP("gpio2agrp", gpio2a_9312_pins, EP93XX_SYSCON_DEVCFG_GONK,
>  						EP93XX_SYSCON_DEVCFG_GONK),
patchwork-bot+linux-soc@kernel.org Sept. 16, 2024, 12:50 p.m. UTC | #2
Hello:

This patch was applied to soc/soc.git (for-next)
by Arnd Bergmann <arnd@arndb.de>:

On Fri, 13 Sep 2024 07:58:29 +0300 you wrote:
> Fix raster_on_sdram0_pins used in PMX_GROUP() instead of
> raster_on_sdram3_pins as it should be.
> 
> Fixes: d1661439f5a3 ("pinctrl: add a Cirrus ep93xx SoC pin controller")
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202409130426.1OzNAC9M-lkp@intel.com/
> Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me>
> 
> [...]

Here is the summary with links:
  - pinctrl: ep93xx: Fix raster pins typo
    https://git.kernel.org/soc/soc/c/7427c5b34fbe

You are awesome, thank you!
diff mbox series

Patch

diff --git a/drivers/pinctrl/pinctrl-ep93xx.c b/drivers/pinctrl/pinctrl-ep93xx.c
index 458cb0e99c65..abafbbb8fd6a 100644
--- a/drivers/pinctrl/pinctrl-ep93xx.c
+++ b/drivers/pinctrl/pinctrl-ep93xx.c
@@ -714,7 +714,7 @@  static const struct ep93xx_pin_group ep9307_pin_groups[] = {
 	PMX_GROUP("i2s_on_ac97", ac97_ep9301_pins, EP93XX_SYSCON_DEVCFG_I2SONAC97,
 						EP93XX_SYSCON_DEVCFG_I2SONAC97),
 	PMX_GROUP("rasteronsdram0grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3, 0),
-	PMX_GROUP("rasteronsdram3grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
+	PMX_GROUP("rasteronsdram3grp", raster_on_sdram3_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
 							EP93XX_SYSCON_DEVCFG_RASONP3),
 	PMX_GROUP("gpio2agrp", gpio2a_9307_pins, EP93XX_SYSCON_DEVCFG_GONK,
 						EP93XX_SYSCON_DEVCFG_GONK),
@@ -1159,7 +1159,7 @@  static const struct ep93xx_pin_group ep9312_pin_groups[] = {
 	PMX_GROUP("i2s_on_ac97", ac97_ep9312_pins, EP93XX_SYSCON_DEVCFG_I2SONAC97,
 						EP93XX_SYSCON_DEVCFG_I2SONAC97),
 	PMX_GROUP("rasteronsdram0grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3, 0),
-	PMX_GROUP("rasteronsdram3grp", raster_on_sdram0_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
+	PMX_GROUP("rasteronsdram3grp", raster_on_sdram3_pins, EP93XX_SYSCON_DEVCFG_RASONP3,
 							EP93XX_SYSCON_DEVCFG_RASONP3),
 	PMX_GROUP("gpio2agrp", gpio2a_9312_pins, EP93XX_SYSCON_DEVCFG_GONK,
 						EP93XX_SYSCON_DEVCFG_GONK),