linux-kernel - Re: [PATCH] i2c: qcom-geni: fix I2C frequency table to achieve accurate bus rates

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <sdhflgjsnvn7l2pix7e5k5yijycy53tmpg4w7722icp6na5rcx@kec7ra7tc4xc>
Date: Fri, 16 May 2025 10:46:15 +0200
From: Andi Shyti <andi.shyti@...nel.org>
To: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
Cc: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@....qualcomm.com>, 
	Viken Dadhaniya <quic_vdadhani@...cinc.com>, Manikanta Mylavarapu <quic_mmanikan@...cinc.com>, 
	Bjorn Andersson <andersson@...nel.org>, linux-i2c@...r.kernel.org, linux-arm-msm@...r.kernel.org, 
	linux-kernel@...r.kernel.org, stable@...nel.org
Subject: Re: [PATCH] i2c: qcom-geni: fix I2C frequency table to achieve
 accurate bus rates

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