Subject:[PATCH v2 4/4] board/km: add support for seli8 design based on nxp ls102x

[Aleksandar Gerasimovski <aleksandar.gerasimovski@hitachi-powergrids.com>]

-----Original Message-----
From: Tom Rini <trini@konsulko.com> 
Sent: Mittwoch, 3. Februar 2021 15:08
To: Priyanka Jain <priyanka.jain@nxp.com>
Cc: Aleksandar Gerasimovski <aleksandar.gerasimovski@hitachi-powergrids.com>; Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>; u-boot at lists.denx.de; Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>; Holger Brunck <holger.brunck@hitachi-powergrids.com>; Rainer Boschung <rainer.boschung@hitachi-powergrids.com>; Matteo Ghidoni <matteo.ghidoni@hitachi-powergrids.com>
Subject: Re: Subject:[PATCH v2 4/4] board/km: add support for seli8 design based on nxp ls102x

On Wed, Feb 03, 2021 at 08:15:49AM +0000, Priyanka Jain wrote:
> 
> 
> >-----Original Message-----
> >From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Aleksandar 
> >Gerasimovski
> >Sent: Tuesday, February 2, 2021 2:37 PM
> >To: Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>; 
> >u-boot at lists.denx.de
> >Cc: Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>; 
> >Holger Brunck <holger.brunck@hitachi-powergrids.com>; Rainer Boschung 
> ><rainer.boschung@hitachi-powergrids.com>; Matteo Ghidoni 
> ><matteo.ghidoni@hitachi-powergrids.com>
> >Subject: RE: Subject:[PATCH v2 4/4] board/km: add support for seli8 
> >design based on nxp ls102x
> >
> >
> >-----Original Message-----
> >From: Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>
> >Sent: Dienstag, 2. Februar 2021 07:33
> >To: Aleksandar Gerasimovski <aleksandar.gerasimovski@hitachi- 
> >powergrids.com>; Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>; u- 
> >boot at lists.denx.de
> >Cc: Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>; 
> >Holger Brunck <holger.brunck@hitachi-powergrids.com>; Rainer Boschung 
> ><rainer.boschung@hitachi-powergrids.com>; Matteo Ghidoni 
> ><matteo.ghidoni@hitachi-powergrids.com>
> >Subject: RE: Subject: [PATCH v2 4/4] board/km: add support for seli8 
> >design based on nxp ls102x
> >
> >CAUTION: This email originated from outside of the organization. Do 
> >not click links or open attachments unless you recognize the sender 
> >and know the content is safe.
> >
> >
> >>-----Original Message-----
> >>From: Aleksandar Gerasimovski <aleksandar.gerasimovski@hitachi- 
> >>powergrids.com>
> >>Sent: Tuesday, January 19, 2021 4:11 PM
> >>To: Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>; 
> >>u-boot at lists.denx.de
> >>Cc: Valentin Longchamp <valentin.longchamp@hitachi-powergrids.com>;
> >>Holger Brunck <holger.brunck@hitachi-powergrids.com>; Rainer 
> >>Boschung <rainer.boschung@hitachi-powergrids.com>; Matteo Ghidoni 
> >><matteo.ghidoni@hitachi-powergrids.com>
> >>Subject: Subject: [PATCH v2 4/4] board/km: add support for seli8 
> >>design based on nxp ls102x
> >>
> >>The SELI8 design is a new tdm service unit card for 
> >>Hitachi-Powergrids XMC and FOX product lines.
> >>
> >>It is based on NXP LS1021 SoC and it provides following interfaces:
> >> - IFC interface for NOR, NAND and external FPGA's
> >> - 1 x RGMII ETH for debug purposes
> >> - 2 x SGMII ETH for management communication via back-plane
> >> - 1 x uQE HDLC for management communication via back-plane
> >> - 1 x I2C for peripheral devices
> >> - 1 x SPI for peripheral devices
> >> - 1 x UART for debug logging
> >>
> >>It is foreseen that the design will be later re-used for another XMC 
> >>and FOX service cards with similar SoC requirements.
> >>
> >>Signed-off-by: Rainer Boschung 
> >><rainer.boschung@hitachi-powergrids.com>
> >>Signed-off-by: Matteo Ghidoni 
> >><matteo.ghidoni@hitachi-powergrids.com>
> >>Signed-off-by: Aleksandar Gerasimovski
> >><aleksandar.gerasimovski@hitachi- powergrids.com>
> >>---
> ><snip>
> >Kindly fix below checkpatch errors /warnings
> >WARNING: 'AYSNC' may be misspelled - perhaps 'ASYNC'?
> >#799: FILE: include/configs/km/pg-wcom-ls102xa.h:64:
> >+                                       CSOR_NOR_NOR_MODE_AYSNC_NOR | 
> >+ \
> >
> >ERROR: All commands are managed by Kconfig
> >#953: FILE: include/configs/km/pg-wcom-ls102xa.h:218:
> >+#define CONFIG_CMDLINE_TAG
> >
> >total: 1 errors, 2 warnings, 0 checks, 964 lines checked
> >
> >Regards
> >Priyanka
> >
> >Hi Priyanka,
> >
> >CSOR_NOR_NOR_MODE_AYSNC_NOR is named like in mainline (nxp) see 
> >include/fsl_ifc.h If you asking me to change that than it belongs to 
> >a separate patch, not this topic.
> >
> OK this warning can be ignored
> 
> >CONFIG_CMDLINE_TAG is a whitelisted on mainline, there is no Kconfig for that.
> >
> 
> Tom,
> 
> How to handle this checkpatch error?
> CONFIG_CMDLINE_TAG is a whitelisted in mainline, but checkpatch is throwing error.
> Can I go ahead and accept this patch ?

There's two parts to it.  One part, checkpatch regex is wrong and shouldn't trigger on that, since it's not an actual command.  Second part, uh, does passing ATAGs make sense on that platform?  Is there some unfortunate and needing to be fixed logic in U-Boot that requires CMDLINE_TAG (and the few others) to be enabled for device tree to be passed or something?

--
Tom

Hi Tom,

To answer to the second part of CMDLINE_TAG doubt, in our design (proposed patch) this is inherited from the NXP reference design that is Layerscape  ls1021atwr and if I grep the u-boot I see this set for all layerscape reference boards, I'm not sure if this must to be set but we are trying to stay close to mainline for our designs.

Priyanka, CMDLINE_TAG is already set for all layerscape reference designs so I see no strong reason to be different in our platform.

Regards,
Aleksandar