Hi Michal, On Mon, 11 Sept 2023 at 06:32, Michal Simek <michal.si...@amd.com> wrote: > > tputq() requires variables to have 64bit width that's why make them 64bit > to clean alignment requirement. > > Signed-off-by: Michal Simek <michal.si...@amd.com> > --- > > tools/proftool.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Simon Glass <s...@chromium.org> I would have thought that 'unsigned long' is enough on a 64-bit machines? Are you running on 32-bit? > > diff --git a/tools/proftool.c b/tools/proftool.c > index 101bcb63334e..869a2a32c510 100644 > --- a/tools/proftool.c > +++ b/tools/proftool.c > @@ -1493,7 +1493,8 @@ static int write_pages(struct twriter *tw, enum > out_format_t out_format, > static int write_flyrecord(struct twriter *tw, enum out_format_t out_format, > int *missing_countp, int *skip_countp) > { > - int start, ret, len; > + unsigned long long start, len; > + int ret; > FILE *fout = tw->fout; > char str[200]; > > -- > 2.36.1 > Regards, Simon