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
next prev parent 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