public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Nick Thompson <nick.thompson@gefanuc.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V5 1/4] add TI DA8xx support: DA8xx includes
Date: Wed, 18 Nov 2009 09:23:24 +0000	[thread overview]
Message-ID: <4B03BD0C.7080803@gefanuc.com> (raw)
In-Reply-To: <20091117231047.75FF3F51B07@gemini.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.

  parent reply	other threads:[~2009-11-18  9:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-10 15:09 [U-Boot] [PATCH V5 1/4] add TI DA8xx support: DA8xx includes Nick Thompson
2009-11-12 16:33 ` Paulraj, Sandeep
2009-11-17 23:10   ` Wolfgang Denk
2009-11-17 23:14     ` Paulraj, Sandeep
2009-11-18  9:23     ` Nick Thompson [this message]
2009-11-18 22:42       ` Wolfgang Denk
2009-11-17 23:09 ` Wolfgang Denk
2009-11-18 17:49   ` Scott Wood

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B03BD0C.7080803@gefanuc.com \
    --to=nick.thompson@gefanuc.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox