On Tue, Apr 28, 2015 at 08:47:08PM +0000, Kaukab, Yousaf wrote: > > -----Original Message----- > > From: Felipe Balbi [mailto:[email protected]] > > Sent: Monday, April 27, 2015 10:43 PM > > To: Kaukab, Yousaf > > Cc: [email protected]; [email protected]; [email protected]; > > Herrero, Gregory; [email protected]; > > [email protected]; [email protected]; > > [email protected]; [email protected] > > 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 <[email protected]> > > > --- > > > 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. -- balbi
signature.asc
Description: Digital signature
