[PATCH] build: Optionally use a VERSION file

2023-08-01 Thread Sebastian Huber
Define the RTEMS version in the wscript. Optionally use a VERSION file to change the default values of the wscript. Allow the command line option --rtems-version to override __RTEMS_MAJOR__. Remove support for command line configurable options. Rename internal define RTEMS_VERSION_VC_KEY to RTE

Re: [PATCH] build: Add optional RTEMS_VERSION_VC_KEY

2023-08-01 Thread Sebastian Huber
On 01.08.23 00:43, Chris Johns wrote: I think naming this option RTEMS_VERSION_CONTROL_KEY makes more sense, since the corresponding RTEMS API directive is rtems_version_control_key(). Is VC a well known abbreviation? The name in the API is what it is and I am not suggesting it be changed. Maybe

Fwd: [rtems-bsp-builder] 2023-07-31 17:37:34: Profile(s): everything

2023-08-01 Thread Joel Sherrill
I think this started before the fix was pushed but since there is no hash in this report, it is impossible to be sure. Since it already has to know about the rtems source tree, could it include a hash? How hard is that to implement? --joel -- Forwarded message - From: OAR Tester f

Re: Fwd: [rtems-bsp-builder] 2023-07-31 17:37:34: Profile(s): everything

2023-08-01 Thread Karel Gardas
Joel, not sure about what do you mean by "before the fix was pushed", but certainly RTEMS HEAD does not contain the fix for this yet and fix is already awaiting review here: https://lists.rtems.org/pipermail/devel/2023-July/076043.html If you ok that, I may push... Thanks, Karel On 8/1

Re: Fwd: [rtems-bsp-builder] 2023-07-31 17:37:34: Profile(s): everything

2023-08-01 Thread Joel Sherrill
Yes, go ahead. But would it be more useful it the report included a hash? On Tue, Aug 1, 2023 at 9:00 AM Karel Gardas wrote: > >Joel, > > not sure about what do you mean by "before the fix was pushed", but > certainly RTEMS HEAD does not contain the fix for this yet and fix is > already awa

Re: Fwd: [rtems-bsp-builder] 2023-07-31 17:37:34: Profile(s): everything

2023-08-01 Thread Karel Gardas
On 8/1/23 16:12, Joel Sherrill wrote: Yes, go ahead. Thanks, But would it be more useful it the report included a hash? Of course, but this is single-liner right? Depending on what you like: - git log -n 1 - git log -n 1|grep "^commit"|cut -d ' ' -f 2- Karel On Tue, Aug 1, 2023 at 9:0

CVA6 RISC-V support

2023-08-01 Thread EYSSARTIER Kevin
Classified as: {THALES GROUP LIMITED DISTRIBUTION} Hello, I am working on the port of RTEMS 6 on our RISC-V processor, the CORE-V CVA6 processor GitHub - ThalesGroup/cva6: The CORE-V CVA6 is an Application class 6-stage RISC-V CPU capable of booting Linux Th

[PATCH RTEMS] mvme3100: Add BSP fatal extension

2023-08-01 Thread Vijay Kumar Banerjee
--- bsps/powerpc/mvme3100/start/bspclean.c| 25 +++ .../bsps/powerpc/mvme3100/bspmvme3100.yml | 1 + 2 files changed, 26 insertions(+) create mode 100644 bsps/powerpc/mvme3100/start/bspclean.c diff --git a/bsps/powerpc/mvme3100/start/bspclean.c b/bsps/powerpc/mvme31

Re: [PATCH RTEMS] mvme3100: Add BSP fatal extension

2023-08-01 Thread Joel Sherrill
On Tue, Aug 1, 2023, 11:41 AM Vijay Kumar Banerjee wrote: > --- > bsps/powerpc/mvme3100/start/bspclean.c| 25 +++ > .../bsps/powerpc/mvme3100/bspmvme3100.yml | 1 + > 2 files changed, 26 insertions(+) > create mode 100644 bsps/powerpc/mvme3100/start/bspclean.c > > d

Re: CVA6 RISC-V support

2023-08-01 Thread Joel Sherrill
Thanks for the submission. It's a bit hard to review with the patches as attachments rather than having been sent via git send-email but here goes. Hopefully someone with more RISC-V knowledge can comment also. 0001 - long comment appears to be one line. Please break into multiple lines. 0001 -

