[U-Boot] [PATCH] armv8/ls1043a: Add the OCRAM initialization

[Pratiyush Srivastava]

Clear the content to zero and the ECC error bit of OCRAM1/2.

The OCRAM must be initialized to ZERO by the unit of 8-Byte before
accessing it, or else it will generate ECC error. And the IBR has
accessed the OCRAM before this initialization, so the ECC error
status bit should to be cleared.

Signed-off-by: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>
Signed-off-by: Hou Zhiqiang <zhiqiang@freescale.com>
Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
---
 arch/arm/cpu/armv8/start.S | 39 +++
 1 file changed, 39 insertions(+)

diff --git a/arch/arm/cpu/armv8/start.S b/arch/arm/cpu/armv8/start.S
index 19c771d..24fed46 100644
--- a/arch/arm/cpu/armv8/start.S
+++ b/arch/arm/cpu/armv8/start.S
@@ -11,6 +11,9 @@
 #include 
 #include 
 
+#define DCSR_SYS_DCFG_SBEESR2  0x20140534
+#define DCSR_SYS_DCFG_MBEESR2  0x20140544
+
 /*
  *
  * Startup Code (reset vector)
@@ -259,10 +262,46 @@ WEAK(lowlevel_init)
 #endif /* CONFIG_ARMV8_MULTIENTRY */
 
 2:
+#if defined(CONFIG_FSL_LSCH2) && !defined(CONFIG_SPL_BUILD)
+   bl  fsl_ocram_init
+#endif
mov lr, x29 /* Restore LR */
ret
 ENDPROC(lowlevel_init)
 
+#if defined(CONFIG_FSL_LSCH2) && !defined(CONFIG_SPL_BUILD)
+ENTRY(fsl_ocram_init)
+   mov x28, lr /* Save LR */
+   bl  fsl_clear_ocram
+   bl  fsl_ocram_clear_ecc_err
+   mov lr, x28 /* Restore LR */
+   ret
+ENDPROC(fsl_ocram_init)
+
+ENTRY(fsl_clear_ocram)
+/* Clear OCRAM */
+   ldr x0, =CONFIG_SYS_FSL_OCRAM_BASE
+   ldr x1, =(CONFIG_SYS_FSL_OCRAM_BASE + CONFIG_SYS_FSL_OCRAM_SIZE)
+   mov x2, #0
+clear_loop:
+   str x2, [x0]
+   add x0, x0, #8
+   cmp x0, x1
+   b.loclear_loop
+   ret
+ENDPROC(fsl_clear_ocram)
+
+ENTRY(fsl_ocram_clear_ecc_err)
+   /* OCRAM1/2 ECC status bit */
+   mov w1, #0x60
+   ldr x0, =DCSR_SYS_DCFG_SBEESR2
+   str w1, [x0]
+   ldr x0, =DCSR_SYS_DCFG_MBEESR2
+   str w1, [x0]
+   ret
+ENDPROC(fsl_ocram_init)
+#endif
+
 WEAK(smp_kick_all_cpus)
/* Kick secondary cpus up by SGI 0 interrupt */
 #if defined(CONFIG_GICV2) || defined(CONFIG_GICV3)
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] armv8: fsl-layerscape: Add support of GPIO structure

[Pratiyush Srivastava]

Layerscape supports GPIO registers to conrol GPIO singals.
Add support of GPIO structure to enable GPIO access.

Signed-off-by: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>
Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
---
 arch/arm/cpu/armv8/fsl-layerscape/Kconfig  | 4 
 arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h | 9 +
 2 files changed, 13 insertions(+)

diff --git a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig 
b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
index f8057ba..6af39d9 100644
--- a/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
+++ b/arch/arm/cpu/armv8/fsl-layerscape/Kconfig
@@ -2,6 +2,7 @@ config ARCH_LS1012A
bool "Freescale Layerscape LS1012A SoC"
select SYS_FSL_MMDC
select SYS_FSL_ERRATUM_A010315
+   select SYS_GPIO1_ADDR
 
 config ARCH_LS1043A
