* [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller
@ 2008-08-27 23:39 Prodyut Hazarika
2008-08-29 8:05 ` Stefan Roese
0 siblings, 1 reply; 6+ messages in thread
From: Prodyut Hazarika @ 2008-08-27 23:39 UTC (permalink / raw)
To: u-boot
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller
2008-08-27 23:39 [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller Prodyut Hazarika
@ 2008-08-29 8:05 ` Stefan Roese
2008-08-29 13:51 ` Prodyut Hazarika
0 siblings, 1 reply; 6+ messages in thread
From: Stefan Roese @ 2008-08-29 8:05 UTC (permalink / raw)
To: u-boot
On Thursday 28 August 2008, Prodyut Hazarika wrote:
> >From bdc99201a532b040b3d346fca5acff8332d670e4 Mon Sep 17 00:00:00 2001
>
> From: Prodyut Hazarika <phazarika@amcc.com>
> Date: Wed, 27 Aug 2008 16:26:36 -0700
> Subject: [PATCH] Optimizations/Cleanups for IBM DDR2 memory controller
> Removed Magic numbers from Initialization preload registers
> Tested with Kilauea, Glacier, Canyonlands and Katmai boards
> About 5-7% improvement seen for LMBench memtests
> Signed-off-by: Prodyut Hazarika <phazarika@amcc.com>
Applied to u-boot-ppc4xx repository.
Somehow your patch was malformatted though. Please see the ">From bdc..."
above. This should not be included in the patch mail. Did you create this
patch with git format-patch and send it with git send-email?
And please don't indent the commit text and add one empty line before the
s-o-f line.
I fixed this manually now but please take more care next time.
Thanks.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller
2008-08-29 8:05 ` Stefan Roese
@ 2008-08-29 13:51 ` Prodyut Hazarika
2008-08-30 9:22 ` Stefan Roese
0 siblings, 1 reply; 6+ messages in thread
From: Prodyut Hazarika @ 2008-08-29 13:51 UTC (permalink / raw)
To: u-boot
Hi Stefan,
Thanks.
> Somehow your patch was malformatted though. Please see the ">From bdc..."
> above. This should not be included in the patch mail. Did you create this
> patch with git format-patch and send it with git send-email?
I created the patch using git format-patch. I had applied the patch using
git-am in my local repo before sending, and it applied fine. I used Evolution on Linux
to send the patch using the instructions given in Documentation/email-clients
in Linux kernel tree. I have problems using git send-email because we send
emails using the Exchange server.
> And please don't indent the commit text and add one empty line before the
> s-o-f line.
I will take care of the that.
Regards,
Prodyut Hazarika
==================
Staff S/W Engineer
AMCC
==================
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller
2008-08-29 13:51 ` Prodyut Hazarika
@ 2008-08-30 9:22 ` Stefan Roese
2008-09-02 20:46 ` Jon Loeliger
0 siblings, 1 reply; 6+ messages in thread
From: Stefan Roese @ 2008-08-30 9:22 UTC (permalink / raw)
To: u-boot
Prodyut,
On Friday 29 August 2008, Prodyut Hazarika wrote:
> > Somehow your patch was malformatted though. Please see the ">From bdc..."
> > above. This should not be included in the patch mail. Did you create this
> > patch with git format-patch and send it with git send-email?
>
> I created the patch using git format-patch. I had applied the patch using
> git-am in my local repo before sending, and it applied fine.
So far so good. :)
> I used
> Evolution on Linux to send the patch using the instructions given in
> Documentation/email-clients in Linux kernel tree. I have problems using git
> send-email because we send emails using the Exchange server.
OK, then this Evolution step seems to have introduced this problem. Too bad
that even the instructions in the Linux Documention describe this correctly.
Sorry, I can't help you here, I have no experience with Evolution. Perhaps
somebody else can jump in here.
> > And please don't indent the commit text and add one empty line before the
> > s-o-f line.
>
> I will take care of the that.
Thanks.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller
2008-08-30 9:22 ` Stefan Roese
@ 2008-09-02 20:46 ` Jon Loeliger
2008-09-02 22:38 ` [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 MemoryController Prodyut Hazarika
0 siblings, 1 reply; 6+ messages in thread
From: Jon Loeliger @ 2008-09-02 20:46 UTC (permalink / raw)
To: u-boot
On Sat, 2008-08-30 at 11:22 +0200, Stefan Roese wrote:
> > I used
> > Evolution on Linux to send the patch using the instructions given in
> > Documentation/email-clients in Linux kernel tree. I have problems using git
> > send-email because we send emails using the Exchange server.
>
> OK, then this Evolution step seems to have introduced this problem. Too bad
> that even the instructions in the Linux Documention describe this correctly.
> Sorry, I can't help you here, I have no experience with Evolution. Perhaps
> somebody else can jump in here.
Just make sure to select "preformat" instead of "normal"
before you "insert" -> text file" on your patch.
jdl
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2008-09-02 22:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-27 23:39 [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 Memory Controller Prodyut Hazarika
2008-08-29 8:05 ` Stefan Roese
2008-08-29 13:51 ` Prodyut Hazarika
2008-08-30 9:22 ` Stefan Roese
2008-09-02 20:46 ` Jon Loeliger
2008-09-02 22:38 ` [U-Boot] ppc4xx: Optimizations/Cleanups for IBM DDR2 MemoryController Prodyut Hazarika
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox