* [PATCH v2 1/5] gpiolib: fix whitespaces
2013-01-31 21:46 [PATCH v2 0/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
@ 2013-01-31 21:46 ` Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 2/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
` (3 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Antony Pavlov @ 2013-01-31 21:46 UTC (permalink / raw)
To: barebox
Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
include/gpio.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/gpio.h b/include/gpio.h
index eedb980..794d959 100644
--- a/include/gpio.h
+++ b/include/gpio.h
@@ -6,7 +6,7 @@
#ifndef CONFIG_GPIOLIB
static inline int gpio_request(unsigned gpio, const char *label)
{
- return 0;
+ return 0;
}
static inline void gpio_free(unsigned gpio)
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v2 2/5] gpiolib: add gpio_request_{in,out}put support
2013-01-31 21:46 [PATCH v2 0/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 1/5] gpiolib: fix whitespaces Antony Pavlov
@ 2013-01-31 21:46 ` Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 3/5] gpiolib: gpio_request(): add error message Antony Pavlov
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Antony Pavlov @ 2013-01-31 21:46 UTC (permalink / raw)
To: barebox
gpio_request() followed by gpio_direction_{in,out}put
is a gpio usage pattern (e.g. the similar functions
are used in a ARM boards code).
Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
drivers/gpio/gpio.c | 34 ++++++++++++++++++++++++++++++++++
include/gpio.h | 13 +++++++++++++
2 files changed, 47 insertions(+)
diff --git a/drivers/gpio/gpio.c b/drivers/gpio/gpio.c
index d37f5a0..5876454 100644
--- a/drivers/gpio/gpio.c
+++ b/drivers/gpio/gpio.c
@@ -47,6 +47,40 @@ int gpio_request(unsigned gpio, const char *label)
return 0;
}
+int gpio_request_input(unsigned gpio, const char *label)
+{
+ int ret;
+
+ ret = gpio_request(gpio, label);
+ if (ret)
+ return ret;
+
+ ret = gpio_direction_input(gpio);
+ if (ret) {
+ pr_err("can't configure gpio %d (%s) as input (%d)\n",
+ gpio, label, ret);
+ }
+
+ return ret;
+}
+
+int gpio_request_output(unsigned gpio, const char *label, int value)
+{
+ int ret;
+
+ ret = gpio_request(gpio, label);
+ if (ret)
+ return ret;
+
+ ret = gpio_direction_output(gpio, value);
+ if (ret) {
+ pr_err("can't configure gpio %d (%s) as output (%d)\n",
+ gpio, label, ret);
+ }
+
+ return ret;
+}
+
void gpio_free(unsigned gpio)
{
struct gpio_info *gi = &gpio_desc[gpio];
diff --git a/include/gpio.h b/include/gpio.h
index 794d959..309c1c5 100644
--- a/include/gpio.h
+++ b/include/gpio.h
@@ -9,11 +9,24 @@ static inline int gpio_request(unsigned gpio, const char *label)
return 0;
}
+static inline int gpio_request_input(unsigned gpio, const char *label)
+{
+ return 0;
+}
+
+static inline int gpio_request_output(unsigned gpio, const char *label,
+ int value)
+{
+ return 0;
+}
+
static inline void gpio_free(unsigned gpio)
{
}
#else
int gpio_request(unsigned gpio, const char *label);
+int gpio_request_input(unsigned gpio, const char *label);
+int gpio_request_output(unsigned gpio, const char *label, int value);
void gpio_free(unsigned gpio);
#endif
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v2 3/5] gpiolib: gpio_request(): add error message
2013-01-31 21:46 [PATCH v2 0/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 1/5] gpiolib: fix whitespaces Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 2/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
@ 2013-01-31 21:46 ` Antony Pavlov
2013-02-01 8:13 ` Jean-Christophe PLAGNIOL-VILLARD
2013-01-31 21:46 ` [PATCH v2 4/5] at91sam9*: use gpio_request_output() Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 5/5] animeo_ip: use gpio_request_input() Antony Pavlov
4 siblings, 1 reply; 9+ messages in thread
From: Antony Pavlov @ 2013-01-31 21:46 UTC (permalink / raw)
To: barebox
Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
drivers/gpio/gpio.c | 29 ++++++++++++++++++++++-------
1 file changed, 22 insertions(+), 7 deletions(-)
diff --git a/drivers/gpio/gpio.c b/drivers/gpio/gpio.c
index 5876454..179fc7b 100644
--- a/drivers/gpio/gpio.c
+++ b/drivers/gpio/gpio.c
@@ -29,22 +29,37 @@ int gpio_request(unsigned gpio, const char *label)
struct gpio_chip *chip = gi->chip;
int ret;
- if (!gpio_is_valid(gpio))
- return -EINVAL;
- if (!chip)
- return -EINVAL;
- if (gi->requested)
- return -EBUSY;
+ if (!gpio_is_valid(gpio)) {
+ ret = -EINVAL;
+ goto err;
+ }
+
+ if (!chip) {
+ ret = -EINVAL;
+ goto err;
+ }
+
+ if (gi->requested) {
+ ret = -EBUSY;
+ goto err;
+ }
+
if (chip->ops->request) {
ret = chip->ops->request(chip, gpio - chip->base);
if (ret)
- return ret;
+ goto err;
}
gi->requested = true;
gi->label = xstrdup(label);
return 0;
+
+err:
+ pr_err("can't request gpio %d as %s (%d)\n",
+ gpio, label, ret);
+
+ return ret;
}
int gpio_request_input(unsigned gpio, const char *label)
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 3/5] gpiolib: gpio_request(): add error message
2013-01-31 21:46 ` [PATCH v2 3/5] gpiolib: gpio_request(): add error message Antony Pavlov
@ 2013-02-01 8:13 ` Jean-Christophe PLAGNIOL-VILLARD
2013-02-01 18:09 ` Antony Pavlov
0 siblings, 1 reply; 9+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-02-01 8:13 UTC (permalink / raw)
To: Antony Pavlov; +Cc: barebox
Nack the is need to handle by the caller not here
Best Regards,
J.
>
> Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
> ---
> drivers/gpio/gpio.c | 29 ++++++++++++++++++++++-------
> 1 file changed, 22 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/gpio/gpio.c b/drivers/gpio/gpio.c
> index 5876454..179fc7b 100644
> --- a/drivers/gpio/gpio.c
> +++ b/drivers/gpio/gpio.c
> @@ -29,22 +29,37 @@ int gpio_request(unsigned gpio, const char *label)
> struct gpio_chip *chip = gi->chip;
> int ret;
>
> - if (!gpio_is_valid(gpio))
> - return -EINVAL;
> - if (!chip)
> - return -EINVAL;
> - if (gi->requested)
> - return -EBUSY;
> + if (!gpio_is_valid(gpio)) {
> + ret = -EINVAL;
> + goto err;
> + }
> +
> + if (!chip) {
> + ret = -EINVAL;
> + goto err;
> + }
> +
> + if (gi->requested) {
> + ret = -EBUSY;
> + goto err;
> + }
> +
> if (chip->ops->request) {
> ret = chip->ops->request(chip, gpio - chip->base);
> if (ret)
> - return ret;
> + goto err;
> }
>
> gi->requested = true;
> gi->label = xstrdup(label);
>
> return 0;
> +
> +err:
> + pr_err("can't request gpio %d as %s (%d)\n",
> + gpio, label, ret);
> +
> + return ret;
> }
>
> int gpio_request_input(unsigned gpio, const char *label)
> --
> 1.7.10.4
>
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 3/5] gpiolib: gpio_request(): add error message
2013-02-01 8:13 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2013-02-01 18:09 ` Antony Pavlov
2013-02-01 18:24 ` Jean-Christophe PLAGNIOL-VILLARD
0 siblings, 1 reply; 9+ messages in thread
From: Antony Pavlov @ 2013-02-01 18:09 UTC (permalink / raw)
To: Jean-Christophe PLAGNIOL-VILLARD; +Cc: barebox
On 1 February 2013 12:13, Jean-Christophe PLAGNIOL-VILLARD
<plagnioj@jcrosoft.com> wrote:
> Nack the is need to handle by the caller not here
You want to get duplicated code? Ok, you will get in the "patch v3" series! :)))
>
> Best Regards,
> J.
>>
>> Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
>> ---
>> drivers/gpio/gpio.c | 29 ++++++++++++++++++++++-------
>> 1 file changed, 22 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/gpio/gpio.c b/drivers/gpio/gpio.c
>> index 5876454..179fc7b 100644
>> --- a/drivers/gpio/gpio.c
>> +++ b/drivers/gpio/gpio.c
>> @@ -29,22 +29,37 @@ int gpio_request(unsigned gpio, const char *label)
>> struct gpio_chip *chip = gi->chip;
>> int ret;
>>
>> - if (!gpio_is_valid(gpio))
>> - return -EINVAL;
>> - if (!chip)
>> - return -EINVAL;
>> - if (gi->requested)
>> - return -EBUSY;
>> + if (!gpio_is_valid(gpio)) {
>> + ret = -EINVAL;
>> + goto err;
>> + }
>> +
>> + if (!chip) {
>> + ret = -EINVAL;
>> + goto err;
>> + }
>> +
>> + if (gi->requested) {
>> + ret = -EBUSY;
>> + goto err;
>> + }
>> +
>> if (chip->ops->request) {
>> ret = chip->ops->request(chip, gpio - chip->base);
>> if (ret)
>> - return ret;
>> + goto err;
>> }
>>
>> gi->requested = true;
>> gi->label = xstrdup(label);
>>
>> return 0;
>> +
>> +err:
>> + pr_err("can't request gpio %d as %s (%d)\n",
>> + gpio, label, ret);
>> +
>> + return ret;
>> }
>>
>> int gpio_request_input(unsigned gpio, const char *label)
>> --
>> 1.7.10.4
>>
--
Best regards,
Antony Pavlov
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 3/5] gpiolib: gpio_request(): add error message
2013-02-01 18:09 ` Antony Pavlov
@ 2013-02-01 18:24 ` Jean-Christophe PLAGNIOL-VILLARD
0 siblings, 0 replies; 9+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2013-02-01 18:24 UTC (permalink / raw)
To: Antony Pavlov; +Cc: barebox
On 22:09 Fri 01 Feb , Antony Pavlov wrote:
> On 1 February 2013 12:13, Jean-Christophe PLAGNIOL-VILLARD
> <plagnioj@jcrosoft.com> wrote:
> > Nack the is need to handle by the caller not here
>
> You want to get duplicated code? Ok, you will get in the "patch v3" series! :)))
this is not duplicated code the caller known the dev or the part of the code
that fault
and we do auto request which is silent or other par of the code report the
erroor at higher level to yet I do not want the generic API to print something
Best Regards,
J.
>
> >
> > Best Regards,
> > J.
> >>
> >> Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
> >> ---
> >> drivers/gpio/gpio.c | 29 ++++++++++++++++++++++-------
> >> 1 file changed, 22 insertions(+), 7 deletions(-)
> >>
> >> diff --git a/drivers/gpio/gpio.c b/drivers/gpio/gpio.c
> >> index 5876454..179fc7b 100644
> >> --- a/drivers/gpio/gpio.c
> >> +++ b/drivers/gpio/gpio.c
> >> @@ -29,22 +29,37 @@ int gpio_request(unsigned gpio, const char *label)
> >> struct gpio_chip *chip = gi->chip;
> >> int ret;
> >>
> >> - if (!gpio_is_valid(gpio))
> >> - return -EINVAL;
> >> - if (!chip)
> >> - return -EINVAL;
> >> - if (gi->requested)
> >> - return -EBUSY;
> >> + if (!gpio_is_valid(gpio)) {
> >> + ret = -EINVAL;
> >> + goto err;
> >> + }
> >> +
> >> + if (!chip) {
> >> + ret = -EINVAL;
> >> + goto err;
> >> + }
> >> +
> >> + if (gi->requested) {
> >> + ret = -EBUSY;
> >> + goto err;
> >> + }
> >> +
> >> if (chip->ops->request) {
> >> ret = chip->ops->request(chip, gpio - chip->base);
> >> if (ret)
> >> - return ret;
> >> + goto err;
> >> }
> >>
> >> gi->requested = true;
> >> gi->label = xstrdup(label);
> >>
> >> return 0;
> >> +
> >> +err:
> >> + pr_err("can't request gpio %d as %s (%d)\n",
> >> + gpio, label, ret);
> >> +
> >> + return ret;
> >> }
> >>
> >> int gpio_request_input(unsigned gpio, const char *label)
> >> --
> >> 1.7.10.4
> >>
>
>
>
> --
> Best regards,
> Antony Pavlov
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 4/5] at91sam9*: use gpio_request_output()
2013-01-31 21:46 [PATCH v2 0/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
` (2 preceding siblings ...)
2013-01-31 21:46 ` [PATCH v2 3/5] gpiolib: gpio_request(): add error message Antony Pavlov
@ 2013-01-31 21:46 ` Antony Pavlov
2013-01-31 21:46 ` [PATCH v2 5/5] animeo_ip: use gpio_request_input() Antony Pavlov
4 siblings, 0 replies; 9+ messages in thread
From: Antony Pavlov @ 2013-01-31 21:46 UTC (permalink / raw)
To: barebox
Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
arch/arm/boards/at91sam9261ek/init.c | 18 +-----------------
arch/arm/boards/at91sam9263ek/init.c | 18 +-----------------
arch/arm/boards/at91sam9m10ihd/init.c | 18 +-----------------
3 files changed, 3 insertions(+), 51 deletions(-)
diff --git a/arch/arm/boards/at91sam9261ek/init.c b/arch/arm/boards/at91sam9261ek/init.c
index 2597ff8..220dd2f 100644
--- a/arch/arm/boards/at91sam9261ek/init.c
+++ b/arch/arm/boards/at91sam9261ek/init.c
@@ -153,22 +153,6 @@ static void ek_add_device_udc(void) {}
* LCD Controller
*/
#if defined(CONFIG_DRIVER_VIDEO_ATMEL)
-static int ek_gpio_request_output(int gpio, const char *name)
-{
- int ret;
-
- ret = gpio_request(gpio, name);
- if (ret) {
- pr_err("%s: can not request gpio %d (%d)\n", name, gpio, ret);
- return ret;
- }
-
- ret = gpio_direction_output(gpio, 1);
- if (ret)
- pr_err("%s: can not configure gpio %d as output (%d)\n", name, gpio, ret);
- return ret;
-}
-
/* TFT */
static struct fb_videomode at91_tft_vga_modes[] = {
{
@@ -211,7 +195,7 @@ static struct atmel_lcdfb_platform_data ek_lcdc_data = {
static int at91_lcdc_gpio(void)
{
- return ek_gpio_request_output(AT91_PIN_PA12, "lcdc_tft_power");
+ return gpio_request_output(AT91_PIN_PA12, "lcdc_tft_power", 1);
}
static void ek_add_device_lcdc(void)
diff --git a/arch/arm/boards/at91sam9263ek/init.c b/arch/arm/boards/at91sam9263ek/init.c
index 90fdbec..ed133b9 100644
--- a/arch/arm/boards/at91sam9263ek/init.c
+++ b/arch/arm/boards/at91sam9263ek/init.c
@@ -153,22 +153,6 @@ static void ek_add_device_udc(void) {}
* LCD Controller
*/
#if defined(CONFIG_DRIVER_VIDEO_ATMEL)
-static int ek_gpio_request_output(int gpio, const char *name)
-{
- int ret;
-
- ret = gpio_request(gpio, name);
- if (ret) {
- pr_err("%s: can not request gpio %d (%d)\n", name, gpio, ret);
- return ret;
- }
-
- ret = gpio_direction_output(gpio, 1);
- if (ret)
- pr_err("%s: can not configure gpio %d as output (%d)\n", name, gpio, ret);
- return ret;
-}
-
static struct fb_videomode at91_tft_vga_modes[] = {
{
.name = "TX09D50VM1CCA @ 60",
@@ -208,7 +192,7 @@ static struct atmel_lcdfb_platform_data ek_lcdc_data = {
static void ek_add_device_lcdc(void)
{
- if (ek_gpio_request_output(AT91_PIN_PA30, "lcdc_power"))
+ if (gpio_request_output(AT91_PIN_PA30, "lcdc_power", 1))
return;
at91_add_device_lcdc(&ek_lcdc_data);
diff --git a/arch/arm/boards/at91sam9m10ihd/init.c b/arch/arm/boards/at91sam9m10ihd/init.c
index f258caf..6159bfc 100644
--- a/arch/arm/boards/at91sam9m10ihd/init.c
+++ b/arch/arm/boards/at91sam9m10ihd/init.c
@@ -174,22 +174,6 @@ static int at91sam9m10g45ek_mem_init(void)
mem_initcall(at91sam9m10g45ek_mem_init);
#if defined(CONFIG_DRIVER_VIDEO_ATMEL)
-static int ek_gpio_request_output(int gpio, const char *name)
-{
- int ret;
-
- ret = gpio_request(gpio, name);
- if (ret) {
- pr_err("%s: can not request gpio %d (%d)\n", name, gpio, ret);
- return ret;
- }
-
- ret = gpio_direction_output(gpio, 1);
- if (ret)
- pr_err("%s: can not configure gpio %d as output (%d)\n", name, gpio, ret);
- return ret;
-}
-
static struct fb_videomode at91fb_default_monspecs[] = {
{
.name = "MULTEK",
@@ -230,7 +214,7 @@ static struct atmel_lcdfb_platform_data ek_lcdc_data = {
static void ek_add_device_lcd(void)
{
- if (ek_gpio_request_output(AT91_PIN_PE6, "lcdc_power"))
+ if (gpio_request_output(AT91_PIN_PE6, "lcdc_power", 1))
return;
at91_add_device_lcdc(&ek_lcdc_data);
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v2 5/5] animeo_ip: use gpio_request_input()
2013-01-31 21:46 [PATCH v2 0/5] gpiolib: add gpio_request_{in,out}put support Antony Pavlov
` (3 preceding siblings ...)
2013-01-31 21:46 ` [PATCH v2 4/5] at91sam9*: use gpio_request_output() Antony Pavlov
@ 2013-01-31 21:46 ` Antony Pavlov
4 siblings, 0 replies; 9+ messages in thread
From: Antony Pavlov @ 2013-01-31 21:46 UTC (permalink / raw)
To: barebox
Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
arch/arm/boards/animeo_ip/init.c | 15 ++-------------
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/arch/arm/boards/animeo_ip/init.c b/arch/arm/boards/animeo_ip/init.c
index ada3bb0..a5062f1 100644
--- a/arch/arm/boards/animeo_ip/init.c
+++ b/arch/arm/boards/animeo_ip/init.c
@@ -33,20 +33,9 @@ static int animeo_ip_get_pio_revision(int gpio, char *name)
{
int ret;
- ret = gpio_request(gpio, name);
- if (ret) {
- pr_err("animeo_ip: can not request gpio %d as %s (%d)\n",
- gpio, name, ret);
+ ret = gpio_request_input(gpio, name);
+ if (ret)
return ret;
- }
-
- ret = gpio_direction_input(gpio);
-
- if (ret) {
- pr_err("animeo_ip: can configure gpio %d (%s) as input (%d)\n",
- gpio, name, ret);
- return ret;
- }
return gpio_get_value(gpio);
}
--
1.7.10.4
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 9+ messages in thread