Re: [PATCH 1/1] ALSA: ASoC: DaVinci: Fix SFFSDR init bug.

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

 



On Thursday 15 January 2009, Hugo Villeneuve wrote:
> Removed call to machine_is_sffsdr() in module init
> function because it always returns zero and causes
> GCC to optimise and remove all functions of the module.
> 
> Signed-off-by: Hugo Villeneuve <hugo@xxxxxxxxxxx>

NAK.  Enable the SFF board in Kconfig, and it won't
be returning false.  If you don't enable it, you
can't possibly be running on an SFF board, and so
there's no reason to include any of these function...


> ---
>  sound/soc/davinci/davinci-sffsdr.c |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/sound/soc/davinci/davinci-sffsdr.c b/sound/soc/davinci/davinci-sffsdr.c
> index 1bbde3e..3e8e6cd 100644
> --- a/sound/soc/davinci/davinci-sffsdr.c
> +++ b/sound/soc/davinci/davinci-sffsdr.c
> @@ -115,9 +115,6 @@ static int __init sffsdr_init(void)
>  {
>  	int ret;
>  
> -	if (!machine_is_sffsdr())
> -		return -EINVAL;
> -
>  	sffsdr_snd_device = platform_device_alloc("soc-audio", 0);
>  	if (!sffsdr_snd_device) {
>  		printk(KERN_ERR "platform device allocation failed\n");
> -- 
> 1.5.4.5
> 
> 
> _______________________________________________
> Davinci-linux-open-source mailing list
> Davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
> 
> 


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

  Powered by Linux