<div dir="ltr"><div>Thanks for all the good feedback.</div><div><br></div><div>RE Joel:</div><div>I'll fix my sloppy formatting that you caught and submit a revised patch.  If I'm realistic about my schedule, I probably won't be able to get to it until next week.</div><div>For xttcps_hw.h, there already is one #ifndef __rtems__ around the #includes, but on review there is another spot where I got lazy and used a #if 0.  I'll correct that too.  Other than that, the file is unmodified.</div><div><br></div><div>On the discussion about a shared space, I'll leave that decision up to you.  Tell me what you want and I can adjust as needed, or it could be done in a follow-on patch.</div><div><br></div><div>For the Versal, I've never used that part and am not very familiar with it, but the feedback from Aaron makes it sound like the core is probably pretty similar.  One other possible difference would be in the timers but it does look like the Versal has the same TTCs.  There just may be a small bit of work to set up the clock input to it.</div><div><br></div><div>-Phil<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 15, 2023 at 8:59 AM Aaron Nyholm <<a href="mailto:aaron.nyholm@unfoldedeffective.com">aaron.nyholm@unfoldedeffective.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This looks exciting.<br>
<br>
As for Versal support from Xilinx's Docs "RPU<br>
<br>
The real-time processing unit (RPU) Arm Cortex-R5F processor has faster clocking frequencies than the Zynq UltraScale+ MPSoC. The Versal Arm Cortex-R5F processor supports Vector Floating-Point v3 (VFPv3) whereas the Zynq UltraScale+ MPSoC Arm Cortex-R5F processor supports VFPv2." <a href="https://docs.xilinx.com/r/en-US/ug1273-versal-acap-design/RPU" rel="noreferrer" target="_blank">https://docs.xilinx.com/r/en-US/ug1273-versal-acap-design/RPU</a><br>
<br>
VFPv3 is backwards compatible with VFPv2 (<a href="https://developer.arm.com/documentation/ddi0344/d/programmer-s-model/vfpv3-architecture?lang=en" rel="noreferrer" target="_blank">https://developer.arm.com/documentation/ddi0344/d/programmer-s-model/vfpv3-architecture?lang=en</a>).<br>
<br>
So hopefully reuse for the Versal should relativity straight forward.<br>
<br>
Side note, even though Xilinx says the R5F in the ZynqMP has VFPv2, ARM says to compile with vfpv3_d16 like what is already in the BSP (<a href="https://developer.arm.com/documentation/dui0472/i/CIHGDBHC" rel="noreferrer" target="_blank">https://developer.arm.com/documentation/dui0472/i/CIHGDBHC</a>).<br>
<br>
Thanks, Aaron<br>
<br>
<br>
------- Original Message -------<br>
On Thursday, June 15th, 2023 at 9:17 AM, Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>> wrote:<br>
<br>
<br>
> <br>
> <br>
> On 14/6/2023 6:08 pm, Philip Kirkpatrick wrote:<br>
> <br>
> > This patch adds support for running RTEMS on the RPU (cortex R5) cores of the<br>
> > ZynqMP.<br>
> <br>
> <br>
> Thanks for submitting this BSP. It is exciting to see this work and support<br>
> being added.<br>
> <br>
> How different are the ZynqMP RPU cores and the ones on the Versal?<br>
> <br>
> I have not looked in detail but I know they are both R5 devices and I think we<br>
> should be able to reuse this support. Is placing the RPU pieces under the ZynqMP<br>
> sources what we want or should we consider how we would reuse the RPU BSP on<br>
> other Xilinx devices?<br>
> <br>
> I am leading with this question without reviewing the sources in detail so I<br>
> apologise for this. I am happy to look at Versal support so I am not asking that<br>
> to be done.<br>
> <br>
> Chris<br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>