Re: [PATCH v1] i2c: qcom-geni: Add dynamic transfer timeout based on transfer length and frequency

[Aniket RANDIVE]

On 7/6/2026 11:28 AM, Krzysztof Kozlowski wrote:
> On 05/07/2026 15:46, Aniket Randive wrote:
> > The driver uses a static XFER_TIMEOUT of HZ (1 second) for all transfers
> > regardless of message length or bus frequency, causing unnecessary
> > delays on error paths.
> >
> > Compute the timeout dynamically from message length and bus frequency
> > with a 10x safety margin over the theoretical wire time and a 300ms
> > floor. For GPI multi-descriptor transfers, use the maximum message
> > length across all queued messages as the per-completion timeout.
> >
> > Signed-off-by: Aniket Randive <aniket.randive@xxxxxxxxxxxxxxxx>
> > ---
> >   arch/arm64/configs/defconfig       | 54 +++---------------------------
> >   drivers/i2c/busses/i2c-qcom-geni.c | 37 +++++++++++++++-----
> >   2 files changed, 33 insertions(+), 58 deletions(-)
> >
> > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > index faf146441f97..841108ae2d01 100644
> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -32,7 +32,6 @@ CONFIG_KALLSYMS_ALL=y
> >   CONFIG_PROFILING=y
> >   CONFIG_KEXEC=y
> >   CONFIG_KEXEC_FILE=y
> > -CONFIG_CRASH_DUMP=y
>
>
> Irrelevant patch change. And not really correct - you base this on some
> old tree.
>
> Best regards,
> Krzysztof

Please check latest [PATCH v2] i2c: qcom-geni: Add dynamic transfer 
timeout based on transfer length and frequency.
- Aniket