On Wed, Aug 24, 2011 at 6:43 AM, Luca Ceresoli <luca.ceres...@comelit.it> wrote:
> Tom Rini wrote:
>> Switch from the legacy omap3 mmc driver to the new generic omap hsmmc
>> driver. This patch is based on the work done for Beagle, etc.
>>
>> Signed-off-by: Tom Rini<tr...@ti.com>
>
> Again, please Cc: the board maintainers in the future, thanks.
Oh, sorry, used to other lists where people complain about too many CCs.
[snip]
>> diff --git a/include/configs/dig297.h b/include/configs/dig297.h
>> index b68f073..0e05df3 100644
>> --- a/include/configs/dig297.h
>> +++ b/include/configs/dig297.h
>> @@ -97,8 +97,9 @@
>> #define CONFIG_BAUDRATE 115200
>> #define CONFIG_SYS_BAUDRATE_TABLE {4800, 9600, 19200, 38400, 57600,\
>> 115200}
>> -#define CONFIG_MMC
>> -#define CONFIG_OMAP3_MMC
>> +#define CONFIG_GENERIC_MMC 1
>> +#define CONFIG_MMC 1
>> +#define CONFIG_OMAP_HSMMC 1
>
> IIRC the coding style says you should define the y/n constants without
> a value, e.g.:
> #define CONFIG_GENERIC_MMC
> not:
> #define CONFIG_GENERIC_MMC 1
>
> Apart for this:
> Tested-by: Luca Ceresoli <luca.ceres...@comelit.it>
OK, thanks, I'll respin this patch (all of the other boards were doing
the define CONFIG_FOO 1 everywhere) shortly.
--
Tom
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
