<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 16, 2023 at 7:14 AM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</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">On 15/6/2023 6:16 pm, Philip Kirkpatrick wrote:<br>
> Thanks for all the good feedback.<br>
> <br>
> RE Joel:<br>
> I'll fix my sloppy formatting that you caught and submit a revised patch.  If<br>
> I'm realistic about my schedule, I probably won't be able to get to it until<br>
> next week.<br>
> For xttcps_hw.h, there already is one #ifndef __rtems__ around the #includes,<br>
> but on review there is another spot where I got lazy and used a #if 0.  I'll<br>
> correct that too.  Other than that, the file is unmodified.<br>
> <br>
> On the discussion about a shared space, I'll leave that decision up to you. <br>
> Tell me what you want and I can adjust as needed, or it could be done in a<br>
> follow-on patch.<br>
<br>
Should the RPU BSP be located under bsps/arm/xilinx-rpu?<br></blockquote><div><br></div><div>I went back and forth on that decision and decided to keep them combined since the APU and RPU share a moderate amount of code.  However, I can definitely see an argument that they are different enough to split.  If you want it the other way, I can make that change when I address the other items.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> For the Versal, I've never used that part and am not very familiar with it, but<br>
> the feedback from Aaron makes it sound like the core is probably pretty<br>
> similar.  One other possible difference would be in the timers but it does look<br>
> like the Versal has the same TTCs.  There just may be a small bit of work to set<br>
> up the clock input to it.<br>
<br>
We would handle the Versal once the BSP has a home.<br>
<br>
Chris<br>
</blockquote></div></div>