Hi Ahmad. > >> @@ -1,4 +1,5 @@ > >> obj-y += setup.o > >> +pbl-y += at91_pmc_ll.o > > > > Will this always make my pbl image larger or do we have some linker > > magic that throws away the unused code? > > My understanding is that barebox instructs GCC to place each out-of-line function > into a separate section (with -ffunction-sections) and then the linker prunes all > unreferenced sections. > > > If the pbl image become larger then we need to add this in a conditional manner > > as we cannot allow this for all at91 variants. > > I used to have conditional function definitions in a prior version (albeit > for the SDRAM variants initialization routines) and Sascha commented that > the #ifdefery can be removed as the linker takes care of it. I should have known already that we used --function-sections as I worked with this in the kernel in my former life. Much better than ifdeffery all over the code or in Makefiles. Sam _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox