RE: [PATCH] pinctrl: GPIO direction support for muxing

[Stephen Warren]

Linus Walleij wrote at Monday, November 21, 2011 6:00 AM:
> When requesting a single GPIO pin to be muxed in, some controllers
> will need to poke a different value into the control register
> depending on whether the pin will be used for GPIO output or GPIO
> input. So create pinmux counterparts to gpio_direction_[input|output]
> in the pinctrl framework.

Just a few very minor typos below. Otherwise,

Acked-by: Stephen Warren <swarren@xxxxxxxxxx>

> diff --git a/Documentation/pinctrl.txt b/Documentation/pinctrl.txt
...
> +NOTE that platforms and individual driver shall *NOT* request GPIO pins to be
                                           ^s
> +muxed in. Instead, implement a proper gpiolib driver hand have that driver
                                                        ^ and

> diff --git a/drivers/pinctrl/pinmux.c b/drivers/pinctrl/pinmux.c
...
> @@ -360,7 +421,7 @@ static int acquire_pins(struct pinctrl_dev *pctldev,
> 
>  	/* Try to allocate all pins in this group, one by one */
>  	for (i = 0; i < num_pins; i++) {
> -		ret = pin_request(pctldev, pins[i], func, NULL);
> +	  ret = pin_request(pctldev, pins[i], func, NULL);

Looks like a mistake.

> diff --git a/include/linux/pinctrl/pinmux.h b/include/linux/pinctrl/pinmux.h

> + * @gpio_set_direction: Since controllers may be needing different

s/be needing/need/

-- 
nvpublic

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/