All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller
@ 2018-11-13  7:26 Ashish Kumar
  2018-11-19 20:42 ` York Sun
  0 siblings, 1 reply; 4+ messages in thread
From: Ashish Kumar @ 2018-11-13  7:26 UTC (permalink / raw)
  To: u-boot

Enable AHB support for Flexspi controller interface meaning
memory can be accessed via md command using absolute addresses

Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@nxp.com>
Signed-off-by: Rajat Srivastava <rajat.srivastava@nxp.com>
Signed-off-by: Ashish Kumar <Ashish.Kumar@nxp.com>
---
v3:
Depends upon http://patchwork.ozlabs.org/patch/975009/
v2:
 1. Rename FSPI to FlexSPI in description and comments
 2. s/cmd/command
 3. Add macro and comments to improve readablity of code in soc.c
 4. Add declaration in soc.h for fspi_ahb_init()

 arch/arm/cpu/armv8/fsl-layerscape/Kconfig     | 10 +++++
 arch/arm/cpu/armv8/fsl-layerscape/soc.c       | 44 +++++++++++++++++++
 .../arm/include/asm/arch-fsl-layerscape/soc.h |  7 +++
 3 files changed, 61 insertions(+)

diff --git a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
index 5280d33ec8..70f26973e9 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
+++ b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
@@ -327,6 +327,16 @@ config SYS_FSPI_AHB_INIT
 	  performed. Default LUT programmed in AHB mode is Fast Read command
 	  with 4-byte addressing enabled.
 
+config FSPI_AHB_EN_4BYTE
+	bool "Enable 4-byte Fast Read command for AHB mode"
+	depends on NXP_FSPI
+	default n
+	help
+	  The default setting for FlexSPI AHB bus just supports 3-byte addressing.
+	  But some FlexSPI flash sizes are up to 64MBytes.
+	  This flag enables fast read command for AHB mode and modifies required
+	  LUT to support full FlexSPI flash.
+
 config SYS_CCI400_OFFSET
 	hex "Offset for CCI400 base"
 	depends on SYS_FSL_HAS_CCI400
diff --git a/arch/arm/cpu/armv8/fsl-layerscape/soc.c b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
index 3f15cb08ff..0a0e112a88 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/soc.c
+++ b/arch/arm/cpu/armv8/fsl-layerscape/soc.c
@@ -640,6 +640,47 @@ void fsl_lsch2_early_init_f(void)
 }
 #endif
 
+#ifdef CONFIG_FSPI_AHB_EN_4BYTE
+int fspi_ahb_init(void)
+{
+	/* Enable 4bytes address support and fast read */
+	u32 *fspi_lut, lut_key, *fspi_key;
+
+	fspi_key = (void *)SYS_NXP_FSPI_ADDR + SYS_NXP_FSPI_LUTKEY_BASE_ADDR;
+	fspi_lut = (void *)SYS_NXP_FSPI_ADDR + SYS_NXP_FSPI_LUT_BASE_ADDR;
+
+	lut_key = in_be32(fspi_key);
+
+	if (lut_key == SYS_NXP_FSPI_LUTKEY) {
+		/* That means the register is BE */
+		out_be32(fspi_key, SYS_NXP_FSPI_LUTKEY);
+		/* Unlock the lut table */
+		out_be32(fspi_key + 1, SYS_NXP_FSPI_LUTCR_UNLOCK);
+		/* Create READ LUT */
+		out_be32(fspi_lut, 0x0820040c);
+		out_be32(fspi_lut + 1, 0x24003008);
+		out_be32(fspi_lut + 2, 0x00000000);
+		/* Lock the lut table */
+		out_be32(fspi_key, SYS_NXP_FSPI_LUTKEY);
+		out_be32(fspi_key + 1, SYS_NXP_FSPI_LUTCR_LOCK);
+	} else {
+		/* That means the register is LE */
+		out_le32(fspi_key, SYS_NXP_FSPI_LUTKEY);
+		/* Unlock the lut table */
+		out_le32(fspi_key + 1, SYS_NXP_FSPI_LUTCR_UNLOCK);
+		/* Create READ LUT */
+		out_le32(fspi_lut, 0x0820040c);
+		out_le32(fspi_lut + 1, 0x24003008);
+		out_le32(fspi_lut + 2, 0x00000000);
+		/* Lock the lut table */
+		out_le32(fspi_key, SYS_NXP_FSPI_LUTKEY);
+		out_le32(fspi_key + 1, SYS_NXP_FSPI_LUTCR_LOCK);
+	}
+
+	return 0;
+}
+#endif
+
 #ifdef CONFIG_QSPI_AHB_INIT
 /* Enable 4bytes address support and fast read */
 int qspi_ahb_init(void)
@@ -688,6 +729,9 @@ int board_late_init(void)
 #ifdef CONFIG_QSPI_AHB_INIT
 	qspi_ahb_init();
 #endif
+#ifdef CONFIG_FSPI_AHB_EN_4BYTE
+	fspi_ahb_init();
+#endif
 
 	return 0;
 }
diff --git a/arch/arm/include/asm/arch-fsl-layerscape/soc.h b/arch/arm/include/asm/arch-fsl-layerscape/soc.h
index 61b6e4bf07..84450c76ed 100644
--- a/arch/arm/include/asm/arch-fsl-layerscape/soc.h
+++ b/arch/arm/include/asm/arch-fsl-layerscape/soc.h
@@ -102,6 +102,13 @@ void init_pfe_scfg_dcfg_regs(void);
 int qspi_ahb_init(void);
 #endif
 
+#ifdef CONFIG_FSPI_AHB_EN_4BYTE
+#define SYS_NXP_FSPI_LUTCR_LOCK			0x00000001
+#define SYS_NXP_FSPI_LUTCR_UNLOCK		0x00000002
+#define SYS_NXP_FSPI_LUTKEY			0x5AF05AF0
+int fspi_ahb_init(void);
+#endif
+
 void cpu_name(char *name);
 #ifdef CONFIG_SYS_FSL_ERRATUM_A009635
 void erratum_a009635(void);
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller
  2018-11-13  7:26 [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller Ashish Kumar
@ 2018-11-19 20:42 ` York Sun
  2018-11-20  9:24   ` Ashish Kumar
  0 siblings, 1 reply; 4+ messages in thread
From: York Sun @ 2018-11-19 20:42 UTC (permalink / raw)
  To: u-boot

On 11/12/18 11:26 PM, Ashish Kumar wrote:
> Enable AHB support for Flexspi controller interface meaning
> memory can be accessed via md command using absolute addresses
> 
> Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@nxp.com>
> Signed-off-by: Rajat Srivastava <rajat.srivastava@nxp.com>
> Signed-off-by: Ashish Kumar <Ashish.Kumar@nxp.com>
> ---
> v3:
> Depends upon http://patchwork.ozlabs.org/patch/975009/

Your v3 patch depends on its v2 version. This doesn't make sense.

York

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller
  2018-11-19 20:42 ` York Sun
