On Fri, May 24, 2019 at 6:27 PM Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> wrote: > The Atmel/Microchip EC508A is a I2C device that could be wired into > any platform, and is being used on the Linaro/96boards Secure96 > mezzanine adapter. This means it could be found on any platform, even > on ones that use ACPI enumeration (via PRP0001 devices). So update the > code to enable this use case. > > This involves tweaking the bus rate discovery code to take ACPI probing > into account, which records the maximum bus rate as a property of the > slave device. For the atmel-ecc code, this means that the effective bus > rate should never exceed the maximum rate, unless we are dealing with > buggy firmware. Nonetheless, let's just use the existing plumbing to > discover the bus rate and keep the existing logic intact. > > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Looks good to me. Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij