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
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to