[PATCH] at91sam9260/9g20ek: use -EINVAL for invalid gpio on atmel_mci

[PATCH] at91sam9260/9g20ek: use -EINVAL for invalid gpio on atmel_mci

[Fabio Porcedda]

Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
---
 arch/arm/boards/at91sam9260ek/init.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/boards/at91sam9260ek/init.c b/arch/arm/boards/at91sam9260ek/init.c
index 77ca6bb..dc2976a 100644
--- a/arch/arm/boards/at91sam9260ek/init.c
+++ b/arch/arm/boards/at91sam9260ek/init.c
@@ -156,6 +156,8 @@ static void at91sam9260ek_phy_reset(void)
 static struct atmel_mci_platform_data __initdata ek_mci_data = {
 	.bus_width	= 4,
 	.slot_b		= 1,
+	.detect_pin	= -EINVAL,
+	.wp_pin		= -EINVAL,
 };
 
 static void ek_usb_add_device_mci(void)
-- 
1.8.1.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] at91sam9260/9g20ek: use -EINVAL for invalid gpio on atmel_mci

[Sascha Hauer]

Hi Fabio,

On Mon, Feb 18, 2013 at 10:58:08AM +0100, Fabio Porcedda wrote:
> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
> Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
>  arch/arm/boards/at91sam9260ek/init.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/boards/at91sam9260ek/init.c b/arch/arm/boards/at91sam9260ek/init.c
> index 77ca6bb..dc2976a 100644
> --- a/arch/arm/boards/at91sam9260ek/init.c
> +++ b/arch/arm/boards/at91sam9260ek/init.c
> @@ -156,6 +156,8 @@ static void at91sam9260ek_phy_reset(void)
>  static struct atmel_mci_platform_data __initdata ek_mci_data = {
>  	.bus_width	= 4,
>  	.slot_b		= 1,
> +	.detect_pin	= -EINVAL,
> +	.wp_pin		= -EINVAL,
>  };

Is this for master or for -next?

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] at91sam9260/9g20ek: use -EINVAL for invalid gpio on atmel_mci

[Fabio Porcedda]

On Tue, Feb 19, 2013 at 8:32 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote:
> Hi Fabio,
>
> On Mon, Feb 18, 2013 at 10:58:08AM +0100, Fabio Porcedda wrote:
>> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
>> Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
>> ---
>>  arch/arm/boards/at91sam9260ek/init.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm/boards/at91sam9260ek/init.c b/arch/arm/boards/at91sam9260ek/init.c
>> index 77ca6bb..dc2976a 100644
>> --- a/arch/arm/boards/at91sam9260ek/init.c
>> +++ b/arch/arm/boards/at91sam9260ek/init.c
>> @@ -156,6 +156,8 @@ static void at91sam9260ek_phy_reset(void)
>>  static struct atmel_mci_platform_data __initdata ek_mci_data = {
>>       .bus_width      = 4,
>>       .slot_b         = 1,
>> +     .detect_pin     = -EINVAL,
>> +     .wp_pin         = -EINVAL,
>>  };
>
> Is this for master or for -next?

I think is for master because is a fix.

Best regards
--
Fabio Porcedda

> Sascha
>
>
> --
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] at91sam9260/9g20ek: use -EINVAL for invalid gpio on atmel_mci

[Fabio Porcedda]

On Tue, Feb 19, 2013 at 11:03 AM, Fabio Porcedda
<fabio.porcedda@gmail.com> wrote:
> On Tue, Feb 19, 2013 at 8:32 AM, Sascha Hauer <s.hauer@pengutronix.de> wrote:
>> Hi Fabio,
>>
>> On Mon, Feb 18, 2013 at 10:58:08AM +0100, Fabio Porcedda wrote:
>>> Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com>
>>> Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
>>> ---
>>>  arch/arm/boards/at91sam9260ek/init.c | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/arch/arm/boards/at91sam9260ek/init.c b/arch/arm/boards/at91sam9260ek/init.c
>>> index 77ca6bb..dc2976a 100644
>>> --- a/arch/arm/boards/at91sam9260ek/init.c
>>> +++ b/arch/arm/boards/at91sam9260ek/init.c
>>> @@ -156,6 +156,8 @@ static void at91sam9260ek_phy_reset(void)
>>>  static struct atmel_mci_platform_data __initdata ek_mci_data = {
>>>       .bus_width      = 4,
>>>       .slot_b         = 1,
>>> +     .detect_pin     = -EINVAL,
>>> +     .wp_pin         = -EINVAL,
>>>  };
>>
>> Is this for master or for -next?
>
> I think is for master because is a fix.


I'll send a v2 with all the other boards.

> Best regards
> --
> Fabio Porcedda
>
>> Sascha
>>
>>
>> --
>> Pengutronix e.K.                           |                             |
>> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
>> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
>> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



--
Fabio Porcedda

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox