On 09:43 Mon 06 Jun , Hubert Feurstein wrote: > 2011/6/2 Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx>: > > On 13:48 Wed 01 Jun , Hubert Feurstein wrote: > [snip] > > I've test it on at91sam9263ek and work fine > Great, good to hear ;) I'll add also support for at91sam9m10g45ek > later in an extra commit. > > > > please fix the following comments > > Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx> > > > [snip] > >> +/* Multimedia Card Interface */ > >> +struct atmel_mci_platform_data { > >> + unsigned bus_width; > >> + unsigned host_caps; /* MCI_MODE_* from mci.h */ > >> + unsigned detect_pin; > >> + unsigned wp_pin; > >> +}; > >we can have 2 slot but you allow only one > Would it be ok to add the second slot in an extra commit? ok Best Regards, J. _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox