Dear Stefan,
In message 8ddb672c-956c-68d2-ceb4-d77f96cdf4e4@denx.de you wrote:
Especially the default value of 4 makes no sense to me - whiy is this not 1?
I can't really tell. I can only assume, that it originates from this patch:
Yes, probably.
I think the issue here resutlsfromt he fact that those who worked on the patches came primarily from architectures where this array of banks has been used; for Power Architecture systems this has (for a long, long time) not been the case - there we would map several memory banks (after determining their respecitve sizes) into one contiguous area of memory starting at address 0.
So on all such system the definition of this array meand code and data overhead, as it is not needed.
...thus futher hiding where we just lost another lof of memory, for no advantage.
Sic...
I'm not sure, what you mean with "lost lot of memory"? We carried referenced code for ~2 years, which this patch series now tries to solve.
See above. On most PPC systems this array of sizes and addresses is not needed at all, and _if_ code is changed to use such an array, it should not waste even more memory be defining room for 4 banks where there usually will be only one.
Sorry, I know that you are not to blame for that...
Best regards,
Wolfgang Denk