On 29 February 2016 at 15:25, Simon Glass <s...@chromium.org> wrote: > We can use linker lists instead of explicitly declaring each function. > This makes the code shorter by avoiding switch() statements and lots of > header file declarations. > > While this does clean up the code it introduces a few code issues with SPL. > SPL never needs to print partition information since this all happens from > commands. SPL mostly doesn't need to obtain information about a partition > either, except in a few cases. Add these cases so that the code will be > dropped from each partition driver when not needed. This avoids code bloat. > > I think this is still a win, since it is not a bad thing to be explicit > about which features are used in SPL. But others may like to weigh in. > > Signed-off-by: Simon Glass <s...@chromium.org> > Reviewed-by: Tom Rini <tr...@konsulko.com> > --- > > Changes in v2: > - Reorder the linker list so that EFI comes before DOS, and add a comment > > disk/part.c | 184 > +++++++++++++++++------------------------------------- > disk/part_amiga.c | 16 +++-- > disk/part_dos.c | 9 ++- > disk/part_efi.c | 15 ++++- > disk/part_iso.c | 16 +++-- > disk/part_mac.c | 16 +++-- > include/part.h | 79 ++++++++++++++--------- > 7 files changed, 162 insertions(+), 173 deletions(-)
Applied to u-boot-dm/next. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
