On Wed, Apr 26, 2023 at 03:09:04PM +0530, Nitin Yadav wrote: > Update the board_fit_config_name_match() to choose the right > dtb based on the board name read from EEPROM. Update the > board_fit_config_name_match() to choose the right dtb based > on the board name read from EEPROM. Also restrict multiple > EEPROM reads by verifying if EEPROM is already read. > > Signed-off-by: Nitin Yadav <n-ya...@ti.com> > --- > board/ti/am62x/evm.c | 148 +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 148 insertions(+)
I've thought about this a bit, and I think this is starting us down the
wrong path in terms of keeping the board code clean and easy to follow
and modify for custom platforms. And I have a new idea. This patch has a
few things in it:
- Code which I think could be moved to the "has TI-style EEPROM" such as
serial# setting, which should be in the common file instead.
- Run-time board detection stuff.
And it's point two I think we should do differently. Lets just have
am62x_evm_sk_{a52,r5}_defconfig files that just support that platform,
single DT. We can still have a general if-check and call for the EEPROM
and to set serial# based on that, but it'll keep most of the code fairly
clean and minimal, and going forward just require new dts/defconfigs.
--
Tom
signature.asc
Description: PGP signature
