Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-15 Thread Simon Glass
Hi Heinrich, On Wed, 14 Aug 2024 at 06:59, Heinrich Schuchardt wrote: > > On 14.08.24 14:40, Simon Glass wrote: > > Hi Heinrich, > > > > On Wed, 14 Aug 2024 at 01:14, Heinrich Schuchardt > > wrote: > >> > >> uint64_t is defined as unsigned long long on 32bit ARM. > >> Convert uint64_t values to

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Heinrich Schuchardt
On 14.08.24 14:40, Simon Glass wrote: Hi Heinrich, On Wed, 14 Aug 2024 at 01:14, Heinrich Schuchardt wrote: uint64_t is defined as unsigned long long on 32bit ARM. Convert uint64_t values to unsigned long long and use %llX for printing. tools/mkeficapsule.c: In function ‘dump_capsule_au

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Simon Glass
Hi Heinrich, On Wed, 14 Aug 2024 at 01:14, Heinrich Schuchardt wrote: > > uint64_t is defined as unsigned long long on 32bit ARM. > Convert uint64_t values to unsigned long long and use %llX for printing. > > tools/mkeficapsule.c: In function ‘dump_capsule_auth_header’: > tools/mkeficapsu

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Heinrich Schuchardt
On 14.08.24 13:40, Ilias Apalodimas wrote: Hi Heinrich, On Wed, 14 Aug 2024 at 10:14, Heinrich Schuchardt wrote: uint64_t is defined as unsigned long long on 32bit ARM. Convert uint64_t values to unsigned long long and use %llX for printing. Why do we need to convert? uint64_t is a fixed wi

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Sughosh Ganu
On Wed, 14 Aug 2024 at 17:10, Ilias Apalodimas wrote: > > Hi Heinrich, > > On Wed, 14 Aug 2024 at 10:14, Heinrich Schuchardt > wrote: > > > > uint64_t is defined as unsigned long long on 32bit ARM. > > Convert uint64_t values to unsigned long long and use %llX for printing. > > Why do we need to

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Ilias Apalodimas
Hi Heinrich, On Wed, 14 Aug 2024 at 10:14, Heinrich Schuchardt wrote: > > uint64_t is defined as unsigned long long on 32bit ARM. > Convert uint64_t values to unsigned long long and use %llX for printing. Why do we need to convert? uint64_t is a fixed width type Thanks /Ilias > > tools/mkef

Re: [PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Sughosh Ganu
On Wed, 14 Aug 2024 at 12:44, Heinrich Schuchardt wrote: > > uint64_t is defined as unsigned long long on 32bit ARM. > Convert uint64_t values to unsigned long long and use %llX for printing. > > tools/mkeficapsule.c: In function ‘dump_capsule_auth_header’: > tools/mkeficapsule.c:694:66: w

[PATCH 1/1] tools/mkeficapsule: correct printf codes

2024-08-14 Thread Heinrich Schuchardt
uint64_t is defined as unsigned long long on 32bit ARM. Convert uint64_t values to unsigned long long and use %llX for printing. tools/mkeficapsule.c: In function ‘dump_capsule_auth_header’: tools/mkeficapsule.c:694:66: warning: format ‘%lX’ expects argument of type ‘long unsigned int’