From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Green Subject: Re: RFC: Platform data for onboard USB assets Date: Fri, 18 Mar 2011 22:39:09 +0000 Message-ID: <4D83DF0D.5050006@linaro.org> References: <20110311165642.GA9996@kroah.com> <20110317214042.GQ31411@opensource.wolfsonmicro.com> <20110317214736.GA29014@kroah.com> <201103172333.01474.arnd@arndb.de> <4D82979B.2050003@linaro.org> <1300487863.22236.1289.camel@pasglop> Reply-To: andy.green-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1300487863.22236.1289.camel@pasglop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Benjamin Herrenschmidt Cc: Nicolas Pitre , Linux USB list , Greg KH , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Mark Brown , lkml List-Id: devicetree@vger.kernel.org On 03/18/2011 10:37 PM, Somebody in the thread at some point said: > On Thu, 2011-03-17 at 23:22 +0000, Andy Green wrote: >> >> 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. > > Except that vid/pid are the -WRONG- way to do that. Your device is > on-board, the proper way to identify it uniquely is it's topological > location, ie path of HW port numbers. Thanks Ben. I'll go change that VID/PID stuff in my patches right away. -Andy