public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] update config for mvBL-M7 (MPC8343)
Date: Fri, 17 Jul 2009 16:32:03 +0200	[thread overview]
Message-ID: <4A608B63.1060901@denx.de> (raw)
In-Reply-To: <1247840636.3953.78.camel@swa-m460>

Hello Andr?,

Andr? Schwarz wrote:
> Heiko,
> 
> this patch is no longer valid since I already posted v2.

Ah, ok.

> But your comments are correct ...
> 
> Will wait for WD's comments and send v3.

OK, one more minor comment ...

> On Fri, 2009-07-17 at 16:15 +0200, Heiko Schocher wrote:
>> Hello Andr?,
>>
>> Andr? Schwarz wrote:
>>> This patch fixes DDR-II settings and adds I2C support.
>>>
>>>
>>> Signed-off-by: Andr? Schwarz <andre.schwarz@matrix-vision.de>
>>> ---
>>>
>>>  include/configs/MVBLM7.h |   48
>>> ++++++++++++++++++++-------------------------
>>>  1 files changed, 21 insertions(+), 27 deletions(-)
>>>
>>> diff --git a/include/configs/MVBLM7.h b/include/configs/MVBLM7.h
>>> index ac8cb57..6e347e8 100644
>>> --- a/include/configs/MVBLM7.h
>>> +++ b/include/configs/MVBLM7.h
>>> @@ -47,12 +47,14 @@
>>>  #define CONFIG_MPC8XXX_SPI
>>>  #define CONFIG_HARD_SPI
>>>  #define MVBLM7_MMC_CS   0x04000000
>>> +#define CONFIG_MISC_INIT_R
>> So misc_init_r () gets called, but I didn;t find this in your
>> patch? May I miss something ...
> 
> It's included in the other "common code" patch.

Hmm... but this will break git-bisect compatibility, and if we
have such a tool for searching bugs, we should make patches which
are "git bisect compatible" ... so please look, that patches compile
clean ...

bye
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

  reply	other threads:[~2009-07-17 14:32 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-17 13:10 [U-Boot] [PATCH 1/2] update config for mvBL-M7 (MPC8343) André Schwarz
2009-07-17 14:15 ` Heiko Schocher
2009-07-17 14:23   ` André Schwarz
2009-07-17 14:32     ` Heiko Schocher [this message]
2009-07-17 14:40       ` André Schwarz
2009-07-17 14:44     ` Wolfgang Denk
2009-07-17 14:54       ` André Schwarz
2009-07-17 17:33         ` Kim Phillips

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=4A608B63.1060901@denx.de \
    --to=hs@denx.de \
    --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