Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Becky Bruce
On Nov 6, 2008, at 2:04 PM, Jon Loeliger wrote: > On Wed, 2008-11-05 at 14:55 -0600, Becky Bruce wrote: >> The memory map on the 8641hpcn is modified to look more like >> the 85xx boards; this is a step towards a more standardized >> layout going forward. As part of this change, we now relocate >

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Becky Bruce
On Nov 6, 2008, at 4:30 PM, Jon Loeliger wrote: > On Thu, 2008-11-06 at 16:16 -0600, Scott Wood wrote: >> Becky Bruce wrote: >>> We *do* need a comment in the release notes for this revision of u- >>> boot >>> that the map for 8641 has changed so it doesn't catch anyone by >>> surprise. >> >>

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Jon Loeliger
On Thu, 2008-11-06 at 16:16 -0600, Scott Wood wrote: > Becky Bruce wrote: > > We *do* need a comment in the release notes for this revision of u-boot > > that the map for 8641 has changed so it doesn't catch anyone by surprise. > > How about a runtime check in the board fdt code to print a warnin

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Scott Wood
Becky Bruce wrote: > On Nov 6, 2008, at 4:16 PM, Scott Wood wrote: >> How about a runtime check in the board fdt code to print a warning if >> it finds an old device tree? Unfortunately, using new dts with old >> u-boot isn't as easy to detect. > > Where exactly were you thinking this would go?

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Becky Bruce
On Nov 6, 2008, at 4:16 PM, Scott Wood wrote: > Becky Bruce wrote: >> We *do* need a comment in the release notes for this revision of u- >> boot that the map for 8641 has changed so it doesn't catch anyone >> by surprise. > > How about a runtime check in the board fdt code to print a warning

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Scott Wood
Becky Bruce wrote: > We *do* need a comment in the release notes for this revision of u-boot > that the map for 8641 has changed so it doesn't catch anyone by surprise. How about a runtime check in the board fdt code to print a warning if it finds an old device tree? Unfortunately, using new dt

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Becky Bruce
On Nov 6, 2008, at 2:04 PM, Jon Loeliger wrote: > On Wed, 2008-11-05 at 14:55 -0600, Becky Bruce wrote: >> The memory map on the 8641hpcn is modified to look more like >> the 85xx boards; this is a step towards a more standardized >> layout going forward. As part of this change, we now relocate >

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Becky Bruce
On Nov 6, 2008, at 1:28 PM, Scott Wood wrote: > Becky Bruce wrote: >> The memory map on the 8641hpcn is modified to look more like >> the 85xx boards; this is a step towards a more standardized >> layout going forward. As part of this change, we now relocate >> the flash. >> The regions for some

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Jon Loeliger
On Wed, 2008-11-05 at 14:55 -0600, Becky Bruce wrote: > The memory map on the 8641hpcn is modified to look more like > the 85xx boards; this is a step towards a more standardized > layout going forward. As part of this change, we now relocate > the flash. > > The regions for some of the mappings w

Re: [U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-06 Thread Scott Wood
Becky Bruce wrote: > The memory map on the 8641hpcn is modified to look more like > the 85xx boards; this is a step towards a more standardized > layout going forward. As part of this change, we now relocate > the flash. > > The regions for some of the mappings were far larger than they > needed t

[U-Boot] [PATCH 8/9] mpc8641: Change 32-bit memory map

2008-11-05 Thread Becky Bruce
The memory map on the 8641hpcn is modified to look more like the 85xx boards; this is a step towards a more standardized layout going forward. As part of this change, we now relocate the flash. The regions for some of the mappings were far larger than they needed to be. I have reduced the mapping