From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nick Thompson <nick.thompson@gefanuc.com>
Date: Wed, 18 Nov 2009 09:23:24 +0000
Subject: [U-Boot] [PATCH V5 1/4] add TI DA8xx support: DA8xx includes
In-Reply-To: <20091117231047.75FF3F51B07@gemini.denx.de>
References: <4AF9823C.8090803@gefanuc.com>
	<0554BEF07D437848AF01B9C9B5F0BC5D93F2F002@dlee01.ent.ti.com>
	<20091117231047.75FF3F51B07@gemini.denx.de>
Message-ID: <4B03BD0C.7080803@gefanuc.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 17/11/09 23:10, Wolfgang Denk wrote:
> Dear "Paulraj, Sandeep",
> 
> In message <0554BEF07D437848AF01B9C9B5F0BC5D93F2F002@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.thompson@gefanuc.com>
>>> Signed-off-by: Sekhar Nori <nsekhar@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.