[linux-sunxi] 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

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

2014-07-28 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

[linux-sunxi] 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

[linux-sunxi] 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

[linux-sunxi] 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

[linux-sunxi] 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

[linux-sunxi] 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

[linux-sunxi] 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,

[linux-sunxi] 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

[linux-sunxi] 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

[linux-sunxi] 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.