Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-21 Thread Matt Porter
On Mon, Jan 21, 2013 at 03:16:32AM +, Vinod Koul wrote: > On Sun, Jan 20, 2013 at 11:51:08AM -0500, Matt Porter wrote: > > The explanation in the cover letter mentions that dmaengine_slave_config() > > is > > required to be called prior to dmaengine_get_channel_caps(). If we > > switch to the

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-21 Thread Matt Porter
On Mon, Jan 21, 2013 at 03:16:32AM +, Vinod Koul wrote: On Sun, Jan 20, 2013 at 11:51:08AM -0500, Matt Porter wrote: The explanation in the cover letter mentions that dmaengine_slave_config() is required to be called prior to dmaengine_get_channel_caps(). If we switch to the

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Vinod Koul
On Sun, Jan 20, 2013 at 11:51:08AM -0500, Matt Porter wrote: > The explanation in the cover letter mentions that dmaengine_slave_config() is > required to be called prior to dmaengine_get_channel_caps(). If we > switch to the alternative API, then that would go away including the > dependency on

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Matt Porter
On Sun, Jan 20, 2013 at 01:03:21PM +, Vinod Koul wrote: > On Thu, Jan 10, 2013 at 02:07:05PM -0500, Matt Porter wrote: > > Implement device_channel_caps(). > > > > EDMA has a finite set of PaRAM slots available for linking > > a multi-segment SG transfer. In order to prevent any one > >

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Vinod Koul
On Thu, Jan 10, 2013 at 02:07:05PM -0500, Matt Porter wrote: > Implement device_channel_caps(). > > EDMA has a finite set of PaRAM slots available for linking > a multi-segment SG transfer. In order to prevent any one > channel from consuming all PaRAM slots to fulfill a large SG > transfer, the

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Vinod Koul
On Thu, Jan 10, 2013 at 02:07:05PM -0500, Matt Porter wrote: Implement device_channel_caps(). EDMA has a finite set of PaRAM slots available for linking a multi-segment SG transfer. In order to prevent any one channel from consuming all PaRAM slots to fulfill a large SG transfer, the driver

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Matt Porter
On Sun, Jan 20, 2013 at 01:03:21PM +, Vinod Koul wrote: On Thu, Jan 10, 2013 at 02:07:05PM -0500, Matt Porter wrote: Implement device_channel_caps(). EDMA has a finite set of PaRAM slots available for linking a multi-segment SG transfer. In order to prevent any one channel from

Re: [PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-20 Thread Vinod Koul
On Sun, Jan 20, 2013 at 11:51:08AM -0500, Matt Porter wrote: The explanation in the cover letter mentions that dmaengine_slave_config() is required to be called prior to dmaengine_get_channel_caps(). If we switch to the alternative API, then that would go away including the dependency on

[PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-10 Thread Matt Porter
Implement device_channel_caps(). EDMA has a finite set of PaRAM slots available for linking a multi-segment SG transfer. In order to prevent any one channel from consuming all PaRAM slots to fulfill a large SG transfer, the driver reports a static per-channel max number of SG segments it will

[PATCH v2 2/3] dma: edma: add device_channel_caps() support

2013-01-10 Thread Matt Porter
Implement device_channel_caps(). EDMA has a finite set of PaRAM slots available for linking a multi-segment SG transfer. In order to prevent any one channel from consuming all PaRAM slots to fulfill a large SG transfer, the driver reports a static per-channel max number of SG segments it will