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>