> -----Original Message----- > From: Felipe Balbi [mailto:ba...@ti.com] > Sent: Thursday, September 03, 2015 9:25 PM > To: Badola Nikhil-B46172 <nikhil.bad...@freescale.com> > Cc: linux-ker...@vger.kernel.org; linux-...@vger.kernel.org; linux- > o...@vger.kernel.org; ba...@ti.com; gre...@linuxfoundation.org > Subject: Re: [PATCH 2/3][v3] drivers: usb: dwc3: Add frame length > adjustment quirk > > On Thu, Sep 03, 2015 at 11:34:04AM +0530, Nikhil Badola wrote: > > Add adjust_frame_length_quirk for writing to fladj register which > > adjusts (micro)frame length to value provided by > > "snps,quirk-frame-length-adjustment" property thus avoiding USB 2.0 > > devices to time-out over a longer run > > > > Signed-off-by: Nikhil Badola <nikhil.bad...@freescale.com> > > --- > > Changes for v3: > > - removed unnecessary if(fladj) condition > > - removed stray newline > > > > drivers/usb/dwc3/core.c | 34 > ++++++++++++++++++++++++++++++++++ > > drivers/usb/dwc3/core.h | 5 +++++ > > drivers/usb/dwc3/platform_data.h | 2 ++ > > 3 files changed, 41 insertions(+) > > > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index > > 064123e..75a17bf 100644 > > --- a/drivers/usb/dwc3/core.c > > +++ b/drivers/usb/dwc3/core.c > > @@ -143,6 +143,32 @@ static int dwc3_soft_reset(struct dwc3 *dwc) > > return 0; > > } > > > > +/* > > + * dwc3_frame_length_adjustment - Adjusts frame length if required > > + * @dwc3: Pointer to our controller context structure > > + * @fladj: Value of GFLADJ_30MHZ to adjust frame length */ static > > +void dwc3_frame_length_adjustment(struct dwc3 *dwc, u32 fladj) { > > + if (dwc->revision < DWC3_REVISION_250A) > > + return; > > + > > + if (fladj == 0) > > + return; > > + > > + u32 reg; > > + u32 dft; > > are you seriously telling me that GCC didn't warn you about mixing > declaration and code ?
That's really careless of me. Somehow this warning skipped my attention. Apologies for this silly thing. I will resend the patch. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
