[U-Boot] [PATCH] Fix typo inside atmel_mci driver
Reinhard Meyer
reinhard.meyer at emk-elektronik.de
Sat Aug 7 01:40:48 CEST 2010
Wolfgang Denk wrote:
> Dear Claudio Mignanti,
>
> In message <AANLkTikkNw19U8yiQokqyE-WxjuO4b8RwqO-IBPa18W3 at mail.gmail.com> you wrote:
>
>> The function get_mci_clk_rate is used inside atmel_mci driver
>> but never defined. Fix this typo.
>>
Thats NOT a typo. The current atmel_mci.c is for AVR32 ONLY.
>
> Well, it is certainly defined in
> arch/avr32/include/asm/arch-at32ap700x/clk.h:
>
> 74 #ifdef AT32AP700x_CHIP_HAS_MMCI
> 75 static inline unsigned long get_mci_clk_rate(void)
> 76 {
> 77 return get_pbb_clk_rate();
> 78 }
> 79 #endif
>
>
>> --- a/drivers/mmc/atmel_mci.c
>> +++ b/drivers/mmc/atmel_mci.c
>> @@ -67,7 +67,7 @@ static void mci_set_mode(unsigned long hz, unsigned
>> long blklen)
>> unsigned long bus_hz;
>> unsigned long clkdiv;
>>
>> - bus_hz = get_mci_clk_rate();
>> + bus_hz = get_mck_clk_rate();
>>
>
> Are you sure this patch is working and does not break anything?
>
It will break on the AVR32 architecture.
Besides, the current atmel_mci.c driver WILL NOT work on the AT91SAM9.
My patch from 07-Jul-10 fixes all issues and makes it work on AT91SAM9.
Reinhard
More information about the U-Boot
mailing list