Re: [PATCH 06/10] saa7134: make module parameters boolean
Em 19-11-2010 21:43, David Härdeman escreveu: int to bool conversion for module parameters which are truely boolean. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/video/saa7134/saa7134-input.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video/saa7134/saa7134-input.c index 8b80efb..aea74e2 100644 --- a/drivers/media/video/saa7134/saa7134-input.c +++ b/drivers/media/video/saa7134/saa7134-input.c @@ -29,12 +29,12 @@ #define MODULE_NAME saa7134 -static unsigned int disable_ir; -module_param(disable_ir, int, 0444); +static bool disable_ir; +module_param(disable_ir, bool, 0444); MODULE_PARM_DESC(disable_ir,disable infrared remote support); -static unsigned int ir_debug; -module_param(ir_debug, int, 0644); +static bool ir_debug; +module_param(ir_debug, bool, 0644); MODULE_PARM_DESC(ir_debug,enable debug messages [IR]); Not sure about this patch. Debug parameters may be useful to be integer, as we may need to add more levels to the debug. Yet, maybe the better would be to just use dynamic_printk for all those stuff (e. g. converting them to use dev_dbg), as we can enable/disable the debug messages per-line, via debugfs. Comments? Cheers, mauro -- 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
Re: [PATCH 06/10] saa7134: make module parameters boolean
On Mon, Nov 22, 2010 at 03:40:13PM -0200, Mauro Carvalho Chehab wrote: Em 19-11-2010 21:43, David Härdeman escreveu: int to bool conversion for module parameters which are truely boolean. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/video/saa7134/saa7134-input.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video/saa7134/saa7134-input.c index 8b80efb..aea74e2 100644 --- a/drivers/media/video/saa7134/saa7134-input.c +++ b/drivers/media/video/saa7134/saa7134-input.c @@ -29,12 +29,12 @@ #define MODULE_NAME saa7134 -static unsigned int disable_ir; -module_param(disable_ir, int, 0444); +static bool disable_ir; +module_param(disable_ir, bool, 0444); MODULE_PARM_DESC(disable_ir,disable infrared remote support); -static unsigned int ir_debug; -module_param(ir_debug, int, 0644); +static bool ir_debug; +module_param(ir_debug, bool, 0644); MODULE_PARM_DESC(ir_debug,enable debug messages [IR]); Not sure about this patch. Debug parameters may be useful to be integer, as we may need to add more levels to the debug. Yet, maybe the better would be to just use dynamic_printk for all those stuff (e. g. converting them to use dev_dbg), as we can enable/disable the debug messages per-line, via debugfs. Comments? Not an important patch. Do whatever you feel suits your style. -- David Härdeman -- 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 06/10] saa7134: make module parameters boolean
int to bool conversion for module parameters which are truely boolean. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/video/saa7134/saa7134-input.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video/saa7134/saa7134-input.c index 8b80efb..aea74e2 100644 --- a/drivers/media/video/saa7134/saa7134-input.c +++ b/drivers/media/video/saa7134/saa7134-input.c @@ -29,12 +29,12 @@ #define MODULE_NAME saa7134 -static unsigned int disable_ir; -module_param(disable_ir, int, 0444); +static bool disable_ir; +module_param(disable_ir, bool, 0444); MODULE_PARM_DESC(disable_ir,disable infrared remote support); -static unsigned int ir_debug; -module_param(ir_debug, int, 0644); +static bool ir_debug; +module_param(ir_debug, bool, 0644); MODULE_PARM_DESC(ir_debug,enable debug messages [IR]); static int pinnacle_remote; -- 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
Re: [PATCH 06/10] saa7134: make module parameters boolean
Am Samstag, den 20.11.2010, 00:43 +0100 schrieb David Härdeman: int to bool conversion for module parameters which are truely boolean. Signed-off-by: David Härdeman da...@hardeman.nu --- drivers/media/video/saa7134/saa7134-input.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/video/saa7134/saa7134-input.c b/drivers/media/video/saa7134/saa7134-input.c index 8b80efb..aea74e2 100644 --- a/drivers/media/video/saa7134/saa7134-input.c +++ b/drivers/media/video/saa7134/saa7134-input.c @@ -29,12 +29,12 @@ #define MODULE_NAME saa7134 -static unsigned int disable_ir; -module_param(disable_ir, int, 0444); +static bool disable_ir; +module_param(disable_ir, bool, 0444); MODULE_PARM_DESC(disable_ir,disable infrared remote support); -static unsigned int ir_debug; -module_param(ir_debug, int, 0644); +static bool ir_debug; +module_param(ir_debug, bool, 0644); MODULE_PARM_DESC(ir_debug,enable debug messages [IR]); static int pinnacle_remote; Hi, not exactly that one, but given all the previous changes, I wonder if there will ever be some tested-by: from someone ... Cheers, Hermann -- 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