Re: [PATCH] gpio: tegra: read output value when gpio is set in direction_out

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

On 11/07/2012 11:27 PM, Laxman Dewangan wrote:
> Read the output value when gpio is set for the output mode for
> gpio_get_value(). Reading input value in direction out does not
> give correct value.

That's an unfortunate HW design, but oh well. Do you have any idea why
reading the input register doesn't work? If you look at the Tegra20 TRM,
page 666 figure 32 "SFIO/GPIO Pin Multiplexing Architecture", there's
not indication that the input path wouldn't work if the output path is
active. Perhaps the issue is in the GPIO module not the pinmux module?

> diff --git a/drivers/gpio/gpio-tegra.c b/drivers/gpio/gpio-tegra.c

>  static int tegra_gpio_get(struct gpio_chip *chip, unsigned offset)
>  {
> +	int bit_val = BIT(GPIO_BIT(offset));
> +
> +	/* If gpio is in output mode then read from the out value */
> +	if (tegra_gpio_readl(GPIO_OE(offset)) & bit_val)
> +		return !!(tegra_gpio_readl(GPIO_OUT(offset)) & bit_val);
> +
>  	return (tegra_gpio_readl(GPIO_IN(offset)) >> GPIO_BIT(offset)) & 0x1;
>  }

Any chance of using the same kind of logic to isolate the bit value? One
branch above does !!(val & mask) and the other (val >> shift) & 1.