[PATCH v4 2/3] buildman: differentiate between riscv32, riscv64
Simon Glass
sjg at chromium.org
Mon Oct 3 18:44:11 CEST 2022
Hi Heinrich,
On Mon, 3 Oct 2022 at 10:08, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> riscv32 needs a different toolchain than riscv64
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
> v4:
> no change
> v3:
> new patch
> ---
> tools/buildman/boards.py | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/tools/buildman/boards.py b/tools/buildman/boards.py
> index 8a0971aa40..cdc4d9ffd2 100644
> --- a/tools/buildman/boards.py
> +++ b/tools/buildman/boards.py
> @@ -263,6 +263,17 @@ class KconfigScanner:
> if params['arch'] == 'arm' and params['cpu'] == 'armv8':
> params['arch'] = 'aarch64'
>
> + # fix-up for riscv
> + if params['arch'] == 'riscv':
> + try:
> + value = self._conf.syms.get('ARCH_RV32I').str_value
> + except:
> + value = ''
> + if value == 'y':
> + params['arch'] = 'riscv32'
> + else:
> + params['arch'] = 'riscv64'
> +
> return params
Reviewed-by: Simon Glass <sjg at chromium.org>
I think I understand this better. Can we detect a boards.cfg file
which needs updating? Or perhaps the existing output_is_new() function
is good enough?
Regards.
Simon
More information about the U-Boot
mailing list