* [PATCH] intelfbdrv.c: bailearly is an int module_param
[not found] <20120103131748.GA11575@elgon.mountain>
@ 2012-01-04 2:41 ` Rusty Russell
0 siblings, 0 replies; only message in thread
From: Rusty Russell @ 2012-01-04 2:41 UTC (permalink / raw)
To: Maik Broemme; +Cc: Florian Tobias Schandinat, linux-fbdev, linux-kernel
Dan Carpenter points out that it's an int, not a bool:
intelfbdrv.c:818: if (bailearly = 1)
intelfbdrv.c:828: if (bailearly = 2)
intelfbdrv.c:836: if (bailearly = 3)
intelfbdrv.c:842: if (bailearly = 4)
intelfbdrv.c:851: if (bailearly = 5)
intelfbdrv.c:859: if (bailearly = 6)
intelfbdrv.c:866: bailearly > 6 ? bailearly - 6 : 0);
intelfbdrv.c:874: if (bailearly = 18)
intelfbdrv.c:886: if (bailearly = 19)
intelfbdrv.c:893: if (bailearly = 20)
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Cc: Dan Carpenter <dan.carpenter@oracle.com>
---
drivers/video/intelfb/intelfbdrv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/video/intelfb/intelfbdrv.c b/drivers/video/intelfb/intelfbdrv.c
--- a/drivers/video/intelfb/intelfbdrv.c
+++ b/drivers/video/intelfb/intelfbdrv.c
@@ -263,7 +263,7 @@ module_param(probeonly, bool, 0);
MODULE_PARM_DESC(probeonly, "Do a minimal probe (debug)");
module_param(idonly, bool, 0);
MODULE_PARM_DESC(idonly, "Just identify without doing anything else (debug)");
-module_param(bailearly, bool, 0);
+module_param(bailearly, int, 0);
MODULE_PARM_DESC(bailearly, "Bail out early, depending on value (debug)");
module_param(mode, charp, S_IRUGO);
MODULE_PARM_DESC(mode,
^ permalink raw reply [flat|nested] only message in thread