[patch 05/20] chardev: GPIO for SCx200: shrink scx200_gpio_dump

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





Shrink scx200_gpio_dump to a single printk with ternary ops.
- still coddled in its if 0  (merge with later patch ?)
- maybe do pdev_info here too.

Signed-off-by:  Jim Cromie <jim.cromie@xxxxxxxxx>
---
$ diffstat patch.dump-diet
scx200.c |   39 +++++++++++----------------------------
1 files changed, 11 insertions(+), 28 deletions(-)


diff -ruNp -X dontdiff -X exclude-diffs ab-4/arch/i386/kernel/scx200.c ab-5/arch/i386/kernel/scx200.c
--- ab-4/arch/i386/kernel/scx200.c	2006-06-02 13:36:04.000000000 -0600
+++ ab-5/arch/i386/kernel/scx200.c	2006-06-02 13:36:23.000000000 -0600
@@ -108,34 +108,17 @@ u32 scx200_gpio_configure(unsigned index
#if 0
void scx200_gpio_dump(unsigned index)
{
-	u32 config = scx200_gpio_configure(index, ~0, 0);
-	printk(KERN_DEBUG "GPIO%02u: 0x%08lx", index, (unsigned long)config);
-	
- if (config & 1) - printk(" OE"); /* output enabled */
-	else
-		printk(" TS"); /* tristate */
- if (config & 2) - printk(" PP"); /* push pull */
-	else
-		printk(" OD"); /* open drain */
- if (config & 4) - printk(" PUE"); /* pull up enabled */
-	else
-		printk(" PUD"); /* pull up disabled */
- if (config & 8) - printk(" LOCKED"); /* locked */ - if (config & 16) - printk(" LEVEL"); /* level input */
-	else
-		printk(" EDGE"); /* edge input */
- if (config & 32) - printk(" HI"); /* trigger on rising edge */
-	else
-		printk(" LO"); /* trigger on falling edge */
- if (config & 64) - printk(" DEBOUNCE"); /* debounce */
-	printk("\n");
+        u32 config = scx200_gpio_configure(index, ~0, 0);
+
+        printk(KERN_INFO NAME ": GPIO-%02u: 0x%08lx %s %s %s %s %s %s %s\n",
+               index, (unsigned long) config,
+               (config & 1) ? "OE"      : "TS",		/* output enabled / tristate */
+               (config & 2) ? "PP"      : "OD",		/* push pull / open drain */
+               (config & 4) ? "PUE"     : "PUD",	/* pull up enabled/disabled */
+               (config & 8) ? "LOCKED"  : "",		/* locked / unlocked */
+               (config & 16) ? "LEVEL"  : "EDGE",	/* level/edge input */
+ (config & 32) ? "HI" : "LO", /* trigger on rising/falling edge */ + (config & 64) ? "DEBOUNCE" : ""); /* debounce */
}
#endif  /*  0  */




--
Kernelnewbies: Help each other learn about the Linux kernel.
Archive:       http://mail.nl.linux.org/kernelnewbies/
FAQ:           http://kernelnewbies.org/faq/


[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux