Messages in this thread Patch in this message | | | Date | Sat, 17 Jun 2006 12:28:05 -0600 | From | Jim Cromie <> | Subject | [patch -mm 06/20] chardev: GPIO for SCx200 & PC-8736x: add 'v' command to device-file |
| |
6/20. patch.viewpins
Add a new driver command: 'v' which calls gpio_dump() on the pin. The output goes to the log, like all other INFO messages in the original driver. Giving the user control over the feedback they 'need' is construed to be a user-friendly feature, and allows us (later) to dial down many INFO messages to DEBUG log-level.
Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
---
diffstat gpio-scx/patch.viewpins arch/i386/kernel/scx200.c | 3 +-- drivers/char/scx200_gpio.c | 16 ++++++++++++++++ 2 files changed, 17 insertions(+), 2 deletions(-)
diff -ruNp -X dontdiff -X exclude-diffs ax-5/arch/i386/kernel/scx200.c ax-6/arch/i386/kernel/scx200.c --- ax-5/arch/i386/kernel/scx200.c 2006-06-17 01:10:02.000000000 -0600 +++ ax-6/arch/i386/kernel/scx200.c 2006-06-17 01:13:26.000000000 -0600 @@ -105,7 +105,6 @@ u32 scx200_gpio_configure(unsigned index return config; } -#if 0 void scx200_gpio_dump(unsigned index) { u32 config = scx200_gpio_configure(index, ~0, 0); @@ -120,7 +119,6 @@ void scx200_gpio_dump(unsigned index) (config & 32) ? "HI" : "LO", /* trigger on rising/falling edge */ (config & 64) ? "DEBOUNCE" : ""); /* debounce */ } -#endif /* 0 */ static int __init scx200_init(void) { @@ -141,4 +139,5 @@ module_exit(scx200_cleanup); EXPORT_SYMBOL(scx200_gpio_base); EXPORT_SYMBOL(scx200_gpio_shadow); EXPORT_SYMBOL(scx200_gpio_configure); +EXPORT_SYMBOL(scx200_gpio_dump); EXPORT_SYMBOL(scx200_cb_base); diff -ruNp -X dontdiff -X exclude-diffs ax-5/drivers/char/scx200_gpio.c ax-6/drivers/char/scx200_gpio.c --- ax-5/drivers/char/scx200_gpio.c 2006-06-17 01:04:20.000000000 -0600 +++ ax-6/drivers/char/scx200_gpio.c 2006-06-17 01:13:26.000000000 -0600 @@ -41,6 +41,7 @@ static ssize_t scx200_gpio_write(struct { unsigned m = iminor(file->f_dentry->d_inode); size_t i; + int err = 0; for (i = 0; i < len; ++i) { char c; @@ -77,8 +78,23 @@ static ssize_t scx200_gpio_write(struct printk(KERN_INFO NAME ": GPIO%d pull up disabled\n", m); scx200_gpio_configure(m, ~4, 0); break; + + case 'v': + /* View Current pin settings */ + scx200_gpio_dump(m); + break; + case '\n': + /* end of settings string, do nothing */ + break; + default: + printk(KERN_ERR NAME + ": GPIO-%2d bad setting: chr<0x%2x>\n", m, + (int)c); + err++; } } + if (err) + return -EINVAL; /* full string handled, report error */ return len; }
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |