On Mon, Oct 19, 2020 at 11:23 AM Jorge Ramirez-Ortiz <jo...@foundries.io> wrote: > > In order to be able to run the I2C bus at 400Khz, the chip errata[1] > recommends that the peripheral clock runs out of the 24MHz oscillator.
I would suggest adding the optee related motivation that you explained earlier here in the commit log too. > [1] Rev 2, 10/2019, ERR007805 Not clear what document this refers to. > /* Set perclk to source from OSC 24MHz */ > - if (is_mx6sl()) > + if (is_mx6sl() || ERRATA_ERR007805) This gives the impression that imx6sl is not affected by the erratum, but in fact it is: https://www.nxp.com/docs/en/errata/IMX6SLCE.pdf You could define it like this: #define has_err007805() (is_mx6sl() || is_mx6dl() || is_mx6solo() || is_mx6ull())