[PATCH rtems-lwip v3 1/7] lwip.py: Change arch and bsp check method

[Duc Doan]

I realized that I am not using the latest rtems, so I did not see those BSPs. Should I create another patch to add the correct BSPs?

Thanks,

Duc
________________________________
From: devel <devel-bounces at rtems.org> on behalf of Kinsey Moore <kinsey.moore at oarcorp.com>
Sent: Thursday, September 8, 2022 12:52:15 PM
To: devel at rtems.org <devel at rtems.org>
Subject: Re: [PATCH rtems-lwip v3 1/7] lwip.py: Change arch and bsp check method

On 9/8/2022 11:34, Duc Doan wrote:
> ---
>   lwip.py | 31 ++++++++++++++++++-------------
>   1 file changed, 18 insertions(+), 13 deletions(-)
>
> diff --git a/lwip.py b/lwip.py
> index 84eef2c..1f0b8e2 100644
> --- a/lwip.py
> +++ b/lwip.py
> @@ -99,6 +99,8 @@ def build(bld):
>       drv_incl = []
>       arch_lib_path = rtems.arch_bsp_lib_path(bld.env.RTEMS_VERSION,
>                                               bld.env.RTEMS_ARCH_BSP)
...
>       is_qemu = False
> -    if bld.env.RTEMS_ARCH_BSP.startswith('aarch64-rtems6-xilinx_zynqmp'):
> +    if arch == 'aarch64' and bsp in ['xilinx_zynqmp_ultra96']:
>           is_xilinx_bsp = True
>           is_aarch64_bsp = True
> -    if bld.env.RTEMS_ARCH_BSP.endswith('_qemu'):
> +    if bsp in ['xilinx_zynq_a9_qemu']:
>           is_qemu = True
>       if is_xilinx_bsp:
>           drv_incl.extend(xilinx_drv_incl)

The BSPs mentioned here for AArch64 are actually ARM BSPs. The proper
set of BSPs to be used here are:

xilinx_zynqmp_lp64_qemu

xilinx_zynqmp_lp64_zu3eg

xilinx_zynqmp_ilp32_qemu

xilinx_zynqmp_ilp32_zu3eg


At some point, the lwIP build needs to get better about managing which BSPs it supports, but that's not a task for you here and now.


Kinsey

_______________________________________________
devel mailing list
devel at rtems.org
http://lists.rtems.org/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20220908/4a23367c/attachment.htm>