[U-Boot] [PATCH] Ethernut 5: fix build error for board_mmc_getcd()

[U-Boot] [PATCH] Ethernut 5: fix build error for board_mmc_getcd()

[Prabhakar Lad]

Fix build error for ethernut5 and mx6qarm2 board due
to prototype change for function  board_mmc_getcd()
ethernut5.c:238: error: conflicting types for 'board_mmc_getcd'
u-boot/include/mmc.h:318: note: previous declaration of 'board_mmc_getcd' was here
make[2]: *** [ethernut5.o] Error 1

Signed-off-by: Prabhakar Lad <prabhakar.csengg@gmail.com>
---
 board/egnite/ethernut5/ethernut5.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/board/egnite/ethernut5/ethernut5.c b/board/egnite/ethernut5/ethernut5.c
index e42e91e..fd021a3 100644
--- a/board/egnite/ethernut5/ethernut5.c
+++ b/board/egnite/ethernut5/ethernut5.c
@@ -235,10 +235,9 @@ int board_mmc_init(bd_t *bd)
 	return atmel_mci_init((void *)ATMEL_BASE_MCI);
 }
 
-int board_mmc_getcd(u8 *cd, struct mmc *mmc)
+int board_mmc_getcd(struct mmc *mmc)
 {
-	*cd = at91_get_pio_value(CONFIG_SYS_MMC_CD_PIN) ? 1 : 0;
-	return 0;
+	return !at91_get_pio_value(CONFIG_SYS_MMC_CD_PIN);
 }
 #endif
 
-- 
1.7.4.1

[U-Boot] [PATCH] Ethernut 5: fix build error for board_mmc_getcd()

[Thierry Reding]

* Prabhakar Lad wrote:
> Fix build error for ethernut5 and mx6qarm2 board due
> to prototype change for function  board_mmc_getcd()
> ethernut5.c:238: error: conflicting types for 'board_mmc_getcd'
> u-boot/include/mmc.h:318: note: previous declaration of 'board_mmc_getcd' was here
> make[2]: *** [ethernut5.o] Error 1
> 
> Signed-off-by: Prabhakar Lad <prabhakar.csengg@gmail.com>
> ---
>  board/egnite/ethernut5/ethernut5.c |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/board/egnite/ethernut5/ethernut5.c b/board/egnite/ethernut5/ethernut5.c
> index e42e91e..fd021a3 100644
> --- a/board/egnite/ethernut5/ethernut5.c
> +++ b/board/egnite/ethernut5/ethernut5.c
> @@ -235,10 +235,9 @@ int board_mmc_init(bd_t *bd)
>  	return atmel_mci_init((void *)ATMEL_BASE_MCI);
>  }
>  
> -int board_mmc_getcd(u8 *cd, struct mmc *mmc)
> +int board_mmc_getcd(struct mmc *mmc)
>  {
> -	*cd = at91_get_pio_value(CONFIG_SYS_MMC_CD_PIN) ? 1 : 0;
> -	return 0;
> +	return !at91_get_pio_value(CONFIG_SYS_MMC_CD_PIN);
>  }
>  #endif

Reviewed-by: Thierry Reding <thierry.reding@avionic-design.de>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120206/953a038a/attachment.pgp>

[U-Boot] [PATCH] Ethernut 5: fix build error for board_mmc_getcd()

[Albert ARIBAUD]

Hi Prabhakar,

Le 06/02/2012 13:49, Prabhakar Lad a ?crit :
> Fix build error for ethernut5 and mx6qarm2 board due
> to prototype change for function  board_mmc_getcd()
> ethernut5.c:238: error: conflicting types for 'board_mmc_getcd'
> u-boot/include/mmc.h:318: note: previous declaration of 'board_mmc_getcd' was here
> make[2]: *** [ethernut5.o] Error 1
>
> Signed-off-by: Prabhakar Lad<prabhakar.csengg@gmail.com>
> ---
>   board/egnite/ethernut5/ethernut5.c |    5 ++---
>   1 files changed, 2 insertions(+), 3 deletions(-)

This patch touches only Ethernut 5, and mx6qarm2 currently builds clean. 
Can you please elaborate on what your issue is with mx6qarm2 and 
resubmit with a corrected summary?

Amicalement,
-- 
Albert.

[U-Boot] [PATCH] Ethernut 5: fix build error for board_mmc_getcd()

[Prabhakar Lad]

Albert,

On Sun, Feb 19, 2012 at 3:19 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net>wrote:

> Hi Prabhakar,
>
> Le 06/02/2012 13:49, Prabhakar Lad a ?crit :
>
>  Fix build error for ethernut5 and mx6qarm2 board due
>> to prototype change for function  board_mmc_getcd()
>> ethernut5.c:238: error: conflicting types for 'board_mmc_getcd'
>> u-boot/include/mmc.h:318: note: previous declaration of 'board_mmc_getcd'
>> was here
>> make[2]: *** [ethernut5.o] Error 1
>>
>> Signed-off-by: Prabhakar Lad<prabhakar.csengg@gmail.com**>
>> ---
>>  board/egnite/ethernut5/**ethernut5.c |    5 ++---
>>  1 files changed, 2 insertions(+), 3 deletions(-)
>>
>
> This patch touches only Ethernut 5, and mx6qarm2 currently builds clean.
> Can you please elaborate on what your issue is with mx6qarm2 and resubmit
> with a corrected summary?
>
>    That was a typo, this patch only touches to Ethernut 5, I Had also
submitted
   a patch for mx6qarm2(Link <http://patchwork.ozlabs.org/patch/139726/>)
which had a same issue but Dirk had already submitted on it,
  i'll resubmit this patch with the correct summary.

WBR,
--Prabhakar Lad

Amicalement,
> --
> Albert.
>