Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Tom Rini
On Thu, Dec 13, 2012 at 03:24:08PM +0100, Richard Genoud wrote: > 2012/12/13 Marek Vasut : > > Dear Tom Rini, > > > > [...] > > > >> Note that we don't use --gc-sections on all archs so I'm not sure we > >> discard the unused VFAT functions on say ARM. > > > > Valid point, Albert? > > > > Best rega

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Richard Genoud
2012/12/13 Marek Vasut : > Dear Tom Rini, > > [...] > >> Note that we don't use --gc-sections on all archs so I'm not sure we >> discard the unused VFAT functions on say ARM. > > Valid point, Albert? > > Best regards, > Marek Vasut I check and the code is discarded (on ARM at91sam9x5ek). This is t

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Marek Vasut
Dear Tom Rini, [...] > Note that we don't use --gc-sections on all archs so I'm not sure we > discard the unused VFAT functions on say ARM. Valid point, Albert? Best regards, Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.d

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Marek Vasut
Dear Richard Genoud, > 2012/12/13 Marek Vasut : > > Dear Richard Genoud, > > > >> 2012/12/13 Marek Vasut : > >> > Dear Richard Genoud, > >> > > >> >> ifdefs in the code are making it harder to read. > >> >> The use of simple if(VFAT_ENABLED) makes no more code and is cleaner. > >> >> (the code i

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Richard Genoud
2012/12/13 Marek Vasut : > Dear Richard Genoud, > >> 2012/12/13 Marek Vasut : >> > Dear Richard Genoud, >> > >> >> ifdefs in the code are making it harder to read. >> >> The use of simple if(VFAT_ENABLED) makes no more code and is cleaner. >> >> (the code is discarded by the compiler and linker ins

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Richard Genoud
2012/12/13 Tom Rini : > Note that we don't use --gc-sections on all archs so I'm not sure we > discard the unused VFAT functions on say ARM. > I tested it on at91sam9x5ek, and the vfat functions are not present in the System.map. The problem is in my commit message: the linker doesn"t have anythin

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Marek Vasut
Dear Richard Genoud, > 2012/12/13 Marek Vasut : > > Dear Richard Genoud, > > > >> ifdefs in the code are making it harder to read. > >> The use of simple if(VFAT_ENABLED) makes no more code and is cleaner. > >> (the code is discarded by the compiler and linker instead of the > >> preprocessor.) >

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/13/12 05:47, Richard Genoud wrote: > ifdefs in the code are making it harder to read. The use of simple > if(VFAT_ENABLED) makes no more code and is cleaner. (the code is > discarded by the compiler and linker instead of the preprocessor.) > > a

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Richard Genoud
2012/12/13 Marek Vasut : > Dear Richard Genoud, > >> ifdefs in the code are making it harder to read. >> The use of simple if(VFAT_ENABLED) makes no more code and is cleaner. >> (the code is discarded by the compiler and linker instead of the >> preprocessor.) >> >> and bonus, now the code compiles

Re: [U-Boot] [PATCH 1/2] FAT: remove ifdefs to make the code more readable

2012-12-13 Thread Marek Vasut
Dear Richard Genoud, > ifdefs in the code are making it harder to read. > The use of simple if(VFAT_ENABLED) makes no more code and is cleaner. > (the code is discarded by the compiler and linker instead of the > preprocessor.) > > and bonus, now the code compiles even if CONFIG_SUPPORT_VFAT is n