Hi Wolfram, On Mon, Feb 4, 2019 at 11:00 PM Wolfram Sang <[email protected]> wrote: > We could finally measure the clock on an Ebisu board. The new formula > gives way better results, i.e. 100kHz instead of 106kHz and 400kHz > instead of 387kHz. Switch to these formulas for all Gen3 SoCs. > > Signed-off-by: Wolfram Sang <[email protected]>
Thanks for your patch! Reviewed-by: Geert Uytterhoeven <[email protected]> > --- a/drivers/i2c/busses/i2c-sh_mobile.c > +++ b/drivers/i2c/busses/i2c-sh_mobile.c > @@ -806,11 +806,11 @@ static const struct of_device_id sh_mobile_i2c_dt_ids[] > = { > { .compatible = "renesas,iic-r8a7792", .data = &fast_clock_dt_config > }, > { .compatible = "renesas,iic-r8a7793", .data = &fast_clock_dt_config > }, > { .compatible = "renesas,iic-r8a7794", .data = &fast_clock_dt_config > }, > - { .compatible = "renesas,iic-r8a7795", .data = &fast_clock_dt_config > }, > - { .compatible = "renesas,iic-r8a77990", .data = &fast_clock_dt_config > }, > + { .compatible = "renesas,iic-r8a7795", .data = > &v2_freq_calc_dt_config }, > + { .compatible = "renesas,iic-r8a77990", .data = > &v2_freq_calc_dt_config }, > { .compatible = "renesas,iic-sh73a0", .data = &fast_clock_dt_config }, > { .compatible = "renesas,rcar-gen2-iic", .data = > &fast_clock_dt_config }, > - { .compatible = "renesas,rcar-gen3-iic", .data = > &fast_clock_dt_config }, > + { .compatible = "renesas,rcar-gen3-iic", .data = > &v2_freq_calc_dt_config }, > { .compatible = "renesas,rmobile-iic", .data = &default_dt_config }, > {}, I guess you want to make a similar change for R-Car Gen2 (both SoC-specific and family-specific)? Currently only R-Car H2 uses the new formula, which should apply to all R-Car Gen2/Gen3 SoCs. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
