[PATCH rtems-lwip - v1 06/11] FIX: printf format spec compiler warning due to uintptr having 64bits on 64bit machines
Bernd Moessner
berndmoessner80 at gmail.com
Thu Jan 4 22:27:51 UTC 2024
On 04.01.2024 23:22, Kinsey Moore wrote:
> Comments inline.
>
> On Thu, Jan 4, 2024 at 12:54 PM <berndmoessner80 at gmail.com> wrote:
>
> From: Bernd Moessner <berndmoessner80 at gmail.com>
>
> ---
> .../lwip211/src/contrib/ports/xilinx/netif/xadapter.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git
> a/embeddedsw/ThirdParty/sw_services/lwip211/src/contrib/ports/xilinx/netif/xadapter.c
> b/embeddedsw/ThirdParty/sw_services/lwip211/src/contrib/ports/xilinx/netif/xadapter.c
> index 9594ff5..98e7a8e 100644
> ---
> a/embeddedsw/ThirdParty/sw_services/lwip211/src/contrib/ports/xilinx/netif/xadapter.c
> +++
> b/embeddedsw/ThirdParty/sw_services/lwip211/src/contrib/ports/xilinx/netif/xadapter.c
> @@ -184,8 +184,9 @@ xemac_add(struct netif *netif,
> #endif
> #endif
> default:
> - xil_printf("unable to determine
> type of EMAC with baseaddress 0x%08x\r\n",
> + xil_printf("unable to determine
> type of EMAC with baseaddress %" PRIXPTR,
> mac_baseaddr);
> + xil_printf("\r\b");
>
>
> The second line has a \b instead of a \n.
>
> Beyond that typo, changes in third-party sources in rtems-lwIP are
> treated just like they are in rtems-libbsd. Changes are additive only
> and gated by __rtems__. So if you wanted to change line A to line B,
> you'd have this construct:
>
> #ifdef __rtems__
> line B
> #else
> line A
> #endif
>
>
Okay, if fixed it in a later patch. I`ll merge the patches and add the
gates. I guess there are some other places where I ll have to add the
gates. I have reported one of the issues already to xilinx. I`ll report
all of them to Xilinx and until they've patched their sources we need to
stick to the gates.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20240104/fcb83d22/attachment-0001.htm>
More information about the devel
mailing list