* [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output
@ 2015-06-09 9:15 Axel Lin
2015-06-09 12:58 ` Bhuvanchandra DV
0 siblings, 1 reply; 5+ messages in thread
From: Axel Lin @ 2015-06-09 9:15 UTC (permalink / raw)
To: u-boot
Pass correct gpio argument to gpio_set_value().
The calcualation of gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
is required for calling imx_iomux_gpio_* functions so move them close to
improve readability.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
Hi Bhuvanchandra,
I think current code does not pass correct gpio argument to gpio_set_value()
in vybrid_gpio_direction_output(). It only works if gpios->chip is 0.
I don't have the h/w to test, can you double check this?
Thanks,
Axel
drivers/gpio/vybrid_gpio.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/gpio/vybrid_gpio.c b/drivers/gpio/vybrid_gpio.c
index 6eaf0a9..33bbf54 100644
--- a/drivers/gpio/vybrid_gpio.c
+++ b/drivers/gpio/vybrid_gpio.c
@@ -36,8 +36,9 @@ static int vybrid_gpio_direction_output(struct udevice *dev, unsigned gpio,
{
const struct vybrid_gpios *gpios = dev_get_priv(dev);
- gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
gpio_set_value(gpio, value);
+
+ gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
imx_iomux_gpio_set_direction(gpio, VF610_GPIO_DIRECTION_OUT);
return 0;
@@ -54,6 +55,7 @@ static int vybrid_gpio_set_value(struct udevice *dev, unsigned gpio,
int value)
{
const struct vybrid_gpios *gpios = dev_get_priv(dev);
+
if (value)
writel((1 << gpio), &gpios->reg->gpio_psor);
else
@@ -68,7 +70,6 @@ static int vybrid_gpio_get_function(struct udevice *dev, unsigned gpio)
u32 g_state = 0;
gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
-
imx_iomux_gpio_get_function(gpio, &g_state);
if (((g_state & (0x07 << PAD_MUX_MODE_SHIFT)) >> PAD_MUX_MODE_SHIFT) > 0)
--
2.1.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output
2015-06-09 9:15 [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output Axel Lin
@ 2015-06-09 12:58 ` Bhuvanchandra DV
2015-06-09 13:19 ` Axel Lin
0 siblings, 1 reply; 5+ messages in thread
From: Bhuvanchandra DV @ 2015-06-09 12:58 UTC (permalink / raw)
To: u-boot
Hello Axel,
On 06/09/2015 02:45 PM, Axel Lin wrote:
> Pass correct gpio argument to gpio_set_value().
> The calcualation of gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
> is required for calling imx_iomux_gpio_* functions so move them close to
> improve readability.
>
> Signed-off-by: Axel Lin <axel.lin@ingics.com>
> ---
> Hi Bhuvanchandra,
> I think current code does not pass correct gpio argument to gpio_set_value()
> in vybrid_gpio_direction_output(). It only works if gpios->chip is 0.
> I don't have the h/w to test, can you double check this?
gpio_set_value() needs the actual gpio number to be passed not the gpio
offset of gpio chip.
The calculation is for getting the actual gpio number from gpio offset,
which we will need for both imx_iomux_gpio* and gpio_*_* functions.
Tested on Colibri VF50 and VF61 modules.
> Thanks,
> Axel
> drivers/gpio/vybrid_gpio.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpio/vybrid_gpio.c b/drivers/gpio/vybrid_gpio.c
> index 6eaf0a9..33bbf54 100644
> --- a/drivers/gpio/vybrid_gpio.c
> +++ b/drivers/gpio/vybrid_gpio.c
> @@ -36,8 +36,9 @@ static int vybrid_gpio_direction_output(struct udevice *dev, unsigned gpio,
> {
> const struct vybrid_gpios *gpios = dev_get_priv(dev);
>
> - gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
> gpio_set_value(gpio, value);
> +
> + gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
> imx_iomux_gpio_set_direction(gpio, VF610_GPIO_DIRECTION_OUT);
>
> return 0;
> @@ -54,6 +55,7 @@ static int vybrid_gpio_set_value(struct udevice *dev, unsigned gpio,
> int value)
> {
> const struct vybrid_gpios *gpios = dev_get_priv(dev);
> +
> if (value)
> writel((1 << gpio), &gpios->reg->gpio_psor);
> else
> @@ -68,7 +70,6 @@ static int vybrid_gpio_get_function(struct udevice *dev, unsigned gpio)
> u32 g_state = 0;
>
> gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
> -
> imx_iomux_gpio_get_function(gpio, &g_state);
>
> if (((g_state & (0x07 << PAD_MUX_MODE_SHIFT)) >> PAD_MUX_MODE_SHIFT) > 0)
>
Best regards,
Bhuvan
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output
2015-06-09 12:58 ` Bhuvanchandra DV
@ 2015-06-09 13:19 ` Axel Lin
2015-06-09 13:53 ` Bhuvanchandra DV
0 siblings, 1 reply; 5+ messages in thread
From: Axel Lin @ 2015-06-09 13:19 UTC (permalink / raw)
To: u-boot
2015-06-09 20:58 GMT+08:00 Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>:
> Hello Axel,
>
> On 06/09/2015 02:45 PM, Axel Lin wrote:
>>
>> Pass correct gpio argument to gpio_set_value().
>> The calcualation of gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
>> is required for calling imx_iomux_gpio_* functions so move them close to
>> improve readability.
>>
>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>> ---
>> Hi Bhuvanchandra,
>> I think current code does not pass correct gpio argument to
>> gpio_set_value()
>> in vybrid_gpio_direction_output(). It only works if gpios->chip is 0.
>> I don't have the h/w to test, can you double check this?
>
> gpio_set_value() needs the actual gpio number to be passed not the gpio
> offset of gpio chip.
Are you sure?
Please take a look at gpio_get_value()/gpio_set_value() implement in
drivers/gpio/gpio-uclass.c.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output
2015-06-09 13:19 ` Axel Lin
@ 2015-06-09 13:53 ` Bhuvanchandra DV
2015-06-09 14:03 ` Axel Lin
0 siblings, 1 reply; 5+ messages in thread
From: Bhuvanchandra DV @ 2015-06-09 13:53 UTC (permalink / raw)
To: u-boot
On 06/09/2015 06:49 PM, Axel Lin wrote:
> 2015-06-09 20:58 GMT+08:00 Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>:
>> Hello Axel,
>>
>> On 06/09/2015 02:45 PM, Axel Lin wrote:
>>>
>>> Pass correct gpio argument to gpio_set_value().
>>> The calcualation of gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
>>> is required for calling imx_iomux_gpio_* functions so move them close to
>>> improve readability.
>>>
>>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>>> ---
>>> Hi Bhuvanchandra,
>>> I think current code does not pass correct gpio argument to
>>> gpio_set_value()
>>> in vybrid_gpio_direction_output(). It only works if gpios->chip is 0.
>>> I don't have the h/w to test, can you double check this?
>>
>> gpio_set_value() needs the actual gpio number to be passed not the gpio
>> offset of gpio chip.
>
> Are you sure?
> Please take a look at gpio_get_value()/gpio_set_value() implement in
> drivers/gpio/gpio-uclass.c.
Yes,
Toggling GPIO_34 => GPIO_01 2nd instance, with debug prints:
Colibri VFxx # gpio toggle 34
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: vybrid_gpio_direction_output: gpio(before calculation): 2
DEBUG: vybrid_gpio_direction_output: gpio(after calculation): 34
DEBUG: gpio_set_value: gpio: 34
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: dm_gpio_set_value: gpio_offset: 2
gpio: pin 34 (gpio 34) value is 0
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
Colibri VFxx # gpio toggle 34
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: vybrid_gpio_direction_output: gpio(before calculation): 2
DEBUG: vybrid_gpio_direction_output: gpio(after calculation): 34
DEBUG: gpio_set_value: gpio: 34
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
DEBUG: dm_gpio_set_value: desc_offset: 2
gpio: pin 34 (gpio 34) value is 1
DEBUG: gpio_to_device: gpio: 34
DEBUG: gpio_to_device: desc_offset: 2
>
Best regards,
Bhuvan
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output
2015-06-09 13:53 ` Bhuvanchandra DV
@ 2015-06-09 14:03 ` Axel Lin
0 siblings, 0 replies; 5+ messages in thread
From: Axel Lin @ 2015-06-09 14:03 UTC (permalink / raw)
To: u-boot
2015-06-09 21:53 GMT+08:00 Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>:
> On 06/09/2015 06:49 PM, Axel Lin wrote:
>>
>> 2015-06-09 20:58 GMT+08:00 Bhuvanchandra DV
>> <bhuvanchandra.dv@toradex.com>:
>>>
>>> Hello Axel,
>>>
>>> On 06/09/2015 02:45 PM, Axel Lin wrote:
>>>>
>>>>
>>>> Pass correct gpio argument to gpio_set_value().
>>>> The calcualation of gpio = gpio + (gpios->chip * VYBRID_GPIO_COUNT);
>>>> is required for calling imx_iomux_gpio_* functions so move them close to
>>>> improve readability.
>>>>
>>>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>>>> ---
>>>> Hi Bhuvanchandra,
>>>> I think current code does not pass correct gpio argument to
>>>> gpio_set_value()
>>>> in vybrid_gpio_direction_output(). It only works if gpios->chip is 0.
>>>> I don't have the h/w to test, can you double check this?
>>>
>>>
>>> gpio_set_value() needs the actual gpio number to be passed not the gpio
>>> offset of gpio chip.
>>
>>
>> Are you sure?
>> Please take a look at gpio_get_value()/gpio_set_value() implement in
>> drivers/gpio/gpio-uclass.c.
>
> Yes,
Ah, my fault actually.
Sorry for the noise.
Regards,
Axel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-06-09 14:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-09 9:15 [U-Boot] [PATCH RFT] gpio: vybrid: Fix up setting output value in vybrid_gpio_direction_output Axel Lin
2015-06-09 12:58 ` Bhuvanchandra DV
2015-06-09 13:19 ` Axel Lin
2015-06-09 13:53 ` Bhuvanchandra DV
2015-06-09 14:03 ` Axel Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox