[RFC PATCH] board: ti: common: board_detect: Fix EEPROM read quirk for 2-byte

Matwey V. Kornilov matwey.kornilov at gmail.com
Wed Nov 30 16:38:43 CET 2022


ср, 30 нояб. 2022 г. в 18:36, Robert Nelson <robertcnelson at gmail.com>:
>
> > >>> Tested on J721E, J7200, DRA7xx, AM64x
> > >>
> > >> I'll try to test this on the AM335x boards I have as soon as possible.
> > >
> > > I wonder if we should re-factor this code a bit and not have a single
> > > ti_i2c_eeprom_get but instead build for whichever sets of quirks a given
> > > family of boards has with their EEPROMs. I really worry that we're going
> > > to find that this change here breaks yet another different EEPROM than
> > > before.
> > >
> >
> > Yes that does make sense... considering a new behavior of EEPROM keeps
> > showing up. I will try refactoring the logic that way.
>
> Due to part shorages, sadly the BeagleBone AI64 (J721E) has both 1byte
> and 2byte eeproms in user hands today..
>
> While I think most other previous designs have stuck with one type of
> eeprom throughout their production life.  So just one big outlier that
> I personally know of..

As was mentioned during the work on bf6376642fe8 ("board: ti: common:
board_detect: Fix EEPROM read quirk"), BeagleBone Black produced by
different companies has different kinds of EEPROMs.

>
> Regards,
>
> --
> Robert Nelson
> https://rcn-ee.com/



-- 
With best regards,
Matwey V. Kornilov


More information about the U-Boot mailing list