Messages in this thread | | | Subject | Re: [PATCH v1 6/6] usb: dwc3: core: Notify USB3 PHY as well for DRD modes | From | Manu Gautam <> | Date | Mon, 24 Jul 2017 15:29:29 +0530 |
| |
On 7/21/2017 10:39 PM, Stephen Boyd wrote: > On 07/21/2017 04:02 AM, Manu Gautam wrote: >> Driver currently notifies only USB3 PHY for mode change. >> Extend this to USB3 PHY so that driver based on the mode >> can release system resources - clocks, regulators etc. >> and can even turn off PHY during runtime suspend. >> >> Signed-off-by: Manu Gautam <mgautam@codeaurora.org> > This patch is confusing because you say "notify for usb3 phy", but then > combine a change to remove the NULL pointer check on phy_set_mode() for > the usb2 phy. Please describe everything that's happening in a patch in > the commit text.
Sure, will add a comment for that. It was done to align with other phy_ops usage in the driver. Also, these wrapper functions already have a NULL pointer check.
| |