Le samedi 26 janvier 2008 à 01:01 +0100, Wolfgang Denk a écrit :
In message 1201288627.12296.4.camel@voyager.dsnet you wrote:
As I said in my original posting:
For size reasons, the patch has been hand edited, splitted in several chunks, and the AT91CAP9.h file (440 kB !) is not in the patch at all.
Will you pelase check this file? Only the insane include/asm-arm/mach-types.h has a similar size - al other files are usually much smaller.
Is this stuff really all needed?
This header file comes directly from Atmel: it contains the register definitions for _all_ the hardware on the AT91CAP9.
Of course, u-boot does only need a (small) subset of it. If you prefer, I can simplify it.
==> Which will make it harder to maintain. These files are automatically generated from the chip database, and most of the stuff can conceivably be used by U-Boot. -- Even if they are not used today.
If a bug is detected in the file, then the place to update is inside the database. Obviously ansy script generating the data, should be modified to generate something which can be directly used by U-Boot without any further modification.
It may make sense to split up the file as well, as Haavard proposes, but preferably, the split should be done at the source. I will forward this request to the AT91 & CAP teams
Thanks,
Stelian.