Re: [PATCH 3/4] tda8290 enable deemphasis_50 module parameter.

2009-09-24 Thread Michael Krufky
On Tue, Sep 22, 2009 at 5:07 PM,   wrote:
>
> This adds a forgotten module_param macro needed to set a deemphasis of 50us.
> It is the standard setting for commercial FM radio broadcasts outside the US.
>
> Signed-off-by: henk.vergo...@gmail.com
>
> diff -r 29e4ba1a09bc linux/drivers/media/common/tuners/tda8290.c
> --- a/linux/drivers/media/common/tuners/tda8290.c       Sat Sep 19 09:45:22 
> 2009 -0300
> +++ b/linux/drivers/media/common/tuners/tda8290.c       Tue Sep 22 22:06:31 
> 2009 +0200
> @@ -34,6 +34,7 @@
>  MODULE_PARM_DESC(debug, "enable verbose debug messages");
>
>  static int deemphasis_50;
> +module_param(deemphasis_50, int, 0644);
>  MODULE_PARM_DESC(deemphasis_50, "0 - 75us deemphasis; 1 - 50us deemphasis");
>
>  /* -- */
>

Reviewed-by: Michael Krufky 

Mauro, this patch is OK to merge ASAP -- if it's not merged already by
the time I have the tda18271 fixes ready, then I'll send this to you
in the same pull request.

Thanks for this fix, Henk.

Regards,

 Mike Krufky
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 3/4] tda8290 enable deemphasis_50 module parameter.

2009-09-22 Thread spam

This adds a forgotten module_param macro needed to set a deemphasis of 50us.
It is the standard setting for commercial FM radio broadcasts outside the US.

Signed-off-by: henk.vergo...@gmail.com

diff -r 29e4ba1a09bc linux/drivers/media/common/tuners/tda8290.c
--- a/linux/drivers/media/common/tuners/tda8290.c   Sat Sep 19 09:45:22 
2009 -0300
+++ b/linux/drivers/media/common/tuners/tda8290.c   Tue Sep 22 22:06:31 
2009 +0200
@@ -34,6 +34,7 @@
 MODULE_PARM_DESC(debug, "enable verbose debug messages");
 
 static int deemphasis_50;
+module_param(deemphasis_50, int, 0644);
 MODULE_PARM_DESC(deemphasis_50, "0 - 75us deemphasis; 1 - 50us deemphasis");
 
 /* -- */
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html