Re: [PATCH rtems v1 2/2] bsps/zynqmp: Added I2C support for ZynqMP

2021-08-24 Thread Chris Johns
On 25/8/21 12:05 am, Stephen Clark wrote: > This approach was also used in bsps/arm/xilinx-zynq/include/bsp/i2c.h. I kept > it specifically for consistency; I assumed it was the standard approach, but > your response makes me think it's not. I am also not sure and why I asked :) > Is there a

Re: [PATCH] tester: Add MicroBlaze KCU105 QEMU BSP

2021-08-24 Thread Chris Johns
On 25/8/21 7:46 am, Alex White wrote: > On Mon, Aug 23, 2021 at 9:29 PM Chris Johns wrote: >> >> Hi, >> >> Could you please explain this file? >> >> Where is the source? >> >> Why would we allow a binay blob into the tester like this? > > Hi Chris, > > Running MicroBlaze tests on QEMU requires

Re: [PATCH] tester: Add MicroBlaze KCU105 QEMU BSP

2021-08-24 Thread Alex White
On Mon, Aug 23, 2021 at 9:29 PM Chris Johns wrote: > > Hi, > > Could you please explain this file? > > Where is the source? > > Why would we allow a binay blob into the tester like this? Hi Chris, Running MicroBlaze tests on QEMU requires an appropriate DTB file to pass to QEMU via the

[PATCH] tester: Limit simultaneous QEMU jobs to 1

2021-08-24 Thread Kinsey Moore
There is an emulation problem between RTEMS and QEMU that causes several sets of tests to fail in circumstances where there is high load on the host machine. Running multiple instances of QEMU simultaneously exacerbates this problem which includes all of the spintrcritical tests, psx12, and some

[PATCH v1 0/3] Uninitialized scalar field Coverity issues

2021-08-24 Thread Ryan Long
Hi, To fix these issues I just needed to add the missing member variables from the initialization lists. Thanks, Ryan Ryan Long (3): rtems-exeinfo.cpp: Initialize byteorder GcovData.cc: Initialize gcnoPreamble rld-dwarf.cpp: Initialize member variables linkers/rtems-exeinfo.cpp | 3

[PATCH v1 3/3] rld-dwarf.cpp: Initialize member variables

2021-08-24 Thread Ryan Long
Initialize member variables not listed. CID 1503019: Uninitialized scalar field. Closes #4500 --- rtemstoolkit/rld-dwarf.cpp | 12 1 file changed, 12 insertions(+) diff --git a/rtemstoolkit/rld-dwarf.cpp b/rtemstoolkit/rld-dwarf.cpp index 1eae50c..2d6f306 100644 ---

[PATCH v1 1/3] rtems-exeinfo.cpp: Initialize byteorder

2021-08-24 Thread Ryan Long
CID 1471637: Uninitialized scalar field Closes #4499 --- linkers/rtems-exeinfo.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/linkers/rtems-exeinfo.cpp b/linkers/rtems-exeinfo.cpp index caae168..e54f9b2 100644 --- a/linkers/rtems-exeinfo.cpp +++

[PATCH v1 2/3] GcovData.cc: Initialize gcnoPreamble

2021-08-24 Thread Ryan Long
CID 1506206: Uninitialized scalar field Closes #4487 --- tester/covoar/GcovData.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/tester/covoar/GcovData.cc b/tester/covoar/GcovData.cc index 02e7489..723e9af 100644 --- a/tester/covoar/GcovData.cc +++ b/tester/covoar/GcovData.cc @@ -21,6

RE: [PATCH rtems v1 2/2] bsps/zynqmp: Added I2C support for ZynqMP

2021-08-24 Thread Stephen Clark
Chris, This approach was also used in bsps/arm/xilinx-zynq/include/bsp/i2c.h. I kept it specifically for consistency; I assumed it was the standard approach, but your response makes me think it's not. Is there a case to be made for breaking the register functions in both i2c.h files out into

Re: [PATCH 0/5] Preliminary Exception Manager Work

2021-08-24 Thread Kinsey Moore
On 8/23/2021 21:41, Chris Johns wrote: On 24/8/21 9:50 am, Kinsey Moore wrote: This patch set contains the result of the Exception Manager work I proposed a while back to manage handling of machine exceptions along with a general feature for mapping exceptions to POSIX signals without delving