On Wed, Sep 03, 2025 at 05:12:44PM +0200, Konrad Dybcio wrote: > On 9/3/25 5:08 PM, Wasim Nazir wrote: > > On Wed, Sep 03, 2025 at 07:41:30PM +0530, Wasim Nazir wrote: > >> On Wed, Sep 03, 2025 at 02:29:11PM +0200, Konrad Dybcio wrote: > >>> On 9/3/25 1:47 PM, Wasim Nazir wrote: > >>>> From: Monish Chunara <quic_mchunara@xxxxxxxxxxx> > >>>> > >>>> Define the nvmem layout on the EEPROM connected via I2C to enable > >>>> structured storage and access to board-specific configuration data, > >>>> such as MAC addresses for Ethernet. > >>> > >>> The commit subject should emphasize the introduction of the EEPROM > >>> itself, with the layout being a minor detail, yet the description of > >>> its use which you provided is important and welcome > >>> > >> > >> Thanks, Konrad, for pointing this out. I’ll update it in the next > >> series. > > > > Moreover, I notice that compatible definition is missing for this > > EEPROM. I will add it in next series. > > I think the pattern match in at24.yaml should catch it > The EEPROM used on this platform is from Giantec, which requires a dedicated compatible string. While the generic "atmel,24c256" compatible is already supported in at24.yaml. -- Regards, Wasim
