> -----Original Message-----
> From: York Sun
> Sent: Wednesday, June 14, 2017 2:23 AM
> To: Santan Kumar <santan.ku...@nxp.com>; u-boot@lists.denx.de
> Cc: Yogesh Narayan Gaur <yogeshnarayan.g...@nxp.com>; Priyanka Jain
> <priyanka.j...@nxp.com>
> Subject: Re: [PATCH 1/1] board/ls2081ardb: Update qspi flash type and
> checkboard
>
> On 06/08/2017 11:16 PM, Santan Kumar wrote:
> > Patch makes below changes:
> > -As per updated board design, different QSPI flash is connected on boards,
> > hence change QSPI flash type from n25q512a to s25fs512ss
> > -Remove deprecated config : CONFIG_DISPLAY_BOARDINFO_LATE
> > -Increase env size from 64kb to 256kb for qspi boot
> > -Update checkboard() to detect boot source and display message
>
> Please separate common change from board specific change.
>
> >
> > Signed-off-by: Santan Kumar <santan.ku...@nxp.com>
> > Signed-off-by: Yogesh Gaur <yogeshnarayan.g...@nxp.com>
> > Signed-off-by: Priyanka Jain <priyanka.j...@nxp.com>
> > ---
> > arch/arm/dts/fsl-ls2081a-rdb.dts | 4 ++--
> > board/freescale/ls2080ardb/ls2080ardb.c | 15 +++------------
> > include/configs/ls2080a_common.h | 2 +-
> > include/configs/ls2080ardb.h | 7 +------
> > 4 files changed, 7 insertions(+), 21 deletions(-)
> >
> > diff --git a/arch/arm/dts/fsl-ls2081a-rdb.dts b/arch/arm/dts/fsl-ls2081a-
> rdb.dts
> > index 6489362..aa4aa68 100644
> > --- a/arch/arm/dts/fsl-ls2081a-rdb.dts
> > +++ b/arch/arm/dts/fsl-ls2081a-rdb.dts
> > @@ -41,7 +41,7 @@
> > bus-num = <0>;
> > status = "okay";
> >
> > - qflash0: n25q512a@0 {
> > + qflash0: s25fs512s@0 {
>
> Is there any version of LS2081ARDB with n25q512a?
No, All ls2081ardb boards will have s25fs512s device
>
> > #address-cells = <1>;
> > #size-cells = <1>;
> > compatible = "spi-flash";
> > @@ -49,7 +49,7 @@
> > reg = <0>;
> > };
> >
> > - qflash1: n25q512a@1 {
> > + qflash1: s25fs512s@1 {
> > #address-cells = <1>;
> > #size-cells = <1>;
> > compatible = "spi-flash";
> > diff --git a/board/freescale/ls2080ardb/ls2080ardb.c
> b/board/freescale/ls2080ardb/ls2080ardb.c
> > index 210142c..9d21d1e 100644
> > --- a/board/freescale/ls2080ardb/ls2080ardb.c
> > +++ b/board/freescale/ls2080ardb/ls2080ardb.c
> > @@ -71,36 +71,27 @@ int checkboard(void)
> > #ifdef CONFIG_TARGET_LS2081ARDB
> > #ifdef CONFIG_FSL_QIXIS
> > sw = QIXIS_READ(arch);
> > - printf("Board Arch: V%d, ", sw >> 4);
> > printf("Board version: %c, ", (sw & 0xf) + 'A');
> >
>
> What's the reason to drop "Board Arch"?
Based on feedback from board team, this is not required for RDB boards
Arch contains Qixis architecture version to use common Qixis image across
multipl platforms.
This field is relevant for QDS boards. For RDB boards it is always constant as
'1'
>
> > sw = QIXIS_READ(brdcfg[0]);
> > - sw = (sw & QIXIS_QMAP_MASK) >> QIXIS_QMAP_SHIFT;
> > + sw = sw >> QIXIS_QMAP_SHIFT;
> > switch (sw) {
> > case 0:
> > + case 4:
> > puts("boot from QSPI DEV#0\n");
> > - puts("QSPI_CSA_1 mapped to QSPI DEV#1\n");
> > break;
> > case 1:
> > puts("boot from QSPI DEV#1\n");
> > - puts("QSPI_CSA_1 mapped to QSPI DEV#0\n");
> > break;
> > case 2:
> > - puts("boot from QSPI EMU\n");
> > - puts("QSPI_CSA_1 mapped to QSPI DEV#0\n");
> > - break;
> > case 3:
> > puts("boot from QSPI EMU\n");
> > - puts("QSPI_CSA_1 mapped to QSPI DEV#1\n");
> > - break;
> > - case 4:
> > - puts("boot from QSPI DEV#0\n");
> > - puts("QSPI_CSA_1 mapped to QSPI EMU\n");
> > break;
> > default:
> > printf("invalid setting of SW%u\n", sw);
> > break;
> > }
> > + printf("FPGA: v%d.%d\n", QIXIS_READ(scver), QIXIS_READ(tagdata));
>
> What are you changing in this function?
>
> York
Sw value is updated to sw = sw >> QIXIS_QMP_SHIFT
Instead of sw = (sw & QIXIS_QMAP_MASK) >> QIXIS_QMAP_SHIFT
Also With earlier version of prints, u-boot boot messages were confusing.
So, tried to make the bootlogs similar to Nor boot bootlogs to make them more
user friendly.
Priyanka
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
