Re: [RFC V2 PATCH 0/2] Defer HCD init until the external PHY, bound to the HCD is ready

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/27/2013 01:44 AM, Greg KH wrote:
On Wed, Nov 27, 2013 at 01:31:06AM +0400, Valentine wrote:
On 11/07/2013 03:14 PM, Valentine Barshak wrote:
This adds remove_phy flag to the HCD structure. If the flag is
set and if hcd->phy is valid, the phy is shutdown and released
whenever usb_add_hcd fails or usb_hcd_remove is called.
This also adds generic external phy support that allows
the HCD driver to search for a USB phy, bound to the HCD,
when the HC is being added.

If the USB PHY is found, it is initialized and the remove_phy flag
is set. In case PHY is not ready, the usb_add_hcd function returns
the -EPROBE_DEFER error code which defers HCD probing till the PHY
becomes ready.
If no PHY is bound to the HCD, or it has been initialized by
the glue-driver before calling usb_add_hcd(), the HCD is
added as usual.

This approach can be used to initialize the external PHY for
the R-Car PCI USB hosts, that share USB ports with USBHS/USBSS devices.
All we need to do is to make R-Car Gen2 platform code bind the
Gen2 USB phy device to the PCI HC devices, and make the phy-rcar-gen2-usb
driver register USB phy with usb_add_phy_dev() AOT usb_add_phy() callback.

Changes from previous version:
* Used #ifdef CONFIG_USB_PHY instead of introducing new config option.

Valentine Barshak (2):
   usb: hcd: Remove USB phy if needed
   usb: hcd: Initialize USB phy if needed

  drivers/usb/core/hcd.c  | 34 +++++++++++++++++++++++++++++++++-
  include/linux/usb/hcd.h |  1 +
  2 files changed, 34 insertions(+), 1 deletion(-)


Greg, are you going to take these or is there anything else I have to do?

I don't see anything to "take" here, sorry.  You sent some patches with
"Request for Comments", which usually means, "don't apply them, I just
want people to look at them", so I don't apply them...

If you want me to apply them, please resend without that.

Also, I'd like to get Alan's ACK on these, as they touch the core HCD
code.

thanks,

Alan's ack is already there.
Just wanted to make sure you don't need anything else.


greg k-h


I'll resend in a bit.
Thanks,
Val.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux