Messages in this thread | | | Date | Tue, 02 Jun 2009 10:49:12 +0200 | From | Nicolas Ferre <> | Subject | Re: [PATCH] New AT91 MCI Driver that supports both MCI slots used at the same time |
| |
Rob Emanuele : > Greetings, > > This patch creates a new AT91 Multimedia Card Interface (MCI) driver > that supports using both MCI slots at the same time. I'm looking for > others to test this patch on other boards within the same family of > chips. > > This driver is a port the Atmel AVR32 MCI driver which uses similar silicon.
I am very interested with this work. But, you will have to tell us what is the precise relation between this code and the Atmel AVR32 MCI driver. Why did you rename all function and variables to at91... instead of trying to modify the atmel-mci driver ? Was the needed behavior so far from the original code that you decided to create another completely new one ? Please enlighten us.
Moreover, it seems that in your code, DMA is not supported so atmel-mci without the DMA option enabled should be working without modification at all (am I missing something ?).
> In addition it modifies the AT91SAM9G20-EK's board platform config to > support having MCI Slot A hooked to an SD Slot. This was the board > that this was developed on. To support MCI Slot A it was needed to not > enable the LEDs on the AT91SAM9G20-EK board if SD/MMC Slot A is > enabled. The Slot A pins overlap the LEDs on Rev A and B boards.
This is a different board, so you will have to create a different board source file (board-my9g20customboard.c) or find a clever way of sharing code (true, can be preferable as only a few lines are changing).
Best regards, -- Nicolas Ferre
| |