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

