Dear Grant, In message <fa686aa40912301457l704c324bk511a207d07218525@xxxxxxxxxxxxxx> you wrote: > > Unfortunately, the wrapper script is also being used to do things that > are completely unrelated to creating wrapper binaries. FIT images > (and uImages) don't use any of the wrapper bits at all. In fact, as > seen in this patch, generating them involves bailing out of the > wrapper script early to avoid linking the wrapper bits. I think for > all types of uImages, the wrapper script is being misused and I don't > like the extra complexity that it adds. Agreed. > Rather than adding new paths to arch/powerpc/boot/wrapper, I would > rather see a new script used for generating FIT image that isn't > complicated by all the current wrapper cruft. Also, the Makefile rule > doesn't need to depend on $(wrapperbits) which means faster build > times when only building uImages. > > Bonus points if you also convert the uImage target to use the new > script; but I'm not demanding that you do that yet. I think if this is handles in a separate script, the legacy uImage support must be handled in this new script, too. Otherwise it would be too difficult to understanmd the relation of the two image formats. > Finally, you need to add documentation about the new target to > Documentation/powerpc/bootwrapper.txt. Let's keep in mind that the uImage formats (both the old legacy and the new FIT format) are inherently architecture independent. We want to use this same mechanism for example on ARM, and on other architectures as well. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@xxxxxxx How many Unix hacks does it take to change a light bulb? Let's see, can you use a shell script for that or does it need a C program? -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
