Dear Stefan Roese,
In message 201009030946.22440.sr@denx.de you wrote:
IIRC, then the main reason for this is the code size reduction. Using your version would increase the code size quite a bit. But I concur, it would be much better readable. So just let me know, and I'll try to come up with a patch cleaning this up a bit.
Maybe some common parts (like "AMCC PowerPC ") could be split off.
For the rest, the compelex code needs also space, so I'm not sure about the end result.
As described in some comments, the problem is that some SoC's have the same PVR (for example 440EP rev C and 440GR rev B have same PVR). I have no idea how this identification could be solved without such ifdef's.
I see.
But you are right. Some of the ifdef's can be removed. Again, this would also result in a bigger image size though.
Really? We should measure the effect.
Best regards,
Wolfgang Denk