On Wed, Jun 12, 2024 at 09:52:04PM +0800, joswang wrote: > On Wed, Jun 12, 2024 at 3:58 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > > On Tue, Jun 11, 2024 at 10:29:53PM +0800, joswang wrote: > > > From: Jos Wang <joswang@xxxxxxxxxx> > > > > > > This is a workaround for STAR 4846132, which only affects > > > DWC_usb31 version2.00a operating in host mode. > > > > > > There is a problem in DWC_usb31 version 2.00a operating > > > in host mode that would cause a CSR read timeout When CSR > > > read coincides with RAM Clock Gating Entry. By disable > > > Clock Gating, sacrificing power consumption for normal > > > operation. > > > > > > Signed-off-by: Jos Wang <joswang@xxxxxxxxxx> > > > --- > > > v1 -> v2: > > > - add "dt-bindings: usb: dwc3: Add snps,p2p3tranok quirk" patch > > > v2 -> v3: > > > - code refactor > > > - modify comment, add STAR number, workaround applied in host mode > > > - modify commit message, add STAR number, workaround applied in host mode > > > - modify Author Jos Wang > > > --- > > > drivers/usb/dwc3/core.c | 20 +++++++++++++++++++- > > > 1 file changed, 19 insertions(+), 1 deletion(-) > > > > Should this have a cc: stable line? > > > > thanks, > > > > greg k-h > > I have a question here, please help me confirm > 1. Cc: stable@xxxxxxxxxxxxxxx or Cc: stable@xxxxxxxxxx ? > 2. Do I need to modify the commit message, for example: > Cc: stable@xxxxxxxxxx > Signed-off-by: Jos Wang <joswang@xxxxxxxxxx> > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Jos Wang <joswang@xxxxxxxxxx> Please read: https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html for how to do this properly.