Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Mark Brown
On Thu, Nov 29, 2012 at 12:00:00PM +, Lee Jones wrote: > On Thu, 29 Nov 2012, Mark Brown wrote: > > Right, which is why this mostly works, but it's still better to provide > > an actual compatible string which we can be 100% certain will avoid > > conflicts. This is very low cost when one is

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Lee Jones
On Thu, 29 Nov 2012, Mark Brown wrote: > On Thu, Nov 29, 2012 at 10:08:08AM +, Lee Jones wrote: > > On Wed, 28 Nov 2012, Mark Brown wrote: > > > > The I2C subsystem guesses at a compatible string by default but it's > > > much better to explicitly set one as conflicts do arise from time to >

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Mark Brown
On Thu, Nov 29, 2012 at 10:08:08AM +, Lee Jones wrote: > On Wed, 28 Nov 2012, Mark Brown wrote: > > The I2C subsystem guesses at a compatible string by default but it's > > much better to explicitly set one as conflicts do arise from time to > > time (eg, Wolfson parts are called WM but

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Lee Jones
On Wed, 28 Nov 2012, Mark Brown wrote: > On Wed, Nov 28, 2012 at 08:57:31AM +, Lee Jones wrote: > > > > I applied these 3 patches, but for DT we also need to specify compatible > > > ID and set up of_match_table pointer. > > > Why do you need a compatible string? > > The I2C subsystem

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Lee Jones
On Wed, 28 Nov 2012, Mark Brown wrote: On Wed, Nov 28, 2012 at 08:57:31AM +, Lee Jones wrote: I applied these 3 patches, but for DT we also need to specify compatible ID and set up of_match_table pointer. Why do you need a compatible string? The I2C subsystem guesses at a

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Mark Brown
On Thu, Nov 29, 2012 at 10:08:08AM +, Lee Jones wrote: On Wed, 28 Nov 2012, Mark Brown wrote: The I2C subsystem guesses at a compatible string by default but it's much better to explicitly set one as conflicts do arise from time to time (eg, Wolfson parts are called WM but the WM

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Lee Jones
On Thu, 29 Nov 2012, Mark Brown wrote: On Thu, Nov 29, 2012 at 10:08:08AM +, Lee Jones wrote: On Wed, 28 Nov 2012, Mark Brown wrote: The I2C subsystem guesses at a compatible string by default but it's much better to explicitly set one as conflicts do arise from time to time

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-29 Thread Mark Brown
On Thu, Nov 29, 2012 at 12:00:00PM +, Lee Jones wrote: On Thu, 29 Nov 2012, Mark Brown wrote: Right, which is why this mostly works, but it's still better to provide an actual compatible string which we can be 100% certain will avoid conflicts. This is very low cost when one is

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Mark Brown
On Wed, Nov 28, 2012 at 08:57:31AM +, Lee Jones wrote: > > I applied these 3 patches, but for DT we also need to specify compatible > > ID and set up of_match_table pointer. > Why do you need a compatible string? The I2C subsystem guesses at a compatible string by default but it's much

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Lee Jones
Author: Lee Jones Date: Fri Sep 28 14:35:43 2012 +0100 Input: bu21013_ts - Add support for Device Tree booting Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously expected to be passed

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Lee Jones
Hi Dmitry, > On Tue, Nov 27, 2012 at 01:13:10PM +, Lee Jones wrote: > > Now we can register the BU21013_ts touch screen when booting with > > Device Tree enabled. Here we parse all the necessary components > > previously expected to be passed from platform data. > > I applied these 3

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Lee Jones
Hi Dmitry, On Tue, Nov 27, 2012 at 01:13:10PM +, Lee Jones wrote: Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously expected to be passed from platform data. I applied these 3 patches, but for

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Lee Jones
Author: Lee Jones lee.jo...@linaro.org Date: Fri Sep 28 14:35:43 2012 +0100 Input: bu21013_ts - Add support for Device Tree booting Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-28 Thread Mark Brown
On Wed, Nov 28, 2012 at 08:57:31AM +, Lee Jones wrote: I applied these 3 patches, but for DT we also need to specify compatible ID and set up of_match_table pointer. Why do you need a compatible string? The I2C subsystem guesses at a compatible string by default but it's much better

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-27 Thread Dmitry Torokhov
Hi Lee, On Tue, Nov 27, 2012 at 01:13:10PM +, Lee Jones wrote: > Now we can register the BU21013_ts touch screen when booting with > Device Tree enabled. Here we parse all the necessary components > previously expected to be passed from platform data. I applied these 3 patches, but for DT we

[PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-27 Thread Lee Jones
Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously expected to be passed from platform data. Cc: Dmitry Torokhov Cc: linux-in...@vger.kernel.org Acked-by: Arnd Bergmann Acked-by: Linus Walleij

[PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-27 Thread Lee Jones
Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously expected to be passed from platform data. Cc: Dmitry Torokhov dmitry.torok...@gmail.com Cc: linux-in...@vger.kernel.org Acked-by: Arnd Bergmann

Re: [PATCH 3/3] Input: bu21013_ts - Add support for Device Tree booting

2012-11-27 Thread Dmitry Torokhov
Hi Lee, On Tue, Nov 27, 2012 at 01:13:10PM +, Lee Jones wrote: Now we can register the BU21013_ts touch screen when booting with Device Tree enabled. Here we parse all the necessary components previously expected to be passed from platform data. I applied these 3 patches, but for DT we