public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] Ethernut 5: fix build error
@ 2012-02-23 11:53 Prabhakar Lad
  2012-02-29  0:07 ` Albert ARIBAUD
  0 siblings, 1 reply; 2+ messages in thread
From: Prabhakar Lad @ 2012-02-23 11:53 UTC (permalink / raw)
  To: u-boot

Fix build error for ethernut5 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>
Cc: Tim Schendekehl <tim.schendekehl@egnite.de>
Cc: Thierry Reding <thierry.reding@avionic-design.de>
Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>
Cc: egnite GmbH <info@egnite.de>
---
 Changes for v2:
  Description change.

 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

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [U-Boot] [PATCH v2] Ethernut 5: fix build error
  2012-02-23 11:53 [U-Boot] [PATCH v2] Ethernut 5: fix build error Prabhakar Lad
@ 2012-02-29  0:07 ` Albert ARIBAUD
  0 siblings, 0 replies; 2+ messages in thread
From: Albert ARIBAUD @ 2012-02-29  0:07 UTC (permalink / raw)
  To: u-boot

Hi Prabhakar,

Le 23/02/2012 12:53, Prabhakar Lad a ?crit :
> Fix build error for ethernut5 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>
> Cc: Tim Schendekehl<tim.schendekehl@egnite.de>
> Cc: Thierry Reding<thierry.reding@avionic-design.de>
> Cc: Albert ARIBAUD<albert.u.boot@aribaud.net>
> Cc: egnite GmbH<info@egnite.de>
> ---
>   Changes for v2:
>    Description change.
>
>   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

Applied to u-boot-arm/master as a bugfix, thanks!

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-02-29  0:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-23 11:53 [U-Boot] [PATCH v2] Ethernut 5: fix build error Prabhakar Lad
2012-02-29  0:07 ` Albert ARIBAUD

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox