Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Felipe Balbi
Hi, On Wed, Jun 26, 2013 at 05:33:26PM +0200, Sebastian Andrzej Siewior wrote: I've been thinkig about creating two child nodes for the independent musb controllers on the am33. I've been thinking about the following: diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi

Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Sebastian Andrzej Siewior
On 06/27/2013 08:51 AM, Felipe Balbi wrote: Hi, Hi Felipe, the patch is alright, but what about the giant amoutn of function pointers we have ? Are you planning to use of_dev_auxdata ?? I didn't plan to use of_dev_auxdata. What do you mean by giant amount of function pointers? Sebastian --

Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Felipe Balbi
Hi, On Thu, Jun 27, 2013 at 09:31:34AM +0200, Sebastian Andrzej Siewior wrote: the patch is alright, but what about the giant amoutn of function pointers we have ? Are you planning to use of_dev_auxdata ?? I didn't plan to use of_dev_auxdata. What do you mean by giant amount of function

Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Benoit Cousson
Hi Sebastian, On 06/26/2013 05:33 PM, Sebastian Andrzej Siewior wrote: I've been thinkig about creating two child nodes for the independent musb controllers on the am33. I've been thinking about the following: diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index

Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Sebastian Andrzej Siewior
On 06/27/2013 01:23 PM, Benoit Cousson wrote: Hi Sebastian, Hi Benoit, BTW, why do have so many DMA compared to the previous version? I added them, the previous had none and is PIO only. I currently use three cells per dma channel (the posted example had two). In general I think I have to

Re: [RFC] am33xx: create child nodes for the two musb controllers

2013-06-27 Thread Felipe Balbi
On Thu, Jun 27, 2013 at 01:23:59PM +0200, Benoit Cousson wrote: Hi Sebastian, On 06/26/2013 05:33 PM, Sebastian Andrzej Siewior wrote: I've been thinkig about creating two child nodes for the independent musb controllers on the am33. I've been thinking about the following: diff --git

[RFC] am33xx: create child nodes for the two musb controllers

2013-06-26 Thread Sebastian Andrzej Siewior
I've been thinkig about creating two child nodes for the independent musb controllers on the am33. I've been thinking about the following: diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi index 8e1248f..6aa9506 100644 --- a/arch/arm/boot/dts/am33xx.dtsi +++