Hi Simon, > -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf Of Simon Glass > Sent: Monday, April 24, 2017 11:38 AM > To: Peng Fan <[email protected]> > Cc: U-Boot Mailing List <[email protected]>; Tom Rini > <[email protected]>; Eddie Cai <[email protected]>; Jagan Teki > <[email protected]>; york sun <[email protected]>; Robert P. J. Day > <[email protected]>; Michal Simek <[email protected]> > Subject: Re: [PATCH 2/2] asm-generic: global_data: change timebase_l/h to > unsigned int > > Hi Peng, > > On 19 April 2017 at 19:10, Peng Fan <[email protected]> wrote: > > Change type of timebase_l/h to unsigned int. > > What is the motivation for this, please?
From lib/time.c: ((uint64_t)gd->timebase_h << 32) | gd->timebase_l; This piece code is based on that timebase_h and timebase_l are 32bits width, I think. But unsigned long are 64bits width on ARM64. I am not sure, but I suppose same case on other archs. Thanks, Peng. > > > > > Signed-off-by: Peng Fan <[email protected]> > > Cc: Simon Glass <[email protected]> > > Cc: Eddie Cai <[email protected]> > > Cc: Jagan Teki <[email protected]> > > Cc: York Sun <[email protected]> > > Cc: "Robert P. J. Day" <[email protected]> > > Cc: Michal Simek <[email protected]> > > Cc: Tom Rini <[email protected]> > > --- > > > > Buildman shows the following build failure, but not related this patch. > > aarch64: + orangepi_pc2 > > x86: + qemu-x86_64 qemu-x86_efi_payload64 chromebook_link64 > > > > include/asm-generic/global_data.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > Regards, > Simon _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