@ 2018-11-20  9:24   ` Ashish Kumar
  2018-11-20 16:54     ` York Sun
  0 siblings, 1 reply; 4+ messages in thread
From: Ashish Kumar @ 2018-11-20  9:24 UTC (permalink / raw)
  To: u-boot



> -----Original Message-----
> From: York Sun
> Sent: Tuesday, November 20, 2018 2:13 AM
> To: Ashish Kumar <ashish.kumar@nxp.com>; u-boot at lists.denx.de
> Cc: Yogesh Narayan Gaur <yogeshnarayan.gaur@nxp.com>; Rajat Srivastava
> <rajat.srivastava@nxp.com>
> Subject: Re: [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi
> controller
> 
> On 11/12/18 11:26 PM, Ashish Kumar wrote:
> > Enable AHB support for Flexspi controller interface meaning memory can
> > be accessed via md command using absolute addresses
> >
> > Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@nxp.com>
> > Signed-off-by: Rajat Srivastava <rajat.srivastava@nxp.com>
> > Signed-off-by: Ashish Kumar <Ashish.Kumar@nxp.com>
> > ---
> > v3:
> > Depends upon http://patchwork.ozlabs.org/patch/975009/
Hi York,

Sorry that  was typo, it was meant to be http://patchwork.ozlabs.org/patch/974301/.
But unfortunately after lot of delay in review on NXP-FlexSPI driver, it has been asked to re-write to a new frame-work. I think I should defer this http://patchwork.ozlabs.org/patch/975008/ and http://patchwork.ozlabs.org/patch/996846/ for the time being until 974301 is re-written. What do you suggest?

Regards
Ashish
> 
> Your v3 patch depends on its v2 version. This doesn't make sense.
> 
> York

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller
  2018-11-20  9:24   ` Ashish Kumar
@ 2018-11-20 16:54     ` York Sun
  0 siblings, 0 replies; 4+ messages in thread
From: York Sun @ 2018-11-20 16:54 UTC (permalink / raw)
  To: u-boot

On 11/20/18 1:24 AM, Ashish Kumar wrote:
> 
> 
>> -----Original Message-----
>> From: York Sun
>> Sent: Tuesday, November 20, 2018 2:13 AM
>> To: Ashish Kumar <ashish.kumar@nxp.com>; u-boot at lists.denx.de
>> Cc: Yogesh Narayan Gaur <yogeshnarayan.gaur@nxp.com>; Rajat Srivastava
>> <rajat.srivastava@nxp.com>
>> Subject: Re: [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi
>> controller
>>
>> On 11/12/18 11:26 PM, Ashish Kumar wrote:
>>> Enable AHB support for Flexspi controller interface meaning memory can
>>> be accessed via md command using absolute addresses
>>>
>>> Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur@nxp.com>
>>> Signed-off-by: Rajat Srivastava <rajat.srivastava@nxp.com>
>>> Signed-off-by: Ashish Kumar <Ashish.Kumar@nxp.com>
>>> ---
>>> v3:
>>> Depends upon http://patchwork.ozlabs.org/patch/975009/
> Hi York,
> 
> Sorry that  was typo, it was meant to be http://patchwork.ozlabs.org/patch/974301/.
> But unfortunately after lot of delay in review on NXP-FlexSPI driver, it has been asked to re-write to a new frame-work. I think I should defer this http://patchwork.ozlabs.org/patch/975008/ and http://patchwork.ozlabs.org/patch/996846/ for the time being until 974301 is re-written. What do you suggest?
> 

You still need FlexSPI driver, don't you? Please work with Jagan on the
driver.

York

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-11-20 16:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-13  7:26 [U-Boot] [PATCH v3] fsl-lsch3: soc: Enable AHB read support for Flexspi controller Ashish Kumar
2018-11-19 20:42 ` York Sun
2018-11-20  9:24   ` Ashish Kumar
2018-11-20 16:54     ` York Sun

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.