Re: [PATCHv2 0/5] *** SUBJECT HERE ***ASoC: tlv320dac33: FIFO caused delay reporting and fixes

[Liam Girdwood <lrg@slimlogic.co.uk>]

On Fri, 2010-04-23 at 10:09 +0300, Peter Ujfalusi wrote:
> Hello,
> 
> Changelog:
> - Calculated delay instead of msleep(2) in patch 1
> - ns changed to us in patch 5 (in the macro for calculation)
> 
> The main feature of this series is the FIFO caused delay reporting on the DAC33
> codec.
> The main patch is quite big, but I don't think it can be divided further.
> 
> Short details of the actual implementation is in the commit message(s), and
> I will not copy it here.
> 
> The values reported by the delay call has been verified by using one GPIO as a
> signal, and printout from the kernel at the same time with the delay, than
> based on the time scale on the scope several (>100) cases has been hand
> calculated and the results were compared to the kernel reported values.
> The difference were around (or less) the 10 samples, but during the bursts it
> is much harder to count, but the reported value was reasonable accurate there
> as well.
> 
> ---
> Peter Ujfalusi (5):
>   ASoC: tlv320dac33: Fix for early interrupt in FIFO Mode1
>   ASoC: tlv320dac33: Skip calculations in FIFO Bypass mode
>   ASoC: tlv320dac33: Change magic numbers used in Mode7
>   ASoC: tlv320dac33: Calculate the interface speed during bursts
>   ASoC: tlv320dac33: FIFO caused delay reporting
> 
>  sound/soc/codecs/tlv320dac33.c |  250 ++++++++++++++++++++++++++++++++++++++--
>  1 files changed, 238 insertions(+), 12 deletions(-)

Applied.

Thanks

Liam

-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk