Re: [PATCH] i2c: qcom-geni: fix I2C frequency table to achieve accurate bus rates
From: Andi Shyti
Date: Fri May 16 2025 - 04:46:32 EST
Hi Mukesh,
On Fri, May 16, 2025 at 12:23:46AM +0530, Mukesh Kumar Savaliya wrote:
> > > /* source_clock = 32 MHz */
> > > static const struct geni_i2c_clk_fld geni_i2c_clk_map_32mhz[] = {
> > > - { I2C_MAX_STANDARD_MODE_FREQ, 8, 14, 18, 40 },
> > > - { I2C_MAX_FAST_MODE_FREQ, 4, 3, 11, 20 },
> > > - { I2C_MAX_FAST_MODE_PLUS_FREQ, 2, 3, 6, 15 },
> > > + { I2C_MAX_STANDARD_MODE_FREQ, 8, 14, 18, 38 },
> > > + { I2C_MAX_FAST_MODE_FREQ, 4, 3, 9, 19 },
> > > + { I2C_MAX_FAST_MODE_PLUS_FREQ, 2, 3, 5, 15 },
> >
> > argh!
> >
> > Can someone from Qualcomm look at this fix? Mukesh, Viken?
> Yes, I am reviewing internally and verifying the same with latest Guidance
> and updates. Let me update on this once reviewed.
thanks a lot! It's very much appreciated!
Andi