[U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Haiying.Wang
This patchset adds support for TPL(Tertiary Program Loader) and P1021MDS board. It is a rework of patchset at http://lists.denx.de/pipermail/u-boot/2010-December/082881.html, addresses the comments from the list and is based on the top of the tree. It needs to be applied after patch

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Scott Wood
On Mon, 31 Jan 2011 20:39:51 +0100 Wolfgang Denk w...@denx.de wrote: Dear haiying.w...@freescale.com, In message 1296499317-26616-1-git-send-email-haiying.w...@freescale.com you wrote: This patchset adds support for TPL(Tertiary Program Loader) and P1021MDS board. It is a rework of

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Wolfgang Denk
Dear Scott Wood, In message 20110131141332.5a4a2...@udp111988uds.am.freescale.net you wrote: I think these patches are incorrectly split. I think the intent was to split the arch-neutral stuff from the 85xx stuff from the board stuff -- you'd rather they be all bunched together? No, of

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Wolfgang Denk
Dear Scott Wood, In message 20110131143141.2959d...@udp111988uds.am.freescale.net you wrote: I'm confused. You say of course not all together, but the first one you say to include with the second, and the second you say to include with the third. I did not say this. If you're suggesting

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Wolfgang Denk
Dear Kumar Gala, In message c6334d93-a826-4c68-9477-2baeee681...@kernel.crashing.org you wrote: ... I'm in agreement with Scott on this. I believe we've taken this a bit too far about dead code. It should be reasonable in a patch series to have code that will be used in a subsequent patch.

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Scott Wood
On Mon, 31 Jan 2011 21:50:57 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20110131143141.2959d...@udp111988uds.am.freescale.net you wrote: I'm confused. You say of course not all together, but the first one you say to include with the second, and the second you

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Scott Wood
On Mon, 31 Jan 2011 22:34:34 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20110131151506.700dd...@udp111988uds.am.freescale.net you wrote: For example, why must we add the Makefile changes in the first step, when all the code it references is still missing?

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Wolfgang Denk
Dear Scott Wood, In message 20110131160713.0b78c...@udp111988uds.am.freescale.net you wrote: Do you want the README changes to be a separate patch from the board/makefile changes? Did you not just explain that this would make no sense? Best regards, Wolfgang Denk -- DENX Software

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Scott Wood
On Mon, 31 Jan 2011 23:40:41 +0100 Wolfgang Denk w...@denx.de wrote: Dear Scott Wood, In message 20110131160713.0b78c...@udp111988uds.am.freescale.net you wrote: Do you want the README changes to be a separate patch from the board/makefile changes? Did you not just explain that this

Re: [U-Boot] [PATCH0/6] patchset to support TPL and P1021MDS board

2011-01-31 Thread Kumar Gala
I've published some related cleanup patches and push those patches into u-boot-85xx.git 'dev' branch. You should be able to: * drop board_lmb_reserve() * remove config.mk and CONFIG_SYS_LDSCRIPT in config.h * remove fsl_ddr_get_mem_data_rate(), fsl_ddr_get_spd() [need to rename