On Wed, 5 Apr 2023 at 06:45, Andrew Davis <a...@ti.com> wrote: > > The header binman_sym.h depends on ulong typedef but does not include > types.h. This means the header must be included after including types.h > or a header that includes it. > > We could include types.h but instead let's just switch from ulong > to directly using unsigned long. This removes the need for typedef'ing > it in some of the tests, so also remove those. > > Signed-off-by: Andrew Davis <a...@ti.com> > --- > include/binman_sym.h | 8 ++++---- > tools/binman/test/blob_syms.c | 2 -- > tools/binman/test/u_boot_binman_syms.c | 2 -- > tools/binman/test/u_boot_binman_syms_size.c | 2 -- > 4 files changed, 4 insertions(+), 10 deletions(-) >
Reviewed-by: Simon Glass <s...@chromium.org> Applied to u-boot-dm, thanks!