Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-03-01 Thread Pali Rohár
On Sunday 28 February 2021 22:54:27 Luka Kovacic wrote: > Hello Pali, > > On Sat, Feb 27, 2021 at 2:09 AM Pali Rohár wrote: > > > > On Monday 15 February 2021 20:59:32 Luka Kovacic wrote: > > > The hw_info command is implemented to enable parsing Marvell hw_info > > > formatted environments. This

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Heinrich Schuchardt
Am 27. Februar 2021 14:05:50 MEZ schrieb Marek Behun : >Luka, Tom, > >I am very against this. > >there already is `mac` command in cmd/mac.c with following help > display and program the system ID and MAC addresses in EEPROM > [read|save|id|num|errata|date|ports|port_number] > mac read >

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Luka Kovacic
Hello Marek, On Sat, Feb 27, 2021 at 2:05 PM Marek Behun wrote: > > Luka, Tom, > > I am very against this. > > there already is `mac` command in cmd/mac.c with following help >display and program the system ID and MAC addresses in EEPROM >[read|save|id|num|errata|date|ports|port_number] >

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-28 Thread Luka Kovacic
Hello Pali, On Sat, Feb 27, 2021 at 2:09 AM Pali Rohár wrote: > > On Monday 15 February 2021 20:59:32 Luka Kovacic wrote: > > The hw_info command is implemented to enable parsing Marvell hw_info > > formatted environments. This format is often used on Marvell Armada A37XX > > based devices to sto

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-27 Thread Marek Behun
Luka, Tom, I am very against this. there already is `mac` command in cmd/mac.c with following help display and program the system ID and MAC addresses in EEPROM [read|save|id|num|errata|date|ports|port_number] mac read - read EEPROM content into memory data structure mac save

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-26 Thread Pali Rohár
On Monday 15 February 2021 20:59:32 Luka Kovacic wrote: > The hw_info command is implemented to enable parsing Marvell hw_info > formatted environments. This format is often used on Marvell Armada A37XX > based devices to store parameters like the board serial number, factory > MAC addresses and so

Re: [PATCH v2 1/3] cmd: mvebu: Implement the Marvell hw_info command

2021-02-15 Thread Tom Rini
On Mon, Feb 15, 2021 at 08:59:32PM +0100, Luka Kovacic wrote: > The hw_info command is implemented to enable parsing Marvell hw_info > formatted environments. This format is often used on Marvell Armada A37XX > based devices to store parameters like the board serial number, factory > MAC addresses