Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 2 Dec 2019 19:57:20 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r355298 - head/sys/dev/gpio
Message-ID:  <201912021957.xB2JvKAj044550@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Mon Dec  2 19:57:20 2019
New Revision: 355298
URL: https://svnweb.freebsd.org/changeset/base/355298

Log:
  Do not initialize the flags field in struct gpiobus_pin from the flags in
  struct gpio_pin.  It turns out these two sets of flags are completely
  unrelated to each other.
  
  Also, update the comment for GPIO_ACTIVE_LOW to reflect the fact that it
  does get set, somewhat unobviously, by code that parses FDT data.  The bits
  from the FDT cell containing flags are just copied to gpiobus_pin.flags, so
  there's never any obvious reference to the symbol GPIO_ACTIVE_LOW being
  stored into the flags field.

Modified:
  head/sys/dev/gpio/gpiobus.c

Modified: head/sys/dev/gpio/gpiobus.c
==============================================================================
--- head/sys/dev/gpio/gpiobus.c	Mon Dec  2 17:53:32 2019	(r355297)
+++ head/sys/dev/gpio/gpiobus.c	Mon Dec  2 19:57:20 2019	(r355298)
@@ -77,7 +77,17 @@ static int gpiobus_pin_set(device_t, device_t, uint32_
 static int gpiobus_pin_get(device_t, device_t, uint32_t, unsigned int*);
 static int gpiobus_pin_toggle(device_t, device_t, uint32_t);
 
-#define	GPIO_ACTIVE_LOW 1 /* XXX Note that nothing currently sets this flag */
+/*
+ * gpiobus_pin flags
+ *  The flags in struct gpiobus_pin are not related to the flags used by the
+ *  low-level controller driver in struct gpio_pin.  Currently, only pins
+ *  acquired via FDT data have gpiobus_pin.flags set, sourced from the flags in
+ *  the FDT properties.  In theory, these flags are defined per-platform.  In
+ *  practice they are always the flags from the dt-bindings/gpio/gpio.h file.
+ *  The only one of those flags we currently support is for handling active-low
+ *  pins, so we just define that flag here instead of including a GPL'd header.
+ */
+#define	GPIO_ACTIVE_LOW 1
 
 /*
  * XXX -> Move me to better place - gpio_subr.c?
@@ -151,7 +161,7 @@ gpio_pin_get_by_bus_pinnum(device_t busdev, uint32_t p
 
 	pin->dev = device_get_parent(busdev);
 	pin->pin = pinnum;
-	GPIO_PIN_GETFLAGS(pin->dev, pin->pin, &pin->flags);
+	pin->flags = 0;
 
 	*ppin = pin;
 	return (0);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201912021957.xB2JvKAj044550>