bool "Freescale Layerscape LS1043A SoC"
@@ -15,3 +16,6 @@ config SYS_FSL_MMDC
 
 config SYS_FSL_ERRATUM_A010315
bool "Workaround for PCIe erratum A010315"
+
+config SYS_GPIO1_ADDR
+   bool "Support for GPIO structure to enable GPIO access"
diff --git a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h 
b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
index df51871..d9d7cb6 100644
--- a/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
+++ b/arch/arm/include/asm/arch-fsl-layerscape/immap_lsch2.h
@@ -584,6 +584,15 @@ struct ccsr_cci400 {
u8 res_e004[0x1 - 0xe004];
 };
 
+typedef struct ccsr_gpio {
+   u32 gpdir;
+   u32 gpodr;
+   u32 gpdat;
+   u32 gpier;
+   u32 gpimr;
+   u32 gpicr;
+} ccsr_gpio_t;
+
 /* MMU 500 */
 #define SMMU_SCR0  (SMMU_BASE + 0x0)
 #define SMMU_SCR1  (SMMU_BASE + 0x4)
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] armv8:ls1012a: Update bootargs for fast-boot

[Pratiyush Srivastava]

Add optimization parameters like "quiet" in bootargs to reduce the system
boot time

Signed-off-by: Pratiyush Mohan Srivastava 
Signed-off-by: Harninder Rai 
---
 include/configs/ls1012a_common.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/ls1012a_common.h b/include/configs/ls1012a_common.h
index 5fb6c47..a0ef2cd 100644
--- a/include/configs/ls1012a_common.h
+++ b/include/configs/ls1012a_common.h
@@ -120,7 +120,7 @@
"console=ttyAMA0,38400n8\0"
 
 #define CONFIG_BOOTARGS"console=ttyS0,115200 root=/dev/ram0 " \
-   "earlycon=uart8250,mmio,0x21c0500"
+   "earlycon=uart8250,mmio,0x21c0500 quiet 
lpj=25"
 #define CONFIG_BOOTCOMMAND "sf probe 0:0; sf read $kernel_load "\
"$kernel_start $kernel_size && "\
"bootm $kernel_load"
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH][v3] armv8: ls1012a: Updating CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

Remove ramdisk_addr, ramdisk_size and update UART baud-rate.

Signed-off-by: Prabhakar Kushwaha 
Signed-off-by: Pratiyush Mohan Srivastava 
---
Changes for v3 :Incorporated York's comments
- Removed hwconfig variable

Changes for v2 :
- Removed "initrd_high=0x\0"
- Removed console variable

 include/configs/ls1012a_common.h | 5 -
 1 file changed, 5 deletions(-)

diff --git a/include/configs/ls1012a_common.h b/include/configs/ls1012a_common.h
index 5fb6c47..58ca75e 100644
--- a/include/configs/ls1012a_common.h
+++ b/include/configs/ls1012a_common.h
@@ -105,19 +105,14 @@
 
 /* Initial environment variables */
 #define CONFIG_EXTRA_ENV_SETTINGS  \
-   "initrd_high=0x\0"  \
"verify=no\0"   \
-   "hwconfig=fsl_ddr:bank_intlv=auto\0"\
"loadaddr=0x8010\0" \
"kernel_addr=0x10\0"\
-   "ramdisk_addr=0x80\0"   \
-   "ramdisk_size=0x200\0"  \
"fdt_high=0x\0" \
"initrd_high=0x\0"  \
"kernel_start=0xa0\0"   \
"kernel_load=0xa000\0"  \
"kernel_size=0x280\0"   \
-   "console=ttyAMA0,38400n8\0"
 
 #define CONFIG_BOOTARGS"console=ttyS0,115200 root=/dev/ram0 " \
"earlycon=uart8250,mmio,0x21c0500"
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH][v4] board: ls1012afrdm: overwrite CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

LS1012AFRDM has 512MB of DDR.
So update Kernel load address as 0x9600 instead of default
0xa000.

Signed-off-by: Prabhakar Kushwaha 
Signed-off-by: Pratiyush Mohan Srivastava 
---
Changes for v4: Incorporated York's Comments
- Removed hwconfig variable 

Changes for v3:
- Rebased v2 patch to master
- Removed "initrd_high=0x\0"
- Removed console variable "console=ttyAMA0,115200n8\0" 

Changes for v2: Incorporated York's comments
- Removed ramdisk_addr, ramdisk_size
- Updated UART baud-rate.

 include/configs/ls1012afrdm.h | 11 +++
 1 file changed, 11 insertions(+)

diff --git a/include/configs/ls1012afrdm.h b/include/configs/ls1012afrdm.h
index 612f243..f6f88e8 100644
--- a/include/configs/ls1012afrdm.h
+++ b/include/configs/ls1012afrdm.h
@@ -20,6 +20,17 @@
 #define CONFIG_SYS_MEMTEST_START   0x8000
 #define CONFIG_SYS_MEMTEST_END 0x9fff
 
+#undef CONFIG_EXTRA_ENV_SETTINGS
+#define CONFIG_EXTRA_ENV_SETTINGS  \
+   "verify=no\0"   \
+   "loadaddr=0x8010\0" \
+   "kernel_addr=0x10\0"\
+   "fdt_high=0x\0" \
+   "initrd_high=0x\0"  \
+   "kernel_start=0xa0\0"   \
+   "kernel_load=0x9600\0"  \
+   "kernel_size=0x280\0"
+
 /*
 * USB
 */
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH][v2] armv8: ls1012a: Updating CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

> -Original Message-
> From: york sun
> Sent: Thursday, October 06, 2016 10:19 PM
> To: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>; u-
> b...@lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: Re: [PATCH][v2] armv8: ls1012a: Updating
> CONFIG_EXTRA_ENV_SETTINGS
> 
> On 10/04/2016 04:38 AM, Pratiyush Mohan Srivastava wrote:
> > Remove ramdisk_addr, ramdisk_size and update UART baud-rate.
> >
> > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > Signed-off-by: Pratiyush Mohan Srivastava
> <pratiyush.srivast...@nxp.com>
> > ---
> > Changes for v2 :
> > - Removed "initrd_high=0x\0"
> > - Removed console variable
> >
> >  include/configs/ls1012a_common.h | 4 
> >  1 file changed, 4 deletions(-)
> >
> > diff --git a/include/configs/ls1012a_common.h
> b/include/configs/ls1012a_common.h
> > index af45993..02cbf5d 100644
> > --- a/include/configs/ls1012a_common.h
> > +++ b/include/configs/ls1012a_common.h
> > @@ -106,19 +106,15 @@
> >
> >  /* Initial environment variables */
> >  #define CONFIG_EXTRA_ENV_SETTINGS  \
> > -   "initrd_high=0x\0"  \
> > "verify=no\0"   \
> > "hwconfig=fsl_ddr:bank_intlv=auto\0"\
> 
> I realize this hwconfig is not used. Please remove.
[Pratiyush>] Ok .I will remove it.
> 
> York
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH][v3] board: ls1012afrdm: overwrite CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

> -Original Message-
> From: york sun
> Sent: Thursday, October 06, 2016 10:02 PM
> To: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>; u-
> b...@lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: Re: [PATCH][v3] board: ls1012afrdm: overwrite
> CONFIG_EXTRA_ENV_SETTINGS
> 
> On 10/04/2016 04:38 AM, Pratiyush Mohan Srivastava wrote:
> > LS1012AFRDM has 512MB of DDR.
> > So update Kernel load address as 0x9600 instead of default
> > 0xa000.
> >
> > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > Signed-off-by: Pratiyush Mohan Srivastava
> > <pratiyush.srivast...@nxp.com>
> > ---
> > Changes for v3:
> > - Rebased v2 patch to master
> > - Removed "initrd_high=0x\0"
> > - Removed console variable "console=ttyAMA0,115200n8\0"
> >
> > Changes for v2: Incorporated York's comments
> > - Removed ramdisk_addr, ramdisk_size
> > - Updated UART baud-rate.
> >
> >  include/configs/ls1012afrdm.h | 12 
> >  1 file changed, 12 insertions(+)
> >
> > diff --git a/include/configs/ls1012afrdm.h
> > b/include/configs/ls1012afrdm.h index 612f243..5763d86 100644
> > --- a/include/configs/ls1012afrdm.h
> > +++ b/include/configs/ls1012afrdm.h
> > @@ -20,6 +20,18 @@
> >  #define CONFIG_SYS_MEMTEST_START   0x8000
> >  #define CONFIG_SYS_MEMTEST_END 0x9fff
> >
> > +#undef CONFIG_EXTRA_ENV_SETTINGS
> > +#define CONFIG_EXTRA_ENV_SETTINGS  \
> > +   "verify=no\0"   \
> 
> Why do you need this?
[Pratiyush>] This is needed for fast-boot.
> 
> > +   "hwconfig=fsl_ddr:bank_intlv=auto\0"\
> 
> This board doesn't use Freescale DDR driver. This hwconfig is not used.
[Pratiyush>] ok I will remove it
> 
> > +   "loadaddr=0x8010\0" \
> > +   "kernel_addr=0x10\0"\
> > +   "fdt_high=0x\0" \
> > +   "initrd_high=0x\0"  \
> > +   "kernel_start=0xa0\0"   \
> > +   "kernel_load=0x9600\0"  \
> > +   "kernel_size=0x280\0"
> > +
> 
> York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH][v2] board: ls1012afrdm: overwrite CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

> -Original Message-
> From: york sun
> Sent: Friday, August 05, 2016 10:54 PM
> To: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>; u-
> b...@lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: Re: [PATCH][v2] board: ls1012afrdm: overwrite
> CONFIG_EXTRA_ENV_SETTINGS
> 
> On 08/05/2016 02:47 AM, Pratiyush Mohan Srivastava wrote:
> > From: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> >
> > LS1012AFRDM has 512MB of DDR.
> > So update Kernel load address as 0x9600 instead of default
> > 0xa000.
> >
> > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > Signed-off-by: Pratiyush Mohan Srivastava
> > <pratiyush.srivast...@nxp.com>
> > ---
> > Changes for v2: Incorporated York's comments
> > - Removed ramdisk_addr, ramdisk_size
> > - Updated UART baud-rate.
> >
> > include/configs/ls1012afrdm.h | 14 ++
> >  1 file changed, 14 insertions(+)
> >
> > diff --git a/include/configs/ls1012afrdm.h
> > b/include/configs/ls1012afrdm.h index ad81142..f2c6ca8 100644
> > --- a/include/configs/ls1012afrdm.h
> > +++ b/include/configs/ls1012afrdm.h
> > @@ -39,4 +39,18 @@
> >  #define CONFIG_SYS_MEMTEST_START   0x8000
> >  #define CONFIG_SYS_MEMTEST_END 0x9fff
> >
> > +#undef CONFIG_EXTRA_ENV_SETTINGS
> > +#define CONFIG_EXTRA_ENV_SETTINGS  \
> > +   "initrd_high=0x\0"  \
> 
> What is this?
[Pratiyush>] I used the LS1043 dts as the base for this dts. Since ls10143 had 
this parameter, I copied the same for LS1012. 
> 
> > +   "verify=no\0"   \
> > +   "hwconfig=fsl_ddr:bank_intlv=auto\0"\
> > +   "loadaddr=0x8010\0" \
> > +   "kernel_addr=0x10\0"\
> > +   "fdt_high=0x\0" \
> > +   "initrd_high=0x\0"  \
> > +   "kernel_start=0xa0\0"   \
> > +   "kernel_load=0x9600\0"  \
> > +   "kernel_size=0x280\0"   \
> > +   "console=ttyAMA0,115200n8\0"
> 
> Do you really use ttyAMA0? Isn't it ttyS0?
[Pratiyush>] I will remove the redundant console command.
> 
> York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH][v2] board: ls1012afrdm: overwrite CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

> -Original Message-
> From: Pratiyush Srivastava
> Sent: Tuesday, August 09, 2016 11:45 AM
> To: york sun <york@nxp.com>; u-boot@lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: RE: [PATCH][v2] board: ls1012afrdm: overwrite
> CONFIG_EXTRA_ENV_SETTINGS
> 
> 
> 
> > -Original Message-
> > From: york sun
> > Sent: Friday, August 05, 2016 10:54 PM
> > To: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>; u-
> > b...@lists.denx.de
> > Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > Subject: Re: [PATCH][v2] board: ls1012afrdm: overwrite
> > CONFIG_EXTRA_ENV_SETTINGS
> >
> > On 08/05/2016 02:47 AM, Pratiyush Mohan Srivastava wrote:
> > > From: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > >
> > > LS1012AFRDM has 512MB of DDR.
> > > So update Kernel load address as 0x9600 instead of default
> > > 0xa000.
> > >
> > > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > > Signed-off-by: Pratiyush Mohan Srivastava
> > > <pratiyush.srivast...@nxp.com>
> > > ---
> > > Changes for v2: Incorporated York's comments
> > >   - Removed ramdisk_addr, ramdisk_size
> > >   - Updated UART baud-rate.
> > >
> > > include/configs/ls1012afrdm.h | 14 ++
> > >  1 file changed, 14 insertions(+)
> > >
> > > diff --git a/include/configs/ls1012afrdm.h
> > > b/include/configs/ls1012afrdm.h index ad81142..f2c6ca8 100644
> > > --- a/include/configs/ls1012afrdm.h
> > > +++ b/include/configs/ls1012afrdm.h
> > > @@ -39,4 +39,18 @@
> > >  #define CONFIG_SYS_MEMTEST_START 0x8000
> > >  #define CONFIG_SYS_MEMTEST_END   0x9fff
> > >
> > > +#undef CONFIG_EXTRA_ENV_SETTINGS
> > > +#define CONFIG_EXTRA_ENV_SETTINGS\
> > > + "initrd_high=0x\0"  \
> >
> > What is this?
> [Pratiyush>] I used the LS1043 dts as the base for this dts. Since ls10143 had
> this parameter, I copied the same for LS1012.
[Pratiyush>] I used the LS10143a_common file as a base for this dts. Since 
ls10143 had
> this parameter, I copied the same for LS1012.
Please ignore the previous comment .
> >
> > > + "verify=no\0"   \
> > > + "hwconfig=fsl_ddr:bank_intlv=auto\0"\
> > > + "loadaddr=0x8010\0" \
> > > + "kernel_addr=0x10\0"\
> > > + "fdt_high=0x\0" \
> > > + "initrd_high=0x\0"  \
> > > + "kernel_start=0xa0\0"   \
> > > + "kernel_load=0x9600\0"  \
> > > + "kernel_size=0x280\0"   \
> > > + "console=ttyAMA0,115200n8\0"
> >
> > Do you really use ttyAMA0? Isn't it ttyS0?
> [Pratiyush>] I will remove the redundant console command.
> >
> > York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] armv8: ls1012a: Updating CONFIG_EXTRA_ENV_SETTINGS

[Pratiyush Srivastava]

> -Original Message-
> From: york sun
> Sent: Friday, August 05, 2016 9:47 PM
> To: Pratiyush Srivastava <pratiyush.srivast...@nxp.com>; u-
> b...@lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> Subject: Re: [PATCH] armv8: ls1012a: Updating
> CONFIG_EXTRA_ENV_SETTINGS
> 
> On 08/05/2016 02:47 AM, Pratiyush Mohan Srivastava wrote:
> > Remove ramdisk_addr, ramdisk_size and update UART baud-rate.
> >
> > Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> > Signed-off-by: Pratiyush Mohan Srivastava
> <pratiyush.srivast...@nxp.com>
> > ---
> >  include/configs/ls1012a_common.h | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/include/configs/ls1012a_common.h
> b/include/configs/ls1012a_common.h
> > index fba2fac..78f4f9b 100644
> > --- a/include/configs/ls1012a_common.h
> > +++ b/include/configs/ls1012a_common.h
> > @@ -111,14 +111,12 @@
> > "hwconfig=fsl_ddr:bank_intlv=auto\0"\
> > "loadaddr=0x8010\0" \
> > "kernel_addr=0x10\0"\
> > -   "ramdisk_addr=0x80\0"   \
> > -   "ramdisk_size=0x200\0"  \
> > "fdt_high=0x\0" \
> > "initrd_high=0x\0"  \
> > "kernel_start=0xa0\0"   \
> > "kernel_load=0xa000\0"  \
> > "kernel_size=0x280\0"   \
> > -   "console=ttyAMA0,38400n8\0"
> > +   "console=ttyAMA0,115200n8\0"
> 
> Is console variable correct here?
[Pratiyush>] I will remove it ...
> 
> >
> >  #define CONFIG_BOOTARGS"console=ttyS0,115200
> root=/dev/ram0 " \
> 
> console is different here.
> 
> 
> > "earlycon=uart8250,mmio,0x21c0500"
> >
> 
> York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH ] board: ls2080a: Add "mcinitcmd" env for MC & DPL deployment

