[PATCH] imx: imx6ul: kontron-sl-mx6ul: Disable CONFIG_FSL_QSPI_AHB_FULL_MAP to fix SPI NAND read access

From: Frieder Schrempf frieder.schrempf@kontron.de
The introduction of CONFIG_FSL_QSPI_AHB_FULL_MAP as default in:
def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX")
broke the SPI NAND read access on the Kontron SL i.MX6UL/ULL boards. Reading data from the flash returns garbage instead of the actual content. Fix this for now by disabling the introduced option.
In the long run this should be fixed globally.
Fixes: def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX") Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de --- configs/kontron-sl-mx6ul_defconfig | 1 + 1 file changed, 1 insertion(+)
diff --git a/configs/kontron-sl-mx6ul_defconfig b/configs/kontron-sl-mx6ul_defconfig index aa789fc6e5e..db4b684690f 100644 --- a/configs/kontron-sl-mx6ul_defconfig +++ b/configs/kontron-sl-mx6ul_defconfig @@ -95,6 +95,7 @@ CONFIG_MXC_UART=y CONFIG_SPI=y CONFIG_DM_SPI=y CONFIG_FSL_QSPI=y +# CONFIG_FSL_QSPI_AHB_FULL_MAP is not set CONFIG_MXC_SPI=y CONFIG_IMX_THERMAL=y CONFIG_USB=y

Hi Frieder,
On Mon, Nov 14, 2022 at 9:02 AM Frieder Schrempf frieder@fris.de wrote:
From: Frieder Schrempf frieder.schrempf@kontron.de
The introduction of CONFIG_FSL_QSPI_AHB_FULL_MAP as default in:
def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX")
broke the SPI NAND read access on the Kontron SL i.MX6UL/ULL boards. Reading data from the flash returns garbage instead of the actual content. Fix this for now by disabling the introduced option.
In the long run this should be fixed globally.
Yes, most likely more boards have the same problem.
Reviewed-by: Fabio Estevam festevam@denx.de

Hi Stefano,
On 14.11.22 13:01, Frieder Schrempf wrote:
From: Frieder Schrempf frieder.schrempf@kontron.de
The introduction of CONFIG_FSL_QSPI_AHB_FULL_MAP as default in:
def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX")
broke the SPI NAND read access on the Kontron SL i.MX6UL/ULL boards. Reading data from the flash returns garbage instead of the actual content. Fix this for now by disabling the introduced option.
In the long run this should be fixed globally.
Fixes: def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX") Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de
Can you pick up this patch, please?
Thanks Frieder

On 31.01.23 11:08, Frieder Schrempf wrote:
Hi Stefano,
On 14.11.22 13:01, Frieder Schrempf wrote:
From: Frieder Schrempf frieder.schrempf@kontron.de
The introduction of CONFIG_FSL_QSPI_AHB_FULL_MAP as default in:
def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX")
broke the SPI NAND read access on the Kontron SL i.MX6UL/ULL boards. Reading data from the flash returns garbage instead of the actual content. Fix this for now by disabling the introduced option.
In the long run this should be fixed globally.
Fixes: def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX") Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de
Can you pick up this patch, please?
Yes, it is already marked for the next CI run.
Regards, Stefano
Thanks Frieder

From: Frieder Schrempf frieder.schrempf@kontron.de The introduction of CONFIG_FSL_QSPI_AHB_FULL_MAP as default in: def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX") broke the SPI NAND read access on the Kontron SL i.MX6UL/ULL boards. Reading data from the flash returns garbage instead of the actual content. Fix this for now by disabling the introduced option. In the long run this should be fixed globally. Fixes: def88bce094e ("spi: fsl_qspi: Support to use full AHB space on i.MX") Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de Reviewed-by: Fabio Estevam festevam@denx.de
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
participants (5)
-
Fabio Estevam
-
Frieder Schrempf
-
Frieder Schrempf
-
sbabic@denx.de
-
Stefano Babic