FreeRTOS+TCP UltraScale port requires xuartps.h

Libraries
1

Hello,
I am using the FreeRTOS+TCP UltraScale port. In the UltraScale port, I noticed that x_emacpsif.h includes #include "xuartps.h"

However, in my hardware design the PS UART peripheral is not enabled. Because of that, the BSP does not generatexuartps.h, and the project fails to compile. From what I can see, the Ethernet driver itself does not seem to use any XUartPs symbols directly. If I comment out the #include "xuartps.h" line, the code appears to compile.

Is xuartps.h actually required by the FreeRTOS+TCP UltraScale Ethernet port or is this include just a leftover dependency from older code possibly for debug console support?

Thanks.

2

From a quick glance it seems like not needed. @htibosch What do you think?

3

Hi , I think that you can safely

 #include "xpseudo_asm.h"
 #include "xil_cache.h"
-#include "xuartps.h"
 #include "xscugic.h"

Including xuartps.h is indeed not needed.

4

Thanks @htibosch! @emrecntz Would you please raise a PR for this change?

5

I have raised a PR here:

I removed the unused xuartps.h include from both the Zynq and UltraScale network interface headers.

Thanks