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,  s...@systol-ng.god.lan 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 mkru...@kernellabs.com

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