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@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@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@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?

On Wed, Feb 03, 2021 at 02:28:01PM +0000, Aleksandar Gerasimovski wrote:

-----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@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@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@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@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.

This got me to grep around and yes, there's some unused code here. None of the CONFIG_xxx_TAG stuff is used on aarch64 and these SoCs never ever supported a kernel that used ATAGs and not device tree. So update your platform to drop all of this dead code would be best.

On Thu, Feb 04, 2021 at 09:49:58AM +0000, Aleksandar Gerasimovski wrote:

-----Original Message----- From: Tom Rini trini@konsulko.com Sent: Mittwoch, 3. Februar 2021 15:36 To: Aleksandar Gerasimovski aleksandar.gerasimovski@hitachi-powergrids.com Cc: Priyanka Jain priyanka.jain@nxp.com; Priyanka Jain (OSS) priyanka.jain@oss.nxp.com; u-boot@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 02:28:01PM +0000, Aleksandar Gerasimovski wrote:

...

-----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@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@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@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@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.

This got me to grep around and yes, there's some unused code here. None of the CONFIG_xxx_TAG stuff is used on aarch64 and these SoCs never ever supported a kernel that used ATAGs and not device tree. So update your platform to drop all of this dead code would be best.

-- Tom

Hi Priyanka,

What do you think, can be remove CMDLINE_TAG from our design without consequences? Let's say our design is similar as ls1021atwr (arm32 bit)? Tom mentioned that that's dead code for aarch64, but I grep around as well and I see they are still in use for arm32? I can remove this but don't want to enter in a trap now or in future and layerscape kernel is expecting this.

It's been dead code for arm32 for over a decade at this point as well, please keep in mind.