Re: [PATCH] commands: rename "gpiolib" -> "gpioinfo"

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

 



On 05/15/2014 11:08 AM, Holger Schurig wrote:
Signed-off-by: Holger Schurig <holgerschurig@xxxxxxxxx>

Subject: rename "gpiolib" -> "gpioinfo"

How about joining the gpio related commands to "gpio"
and use e.g. -l for list?

Sebastian

---
  drivers/gpio/gpiolib.c |    8 +++-----
  1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index 54ec5a6..611e41e 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -305,8 +305,6 @@ static int do_gpiolib(int argc, char *argv[])
  {
  	int i;

-	printf("gpiolib: gpio lists\n");
-
  	for (i = 0; i < ARCH_NR_GPIOS; i++) {
  		struct gpio_info *gi = &gpio_desc[i];
  		int val = -1, dir = -1;
@@ -316,7 +314,7 @@ static int do_gpiolib(int argc, char *argv[])

  		/* print chip information and header on first gpio */
  		if (gi->chip->base == i) {
-			printf("\ngpios %u-%u, chip %s:\n",
+			printf("\nGPIOs %u-%u, chip %s:\n",
  				gi->chip->base,
  				gi->chip->base + gi->chip->ngpio - 1,
  				gi->chip->dev->name);
@@ -330,7 +328,7 @@ static int do_gpiolib(int argc, char *argv[])
  			val = gi->chip->ops->get(gi->chip,
  						i - gi->chip->base);

-		printf("  gpio %*d: %*s %*s %*s  %s\n", 4, i,
+		printf("  GPIO %*d: %*s %*s %*s  %s\n", 4, i,
  			3, (dir < 0) ? "unk" : ((dir == GPIOF_DIR_IN) ? "in" : "out"),
  			3, (val < 0) ? "unk" : ((val == 0) ? "lo" : "hi"),
  			9, gi->requested ? "true" : "false",
@@ -340,7 +338,7 @@ static int do_gpiolib(int argc, char *argv[])
  	return 0;
  }

-BAREBOX_CMD_START(gpiolib)
+BAREBOX_CMD_START(gpioinfo)
  	.cmd		= do_gpiolib,
  	BAREBOX_CMD_DESC("list registered GPIOs")
  	BAREBOX_CMD_GROUP(CMD_GRP_INFO)



_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux