[U-Boot] [PATCH v2] ARMV7: Vexpress: Add missing MMC header

[U-Boot] [PATCH v2] ARMV7: Vexpress: Add missing MMC header

[Dirk Behme]

From: Dirk Behme <dirk.behme@googlemail.com>

Add a header file with the missing function prototype to fix

ca9x4_ct_vxp.c: In function 'cpu_mmc_init':
ca9x4_ct_vxp.c:93: warning: implicit declaration of function 'arm_pl180_mmci_init'

introduced by commit "ARMV7: Vexpress: Add MMC support"
(f0c64526b7e51ba997a0f1baf9e74e6d497b957e)

Signed-off-by: Dirk Behme <dirk.behme@googlemail.com>
CC: Andy Fleming <afleming@freescale.com>
CC: Matt Waddel <matt.waddel@linaro.org>

---

Changes in v2: Switch angle braces to quotes.

Note: Maybe it would be better to move

int arm_pl180_mmci_init(void);

from drivers/mmc/arm_pl180_mmci.h to a header in the
include directory?

 board/armltd/vexpress/ca9x4_ct_vxp.c |    1 +
 1 file changed, 1 insertion(+)

Index: u-boot.git/board/armltd/vexpress/ca9x4_ct_vxp.c
===================================================================
--- u-boot.git.orig/board/armltd/vexpress/ca9x4_ct_vxp.c
+++ u-boot.git/board/armltd/vexpress/ca9x4_ct_vxp.c
@@ -38,6 +38,7 @@
 #include <asm/arch/systimer.h>
 #include <asm/arch/sysctrl.h>
 #include <asm/arch/wdt.h>
+#include "../drivers/mmc/arm_pl180_mmci.h"
 
 static ulong timestamp;
 static ulong lastdec;

[U-Boot] [PATCH v2] ARMV7: Vexpress: Add missing MMC header

[Albert ARIBAUD]

Le 23/05/2011 19:40, Dirk Behme a ?crit :
> From: Dirk Behme<dirk.behme@googlemail.com>
>
> Add a header file with the missing function prototype to fix
>
> ca9x4_ct_vxp.c: In function 'cpu_mmc_init':
> ca9x4_ct_vxp.c:93: warning: implicit declaration of function 'arm_pl180_mmci_init'
>
> introduced by commit "ARMV7: Vexpress: Add MMC support"
> (f0c64526b7e51ba997a0f1baf9e74e6d497b957e)
>
> Signed-off-by: Dirk Behme<dirk.behme@googlemail.com>
> CC: Andy Fleming<afleming@freescale.com>
> CC: Matt Waddel<matt.waddel@linaro.org>
>
> ---
>
> Changes in v2: Switch angle braces to quotes.

Applied to u-boot-arm/next, thanks -- that will go in master at next 
merge window.

Amicalement,
-- 
Albert.

[U-Boot] [PATCH v2] ARMV7: Vexpress: Add missing MMC header

[Dirk Behme]

On 31.05.2011 20:18, Albert ARIBAUD wrote:
> Le 23/05/2011 19:40, Dirk Behme a ?crit :
>> From: Dirk Behme<dirk.behme@googlemail.com>
>>
>> Add a header file with the missing function prototype to fix
>>
>> ca9x4_ct_vxp.c: In function 'cpu_mmc_init':
>> ca9x4_ct_vxp.c:93: warning: implicit declaration of function
>> 'arm_pl180_mmci_init'
>>
>> introduced by commit "ARMV7: Vexpress: Add MMC support"
>> (f0c64526b7e51ba997a0f1baf9e74e6d497b957e)
>>
>> Signed-off-by: Dirk Behme<dirk.behme@googlemail.com>
>> CC: Andy Fleming<afleming@freescale.com>
>> CC: Matt Waddel<matt.waddel@linaro.org>
>>
>> ---
>>
>> Changes in v2: Switch angle braces to quotes.
>
> Applied to u-boot-arm/next, thanks -- that will go in master at next
> merge window.

Thanks!

Hmm, but why next? This would mean we release v2011.06 with the above 
warning? I thought that fixing warnings introduced while the merge 
window is one of the things a rc cycle is good for?

Best regards

Dirk

[U-Boot] [PATCH v2] ARMV7: Vexpress: Add missing MMC header

[Albert ARIBAUD]

Hi Dirk,

Le 31/05/2011 20:41, Dirk Behme a ?crit :
> On 31.05.2011 20:18, Albert ARIBAUD wrote:
>> Le 23/05/2011 19:40, Dirk Behme a ?crit :
>>> From: Dirk Behme<dirk.behme@googlemail.com>
>>>
>>> Add a header file with the missing function prototype to fix
>>>
>>> ca9x4_ct_vxp.c: In function 'cpu_mmc_init':
>>> ca9x4_ct_vxp.c:93: warning: implicit declaration of function
>>> 'arm_pl180_mmci_init'
>>>
>>> introduced by commit "ARMV7: Vexpress: Add MMC support"
>>> (f0c64526b7e51ba997a0f1baf9e74e6d497b957e)
>>>
>>> Signed-off-by: Dirk Behme<dirk.behme@googlemail.com>
>>> CC: Andy Fleming<afleming@freescale.com>
>>> CC: Matt Waddel<matt.waddel@linaro.org>
>>>
>>> ---
>>>
>>> Changes in v2: Switch angle braces to quotes.
>>
>> Applied to u-boot-arm/next, thanks -- that will go in master at next
>> merge window.
>
> Thanks!
>
> Hmm, but why next? This would mean we release v2011.06 with the above
> warning? I thought that fixing warnings introduced while the merge
> window is one of the things a rc cycle is good for?

Hmm, you're right, I overlooked the date of the commit that caused the 
warning. Moving from next to master, then, with my apologies.

> Best regards
>
> Dirk

Amicalement,
-- 
Albert.