Re: [PATCH rtems-tools v3] tester/rtemstoolkit: add renode implementation

2023-08-02 Thread Chris Johns
On 2/8/2023 8:08 pm, Muhammad Sulthan Mazaya wrote: > Add licensing for renode files and change the renode_scripts/ folder to > renode/ Where is the license for the .bin file? I only saw a link to the script used to build it but it was not clear what the license is because it downloads Linux and

RE: CVA6 RISC-V support

2023-08-02 Thread EYSSARTIER Kevin
Classified as: {THALES GROUP LIMITED DISTRIBUTION} Thanks for your quick answer Joel. Sorry for not using the “git send-email” command, my company blocks smtp ports on dev machines. Patch 0001: I corrected the comment. The modification in riscv/abi.yml is indeed not needed, it has been removed.

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

2023-08-02 Thread Christian MAUDERER
On 2023-08-02 15:31, Joel Sherrill wrote: On Wed, Aug 2, 2023 at 6:45 AM Christian MAUDERER > wrote: Hello Joel, thanks. Is the ID processed somewhere automatically so that I should use a special format? Otherwise I can just

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

2023-08-02 Thread Joel Sherrill
On Wed, Aug 2, 2023 at 6:45 AM Christian MAUDERER < christian.maude...@embedded-brains.de> wrote: > Hello Joel, > > thanks. Is the ID processed somewhere automatically so that I should use > a special format? Otherwise I can just just add the error message from > covertity: > > ** CID 1539495:

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Sebastian Huber
On 02.08.23 13:53, Karel Gardas wrote: A bit off-topic. On 8/2/23 10:39, Sebastian Huber wrote: Yes, but this would be another patch and it is a bit more work since you have to test the clang support. Is building with clang already supported? I'm curious since this is something I'd like to

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Karel Gardas
A bit off-topic. On 8/2/23 10:39, Sebastian Huber wrote: Yes, but this would be another patch and it is a bit more work since you have to test the clang support. Is building with clang already supported? I'm curious since this is something I'd like to test locally too but neither code nor

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

2023-08-02 Thread Christian MAUDERER
Hello Joel, thanks. Is the ID processed somewhere automatically so that I should use a special format? Otherwise I can just just add the error message from covertity: ** CID 1539495: Integer handling issues (CONSTANT_EXPRESSION_RESULT) /bsps/shared/dev/rtc/mcp7940m.c: 317 in

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

2023-08-02 Thread Joel Sherrill
Ok but out the Coverity Id number in the commit long message On Wed, Aug 2, 2023, 1:17 AM Christian Mauderer < christian.maude...@embedded-brains.de> wrote: > Coverity warns that (buf[...] & 0x7) can't be bigger than 7. This patch > removes the unnecessary comparison. > --- >

Fwd: New Defects reported by Coverity Scan for RTEMS

2023-08-02 Thread Joel Sherrill
New Coverity issue -- Forwarded message - From: Date: Wed, Aug 2, 2023, 12:22 AM Subject: New Defects reported by Coverity Scan for RTEMS To: Hi, Please find the latest report on new defect(s) introduced to RTEMS found with Coverity Scan. 1 new defect(s) introduced to RTEMS

[PATCH rtems-tools v3] tester/rtemstoolkit: add renode implementation

2023-08-02 Thread Muhammad Sulthan Mazaya
Add licensing for renode files and change the renode_scripts/ folder to renode/ --- .../testing/bsps/kendrytek210-renode.ini | 38 tester/rtems/testing/bsps/leon3-renode.ini| 37 tester/rtems/testing/renode.cfg | 64 +

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Chris Johns
On 2/8/2023 6:56 pm, Sebastian Huber wrote: > On 02.08.23 10:50, Chris Johns wrote: >> On 2/8/2023 6:39 pm, Sebastian Huber wrote: >>> On 02.08.23 10:33, Chris Johns wrote: > diff --git a/spec/build/bsps/makeinc.yml b/spec/build/bsps/makeinc.yml > index ac395f2f02..08fc75a8b9 100644 >

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Sebastian Huber
On 02.08.23 10:50, Chris Johns wrote: On 2/8/2023 6:39 pm, Sebastian Huber wrote: On 02.08.23 10:33, Chris Johns wrote: diff --git a/spec/build/bsps/makeinc.yml b/spec/build/bsps/makeinc.yml index ac395f2f02..08fc75a8b9 100644 --- a/spec/build/bsps/makeinc.yml +++

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Chris Johns
On 2/8/2023 6:39 pm, Sebastian Huber wrote: > On 02.08.23 10:33, Chris Johns wrote: >>> diff --git a/spec/build/bsps/makeinc.yml b/spec/build/bsps/makeinc.yml >>> index ac395f2f02..08fc75a8b9 100644 >>> --- a/spec/build/bsps/makeinc.yml >>> +++ b/spec/build/bsps/makeinc.yml >>> @@ -16,14 +16,14 @@

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Chris Johns
On 2/8/2023 6:42 pm, Sebastian Huber wrote: > On 02.08.23 10:40, Chris Johns wrote: >> On 2/8/2023 6:33 pm, Chris Johns wrote: >>> On 2/8/2023 3:49 pm, Sebastian Huber wrote: Replace --rtems-version with a PROGRAM_PREFIX option.  This allows also the use of vendor tools.> --- >> One

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Sebastian Huber
On 02.08.23 10:40, Chris Johns wrote: On 2/8/2023 6:33 pm, Chris Johns wrote: On 2/8/2023 3:49 pm, Sebastian Huber wrote: Replace --rtems-version with a PROGRAM_PREFIX option. This allows also the use of vendor tools.> --- One further thing to consider is if PROGRAM_PREFIX could clash with

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Chris Johns
On 2/8/2023 6:33 pm, Chris Johns wrote: > On 2/8/2023 3:49 pm, Sebastian Huber wrote: >> Replace --rtems-version with a PROGRAM_PREFIX option. This allows also >> the use of vendor tools.> --- One further thing to consider is if PROGRAM_PREFIX could clash with something else in a user's

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Sebastian Huber
On 02.08.23 10:33, Chris Johns wrote: diff --git a/spec/build/bsps/makeinc.yml b/spec/build/bsps/makeinc.yml index ac395f2f02..08fc75a8b9 100644 --- a/spec/build/bsps/makeinc.yml +++ b/spec/build/bsps/makeinc.yml @@ -16,14 +16,14 @@ content: | prefix = ${PREFIX} exec_prefix =

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-02 Thread Chris Johns
On 2/8/2023 3:49 pm, Sebastian Huber wrote: > 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 +++--- >

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

2023-08-02 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