[PATCH v2 0/2] FIX: Re-enable MUX_MMIO on J721E/J7200

[PATCH v2 0/2] FIX: Re-enable MUX_MMIO on J721E/J7200

[Siddharth Vadapalli]

Hello,

This series re-enables MUX_MMIO at U-Boot proper on J721E and J7200
SoCs. The commits which introduced support for Hyperflash boot on both
of these SoCs disabled MUX_MMIO functionality at U-Boot proper, thereby
introducing a regression.

Series is based on commit
2b1c8d3b2d cmd: Fix Kconfig coding style
of the master branch of U-Boot.

v1 of this series was a single patch which was posted to fix the issue
on J7200. The issue was detected when PCIe functionality on J7200 was
tested and found to be broken.
v1:
https://patchwork.ozlabs.org/project/uboot/patch/20250127162509.965644-1-s-vadapalli@ti.com/
Changes since v1:
- Added patch to fix issue on J721E as well.

Regards,
Siddharth.

Siddharth Vadapalli (2):
  configs: j721e_evm_a72_defconfig: Re-enable MUX_MMIO at U-Boot proper
  configs: j7200_evm_a72_defconfig: Re-enable MUX_MMIO at U-Boot proper

 configs/j7200_evm_a72_defconfig | 1 +
 configs/j721e_evm_a72_defconfig | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

-- 
2.43.0

[PATCH v2 1/2] configs: j721e_evm_a72_defconfig: Re-enable MUX_MMIO at U-Boot proper

[Siddharth Vadapalli]

Commit under Fixes introduced support for Hyperflash boot but also disabled
MUX_MMIO support at U-Boot proper. MUX_MMIO is required at U-Boot proper
to setup the SERDES Lane Mapping as described by the device-tree.

Fix this regression by re-enabling MUX_MMIO.

Fixes: fd7fcd4098d5 ("configs: j721e_evm_*_defconfig: Add configs for Hyperflash boot")
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---

This patch has been newly introduced in this series and has no v1.

 configs/j721e_evm_a72_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configs/j721e_evm_a72_defconfig b/configs/j721e_evm_a72_defconfig
index 3928430f16..1ddb1965d8 100644
--- a/configs/j721e_evm_a72_defconfig
+++ b/configs/j721e_evm_a72_defconfig
@@ -151,7 +151,7 @@ CONFIG_SPI_FLASH_MT35XU=y
 # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_SPI_FLASH_MTD=y
 CONFIG_MULTIPLEXER=y
-CONFIG_SPL_MUX_MMIO=y
+CONFIG_MUX_MMIO=y
 CONFIG_PHY_TI_DP83867=y
 CONFIG_PHY_FIXED=y
 CONFIG_TI_AM65_CPSW_NUSS=y
-- 
2.43.0

[PATCH v2 2/2] configs: j7200_evm_a72_defconfig: Re-enable MUX_MMIO at U-Boot proper

[Siddharth Vadapalli]

Commit under Fixes introduced support for Hyperflash boot but also disabled
MUX_MMIO support at U-Boot proper. MUX_MMIO is required at U-Boot proper
to setup the SERDES Lane Mapping as described by the device-tree. On J7200
SoC, PCIe functionality is broken since commit under fixes.

Fix this regression by re-enabling MUX_MMIO.

Fixes: 038f6faea9f0 ("configs: j7200_evm_*_defconfig: Add configs for Hyperflash boot")
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
---

v1:
https://patchwork.ozlabs.org/project/uboot/patch/20250127162509.965644-1-s-vadapalli@ti.com/
Changes since v1:
- Rephrased commit message.

 configs/j7200_evm_a72_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/j7200_evm_a72_defconfig b/configs/j7200_evm_a72_defconfig
index e1ef6e86d4..ac5b10ba14 100644
--- a/configs/j7200_evm_a72_defconfig
+++ b/configs/j7200_evm_a72_defconfig
@@ -144,6 +144,7 @@ CONFIG_SPI_FLASH_S28HX_T=y
 # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
 CONFIG_SPI_FLASH_MTD=y
 CONFIG_MULTIPLEXER=y
+CONFIG_MUX_MMIO=y
 CONFIG_PHY_TI_DP83869=y
 CONFIG_PHY_FIXED=y
 CONFIG_TI_AM65_CPSW_NUSS=y
-- 
2.43.0

Re: [PATCH v2 2/2] configs: j7200_evm_a72_defconfig: Re-enable MUX_MMIO at U-Boot proper

[Dutta, Anurag]

On 01-02-2025 14:48, Siddharth Vadapalli wrote:
> Commit under Fixes introduced support for Hyperflash boot but also disabled
> MUX_MMIO support at U-Boot proper. MUX_MMIO is required at U-Boot proper
> to setup the SERDES Lane Mapping as described by the device-tree. On J7200
> SoC, PCIe functionality is broken since commit under fixes.
>
> Fix this regression by re-enabling MUX_MMIO.
>
> Fixes: 038f6faea9f0 ("configs: j7200_evm_*_defconfig: Add configs for Hyperflash boot")
> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
Reviewed-by: Anurag Dutta <a-dutta@ti.com>
> ---
>
> v1:
> https://patchwork.ozlabs.org/project/uboot/patch/20250127162509.965644-1-s-vadapalli@ti.com/
> Changes since v1:
> - Rephrased commit message.
>
>   configs/j7200_evm_a72_defconfig | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/configs/j7200_evm_a72_defconfig b/configs/j7200_evm_a72_defconfig
> index e1ef6e86d4..ac5b10ba14 100644
> --- a/configs/j7200_evm_a72_defconfig
> +++ b/configs/j7200_evm_a72_defconfig
> @@ -144,6 +144,7 @@ CONFIG_SPI_FLASH_S28HX_T=y
>   # CONFIG_SPI_FLASH_USE_4K_SECTORS is not set
>   CONFIG_SPI_FLASH_MTD=y
>   CONFIG_MULTIPLEXER=y
> +CONFIG_MUX_MMIO=y
>   CONFIG_PHY_TI_DP83869=y
>   CONFIG_PHY_FIXED=y
>   CONFIG_TI_AM65_CPSW_NUSS=y

Re: [PATCH v2 0/2] FIX: Re-enable MUX_MMIO on J721E/J7200

[Tom Rini]

On Sat, 01 Feb 2025 14:47:58 +0530, Siddharth Vadapalli wrote:

> This series re-enables MUX_MMIO at U-Boot proper on J721E and J7200
> SoCs. The commits which introduced support for Hyperflash boot on both
> of these SoCs disabled MUX_MMIO functionality at U-Boot proper, thereby
> introducing a regression.
> 
> Series is based on commit
> 2b1c8d3b2d cmd: Fix Kconfig coding style
> of the master branch of U-Boot.
> 
> [...]

Applied to u-boot/master, thanks!

-- 
Tom