[PATCH 2/2] arm64: dts: imx8mm: Add i.MX8M Mini Toradex Verdin based Menlo board

[Marek Vasut]

On 4/8/22 10:49, Francesco Dolcini wrote:
> On Fri, Apr 08, 2022 at 09:56:09AM +0200, Marcel Ziswiler wrote:
>> On Fri, 2022-04-08 at 08:46 +0200, Francesco Dolcini wrote:
>>> On Thu, Apr 07, 2022 at 10:24:56PM +0200, Marek Vasut wrote:
>>>> +
>>>> +&i2c1 {
>>>> +       /* IMX8MM ERRATA e7805 -- I2C is limited to 384 kHz due to SoC bug */
>>>> +       clock-frequency = <100000>;
>>> should this and the related changes in the other i2c nodes done in the
>>> verdin.dtsi? Marcel? (errata is here:
>>> https://www.nxp.com/docs/en/errata/IMX8MDQLQ_1N14W.pdf)
>>
>> Hehe, good catch. Yeah, we probably should move this one into imx8mm-verdin.dtsi instead. On the other hand, it
>> won't hurt doing it twice resp. this one will always override it like that.
>>
>> Anyway, looks like NXP has not fixed this and is re-using the exact same I2C IP with the same errata (;-p).
> 
> It looks like even i.MX7 is affected, and NXP has a quirk in the
> i2c-imx driver [0]
> 
> [0] https://source.codeaurora.org/external/imx/linux-imx/commit/drivers/i2c/busses/i2c-imx.c?h=lf-5.15.y&id=493b3892ee156af529796641889ca19b971735d2

That's downstream btw, the quirk is not in mainline, but maybe we should 
add a quirk to the i2c-imx.c .

So all of MX8M{M,N,P,Q) , MX7{S,D} , 
MX6{UL{,L,Z},S{,LL,X},S,D,DL,Q,DP,QP} are affected.

i.MX53 is not affected, but maybe the errata wasn't found yet, the 
errata document rev.6 predates 2016 from when the ERR007805 / e7805 
originates in MX6 errata sheets.

MX7ULP, MX8Q, MX8X are not affected? What about MX8ULP ?