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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130430062624.GB4776@arwen.pp.htv.fi>
Date:	Tue, 30 Apr 2013 09:26:24 +0300
From:	Felipe Balbi <balbi@...com>
To:	Linus Torvalds <torvalds@...ux-foundation.org>
CC:	Alan Stern <stern@...land.harvard.edu>,
	Greg KH <gregkh@...uxfoundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	USB list <linux-usb@...r.kernel.org>
Subject: Re: [GIT PATCH] USB patches for 3.10-rc1

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@...land.harvard.edu> 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

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)