Hi, On Tue, Apr 28, 2015 at 09:09:40PM +0000, Kaukab, Yousaf wrote: > > -----Original Message----- > > From: Felipe Balbi [mailto:balbi@xxxxxx] > > Sent: Tuesday, April 28, 2015 10:49 PM > > To: Kaukab, Yousaf > > Cc: balbi@xxxxxx; linux-usb@xxxxxxxxxxxxxxx; john.youn@xxxxxxxxxxxx; > > Herrero, Gregory; r.baldyga@xxxxxxxxxxx; > > dinguyen@xxxxxxxxxxxxxxxxxxxxx; zhangfei.gao@xxxxxxxxxx; > > jwerner@xxxxxxxxxxxx; sergei.shtylyov@xxxxxxxxxxxxxxxxxx > > Subject: Re: [PATCH v4 14/22] usb: dwc2: host: register handle to the phy > > > > On Tue, Apr 28, 2015 at 08:47:08PM +0000, Kaukab, Yousaf wrote: > > > > -----Original Message----- > > > > From: Felipe Balbi [mailto:balbi@xxxxxx] > > > > Sent: Monday, April 27, 2015 10:43 PM > > > > To: Kaukab, Yousaf > > > > Cc: linux-usb@xxxxxxxxxxxxxxx; balbi@xxxxxx; john.youn@xxxxxxxxxxxx; > > > > Herrero, Gregory; r.baldyga@xxxxxxxxxxx; > > > > dinguyen@xxxxxxxxxxxxxxxxxxxxx; zhangfei.gao@xxxxxxxxxx; > > > > jwerner@xxxxxxxxxxxx; sergei.shtylyov@xxxxxxxxxxxxxxxxxx > > > > Subject: Re: [PATCH v4 14/22] usb: dwc2: host: register handle to > > > > the phy > > > > > > > > On Tue, Mar 24, 2015 at 10:01:02AM +0100, Mian Yousaf Kaukab wrote: > > > > > If phy driver is present register hcd handle to it. > > > > > > > > > > Signed-off-by: Mian Yousaf Kaukab <yousaf.kaukab@xxxxxxxxx> > > > > > --- > > > > > drivers/usb/dwc2/hcd.c | 6 ++++++ > > > > > 1 file changed, 6 insertions(+) > > > > > > > > > > diff --git a/drivers/usb/dwc2/hcd.c b/drivers/usb/dwc2/hcd.c index > > > > > 1cca5e0..919658e 100644 > > > > > --- a/drivers/usb/dwc2/hcd.c > > > > > +++ b/drivers/usb/dwc2/hcd.c > > > > > @@ -2913,6 +2913,9 @@ int dwc2_hcd_init(struct dwc2_hsotg *hsotg, > > > > > int > > > > irq) > > > > > /* Don't support SG list at this point */ > > > > > hcd->self.sg_tablesize = 0; > > > > > > > > > > + if (!IS_ERR_OR_NULL(hsotg->uphy)) > > > > > > > > when is uphy NULL ? Aparently, only platform.c sets it to NULL in > > > > case of error, I'd say we should just make NULL a valid PHY pointer, > > > > then we can remove a bunch of these checks scattered around. > > > > > > Here we can't as we want to dereference uphy. > > > However, I like the idea and I guess it entails a separate patch to > > > fix where possible? > > > > yeah, I'll handle that, unless you want to. It should be pretty simple. > > I can do it if it can wait. oh, no rush. But I'd like to get it in time for v4.2 :-) I can definitely wait a couple weeks. -- balbi
Attachment:
signature.asc
Description: Digital signature