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.kau...@intel.com>
> ---
>  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.

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to