Re: [PATCH 2/2] arm64:?==?utf-8?q? dts: rockchip: Make eeprom read-only for Radxa ROCK 5A

[Dragan Simic]

Hello Naoki,

On Monday, November 10, 2025 09:03 CET, FUKAUMI Naoki <naoki at radxa.com> wrote:
> On 11/10/25 16:38, Dragan Simic wrote:
> > On Monday, November 10, 2025 08:05 CET, FUKAUMI Naoki <naoki at radxa.com> wrote:
> >> arm64: dts: rockchip: Make eeprom read-only for Radxa ROCK 3C/5A/5C
> >>
> >> Make the EEPROM[1][2][3] read-only. Board-specific data written to the
> >> EEPROM at the factory includes, but is not limited to, the SKU (~
> >> product name), BOM version (~ board version), and MAC address.
> > 
> > Thanks for the improvements!  I'd just suggest that the opening
> > sentence above is rewritten like this:
> > 
> >    Make the BL24C16 EEPROM chips found on Radxa ROCK 3C, 5A and
> >    5C SBCs [1][2][3] read-only, because they contain factory-
> >    programmed data that isn't supposed to be modified by the end
> >    users.
> > 
> > This would also be perfect timing for Radxa to officially release
> > the layout of the board-specific data that goes into these EPROM
> > chips, so the data can actually be used.
> 
>   https://github.com/radxa/u-boot/blob/next-dev-v2024.10/drivers/misc/radxa-i2c-eeprom.c
> 
> The actual data in the EEPROMs of my boards has some content 
> inconsistencies with the code descriptions, but the format is 
> consistent. The SKU, BOM, and MAC mentioned previously should be available.

I see, thanks.  I'm wondering could U-Boot's drivers/misc/nvmem.c be
used instead.  It seems like the driver linked above includes quite
a lot of code that's already present elsewhere in U-Boot.