[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110323093847.55e9dbba@lxorguk.ukuu.org.uk>
Date: Wed, 23 Mar 2011 09:38:47 +0000
From: Alan Cox <alan@...rguk.ukuu.org.uk>
To: Nicolas Pitre <nicolas.pitre@...aro.org>
Cc: Benjamin Herrenschmidt <benh@...nel.crashing.org>,
andy.green@...aro.org,
Jaswinder Singh <jaswinder.singh@...aro.org>,
Linux USB list <linux-usb@...r.kernel.org>,
lkml <linux-kernel@...r.kernel.org>, arnd@...db.de,
broonie@...nsource.wolfsonmicro.com, roger.quadros@...ia.com,
greg@...ah.com, grant.likely@...retlab.ca
Subject: Re: RFC: Platform data for onboard USB assets
> > Does it make sense however to add platform data for generic probed
> > devices ? I don't think so.
>
> Generally speaking, this wouldn't make sense. but this is a case where
> a generically probed device happens to be used in a very specific
> hardware design with its own quirks. in that very particular case then
> it certainly makes some sense.
If it's a very specific hardware design it can do its own very specific
internal private kernel patch, or little config app in user space. There
isn't a valid reason to inflict that complexity on the other 99.999999%
of users.
> actual device registration. So, while this is true that there is a
> possibility for misuse, in practice this is rather unlikely to go very
> far without being noticed, and therefore this argument alone is rather
> weak in support of a significant world order change.
You mean it'll get missed and years later will get compiled into some
other device and a joker with a custom USB widget will add the ids to his
widget and declare it an exploit ?
The other thing being overlooked is that for board specific horrors you
can still put them in udev quite easily if your udev happens to know how
to parse some kind of firmware provided table. That might even be bits of
your device tree.
The kernel simply shouldn't get involved in machine specific namespace
fights.
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists