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

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

2021-02-15 Thread Luka Kovacic
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 some other information. These parameters are usually written to th