Re: [U-Boot] [PATCH][v2] mpc8260: move FDT memory node fixup into common CPU code.
Hello Marcel, Marcel Ziswiler wrote: Move the memory node fixup of the MPC8260ADS, ids8247, mgcoge and muas3001 boards into common mpc8260 CPU code. Signed-off-by: Marcel Ziswiler marcel.ziswi...@noser.com --- board/freescale/mpc8260ads/mpc8260ads.c | 13 - board/ids8247/ids8247.c | 16 board/keymile/mgcoge/mgcoge.c |8 +--- board/muas3001/muas3001.c | 16 cpu/mpc8260/cpu.c |1 + 5 files changed, 2 insertions(+), 52 deletions(-) Tested on the mgcoge board. (ids8247 and muas3001 I have no access anymore) Tested-by: Heiko Schocher h...@denx.de Thanks bye Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH][v2] mpc8260: move FDT memory node fixup into common CPU code.
Dear Marcel Ziswiler, In message 1252423922.5386.17.ca...@com-21 you wrote: Move the memory node fixup of the MPC8260ADS, ids8247, mgcoge and muas3001 boards into common mpc8260 CPU code. In which way is this patch version 2? I have not seen any v1 of such a patch? Signed-off-by: Marcel Ziswiler marcel.ziswi...@noser.com --- board/freescale/mpc8260ads/mpc8260ads.c | 13 - board/ids8247/ids8247.c | 16 board/keymile/mgcoge/mgcoge.c |8 +--- board/muas3001/muas3001.c | 16 cpu/mpc8260/cpu.c |1 + You want to put the respective board maintainers on Cc: Also, what is the impact for the other MPC826x board, that are not listed here? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de No one can guarantee the actions of another. -- Spock, Day of the Dove, stardate unknown ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH][v2] mpc8260: move FDT memory node fixup into common CPU code.
Dear Marcel Ziswiler, In message 1252440603.5386.40.ca...@com-21 you wrote: On Tue, 2009-09-08 at 20:25 +0200, Wolfgang Denk wrote: In which way is this patch version 2? I have not seen any v1 of such a patch? What about http://article.gmane.org/gmane.comp.boot-loaders.u-boot/67275? A pair of glasses might help (;-p). Heh. If that was sufficient. I'm alrady wearing glasses _and_ contact lenses, and it didn't help. Um... but why didn't you set the In-Reply-To: and References: headers, then? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de 2000 pounds of chinese soup = 1 Won Ton ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH][v2] mpc8260: move FDT memory node fixup into common CPU code.
Dear Wolfgang Denk On Tue, 2009-09-08 at 22:44 +0200, Wolfgang Denk wrote: Heh. If that was sufficient. I'm alrady wearing glasses _and_ contact lenses, and it didn't help. I do know that as I saw you at the EW in Nürnberg this spring. Um... but why didn't you set the In-Reply-To: and References: headers, then? Sorry about that, my understanding was to post new versions of a patch as a new thread. Is that not so? What exactly is that references header you are talking about? Cheers Marcel Ziswiler ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH][v2] mpc8260: move FDT memory node fixup into common CPU code.
On Tue, 2009-09-08 at 23:19 +0200, Wolfgang Denk wrote: No, on contrary. New versions of a patch should always make sure to thread correctly to the old version(s). It seems that preferences here are a matter of taste - my personal preference is to link to the preceeding version of the patch, but most others seem to prefer to link to the first version always. That's kind of what I thought at first as well, but then I saw many others posting patches not adhering to that and still getting your ack no problem: http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/67581 http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/67462 http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/67394 http://article.gmane.org/gmane.comp.boot-loaders.u-boot/67187 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot