Re: [PATCH v6 1/2] i2c: core: Add i2c_update_timeout() helper for dynamic transfer timeouts

[Aniket RANDIVE <aniket.randive@oss.qualcomm.com>]

Hi Wolfram,

Thanks for the suggestion.

I'm okay with the Kconfig-based approach, as it would avoid regressions 
for platforms that prefer the existing static 1 Hz timeout behavior 
while allowing controllers that benefit from dynamic timeout calculation 
to opt in.

I have reviewed the patch series in [1], and my understanding is that it 
does not directly conflict with the dynamic timeout changes. My series 
calculates transfer timeouts based on the configured bus frequency, 
while the referenced work focuses on dynamically changing bus speeds. At 
the moment, I don't see any functional overlap that would require the 
timeout series to be based on top of it.

That said, please let me know if I'm missing any relationship between 
the two series that should be considered before posting the next revision.

Thanks,
Aniket

On 8/2/2026 2:32 AM, Wolfram Sang wrote:
> On Thu, Jul 30, 2026 at 10:23:46PM +0200, Wolfram Sang wrote:
>>
>>> Given the feedback so far, would maintainers prefer that if i drop the core
>>> helper entirely and keep the dynamic timeout calculation local to the
>>> qcom-geni driver or should we continue exploring an opt-in core-based
>>> approach?
>>
>> Well, I suggested a potential way in my last mail (Kconfig option). So,
>> what do you think about it? Does it handle your use case?
> 
> Another bit which may be important for you: there is currently a patch
> set under discussion [1] to allow changing bus speeds dynamically. You
> probably want to base your work on top of it and/or help getting it
> discussed?
> 
> Happy hacking!
> 
> [1] https://lore.kernel.org/all/20260719-i2c-mux-v12-0-a5010d623a57@gmail.com/
> 
>