From: Randy Li
> Sent: 03 September 2016 22:55
...
> +                     if (of_device_is_compatible(np, "rockchip,rk3288-usb")
> +                                     && (NULL != hsotg->phy->ops->reset))
> +                             hsotg->phy->ops->reset(hsotg->phy);
> +

Is this the only place ops->reset() is called?
Probably much better to check it first.

                        if (hsotg->phy->ops->reset
                                        && of_device_is_compatible(np, 
"rockchip,rk3288-usb")
                                hsotg->phy->ops->reset(hsotg->phy);

        David

Reply via email to