Re: [PATCH RTEMS] mvme3100: Add BSP fatal extension

2023-08-01 Thread Vijay Kumar Banerjee
On Tue, Aug 1, 2023, 12:20 Joel Sherrill wrote: > > > On Tue, Aug 1, 2023, 11:41 AM Vijay Kumar Banerjee > wrote: > >> --- >> bsps/powerpc/mvme3100/start/bspclean.c| 25 +++ >> .../bsps/powerpc/mvme3100/bspmvme3100.yml | 1 + >> 2 files changed, 26 insertions(+) >>

[PATCH RTEMS] spec: Remove empty reset from mvme3100

2023-08-01 Thread Vijay Kumar Banerjee
--- spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml b/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml index 1667c1617a..fbb85123f0 100644 --- a/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml ++

Re: [PATCH RTEMS] mvme3100: Add BSP fatal extension

2023-08-01 Thread Joel Sherrill
On Tue, Aug 1, 2023 at 2:52 PM Vijay Kumar Banerjee wrote: > > > On Tue, Aug 1, 2023, 12:20 Joel Sherrill wrote: > >> >> >> On Tue, Aug 1, 2023, 11:41 AM Vijay Kumar Banerjee >> wrote: >> >>> --- >>> bsps/powerpc/mvme3100/start/bspclean.c| 25 +++ >>> .../bsps/powerpc/m

Re: [PATCH RTEMS] mvme3100: Add BSP fatal extension

2023-08-01 Thread Vijay Kumar Banerjee
On Tue, Aug 1, 2023, 15:30 Joel Sherrill wrote: > > > On Tue, Aug 1, 2023 at 2:52 PM Vijay Kumar Banerjee > wrote: > >> >> >> On Tue, Aug 1, 2023, 12:20 Joel Sherrill wrote: >> >>> >>> >>> On Tue, Aug 1, 2023, 11:41 AM Vijay Kumar Banerjee >>> wrote: >>> --- bsps/powerpc/mvme3100/st

Re: [PATCH RTEMS] spec: Remove empty reset from mvme3100

2023-08-01 Thread Chris Johns
OK Thanks Chris On 2/8/2023 6:16 am, Vijay Kumar Banerjee wrote: > --- > spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml > b/spec/build/bsps/powerpc/mvme3100/bspmvme3100.yml > index 1667c1

Re: [PATCH] build: Optionally use a VERSION file

2023-08-01 Thread Chris Johns
On 1/8/2023 5:11 pm, Sebastian Huber wrote: > Define the RTEMS version in the wscript. Optionally use a VERSION file > to change the default values of the wscript. Thanks, this is welcome. I will update the release scripts when merged. > Allow the command line > option --rtems-version to overri

Re: [PATCH] build: Optionally use a VERSION file

2023-08-01 Thread Sebastian Huber
On 02.08.23 06:01, Chris Johns wrote: Allow the command line option --rtems-version to override __RTEMS_MAJOR__. Remove support for command line configurable options. I do not understand this last bit. It says allow --rtems-version and then the command line configuration options are removed? An

Re: [PATCH] build: Optionally use a VERSION file

2023-08-01 Thread Chris Johns
On 2/8/2023 2:46 pm, Sebastian Huber wrote: > On 02.08.23 06:01, Chris Johns wrote: >>> Allow the command line >>> option --rtems-version to override __RTEMS_MAJOR__.  Remove support for >>> command line configurable options. >> I do not understand this last bit. It says allow --rtems-version and t

[PATCH] build: Add PROGRAM_PREFIX option

2023-08-01 Thread Sebastian Huber
Replace --rtems-version with a PROGRAM_PREFIX option. This allows also the use of vendor tools. --- spec/build/bsps/makeinc.yml| 16 spec/build/bsps/maketarget.yml | 22 +++--- spec/build/bsps/optobjcopy.yml | 2 +- spec/build/cpukit/c

[PATCH] bsps/shared: Fix Coverity warning in MCP7940M

2023-08-01 Thread Christian Mauderer
Coverity warns that (buf[...] & 0x7) can't be bigger than 7. This patch removes the unnecessary comparison. --- bsps/shared/dev/rtc/mcp7940m.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bsps/shared/dev/rtc/mcp7940m.c b/bsps/shared/dev/rtc/mcp7940m.c index 78a4f21b58..