Hi Rob, On 9 August 2017 at 17:14, Rob Clark <[email protected]> wrote: > We'll eventually want these in a few places in efi_loader, and also > vsprintf. > > Signed-off-by: Rob Clark <[email protected]> > --- > include/charset.h | 55 ++++++++++++++++++++++++++++++ > lib/Makefile | 1 + > lib/charset.c | 81 > ++++++++++++++++++++++++++++++++++++++++++++ > lib/efi_loader/efi_console.c | 17 ++-------- > 4 files changed, 140 insertions(+), 14 deletions(-) > create mode 100644 include/charset.h > create mode 100644 lib/charset.c
Looks good. Please can you add some tests for these functions? Regards, Simon _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
