Wolfgang Denk wrote:
Dear Claudio Mignanti,
In message AANLkTikkNw19U8yiQokqyE-WxjuO4b8RwqO-IBPa18W3@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