* [PATCHv3 0/3] gpios: Add daisy-chaining and dt support to 74x164
@ 2012-09-10 20:35 Maxime Ripard
2012-09-10 20:35 ` [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining Maxime Ripard
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Maxime Ripard @ 2012-09-10 20:35 UTC (permalink / raw)
To: linux-arm-kernel
Hi all,
This is an attempt at adding the support for chained 8 bit shift registers that
are present in the cfa-10049.
This patchset also adds dt documentation to the 74x164 gpio driver.
Thanks,
Maxime
Changes since v2:
* Fixed a memleak in the _write_config function when having an error.
Maxime Ripard (3):
gpio: 74x164: Add support for the daisy-chaining
gpio: 74x164: dts: Add documentation for the dt binding
ARM: dts: cfa10049: Add the 74HC595 gpio expanders
.../devicetree/bindings/gpio/gpio-74x164.txt | 22 +++++++
arch/arm/boot/dts/imx28-cfa10049.dts | 21 ++++++
drivers/gpio/gpio-74x164.c | 68 ++++++++++++++++++--
3 files changed, 104 insertions(+), 7 deletions(-)
create mode 100644 Documentation/devicetree/bindings/gpio/gpio-74x164.txt
--
1.7.9.5
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining 2012-09-10 20:35 [PATCHv3 0/3] gpios: Add daisy-chaining and dt support to 74x164 Maxime Ripard @ 2012-09-10 20:35 ` Maxime Ripard 2012-09-11 17:05 ` Linus Walleij 2012-09-10 20:35 ` [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding Maxime Ripard 2012-09-10 20:35 ` [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders Maxime Ripard 2 siblings, 1 reply; 12+ messages in thread From: Maxime Ripard @ 2012-09-10 20:35 UTC (permalink / raw) To: linux-arm-kernel The shift registers have an output pin that, when enabled, propagates the values of its internal register to the pins. If another value comes to the register while the output pin is disabled, this new value will makae the older shift into the next register in the chain. This patch adds support for daisy-chaining the registers, using the regular SPI chip select mechanism to manage the output pin, and the registers-number dt property to set the number of chained registers. Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> --- drivers/gpio/gpio-74x164.c | 68 +++++++++++++++++++++++++++++++++++++++----- 1 file changed, 61 insertions(+), 7 deletions(-) diff --git a/drivers/gpio/gpio-74x164.c b/drivers/gpio/gpio-74x164.c index 2e31bd3..ed3e551 100644 --- a/drivers/gpio/gpio-74x164.c +++ b/drivers/gpio/gpio-74x164.c @@ -14,14 +14,18 @@ #include <linux/spi/spi.h> #include <linux/spi/74x164.h> #include <linux/gpio.h> +#include <linux/of_gpio.h> #include <linux/slab.h> #include <linux/module.h> +#define GEN_74X164_NUMBER_GPIOS 8 + struct gen_74x164_chip { struct spi_device *spi; + u8 *buffer; struct gpio_chip gpio_chip; struct mutex lock; - u8 port_config; + u32 registers; }; static struct gen_74x164_chip *gpio_to_74x164_chip(struct gpio_chip *gc) @@ -31,17 +35,47 @@ static struct gen_74x164_chip *gpio_to_74x164_chip(struct gpio_chip *gc) static int __gen_74x164_write_config(struct gen_74x164_chip *chip) { - return spi_write(chip->spi, - &chip->port_config, sizeof(chip->port_config)); + struct spi_message message; + struct spi_transfer *msg_buf; + int i, ret = 0; + + msg_buf = kzalloc(chip->registers * sizeof(struct spi_transfer), + GFP_KERNEL); + if (!msg_buf) + return -ENOMEM; + + spi_message_init(&message); + + /* + * Since the registers are chained, every byte sent will make + * the previous byte shift to the next register in the + * chain. Thus, the first byte send will end up in the last + * register at the end of the transfer. So, to have a logical + * numbering, send the bytes in reverse order so that the last + * byte of the buffer will end up in the last register. + */ + for (i = chip->registers - 1; i >= 0; i--) { + msg_buf[i].tx_buf = chip->buffer +i; + msg_buf[i].len = sizeof(u8); + spi_message_add_tail(msg_buf + i, &message); + } + + ret = spi_sync(chip->spi, &message); + + kfree(msg_buf); + + return ret; } static int gen_74x164_get_value(struct gpio_chip *gc, unsigned offset) { struct gen_74x164_chip *chip = gpio_to_74x164_chip(gc); + u8 bank = offset / 8; + u8 pin = offset % 8; int ret; mutex_lock(&chip->lock); - ret = (chip->port_config >> offset) & 0x1; + ret = (chip->buffer[bank] >> pin) & 0x1; mutex_unlock(&chip->lock); return ret; @@ -51,12 +85,14 @@ static void gen_74x164_set_value(struct gpio_chip *gc, unsigned offset, int val) { struct gen_74x164_chip *chip = gpio_to_74x164_chip(gc); + u8 bank = offset / 8; + u8 pin = offset % 8; mutex_lock(&chip->lock); if (val) - chip->port_config |= (1 << offset); + chip->buffer[bank] |= (1 << pin); else - chip->port_config &= ~(1 << offset); + chip->buffer[bank] &= ~(1 << pin); __gen_74x164_write_config(chip); mutex_unlock(&chip->lock); @@ -75,6 +111,11 @@ static int __devinit gen_74x164_probe(struct spi_device *spi) struct gen_74x164_chip_platform_data *pdata; int ret; + if (!spi->dev.of_node) { + dev_err(&spi->dev, "No device tree data available.\n"); + return -EINVAL; + } + /* * bits_per_word cannot be configured in platform data */ @@ -104,7 +145,20 @@ static int __devinit gen_74x164_probe(struct spi_device *spi) chip->gpio_chip.direction_output = gen_74x164_direction_output; chip->gpio_chip.get = gen_74x164_get_value; chip->gpio_chip.set = gen_74x164_set_value; - chip->gpio_chip.ngpio = 8; + + if (of_property_read_u32(spi->dev.of_node, "registers-number", &chip->registers)) { + dev_err(&spi->dev, "Missing registers-number property in the DT.\n"); + ret = -EINVAL; + goto exit_destroy; + } + + chip->gpio_chip.ngpio = GEN_74X164_NUMBER_GPIOS * chip->registers; + chip->buffer = devm_kzalloc(&spi->dev, chip->gpio_chip.ngpio, GFP_KERNEL); + if (!chip->buffer) { + ret = -ENOMEM; + goto exit_destroy; + } + chip->gpio_chip.can_sleep = 1; chip->gpio_chip.dev = &spi->dev; chip->gpio_chip.owner = THIS_MODULE; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining 2012-09-10 20:35 ` [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining Maxime Ripard @ 2012-09-11 17:05 ` Linus Walleij 2012-09-11 19:35 ` Gabor Juhos 2012-09-11 21:51 ` Linus Walleij 0 siblings, 2 replies; 12+ messages in thread From: Linus Walleij @ 2012-09-11 17:05 UTC (permalink / raw) To: linux-arm-kernel On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > The shift registers have an output pin that, when enabled, propagates > the values of its internal register to the pins. If another value comes > to the register while the output pin is disabled, this new value will > makae the older shift into the next register in the chain. > > This patch adds support for daisy-chaining the registers, using the > regular SPI chip select mechanism to manage the output pin, and the > registers-number dt property to set the number of chained registers. > > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> OK cool, but I just realized that this driver was added just two years ago by Gabor and Miguel, so I'd like to get their feedback before continuing. I know I already merged the DT support and stuff, but better late than never. Please include them on future postings of patches to this driver... (See To: line) Miguel, Gabor: please look at this, I hope you can get hold of the patches. Please also check the stuff I've already merged here: http://git.kernel.org/?p=linux/kernel/git/linusw/linux-gpio.git;a=shortlog;h=refs/heads/devel Yours, Linus Walleij ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining 2012-09-11 17:05 ` Linus Walleij @ 2012-09-11 19:35 ` Gabor Juhos 2012-09-11 21:51 ` Linus Walleij 1 sibling, 0 replies; 12+ messages in thread From: Gabor Juhos @ 2012-09-11 19:35 UTC (permalink / raw) To: linux-arm-kernel 2012.09.11. 19:05 keltez?ssel, Linus Walleij ?rta: > On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > >> The shift registers have an output pin that, when enabled, propagates >> the values of its internal register to the pins. If another value comes >> to the register while the output pin is disabled, this new value will >> makae the older shift into the next register in the chain. >> >> This patch adds support for daisy-chaining the registers, using the >> regular SPI chip select mechanism to manage the output pin, and the >> registers-number dt property to set the number of chained registers. >> >> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > OK cool, but I just realized that this driver was added just two > years ago by Gabor and Miguel, so I'd like to get their feedback > before continuing. > > I know I already merged the DT support and stuff, but better > late than never. > > Please include them on future postings of patches to this > driver... (See To: line) > > Miguel, Gabor: please look at this, I hope you can get hold > of the patches. Please also check the stuff I've already merged > here: > http://git.kernel.org/?p=linux/kernel/git/linusw/linux-gpio.git;a=shortlog;h=refs/heads/devel The merged patches look good in general. The merged patches look good in general. However I don't know anything about the 74x164 chip. The driver contain my copyright only because it was based on another driver which I wrote for a different chip. -Gabor ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining 2012-09-11 17:05 ` Linus Walleij 2012-09-11 19:35 ` Gabor Juhos @ 2012-09-11 21:51 ` Linus Walleij 2012-09-12 7:23 ` Miguel GAIO 1 sibling, 1 reply; 12+ messages in thread From: Linus Walleij @ 2012-09-11 21:51 UTC (permalink / raw) To: linux-arm-kernel On Tue, Sep 11, 2012 at 7:05 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > >> The shift registers have an output pin that, when enabled, propagates >> the values of its internal register to the pins. If another value comes >> to the register while the output pin is disabled, this new value will >> makae the older shift into the next register in the chain. >> >> This patch adds support for daisy-chaining the registers, using the >> regular SPI chip select mechanism to manage the output pin, and the >> registers-number dt property to set the number of chained registers. >> >> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > OK cool, but I just realized that this driver was added just two > years ago by Gabor and Miguel, so I'd like to get their feedback > before continuing. Tentatively applied waiting for feedback from Miguel... Thanks! Linus Walleij ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining 2012-09-11 21:51 ` Linus Walleij @ 2012-09-12 7:23 ` Miguel GAIO 0 siblings, 0 replies; 12+ messages in thread From: Miguel GAIO @ 2012-09-12 7:23 UTC (permalink / raw) To: linux-arm-kernel On Tue, 2012-09-11 at 23:51 +0200, Linus Walleij wrote: > On Tue, Sep 11, 2012 at 7:05 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > > On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard > > <maxime.ripard@free-electrons.com> wrote: > > > >> The shift registers have an output pin that, when enabled, propagates > >> the values of its internal register to the pins. If another value comes > >> to the register while the output pin is disabled, this new value will > >> makae the older shift into the next register in the chain. > >> > >> This patch adds support for daisy-chaining the registers, using the > >> regular SPI chip select mechanism to manage the output pin, and the > >> registers-number dt property to set the number of chained registers. > >> > >> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > > > OK cool, but I just realized that this driver was added just two > > years ago by Gabor and Miguel, so I'd like to get their feedback > > before continuing. > > Tentatively applied waiting for feedback from Miguel... All is ok for me. Miguel > > Thanks! > Linus Walleij ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding 2012-09-10 20:35 [PATCHv3 0/3] gpios: Add daisy-chaining and dt support to 74x164 Maxime Ripard 2012-09-10 20:35 ` [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining Maxime Ripard @ 2012-09-10 20:35 ` Maxime Ripard 2012-09-11 17:22 ` Linus Walleij 2012-09-10 20:35 ` [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders Maxime Ripard 2 siblings, 1 reply; 12+ messages in thread From: Maxime Ripard @ 2012-09-10 20:35 UTC (permalink / raw) To: linux-arm-kernel Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> --- .../devicetree/bindings/gpio/gpio-74x164.txt | 22 ++++++++++++++++++++ 1 file changed, 22 insertions(+) create mode 100644 Documentation/devicetree/bindings/gpio/gpio-74x164.txt diff --git a/Documentation/devicetree/bindings/gpio/gpio-74x164.txt b/Documentation/devicetree/bindings/gpio/gpio-74x164.txt new file mode 100644 index 0000000..cc26080 --- /dev/null +++ b/Documentation/devicetree/bindings/gpio/gpio-74x164.txt @@ -0,0 +1,22 @@ +* Generic 8-bits shift register GPIO driver + +Required properties: +- compatible : Should be "fairchild,74hc595" +- reg : chip select number +- gpio-controller : Marks the device node as a gpio controller. +- #gpio-cells : Should be two. The first cell is the pin number and + the second cell is used to specify the gpio polarity: + 0 = active high + 1 = active low +- registers-number: Number of daisy-chained shift registers + +Example: + +gpio5: gpio5 at 0 { + compatible = "fairchild,74hc595"; + reg = <0>; + gpio-controller; + #gpio-cells = <2>; + registers-number = <4>; + spi-max-frequency = <100000>; +}; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding 2012-09-10 20:35 ` [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding Maxime Ripard @ 2012-09-11 17:22 ` Linus Walleij 2012-09-11 21:52 ` Linus Walleij 0 siblings, 1 reply; 12+ messages in thread From: Linus Walleij @ 2012-09-11 17:22 UTC (permalink / raw) To: linux-arm-kernel On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> Applied (since I already merged the DT code). Yours, Linus Walleij ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding 2012-09-11 17:22 ` Linus Walleij @ 2012-09-11 21:52 ` Linus Walleij 2012-09-12 11:51 ` Maxime Ripard 0 siblings, 1 reply; 12+ messages in thread From: Linus Walleij @ 2012-09-11 21:52 UTC (permalink / raw) To: linux-arm-kernel On Tue, Sep 11, 2012 at 7:22 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: > >> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> > > Applied (since I already merged the DT code). BTW: I don't have patch 3/3 in this series, does it exist? Or can you resend it? 1/3 and 2/3 are applied. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding 2012-09-11 21:52 ` Linus Walleij @ 2012-09-12 11:51 ` Maxime Ripard 0 siblings, 0 replies; 12+ messages in thread From: Maxime Ripard @ 2012-09-12 11:51 UTC (permalink / raw) To: linux-arm-kernel Hi Linus, Le 11/09/2012 23:52, Linus Walleij a ?crit : > On Tue, Sep 11, 2012 at 7:22 PM, Linus Walleij <linus.walleij@linaro.org> wrote: >> On Mon, Sep 10, 2012 at 10:35 PM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> >>> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> >> >> Applied (since I already merged the DT code). > > BTW: I don't have patch 3/3 in this series, does it exist? Or > can you resend it? 1/3 and 2/3 are applied. Hmm, that's odd, but since it was about the CFA-10049 dts, Shawn already applied it, so I guess it's fine. Thanks, Maxime -- Maxime Ripard, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders 2012-09-10 20:35 [PATCHv3 0/3] gpios: Add daisy-chaining and dt support to 74x164 Maxime Ripard 2012-09-10 20:35 ` [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining Maxime Ripard 2012-09-10 20:35 ` [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding Maxime Ripard @ 2012-09-10 20:35 ` Maxime Ripard 2012-09-12 2:30 ` Shawn Guo 2 siblings, 1 reply; 12+ messages in thread From: Maxime Ripard @ 2012-09-10 20:35 UTC (permalink / raw) To: linux-arm-kernel Add the two daisy-chains of 74hc595s shift registers available on the CFA-10049. Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> --- arch/arm/boot/dts/imx28-cfa10049.dts | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/arch/arm/boot/dts/imx28-cfa10049.dts b/arch/arm/boot/dts/imx28-cfa10049.dts index 1f7fa50..05c892e 100644 --- a/arch/arm/boot/dts/imx28-cfa10049.dts +++ b/arch/arm/boot/dts/imx28-cfa10049.dts @@ -27,6 +27,8 @@ fsl,pinmux-ids = < 0x0181 /* MX28_PAD_GPMI_RDN__SSP3_SCK */ 0x01c1 /* MX28_PAD_GPMI_RESETN__SSP3_CMD */ + 0x0111 /* MX28_PAD_GPMI_CE1N__SSP3_D3 */ + 0x01a2 /* MX28_PAD_GPMI_ALE__SSP3_D4 */ >; fsl,drive-strength = <1>; fsl,voltage = <1>; @@ -39,6 +41,25 @@ pinctrl-names = "default"; pinctrl-0 = <&spi3_pins_cfa10049>; status = "okay"; + + gpio5: gpio5 at 0 { + compatible = "fairchild,74hc595"; + gpio-controller; + #gpio-cells = <2>; + reg = <0>; + registers-number = <2>; + spi-max-frequency = <100000>; + }; + + gpio6: gpio6 at 1 { + compatible = "fairchild,74hc595"; + gpio-controller; + #gpio-cells = <2>; + reg = <1>; + registers-number = <4>; + spi-max-frequency = <100000>; + }; + }; }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders 2012-09-10 20:35 ` [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders Maxime Ripard @ 2012-09-12 2:30 ` Shawn Guo 0 siblings, 0 replies; 12+ messages in thread From: Shawn Guo @ 2012-09-12 2:30 UTC (permalink / raw) To: linux-arm-kernel On Mon, Sep 10, 2012 at 10:35:41PM +0200, Maxime Ripard wrote: > Add the two daisy-chains of 74hc595s shift registers available on the > CFA-10049. > > Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com> Applied, thanks. ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2012-09-12 11:51 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-09-10 20:35 [PATCHv3 0/3] gpios: Add daisy-chaining and dt support to 74x164 Maxime Ripard 2012-09-10 20:35 ` [PATCH 1/3] gpio: 74x164: Add support for the daisy-chaining Maxime Ripard 2012-09-11 17:05 ` Linus Walleij 2012-09-11 19:35 ` Gabor Juhos 2012-09-11 21:51 ` Linus Walleij 2012-09-12 7:23 ` Miguel GAIO 2012-09-10 20:35 ` [PATCH 2/3] gpio: 74x164: dts: Add documentation for the dt binding Maxime Ripard 2012-09-11 17:22 ` Linus Walleij 2012-09-11 21:52 ` Linus Walleij 2012-09-12 11:51 ` Maxime Ripard 2012-09-10 20:35 ` [PATCH 3/3] ARM: dts: cfa10049: Add the 74HC595 gpio expanders Maxime Ripard 2012-09-12 2:30 ` Shawn Guo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).