Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-29 Thread Mark Brown
On Mon, Jul 28, 2014 at 05:09:11PM +0200, Maxime Ripard wrote: > On Sun, Jul 27, 2014 at 07:49:42PM +0100, Mark Brown wrote: > > There's supposed to be facilities appearing in the generic clock code > > for specifying default clock tree configurations via the DT - it's quite > > a common

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-29 Thread Mark Brown
On Mon, Jul 28, 2014 at 05:09:11PM +0200, Maxime Ripard wrote: On Sun, Jul 27, 2014 at 07:49:42PM +0100, Mark Brown wrote: There's supposed to be facilities appearing in the generic clock code for specifying default clock tree configurations via the DT - it's quite a common requirement

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-28 Thread Maxime Ripard
Hi Mark, On Sun, Jul 27, 2014 at 07:49:42PM +0100, Mark Brown wrote: > On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: > > On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: > > > > > It feels a little fragile to rely on the organisation of the clock tree > > > > and

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-28 Thread Maxime Ripard
Hi Mark, On Sun, Jul 27, 2014 at 07:49:42PM +0100, Mark Brown wrote: On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: It feels a little fragile to rely on the organisation of the clock tree and the naming

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-27 Thread Mark Brown
On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: > On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: > > > It feels a little fragile to rely on the organisation of the clock tree > > > and the naming thereof. If the IP block is ever reused on an SoC with a > > >

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-27 Thread Mark Brown
On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: It feels a little fragile to rely on the organisation of the clock tree and the naming thereof. If the IP block is ever reused on an SoC with a different clock

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-07 Thread Maxime Ripard
On Sun, Jul 06, 2014 at 05:22:00PM +0200, Arnd Bergmann wrote: > On Friday 04 July 2014, Maxime Ripard wrote: > > > > > It feels a little fragile to rely on the organisation of the clock > > > > > tree > > > > > and the naming thereof. If the IP block is ever reused on an SoC with > > > > > a >

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-07 Thread Arnd Bergmann
On Friday 04 July 2014, Maxime Ripard wrote: > > > > It feels a little fragile to rely on the organisation of the clock tree > > > > and the naming thereof. If the IP block is ever reused on an SoC with a > > > > different clock tree layout then we have to handle things differently. > > > > > >

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-07 Thread Arnd Bergmann
On Friday 04 July 2014, Maxime Ripard wrote: It feels a little fragile to rely on the organisation of the clock tree and the naming thereof. If the IP block is ever reused on an SoC with a different clock tree layout then we have to handle things differently. What do you suggest

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-07 Thread Maxime Ripard
On Sun, Jul 06, 2014 at 05:22:00PM +0200, Arnd Bergmann wrote: On Friday 04 July 2014, Maxime Ripard wrote: It feels a little fragile to rely on the organisation of the clock tree and the naming thereof. If the IP block is ever reused on an SoC with a different clock

Re: [linux-sunxi] [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-05 Thread Emilio López
Hi Maxime, El 30/06/14 10:20, Maxime Ripard escribió: The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the older Allwinner SoCs, it's significantly different, I don't expect it to be possible to

Re: [linux-sunxi] [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-05 Thread Emilio López
Hi Maxime, El 30/06/14 10:20, Maxime Ripard escribió: The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the older Allwinner SoCs, it's significantly different, I don't expect it to be possible to

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-04 Thread Maxime Ripard
Hi Mark, On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: > On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: > > On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: > > > On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: > > > > On Mon, Jun 30,

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-04 Thread Maxime Ripard
Hi Mark, On Tue, Jul 01, 2014 at 01:48:52PM +0100, Mark Rutland wrote: On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-01 Thread Mark Rutland
On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: > On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: > > On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: > > > On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: > > > > Hi Maxime, > > > > > > > >

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-01 Thread Maxime Ripard
On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: > On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: > > On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: > > > Hi Maxime, > > > > > > On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: > > > > The

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-01 Thread Maxime Ripard
On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: Hi Maxime, On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: The Allwinner A31

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-07-01 Thread Mark Rutland
On Tue, Jul 01, 2014 at 08:21:19AM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at 04:33:05PM +0100, Mark Rutland wrote: On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: Hi Maxime, On Mon, Jun 30,

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Mark Rutland
On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: > On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: > > Hi Maxime, > > > > On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: > > > The Allwinner A31 has a 16 channels DMA controller that it shares with the > >

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Maxime Ripard
On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: > Hi Maxime, > > On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: > > The Allwinner A31 has a 16 channels DMA controller that it shares with the > > newer A23. Although sharing some similarities with the DMA controller of

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Mark Rutland
Hi Maxime, On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: > The Allwinner A31 has a 16 channels DMA controller that it shares with the > newer A23. Although sharing some similarities with the DMA controller of the > older Allwinner SoCs, it's significantly different, I don't

[PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Maxime Ripard
The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the older Allwinner SoCs, it's significantly different, I don't expect it to be possible to share the driver for these two. The A31 Controller is

[PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Maxime Ripard
The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the older Allwinner SoCs, it's significantly different, I don't expect it to be possible to share the driver for these two. The A31 Controller is

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Mark Rutland
Hi Maxime, On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the older Allwinner SoCs, it's significantly different, I don't expect

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Maxime Ripard
On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: Hi Maxime, On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23. Although sharing some similarities with the DMA controller of the

Re: [PATCH v10 2/2] dmaengine: sun6i: Add driver for the Allwinner A31 DMA controller

2014-06-30 Thread Mark Rutland
On Mon, Jun 30, 2014 at 04:19:06PM +0100, Maxime Ripard wrote: On Mon, Jun 30, 2014 at 03:20:54PM +0100, Mark Rutland wrote: Hi Maxime, On Mon, Jun 30, 2014 at 02:20:54PM +0100, Maxime Ripard wrote: The Allwinner A31 has a 16 channels DMA controller that it shares with the newer A23.