From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heinrich Schuchardt Date: Sun, 26 Apr 2020 23:14:25 +0200 Subject: [PATCH v2 6/7] x86: Add an indication of 32/64-bit to bdinfo In-Reply-To: References: <20200426193824.171193-1-sjg@chromium.org> <20200426193824.171193-2-sjg@chromium.org> <77ab3c57-a320-b775-59f5-bd852bd7b782@gmx.de> Message-ID: <73930b03-0fd5-2e74-a5db-3ac15a149dfc@gmx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 4/26/20 10:32 PM, Simon Glass wrote: > Hi Heinrich, > > On Sun, 26 Apr 2020 at 14:21, Heinrich Schuchardt wrote: >> >> On 4/26/20 9:38 PM, Simon Glass wrote: >>> It is useful to know what mode U-Boot is running in. Add a message at the >>> end of the 'bdinfo' output. >> >> X86 is not the only architecture that can run both in 32bit and 64bit >> mode. The same is true for ARM and SANDBOX. Shouldn't we provide the >> information for these architecture too? >> >> The Raspberry Pi 3 is an example of an ARM board where building either >> 32bit or 64bit U-Boot is used depending on the bitness of the OS. > > Yes I think that would be good. But do we have a generic way to tell? > sizeof(int) ? sizeof(int) = 4 on aarch64. Using these should be fine: CONFIG_ARM64=y CONFIG_ARCH_RV64I=y CONFIG_SANDBOX64=y (continued below) > > Regards, > Simon > > >> >> Best regards >> >> Heinrich >> >>> >>> Signed-off-by: Simon Glass >>> --- >>> >>> Changes in v2: >>> - Add a new patch to indicate 32/64-bit in bdinfo >>> >>> cmd/bdinfo.c | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c >>> index d6a7175b37..a3129c9fe1 100644 >>> --- a/cmd/bdinfo.c >>> +++ b/cmd/bdinfo.c >>> @@ -388,6 +388,7 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) >>> print_mhz("ethspeed", bd->bi_ethspeed); >>> #endif >>> print_baudrate(); >>> + printf("Build: %d-bit\n", CONFIG_IS_ENABLED(X86_64) ? 64 : 32); "Build" it usually used to refer to a software release. Should this be: printf("bitness = %d\n", ... or simply printf("%d-bit\n", ... Best regards Heinrich >>> >>> return 0; >>> } >>> >>