[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20251224125128.dwiahkuwbpmfiugb@e45ae8f13cb0>
Date: Wed, 24 Dec 2025 20:51:28 +0800
From: Jason Sun <jason.sun689@...il.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
sunqian@...arytech.com, jason.sun689@...il.com
Subject: Re: [PATCH v2] dwc2: Delay resume until device connection is stable
On Wed, Dec 24, 2025 at 09:40:03AM +0100, Greg KH wrote:
>On Wed, Dec 24, 2025 at 04:20:13PM +0800, Jason Sun wrote:
>> Tested on:
>> - Kernel: v5.15.140
>
>That's a very old kernel version. How about 6.18?
>
>thanks,
>
>greg k-h
Hi Greg,
Sorry for the formatting issues in my previous email.
Thanks for the feedback.
At the moment I only have access to this board on a 5.15-based BSP,
so I can only run-time test on v5.15.140. I don't currently have a
working mainline (v6.x) boot chain for this hardware.
The patch applies cleanly and builds on the current usb.git tree
(v6.15-based, build-tested only). The change is based on reviewing
the resume path around VBUS enable, which still proceeds after a fixed
usleep_range(3000, 5000) delay and can observe an unstable CONNSTS
window on our setup.
Please let me know if build-tested on usb.git is acceptable for now,
or if you'd prefer the change to be conditional (e.g. via DT or a
module parameter).
Thanks,
Jason Sun
Powered by blists - more mailing lists