devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andy Green <andy@warmcat.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Greg KH <greg@kroah.com>,
	Grant Likely <grant.likely@secretlab.ca>,
	devicetree-discuss@lists.ozlabs.org,
	Mark Brown <broonie@opensource.wolfsonmicro.com>,
	Nicolas Pitre <nicolas.pitre@linaro.org>,
	Linux USB list <linux-usb@vger.kernel.org>,
	lkml <linux-kernel@vger.kernel.org>
Subject: Re: RFC: Platform data for onboard USB assets
Date: Thu, 17 Mar 2011 23:22:03 +0000	[thread overview]
Message-ID: <4D82979B.2050003@linaro.org> (raw)
In-Reply-To: <201103172333.01474.arnd@arndb.de>

On 03/17/2011 10:33 PM, Somebody in the thread at some point said:
> On Thursday 17 March 2011 22:47:36 Greg KH wrote:
>>>> Patches to fix this, for this specific PandaBoard controller are gladly
>>>> accepted.  What's odd is this is explicitly a Linux development board,
>>>> so you would think that this could have been caught, and fixed, in the
>>>> hardware a long time ago, right?
>>>
>>> The way everyone resolves this stuff is by patching their kernel
>>> locally.
>>
>> Well, that means that the device tree work is going to be useful here,
>> right?  :)
>
> I like the idea. Let's make this the first use case where a lot of

You changed your first opinion about tagging "dynamically probed 
devices" with what is effectively platform_data, cool.

> people will want to have the device tree on ARM. The patch to the
> driver to check for a mac-address property is trivial, and we
> can probably come up with a decent way of parsing the device
> tree for USB devices, after all there is an existing spec for
> it (http://playground.sun.com/1275/bindings/usb/usb-1_0.ps).

It doesn't do it already then.

That spec you pointed to from 1998 is obviously going to be a whole 
subproject doing the binding, it seems to fingerprint devices by VID/PID 
if I understood it.

What's the plan for leveraging that level of generality on "dynamically 
probed devices"?  I mean I know what I want to use this for and the 
platform_data scheme covers all the soldered-on-the-board cases fine.

Is there actually a need for sort of not platform_data but universal 
vid_pid_specific_usb_device_option_data coming from the board definition 
file or bootloader for *pluggable* usb devices?  udev seems to be well 
established doing that already in a generic, not-platform-specific way 
that can go in all distros and so on nicely.  Maybe this is just my 
impoverished imagination and people do want, say, some kinds of USB mice 
to operate at higher DPI or whatever when plugged in a specific board 
because it is that board.

BTW the whole RFC patchset I sent was tested on real Panda, including 
the platform end which actually exists.

-Andy

  reply	other threads:[~2011-03-17 23:22 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20110311165642.GA9996@kroah.com>
     [not found] ` <20110317214042.GQ31411@opensource.wolfsonmicro.com>
     [not found]   ` <20110317214736.GA29014@kroah.com>
     [not found]     ` <20110317214736.GA29014-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2011-03-17 22:33       ` RFC: Platform data for onboard USB assets Arnd Bergmann
2011-03-17 23:22         ` Andy Green [this message]
     [not found]           ` <4D82979B.2050003-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18 15:00             ` Arnd Bergmann
     [not found]               ` <201103181600.09877.arnd-r2nGTMty4D4@public.gmane.org>
2011-03-18 15:15                 ` Mark Brown
2011-03-18 17:52                 ` Andy Green
     [not found]                   ` <4D839BCD.6030202-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18 18:20                     ` David Anders
     [not found]                       ` <4D83A25C.804-l0cyMroinI0@public.gmane.org>
2011-03-18 18:25                         ` Mark Brown
     [not found]                           ` <20110318182518.GA2271-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2011-03-18 20:02                             ` Andy Green
2011-03-18 21:11                               ` Arnd Bergmann
     [not found]                                 ` <201103182211.36869.arnd-r2nGTMty4D4@public.gmane.org>
2011-03-18 21:17                                   ` Andy Green
2011-03-18 20:06                     ` Arnd Bergmann
2011-03-18 21:33                       ` Andy Green
     [not found]                         ` <4D83CF8C.3020605-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18 23:25                           ` Mark Brown
     [not found]                             ` <20110318232553.GA11422-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2011-03-18 23:33                               ` Andy Green
     [not found]                       ` <201103182106.13888.arnd-r2nGTMty4D4@public.gmane.org>
2011-03-18 21:36                         ` Grant Likely
2011-03-18 22:47                         ` Benjamin Herrenschmidt
2011-03-18 21:28                     ` Grant Likely
     [not found]                       ` <AANLkTi=Vy4Cu9rf7SpOFL1umN37bJgXhq9jEQpUrqjgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-03-18 23:04                         ` Andy Green
2011-03-18 22:37             ` Benjamin Herrenschmidt
2011-03-18 22:39               ` Andy Green
     [not found]         ` <201103172333.01474.arnd-r2nGTMty4D4@public.gmane.org>
2011-03-17 22:53           ` Greg KH
     [not found]             ` <20110317225328.GB31581-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2011-03-17 23:18               ` Andy Green
2011-03-17 23:25                 ` Greg KH
     [not found]                   ` <20110317232503.GA14561-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2011-03-18  7:42                     ` Andy Green
     [not found]                       ` <4D830CEC.4040608-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18 22:54                         ` Benjamin Herrenschmidt
2011-03-18 22:57                           ` Andy Green
     [not found]                 ` <4D8296D1.9060106-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18  4:54                   ` Grant Likely
     [not found]                     ` <20110318045401.GA18545-MrY2KI0G/OVr83L8+7iqerDks+cytr/Z@public.gmane.org>
2011-03-18  8:19                       ` Arnd Bergmann
2011-03-17 23:27           ` Grant Likely
2011-03-18  7:49             ` Andy Green
     [not found]               ` <4D830E97.4010403-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2011-03-18  8:25                 ` Arnd Bergmann
     [not found]                   ` <201103180925.30074.arnd-r2nGTMty4D4@public.gmane.org>
2011-03-18  8:38                     ` Andy Green

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D82979B.2050003@linaro.org \
    --to=andy@warmcat.com \
    --cc=andy.green@linaro.org \
    --cc=arnd@arndb.de \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=nicolas.pitre@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).