On 17/11/09 23:10, Wolfgang Denk wrote: > Dear "Paulraj, Sandeep", > > In message <0554bef07d437848af01b9c9b5f0bc5d93f2f...@dlee01.ent.ti.com> you > wrote: >> >>> >>> Provides initial support for TI OMAP-L1x/DA8xx SoC devices. >>> See http://www.ti.com >>> >>> The DA8xx devices are similar to DaVinci devices but have a differing >>> memory map and updated peripheral versions. >>> >>> Signed-off-by: Nick Thompson <nick.thomp...@gefanuc.com> >>> Signed-off-by: Sekhar Nori <nsek...@ti.com> >>> --- >>> Applies to u-boot-ti >>> >>> include/asm-arm/arch-davinci/hardware.h | 237 >>> +++++++++++++++++++++++++++++++ >>> include/asm-arm/arch-davinci/i2c_defs.h | 5 + >>> 2 files changed, 242 insertions(+), 0 deletions(-) >> >> >> >> Pushed to u-boot-ti/master >> >> http://git.denx.de/?p=u-boot/u-boot-ti.git;a=commit;h=32c39e2da8483093dea29858a234ab2ec54e75bd >> >> modified the patch header a little. > > Argh... > > Please do not push stuff to the master branch of the poublic > repository so quickly. Please allow for a few days reviewing first. > > I just NAKed this patch.
Wolfgang, I'm happy to submit a new patch to lowercase these enums if you think that would be a good idea. It might be inviting trouble, but I'm not aware of any outstanding patches that rely on these values directly. I'm not thinking of process here, but we can catch if before it propagates anywhere. Nick. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
