提交 a8b2580b 编写于 作者: M Mariusz Gorski 提交者: Greg Kroah-Hartman

staging: panel: Start making module params read-only

Start decoupling module params from the actual device state,
both for lcd and keypad, by keeping the params read-only
and moving the device state to related structs.
Signed-off-by: NMariusz Gorski <marius.gorski@gmail.com>
Acked-by: NWilly Tarreau <w@1wt.eu>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
上级 2d35bcf6
...@@ -212,6 +212,10 @@ static pmask_t phys_prev; ...@@ -212,6 +212,10 @@ static pmask_t phys_prev;
static char inputs_stable; static char inputs_stable;
/* these variables are specific to the keypad */ /* these variables are specific to the keypad */
static struct {
bool enabled;
} keypad;
static char keypad_buffer[KEYPAD_BUFFER]; static char keypad_buffer[KEYPAD_BUFFER];
static int keypad_buflen; static int keypad_buflen;
static int keypad_start; static int keypad_start;
...@@ -219,6 +223,9 @@ static char keypressed; ...@@ -219,6 +223,9 @@ static char keypressed;
static wait_queue_head_t keypad_read_wait; static wait_queue_head_t keypad_read_wait;
/* lcd-specific variables */ /* lcd-specific variables */
static struct {
bool enabled;
} lcd;
/* contains the LCD config state */ /* contains the LCD config state */
static unsigned long int lcd_flags; static unsigned long int lcd_flags;
...@@ -1393,7 +1400,7 @@ static void panel_lcd_print(const char *s) ...@@ -1393,7 +1400,7 @@ static void panel_lcd_print(const char *s)
const char *tmp = s; const char *tmp = s;
int count = strlen(s); int count = strlen(s);
if (lcd_enabled && lcd_initialized) { if (lcd.enabled && lcd_initialized) {
for (; count-- > 0; tmp++) { for (; count-- > 0; tmp++) {
if (!in_interrupt() && (((count + 1) & 0x1f) == 0)) if (!in_interrupt() && (((count + 1) & 0x1f) == 0))
/* let's be a little nice with other processes /* let's be a little nice with other processes
...@@ -1923,7 +1930,7 @@ static void panel_process_inputs(void) ...@@ -1923,7 +1930,7 @@ static void panel_process_inputs(void)
static void panel_scan_timer(void) static void panel_scan_timer(void)
{ {
if (keypad_enabled && keypad_initialized) { if (keypad.enabled && keypad_initialized) {
if (spin_trylock_irq(&pprt_lock)) { if (spin_trylock_irq(&pprt_lock)) {
phys_scan_contacts(); phys_scan_contacts();
...@@ -1935,7 +1942,7 @@ static void panel_scan_timer(void) ...@@ -1935,7 +1942,7 @@ static void panel_scan_timer(void)
panel_process_inputs(); panel_process_inputs();
} }
if (lcd_enabled && lcd_initialized) { if (lcd.enabled && lcd_initialized) {
if (keypressed) { if (keypressed) {
if (light_tempo == 0 && ((lcd_flags & LCD_FLAG_L) == 0)) if (light_tempo == 0 && ((lcd_flags & LCD_FLAG_L) == 0))
lcd_backlight(1); lcd_backlight(1);
...@@ -2114,7 +2121,7 @@ static void keypad_init(void) ...@@ -2114,7 +2121,7 @@ static void keypad_init(void)
static int panel_notify_sys(struct notifier_block *this, unsigned long code, static int panel_notify_sys(struct notifier_block *this, unsigned long code,
void *unused) void *unused)
{ {
if (lcd_enabled && lcd_initialized) { if (lcd.enabled && lcd_initialized) {
switch (code) { switch (code) {
case SYS_DOWN: case SYS_DOWN:
panel_lcd_print panel_lcd_print
...@@ -2170,13 +2177,13 @@ static void panel_attach(struct parport *port) ...@@ -2170,13 +2177,13 @@ static void panel_attach(struct parport *port)
/* must init LCD first, just in case an IRQ from the keypad is /* must init LCD first, just in case an IRQ from the keypad is
* generated at keypad init * generated at keypad init
*/ */
if (lcd_enabled) { if (lcd.enabled) {
lcd_init(); lcd_init();
if (misc_register(&lcd_dev)) if (misc_register(&lcd_dev))
goto err_unreg_device; goto err_unreg_device;
} }
if (keypad_enabled) { if (keypad.enabled) {
keypad_init(); keypad_init();
if (misc_register(&keypad_dev)) if (misc_register(&keypad_dev))
goto err_lcd_unreg; goto err_lcd_unreg;
...@@ -2184,7 +2191,7 @@ static void panel_attach(struct parport *port) ...@@ -2184,7 +2191,7 @@ static void panel_attach(struct parport *port)
return; return;
err_lcd_unreg: err_lcd_unreg:
if (lcd_enabled) if (lcd.enabled)
misc_deregister(&lcd_dev); misc_deregister(&lcd_dev);
err_unreg_device: err_unreg_device:
parport_unregister_device(pprt); parport_unregister_device(pprt);
...@@ -2202,12 +2209,12 @@ static void panel_detach(struct parport *port) ...@@ -2202,12 +2209,12 @@ static void panel_detach(struct parport *port)
return; return;
} }
if (keypad_enabled && keypad_initialized) { if (keypad.enabled && keypad_initialized) {
misc_deregister(&keypad_dev); misc_deregister(&keypad_dev);
keypad_initialized = 0; keypad_initialized = 0;
} }
if (lcd_enabled && lcd_initialized) { if (lcd.enabled && lcd_initialized) {
misc_deregister(&lcd_dev); misc_deregister(&lcd_dev);
lcd_initialized = 0; lcd_initialized = 0;
} }
...@@ -2283,8 +2290,8 @@ static int __init panel_init_module(void) ...@@ -2283,8 +2290,8 @@ static int __init panel_init_module(void)
break; break;
} }
lcd_enabled = (lcd_type > 0); lcd.enabled = (lcd_type > 0);
keypad_enabled = (keypad_type > 0); keypad.enabled = (keypad_type > 0);
switch (keypad_type) { switch (keypad_type) {
case KEYPAD_TYPE_OLD: case KEYPAD_TYPE_OLD:
...@@ -2309,7 +2316,7 @@ static int __init panel_init_module(void) ...@@ -2309,7 +2316,7 @@ static int __init panel_init_module(void)
return -EIO; return -EIO;
} }
if (!lcd_enabled && !keypad_enabled) { if (!lcd.enabled && !keypad.enabled) {
/* no device enabled, let's release the parport */ /* no device enabled, let's release the parport */
if (pprt) { if (pprt) {
parport_release(pprt); parport_release(pprt);
...@@ -2344,12 +2351,12 @@ static void __exit panel_cleanup_module(void) ...@@ -2344,12 +2351,12 @@ static void __exit panel_cleanup_module(void)
del_timer_sync(&scan_timer); del_timer_sync(&scan_timer);
if (pprt != NULL) { if (pprt != NULL) {
if (keypad_enabled) { if (keypad.enabled) {
misc_deregister(&keypad_dev); misc_deregister(&keypad_dev);
keypad_initialized = 0; keypad_initialized = 0;
} }
if (lcd_enabled) { if (lcd.enabled) {
panel_lcd_print("\x0cLCD driver " PANEL_VERSION panel_lcd_print("\x0cLCD driver " PANEL_VERSION
"\nunloaded.\x1b[Lc\x1b[Lb\x1b[L-"); "\nunloaded.\x1b[Lc\x1b[Lb\x1b[L-");
misc_deregister(&lcd_dev); misc_deregister(&lcd_dev);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册