<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 3:18 AM Philip Kirkpatrick <<a href="mailto:p.kirkpatrick@reflexaerospace.com">p.kirkpatrick@reflexaerospace.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"><div dir="ltr">For the RPU patch, I'll add removing this to my next pass at it.<br></div></blockquote><div><br></div><div>If there is some transformation on the linked executable needed to download it,</div><div>I hope you document that entire process in the Users Guide. :)</div><div><br></div><div>--joel</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 16, 2023 at 6:37 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</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"><br>
<br>
On 16.06.23 06:35, Chris Johns wrote:<br>
> On 16/6/2023 2:33 pm, Sebastian Huber wrote:<br>
>> On 16.06.23 03:46, Chris Johns wrote:<br>
>>> On 16/6/2023 9:04 am, Joel Sherrill wrote:<br>
>>>> On Thu, Jun 15, 2023 at 5:41 PM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a><br>
>>>> <mailto:<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>>> wrote:<br>
>>>><br>
>>>> Hi,<br>
>>>><br>
>>>> Reviewing the RPU patch I noticed a .inc file being added and I wondered<br>
>>>> why so<br>
>>>> I checked the sources and to my surprise found:<br>
>>>><br>
>>>> rtems.git $ find bsps -name \*.inc<br>
>>>> bsps/v850/gdbv850sim/config/v850sim.inc<br>
>>>> bsps/m68k/genmcf548x/config/genmcf548x.inc<br>
>>>> bsps/m68k/gen68340/include/m68340.inc<br>
>>>> bsps/m68k/gen68340/include/m68349.inc<br>
>>>> bsps/arm/altera-cyclone-v/config/altcycv.inc<br>
>>>> bsps/arm/raspberrypi/config/raspberrypi.inc<br>
>>>> bsps/arm/lpc32xx/config/lpc32xx.inc<br>
>>>> bsps/arm/beagle/config/beagle.inc<br>
>>>> bsps/arm/tms570/config/tms570ls3137.inc<br>
>>>> bsps/arm/xilinx-zynqmp/config/xilinx_zynqmp.inc<br>
>>>> bsps/arm/xilinx-zynq/config/xilinx_zynq.inc<br>
>>>> bsps/arm/lpc24xx/config/lpc17xx.inc<br>
>>>> bsps/arm/lpc24xx/config/lpc40xx.inc<br>
>>>> bsps/arm/lpc24xx/config/lpc24xx.inc<br>
>>>> bsps/arm/lm3s69xx/config/lm3s69xx.inc<br>
>>>> bsps/powerpc/gen5200/config/gen5200.inc<br>
>>>> bsps/powerpc/tqm8xx/config/tqm8xx.inc<br>
>>>> bsps/powerpc/mpc55xxevb/config/mpc55xx.inc<br>
>>>> bsps/powerpc/qoriq/config/qoriq.inc<br>
>>>> bsps/powerpc/gen83xx/config/gen83xx.inc<br>
>>>><br>
>>>><br>
>>>> These likely can. Thinking back, we have cfg files for every BSP<br>
>>>> variant within a family. They often set a variable like CPU and CPU_CFLAGS<br>
>>>> and included a shared .inc file. These all appear to be from BSPs which<br>
>>>> had variants. I think they are remnants and missed when the .cfg files<br>
>>>> were<br>
>>>><br>
>>>> Would like someone to concur.<br>
>>>><br>
>>>><br>
>>>> And for .cfg there is:<br>
>>>><br>
>>>> rtems.git $ find bsps -name \*.cfg | wc -l<br>
>>>> 191<br>
>>>><br>
>>>><br>
>>>> Are these installed and used by the application Makefile system. If so, no.<br>
>>> I cannot see any references in spec but Sebastian is the best person to answer<br>
>>> this.<br>
>> They are definitely not used or installed. I think we still have then since they<br>
>> document the post-link actions which are not yet supported by the new build system.<br>
> Ah yes, thanks. What if the files are collected and archived?<br>
<br>
I think we should just remove them. We can use the RTEMS 5 branch to <br>
look up the post-link actions if needed.<br>
<br>
> The RPU BSP is adding a new one. Is this what we want to see happen?<br>
<br>
No, we don't want to see new ones.<br>
<br>
-- <br>
embedded brains GmbH<br>
Herr Sebastian HUBER<br>
Dornierstr. 4<br>
82178 Puchheim<br>
Germany<br>
email: <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
phone: +49-89-18 94 741 - 16<br>
fax: +49-89-18 94 741 - 08<br>
<br>
Registergericht: Amtsgericht München<br>
Registernummer: HRB 157899<br>
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler<br>
Unsere Datenschutzerklärung finden Sie hier:<br>
<a href="https://embedded-brains.de/datenschutzerklaerung/" rel="noreferrer" target="_blank">https://embedded-brains.de/datenschutzerklaerung/</a><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></blockquote></div>
</blockquote></div></div>