BSP .inc files and .cfg files

[Chris Johns]

On 16/6/2023 9:04 am, Joel Sherrill wrote:
> 
> 
> On Thu, Jun 15, 2023 at 5:41 PM Chris Johns <chrisj at rtems.org
> <mailto:chrisj at rtems.org>> wrote:
> 
>     Hi,
> 
>     Reviewing the RPU patch I noticed a .inc file being added and I wondered why so
>     I checked the sources and to my surprise found:
> 
>     rtems.git $ find bsps -name \*.inc
>     bsps/v850/gdbv850sim/config/v850sim.inc
>     bsps/m68k/genmcf548x/config/genmcf548x.inc
>     bsps/m68k/gen68340/include/m68340.inc
>     bsps/m68k/gen68340/include/m68349.inc
>     bsps/arm/altera-cyclone-v/config/altcycv.inc
>     bsps/arm/raspberrypi/config/raspberrypi.inc
>     bsps/arm/lpc32xx/config/lpc32xx.inc
>     bsps/arm/beagle/config/beagle.inc
>     bsps/arm/tms570/config/tms570ls3137.inc
>     bsps/arm/xilinx-zynqmp/config/xilinx_zynqmp.inc
>     bsps/arm/xilinx-zynq/config/xilinx_zynq.inc
>     bsps/arm/lpc24xx/config/lpc17xx.inc
>     bsps/arm/lpc24xx/config/lpc40xx.inc
>     bsps/arm/lpc24xx/config/lpc24xx.inc
>     bsps/arm/lm3s69xx/config/lm3s69xx.inc
>     bsps/powerpc/gen5200/config/gen5200.inc
>     bsps/powerpc/tqm8xx/config/tqm8xx.inc
>     bsps/powerpc/mpc55xxevb/config/mpc55xx.inc
>     bsps/powerpc/qoriq/config/qoriq.inc
>     bsps/powerpc/gen83xx/config/gen83xx.inc
> 
> 
> These likely can. Thinking back, we have cfg files for every BSP
> variant within a family. They often set a variable like CPU and CPU_CFLAGS
> and included a shared .inc file. These all appear to be from BSPs which
> had variants. I think they are remnants and missed when the .cfg files
> were 
> 
> Would like someone to concur. 
> 
> 
>     And for .cfg there is:
> 
>     rtems.git $ find bsps -name \*.cfg | wc -l
>          191
> 
> 
> Are these installed and used by the application Makefile system. If so, no. 

I cannot see any references in spec but Sebastian is the best person to answer this.

Chris

> But if these were just the ".cfg" files I mentioned above, then likely yes. We
> probably had 191 BSPs which we switched to waf.
> 
> 
>     Can these files be removed?
> 
>     Chris
>     _______________________________________________
>     devel mailing list
>     devel at rtems.org <mailto:devel at rtems.org>
>     http://lists.rtems.org/mailman/listinfo/devel
>     <http://lists.rtems.org/mailman/listinfo/devel>
>