Re: [PATCH v2 0/7] ASoC: omap-mcbsp: Cleanup and split core/sidetone support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/7/18 3:23 PM, Peter Ujfalusi wrote:
> Hi,
> 
> Changes since v1:
> - Two patch added to address the code move triggered few warnings from build.
>   One of them is around mcbsp->pdata checks (not valid complaint), the other is
>   that we set -EINVAL to unsigned variables (they were not used in that case)
> 
> While preparing for merging the davinci and omap directories (I will send the
> series for that shortly) I have taken some time to do cleanup on the McBSP
> driver.
> 
> The mcbsp.c/h files were the result when we moved code from arch/arm/plat-omap
> to sound/soc/omap/ a long time ago and it contained code for McBSP core and the
> OMAP3 sidetone functionality.
> 
> With this series I tried to split the core and sidetone code to separate files.
> 
> There were two reasons for this:
> 1. to have the OMAP specific DAI driver files prefixed with omap-* under the new
>    sound/soc/ti/ directory
> 2. easier to find functions as all sidetone code is in separate file.
> 
> Regards,
> Peter
> ---
> Peter Ujfalusi (7):
>   ASoC: omap-mcbsp: Clean up dma_data addr initialization code
>   ASoC: omap-mcbsp: Clean up the interrupt handlers
>   ASoC: omap-mcbsp: Simplify the mcbsp_start/_stop function parameters
>   ASoC: omap-mcbsp: Move out the FIFO check from set_threshold and
>     get_delay
>   ASoC: omap-mcbsp: Re-arrange files for core McBSP and Sidetone
>     function split
>   ASoC: omap-mcbsp: Remove redundant check for mcbsp->pdata
>   ASoC: omap-mcbsp: No need to initialize max_xx_thres when it is not
>     used
> 
Minor comments, no need to resend. Patch 1/7 looked like it could be
splitted into two but don't know is it worth of effort, maybe not. Then
I was thinking can 6-7/7 be moved before 5/7?

I'll try to give a test to the set perhaps during weekend but don't let
that delay the process.

Acked-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxx>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux