[U-Boot-Users] [PATCH] Correction to previous patch of 26th Sept which broke other boards
Correction to previous patch of 26th Sept which broke other boards - apologies to anyone affected.
I was so embarassed I forgot to attach the patch - here it is.
This patch replaces the arm920t.patch submitted on 26th Sept.
The patch was produced using cg-diff against branch based on commit fb34a9a20132216a47c9c91551cec3d7b4ebc0a3 of http://source.denx.net/git/u-boot.git
CHANGELOG * Ensure Integrator boards call platformsetup() rather than lowlevel_init() * Patch by Peter Pearse 29 Sept 2005
Regards
Peter Pearse
In message 89A528FE6DB0FA44877BB2F05B84671803233939@ZIPPY.Emea.Arm.com you wrote:
CHANGELOG
- Ensure Integrator boards call platformsetup() rather than
lowlevel_init()
- Patch by Peter Pearse 29 Sept 2005
Just for my understanding: is this patch related to the list of the following patches you submitted earlier, and if so, how (i. e. shall these older patches still be applied, or not) ?
4278 04/11 "Peter Pearse" [U-Boot-Users] [Patch] * Patch by Peter Pearse, 11 Apr 2005 - add core module targets for integratorap, integratorcp - add ARM946E cpu support - remap SSRAM memory to 0x00000000
4292 04/11 "Peter Pearse" [U-Boot-Users] [Patch] Multicore patch for ARM supplied development boards. * Patch by Peter Pearse, 12 Apr 2005 - add core module targets for integratorap, integratorcp - add ARM946E cpu support - remap SSRAM memory to 0x00000000
4300 04/12 "Peter Pearse" [U-Boot-Users] [Patch] CFI flash for ARM supplied & supported development boards * Patch by Peter Pearse, 12 Apr 2005 Changes all ARM supplied development boards, supported by ARM, to use CFI flash. Adds CFI support for the Versatile/AB926EJ-S.
5542 06/30 "Peter Pearse" [U-Boot-Users] [PATCH] Differing gcc version ARM options * Patch by Peter Pearse, 30th June 2005 - Import cc-option shell function from kernel - Use it to get the correct ARM gcc options for individual cpus
My understanding is that 4292 obsoletes 4278 anyway, but how is the rest (4292, 4300, 5542) related yto your latest patch?
Best regards,
Wolfgang Denk
In message 89A528FE6DB0FA44877BB2F05B84671803233939@ZIPPY.Emea.Arm.com you wrote:
CHANGELOG
- Ensure Integrator boards call platformsetup() rather than
lowlevel_init()
- Patch by Peter Pearse 29 Sept 2005
I reject this patch. I don't want to see any (new) board-dependent code in cpu/arm920t/start.S
Is there any specific reaso why the same (?) function is called lowlevel_init() for some processors but platformsetup() for others?
I think *this* should be cleaned up instead of using both names for one processor.
Best regards,
Wolfgang Denk
participants (2)
-
Peter Pearse -
Wolfgang Denk