RE: [EXT] Re: [net-next PatchV3] octeontx2-pf: Add support to read eeprom information

[Hariprasad Kelam]

> On Tue, Feb 27, 2024 at 02:17:22PM +0530, Hariprasad Kelam wrote:
> > Add support to read/decode EEPROM module information using ethtool.
> > Usage: ethtool -m <interface>
> >
> > Signed-off-by: Hariprasad Kelam <hkelam@xxxxxxxxxxx>
> > Signed-off-by: Sunil Goutham <sgoutham@xxxxxxxxxxx>
> > ---
> > V3 * remove redundant checks as stack is already doing it.
> 
> So still only access to the first 256 bytes, using the old internal API.
> 
Firmware and Kernel shares the data over a shared memory region. 

  Firmware   <-- Shared memory -->   Kernel

As per our design, firmware updates the shared memory region by reading eeprom data from the MAC. 
Upon receiving ethtool request to read eeprom data, Kernel maps this shared memory and copies the eeprom data to the user.

Currently firmware supports updating only the first page of eeprom. Due to this we are limited to support the first page.

, using the old internal API.

While copying the data, the current patch does considers the offset/length fields.
         ethtool -m ethx offset x length x

Could please point us what are we missing here?

Thanks,
Hariprasad k


> Disappointing.
> 
> And the Signed-of-by: appear to be in the wrong order.
> 
>     Andrew