Re: [GIT PATCH] USB patches for 3.10-rc1

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

 



Hi,

On Mon, Apr 29, 2013 at 02:31:59PM -0700, Linus Torvalds wrote:
> On Mon, Apr 29, 2013 at 2:14 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote:
> >
> > What other things seemed odd about Greg's pull request?
> 
> The only other thing I noticed was the new CONFIG_USB_PHY quesiton,
> which is not something that I think is sensible to ask from a user,
> and the help text doesn't really help anything either.
> 
> I think the question may make sense, but the wording does not.
> 
> *EVERYBODY* wants a USB PHY. You can't have USB without a physical
> layer unless it's a purely virtual device. There's one in a EHCI
> controller too. It's like a network chip - without a PHY there's no
> point. Why ask about whether you want to support a phy or not? The
> question makes no sense.

right, the thing is that the standard host-only drivers (EHCI, OHCI,
UHCI and XHCI) still don't know about the PHY layer. They give no
SW visibility on the PHY at all so I felt that adding unused code to the
x86 binaries would worse than just asking the user if they want the PHY
layer enabled or not.

In any case, *HCI drivers need to learn about the PHY layer, at least on
their embedded forms (OMAP, Tegra, Marvel, SuperH, etc) since those will
need SW control of the PHY for e.g. PM, remote wakeup, and few other
cases.

> So I don't think the question should be "do you want a USB PHY". The
> question should be "Do you want a driver for some of the specialized
> external USB controllers" or something like that. Because as it is
> now, anybody who actually reads the question is likely to answer "y",
> I think, even if he just wants one of the *normal* USB chips that
> don't split out the PHY.

makes sense.

> Hmm? Or does PHY have some magic other meaning in USB circles? In

no, it means the same thing.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


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

  Powered by Linux