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

From: Krzysztof Kozlowski

Date: Mon Jul 06 2026 - 01:58:40 EST


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