[Pratiyush Srivastava]

-Original Message-
From: york sun 
Sent: Wednesday, January 20, 2016 9:32 PM
To: Pratiyush Mohan Srivastava <pratiyush.srivast...@freescale.com>; 
u-boot@lists.denx.de
Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Pratiyush Srivastava 
<pratiyush.srivast...@nxp.com>
Subject: Re: [PATCH ] board: ls2080a: Add "mcinitcmd" env for MC & DPL 
deployment

On 01/19/2016 10:59 PM, Pratiyush Mohan Srivastava wrote:
> From: Pratiyush Mohan Srivastava <pratiyush.srivast...@nxp.com>
> 
> Environment variable mcinitcmd is defined to initiate MC and DPL 
> deployment from the location where it is stored(NOR, NAND, SD, SATA, 
> USB)during u-boot booting.If this variable is not defined then macro 
> MC_BOOT_ENV_VAR will be null and MC will not be booted and DPL will not be 
> applied during U-boot booting.
> 
> Signed-off-by: Pratiyush Mohan Srivastava 
> <pratiyush.srivast...@nxp.com>
> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>
> ---
>  arch/arm/cpu/armv8/fsl-layerscape/README.lsch3 | 29 
> ++
>  board/freescale/ls2080aqds/eth.c   |  5 +
>  board/freescale/ls2080ardb/eth_ls2080rdb.c |  5 +
>  3 files changed, 39 insertions(+)
> 



> diff --git a/board/freescale/ls2080aqds/eth.c 
> b/board/freescale/ls2080aqds/eth.c
> index 5b9c2d1..db9de77 100644
> --- a/board/freescale/ls2080aqds/eth.c
> +++ b/board/freescale/ls2080aqds/eth.c
> @@ -20,6 +20,7 @@
>  
>  #include "ls2080aqds_qixis.h"
>  
> +#define MC_BOOT_ENV_VAR "mcinitcmd"
>  
>  #ifdef CONFIG_FSL_MC_ENET
>   /* - In LS2080A there are only 16 SERDES lanes, spread across 2 SERDES 
> banks.
> @@ -714,6 +715,7 @@ void ls2080a_handle_phy_interface_xsgmii(int i)  
> int board_eth_init(bd_t *bis)  {
>   int error;
> + char *mc_boot_env_var;
>  #ifdef CONFIG_FSL_MC_ENET
>   struct ccsr_gur __iomem *gur = (void *)CONFIG_SYS_FSL_GUTS_ADDR;
>   int serdes1_prtcl = (in_le32(>rcwsr[28]) & @@ -781,6 +783,9 @@ 
> int board_eth_init(bd_t *bis)
>   }
>   }
>  
> + mc_boot_env_var = getenv(MC_BOOT_ENV_VAR);
> + if (mc_boot_env_var)
> + run_command_list(mc_boot_env_var, -1, 0);
>   error = cpu_eth_init(bis);

Why not put the code into cpu_eth_init()?

Ans: While deciding on the location of the command "run_command_list" I could 
not find any prior usage in arch/arm folder. Hence I decided that the 
run_command_list command be kept just before calling cpu_eth_init inside board 
folder.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot