From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mugunthan V N Subject: Re: [PATCH 0/4] Add DMA support for ti_am335x_adc driver Date: Thu, 29 Sep 2016 18:31:30 +0530 Message-ID: References: <20160921161134.6951-1-mugunthanvnm@ti.com> <22588f2d-d0de-7911-35ed-92094f7dd28d@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <22588f2d-d0de-7911-35ed-92094f7dd28d-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jonathan Cameron , linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: Tony Lindgren , Rob Herring , Mark Rutland , Russell King , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , Lee Jones , Vignesh R , "Andrew F . Davis" , linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Sekhar Nori , Peter Ujfalusi List-Id: devicetree@vger.kernel.org On Sunday 25 September 2016 03:11 PM, Jonathan Cameron wrote: > On 21/09/16 17:11, Mugunthan V N wrote: >> > The ADC has a 64 work depth fifo length which holds the ADC data >> > till the CPU reads. So when a user program needs a large ADC data >> > to operate on, then it has to do multiple reads to get its >> > buffer. Currently if the application asks for 4 samples per >> > channel with all 8 channels are enabled, kernel can provide only >> > 3 samples per channel when all 8 channels are enabled (logs at >> > [1]). So with DMA support user can request for large number of >> > samples at a time (logs at [2]). >> > >> > Tested the patch on AM437x-gp-evm and AM335x Boneblack with the >> > patch [3] to enable ADC and pushed a branch for testing [4] >> > >> > [1] - http://pastebin.ubuntu.com/23211490/ >> > [2] - http://pastebin.ubuntu.com/23211492/ >> > [3] - http://pastebin.ubuntu.com/23211494/ >> > [4] - git://git.ti.com/~mugunthanvnm/ti-linux-kernel/linux.git iio-dma > Just curious. How fast is the ADC sampling at in these? Never that > obvious for this driver! > > I'm also curious as to whether you started to hit the limits of the > kfifo based interface. Might be worth considering adding alternative > support for the dma buffers interface which is obviously much lower > overhead. > > Good to have this work prior to that as the kfifo stuff is somewhat > easier to use. Currently ADC clock is 3MHz, which can produce a data rate of 225KBps per channel with no open delay and no averaging of samples. So when all 8 Channels are enables the data rate will be 1.75MBps ADC can be operated at 24MHz, which can generate a data rate of 28MBps with all 8 channels enabled and no open delay and averaging, but our target is to get 800K samples per second per channel which has a data rate of 12.5MBps I think with this data rate, DMA will be the best option to implement without any data loss and less cpu overload to read the ADC samples. Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html