Hi, On Wed, Jan 15, 2014 at 8:37 AM, Ma Haijun <[email protected]> wrote: > Hi, > > I think it is better to set the output value first to avoid glitch.
If I understand the user manual correctly, setting the output value before changing the pin to output function first will have no effect. And it is not possible to set the function and the value at the same time, as they are in different registers. Cheers, ChenYu > Regards, > > Ma > >> -----Original Message----- >> From: [email protected] [mailto:[email protected]] >> On Behalf Of Chen-Yu Tsai >> Sent: Tuesday, January 14, 2014 10:50 PM >> To: Maxime Ripard >> Cc: Chen-Yu Tsai; linux-arm-kernel; linux-sunxi; Linus Walleij >> Subject: [linux-sunxi] [PATCH] pinctrl: sunxi: Honor GPIO output initial > vaules >> >> Some GPIO users, such as fixed-regulator, request GPIO output with initial >> value of 1. This was ignored by sunxi driver. >> >> Signed-off-by: Chen-Yu Tsai <[email protected]> >> --- >> drivers/pinctrl/pinctrl-sunxi.c | 17 +++++++++++------ >> 1 file changed, 11 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/pinctrl/pinctrl-sunxi.c > b/drivers/pinctrl/pinctrl-sunxi.c index >> 119d2dd..fc72d10 100644 >> --- a/drivers/pinctrl/pinctrl-sunxi.c >> +++ b/drivers/pinctrl/pinctrl-sunxi.c >> @@ -469,12 +469,6 @@ static int sunxi_pinctrl_gpio_get(struct gpio_chip >> *chip, unsigned offset) >> return val; >> } >> >> -static int sunxi_pinctrl_gpio_direction_output(struct gpio_chip *chip, >> - unsigned offset, int value) >> -{ >> - return pinctrl_gpio_direction_output(chip->base + offset); >> -} >> - >> static void sunxi_pinctrl_gpio_set(struct gpio_chip *chip, >> unsigned offset, int value) >> { >> @@ -498,6 +492,17 @@ static void sunxi_pinctrl_gpio_set(struct gpio_chip >> *chip, >> spin_unlock_irqrestore(&pctl->lock, flags); } >> >> +static int sunxi_pinctrl_gpio_direction_output(struct gpio_chip *chip, >> + unsigned offset, int value) >> +{ >> + int ret = pinctrl_gpio_direction_output(chip->base + offset); >> + >> + if (ret == 0) >> + sunxi_pinctrl_gpio_set(chip, offset, value); >> + >> + return ret; >> +} >> + >> static int sunxi_pinctrl_gpio_of_xlate(struct gpio_chip *gc, >> const struct of_phandle_args *gpiospec, >> u32 *flags) >> -- >> 1.8.5.2 >> -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
