On 30/6/21 2:19 am, Kinsey Moore wrote:
> On 6/29/2021 11:10, Gedare Bloom wrote:
>> On Tue, Jun 29, 2021 at 6:34 AM Kinsey Moore
>> wrote:
>>> I suppose this could have been configured in libbsd (possibly
>>> config.inc) instead if that's what you're getting at. The overall goal
>>> is to be
On 30/6/21 1:21 am, Gedare Bloom wrote:
> On Tue, Jun 29, 2021 at 7:50 AM Sebastian Huber
> wrote:
>>
>> On 29/06/2021 15:48, Kinsey Moore wrote:
>>> On 6/9/2021 00:27, Sebastian Huber wrote:
On 08/06/2021 22:18, Kinsey Moore wrote:
> Provide the options necessary to enable any
looks good
On Tue, Jun 29, 2021 at 3:36 PM Kinsey Moore wrote:
>
> Add tester configurations for ILP32 and LP64 on hardware.
> ---
> .../bsps/xilinx_zynqmp_ilp32_zu3eg.ini| 41 +++
> .../testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini | 41 +++
> 2 files
Add tester configurations for ILP32 and LP64 on hardware.
---
.../bsps/xilinx_zynqmp_ilp32_zu3eg.ini| 41 +++
.../testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini | 41 +++
2 files changed, 82 insertions(+)
create mode 100644
See Also CID 1437638
Closes #4462
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index 5580d415e2..b1bb26572d 100644
---
See Also CID 1437645
Closes #4297
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index bfbc6102a2..5580d415e2 100644
--- a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
+++
See Also CID 1437647
Closes #4460
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index 7747ba9bf8..bfbc6102a2 100644
---
On Tue, Jun 29, 2021 at 2:11 PM Joel Sherrill wrote:
>
> Same issue to improve the commit message as I just emailed. Something
> like this would be better:
>
> cpukit/libmisc/rtems-ftd.c: Fixed error of null pointer dereference
>
check typo on fdt, and see my note, use 'libmisc/rtems-fdt' would
On Tue, Jun 29, 2021 at 2:26 PM Joel Sherrill wrote:
>
> On Tue, Jun 29, 2021 at 3:15 PM Harrison Gerber wrote:
> >
> > Joel,
> >
> > I'm not too sure what is meant by almost duplicating the base file name.
> > The file referenced is 'rtems-fdt.c', which is found in a file called
> >
these two look good now, thanks
On Tue, Jun 29, 2021 at 1:37 PM Ryan Long wrote:
>
> CID 1503008: Division or modulo by zero
> CID 1503015: Division or modulo by zero
>
> Closes #4416
> ---
> linkers/rtems-exeinfo.cpp | 20
> 1 file changed, 16 insertions(+), 4 deletions(-)
PS: loosely related, there is
arm/altera-cyclone-v/optfdten.yml:name: BSP_FDT_IS_SUPPORTED
It looks like other BSPs are deriving this check by conditioning it on
BSP_START_COPY_FDT_FROM_U_BOOT
This could probably be made consistent across these FDT-supporting BSPs.
Gedare
On Tue, Jun 29, 2021
On Tue, Jun 29, 2021 at 12:27 PM Christian Mauderer wrote:
>
> PS: Wouldn't it make sense to move these even further up? There is for
> example ./spec/build/bsps/riscv/riscv/optfdtcpyro.yml too. A short md5
> shows that it's the same file again.
>
Yes. Pranav, can you prepare a patch to move
On Tue, Jun 29, 2021 at 3:15 PM Harrison Gerber wrote:
>
> Joel,
>
> I'm not too sure what is meant by almost duplicating the base file name. The
> file referenced is 'rtems-fdt.c', which is found in a file called 'rtems-fdt'
> inside libmisc, so I thought it was best to have the entire
OK with code. Commit message needs correction per suggestions in other
patches in this series.
--joel
On Tue, Jun 29, 2021 at 2:54 PM Harrison Edward Gerber
wrote:
>
> See Also CID 1437638
>
> Closes #4462
> ---
> cpukit/libmisc/rtems-fdt/rtems-fdt.c | 2 +-
> 1 file changed, 1 insertion(+), 1
Same issue to improve the commit message as I just emailed. Something
like this would be better:
cpukit/libmisc/rtems-ftd.c: Fixed error of null pointer dereference
On Tue, Jun 29, 2021 at 3:07 PM Joel Sherrill wrote:
>
> This definitely moves the dereference before the free() so appears to be
I'm ok with the fix but the commit message could be improved.
Something like this would be better.
cpukit/libmisc/rtems-fdt.c : close() file to avoid leaking descriptor
You almost duplicated the base file name and it isn't leaking the
variable, it is leaking the file descriptor referenced by bf.
This definitely moves the dereference before the free() so appears to be OK.
If Gedare approves, he can merge it.
--joel
On Tue, Jun 29, 2021 at 2:54 PM Harrison Edward Gerber
wrote:
>
> See Also CID 1437647
>
> Closes #4460
> ---
> cpukit/libmisc/rtems-fdt/rtems-fdt.c | 4 ++--
> 1 file
See Also CID 1437638
Closes #4462
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index 5580d415e2..b1bb26572d 100644
---
See Also CID 1437645
Closes #4297
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index bfbc6102a2..5580d415e2 100644
--- a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
+++
See Also CID 1437647
Closes #4460
---
cpukit/libmisc/rtems-fdt/rtems-fdt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
index 7747ba9bf8..bfbc6102a2 100644
---
> -Original Message-
> From: Gedare Bloom
> Sent: Tuesday, June 29, 2021 6:22 PM
> To: Kinsey Moore ; Sommer, Jan
>
> Cc: devel@rtems.org
> Subject: Re: [PATCH] bsps/cadence-spi: Fix moduleid offset
>
> Jan, please confirm. Thanks
>
Looks good to me.
Thanks,
Jan
> On Mon,
CID 1503709: Division or modulo by float zero
Closes #4415
---
tester/covoar/ReportsBase.cc | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tester/covoar/ReportsBase.cc b/tester/covoar/ReportsBase.cc
index b4a755c..328980d 100644
--- a/tester/covoar/ReportsBase.cc
+++
CID 1503008: Division or modulo by zero
CID 1503015: Division or modulo by zero
Closes #4416
---
linkers/rtems-exeinfo.cpp | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/linkers/rtems-exeinfo.cpp b/linkers/rtems-exeinfo.cpp
index 1e6d4b4..6e92206 100644
I'll cast those. Sorry I missed that.
-Original Message-
From: Gedare Bloom
Sent: Tuesday, June 29, 2021 2:03 PM
To: Ryan Long
Cc: devel@rtems.org
Subject: Re: [PATCH v2 2/2] rtems-exeinfo.cpp: Fix division by zero errors
On Tue, Jun 29, 2021 at 11:50 AM Ryan Long wrote:
>
> CID
Also, if you do not have time, we could research a solution ( guidance
is welcome! )
Alan
On Tue, Jun 29, 2021 at 3:09 PM Alan Cudmore wrote:
>
> On Tue, Jun 29, 2021 at 3:01 PM Sebastian Huber
> wrote:
> >
> > On 29/06/2021 20:56, Alan Cudmore wrote:
> > > I understand the move in that commit
Hello mentors,
Here is the code for my pre-commit hook script. How it works is by default,
upon commiting it outputs a warning stating the number of style issues in
case there are mismatches.
The user can trigger the strict mode which gives a more detailed output of
the style issues by running
On Tue, Jun 29, 2021 at 3:01 PM Sebastian Huber
wrote:
>
> On 29/06/2021 20:56, Alan Cudmore wrote:
> > I understand the move in that commit now.
> > Maybe it's not working on the single core models because the code is
> > conditional for ARMV7 + A or ARMV8?
> >
On Tue, Jun 29, 2021 at 11:50 AM Ryan Long wrote:
>
> CID 1503008: Division or modulo by zero
> CID 1503015: Division or modulo by zero
>
> Closes #4416
> ---
> linkers/rtems-exeinfo.cpp | 20
> 1 file changed, 16 insertions(+), 4 deletions(-)
>
> diff --git
On 29/06/2021 20:56, Alan Cudmore wrote:
I understand the move in that commit now.
Maybe it's not working on the single core models because the code is
conditional for ARMV7 + A or ARMV8?
https://git.rtems.org/rtems/tree/bsps/arm/shared/start/start.S#n485
I think the single core RPIs are
Hi Sebastian,
I understand the move in that commit now.
Maybe it's not working on the single core models because the code is
conditional for ARMV7 + A or ARMV8?
https://git.rtems.org/rtems/tree/bsps/arm/shared/start/start.S#n485
I think the single core RPIs are ARM1176JZF-S (ARMv6Z).
Do the
PS: Wouldn't it make sense to move these even further up? There is for
example ./spec/build/bsps/riscv/riscv/optfdtcpyro.yml too. A short md5
shows that it's the same file again.
Best regards
Christian
On 29/06/2021 20:23, Christian Mauderer wrote:
arm/imx7, arm/altcycv_devkit,
arm/imx7, arm/altcycv_devkit, arm/raspberrypi and arm/beagleboneblack
compile with the patch. I tried libbsd media01 on beagleboneblack and it
works.
Best regards
Christian
On 29/06/2021 18:56, Gedare Bloom wrote:
I don't have an arm toolchain, if someone can pick this up and test
compile
Hello Gedare,
I _think_ Sebastian tested it on the same evaluation board that I used
for writing the SD driver: STM32H743I-EVAL2. I'm not entirely sure
whether the "2" is really relevant or whether that's part of an order
number at a distributor.
Best regards
Christian
On 29/06/2021
Hello Stephen,
testing SD drivers is a bit tricky. You can use the media01 and a SD
card. As soon as the driver works, media01 will mount the SD card and
you can write to it.
I wrote a SD driver just recently. What was really useful for me was a
logic analyzer that can be handled by sigrok
CID 1503008: Division or modulo by zero
CID 1503015: Division or modulo by zero
Closes #4416
---
linkers/rtems-exeinfo.cpp | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/linkers/rtems-exeinfo.cpp b/linkers/rtems-exeinfo.cpp
index 1e6d4b4..ec51597 100644
CID 1503709: Division or modulo by float zero
Closes #4415
---
tester/covoar/ReportsBase.cc | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tester/covoar/ReportsBase.cc b/tester/covoar/ReportsBase.cc
index b4a755c..328980d 100644
--- a/tester/covoar/ReportsBase.cc
+++
I see STM32H7 was added to the nexus last year in March. Was
libbsd/networking tested on any hardware devices?
Thanks,
Gedare
___
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
No good reason. I'll make those changes and send V2.
-Original Message-
From: Gedare Bloom
Sent: Tuesday, June 29, 2021 12:13 PM
To: Ryan Long
Cc: devel@rtems.org
Subject: Re: [PATCH v1 1/2] ReportsBase.cc: Remove possible division by zero
On Tue, Jun 29, 2021 at 11:06 AM Ryan Long
On Tue, Jun 29, 2021 at 11:55 AM Matthew Joyce wrote:
>
> Dr. Joel,
>
> Thanks! This made me re-read your discord workflow advice from last
> week, and it brings me to another question...You wrote:
>
> "You don't have to rebuild the entire toolsuite at this point. Just
> add a functional test for
On Tue, Jun 29, 2021 at 11:12 AM Gedare Bloom wrote:
>
> On Tue, Jun 29, 2021 at 11:06 AM Ryan Long wrote:
> >
> > CID 1503709: Division or modulo by float zero
> >
> > Closes #4415
> > ---
> > tester/covoar/ReportsBase.cc | 10 +++---
> > 1 file changed, 7 insertions(+), 3 deletions(-)
> >
On Tue, Jun 29, 2021 at 11:06 AM Ryan Long wrote:
>
> CID 1503008: Division or modulo by zero
> CID 1503015: Division or modulo by zero
>
> Closes #4416
> ---
> linkers/rtems-exeinfo.cpp| 20
> tester/covoar/ReportsBase.cc | 2 +-
> 2 files changed, 17 insertions(+), 5
On Tue, Jun 29, 2021 at 11:06 AM Ryan Long wrote:
>
> CID 1503709: Division or modulo by float zero
>
> Closes #4415
> ---
> tester/covoar/ReportsBase.cc | 10 +++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/tester/covoar/ReportsBase.cc
CID 1503008: Division or modulo by zero
CID 1503015: Division or modulo by zero
Closes #4416
---
linkers/rtems-exeinfo.cpp| 20
tester/covoar/ReportsBase.cc | 2 +-
2 files changed, 17 insertions(+), 5 deletions(-)
diff --git a/linkers/rtems-exeinfo.cpp
CID 1503709: Division or modulo by float zero
Closes #4415
---
tester/covoar/ReportsBase.cc | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/tester/covoar/ReportsBase.cc b/tester/covoar/ReportsBase.cc
index b4a755c..31dc429 100644
--- a/tester/covoar/ReportsBase.cc
I don't have an arm toolchain, if someone can pick this up and test
compile the affected BSPs, it looks ok to me.
On Sun, Jun 27, 2021 at 2:46 PM pranav wrote:
>
> ---
> .../arm/altera-cyclone-v/bspalteracyclonev.yml | 8
> spec/build/bsps/arm/beagle/grp.yml | 8
Dr. Joel,
Thanks! This made me re-read your discord workflow advice from last
week, and it brings me to another question...You wrote:
"You don't have to rebuild the entire toolsuite at this point. Just
add a functional test for sig2str.c to the testsuite (e.g. psxsigNN or
similar based on
Hi Zack,
Please provide a full name in your git-commit author metadata (git-config.user)
Please use a short tag at the start of your commit to identify the
scope, in this case, it will be "posix/timer"
Check typo "addded" in your commit message. I think this commit is
related to a ticket? if it
This should be fixed now.
On Tue, Jun 29, 2021 at 9:10 AM Gedare Bloom wrote:
>
> I'll look at this, I didn't spend much time on the ilp32 or the vck190
> targets. There's probably something that isn't supported in ilp32 mode
> in the boot code.
>
> On Mon, Jun 28, 2021 at 2:53 AM Sebastian
Looks good.
On 6/29/2021 11:04, Gedare Bloom wrote:
---
spec/build/bsps/aarch64/xilinx-versal/abi.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/spec/build/bsps/aarch64/xilinx-versal/abi.yml
b/spec/build/bsps/aarch64/xilinx-versal/abi.yml
index 67340ffa08..90bfca9f76 100644
---
Jan, please confirm. Thanks
On Mon, Jun 28, 2021 at 7:06 AM Kinsey Moore wrote:
>
> Move the moduleid register to the correct offset according to Cadence IP
> documentation.
> ---
> bsps/include/dev/spi/cadence-spi-regs.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git
On 6/29/2021 11:10, Gedare Bloom wrote:
On Tue, Jun 29, 2021 at 6:34 AM Kinsey Moore wrote:
I suppose this could have been configured in libbsd (possibly
config.inc) instead if that's what you're getting at. The overall goal
is to be able to run the tests that use the network on any one of the
ok
On Mon, Jun 28, 2021 at 7:04 AM Kinsey Moore wrote:
>
> The existing fix for the ZynqMP UART hardware bug only caught the vast
> majority of instances where it could occur. To fully fix the data
> corruption, this fix must be applied after every baud rate change. This
> makes the logic reset
Run with --no-clean option and see if you can find the config.log for
the expat package. Otherwise, you might try to build that expat
package by hand to see what is failing.
On Tue, Jun 29, 2021 at 1:43 AM Richi Dubey wrote:
>
> These are the last few lines of the report:
>
>
On Tue, Jun 29, 2021 at 6:34 AM Kinsey Moore wrote:
>
> I suppose this could have been configured in libbsd (possibly
> config.inc) instead if that's what you're getting at. The overall goal
> is to be able to run the tests that use the network on any one of the
> board variants that this BSP
Hi,
I'm working on updating the ZynqMP board's nexus devices so that it will load
SDHCI device drivers, but I'm not sure what the best way to test the drivers
is. I don't see an existing test for SDIO functionality in the libbsd test
suites.
Also, does anyone have any advice for running RTEMS
---
spec/build/bsps/aarch64/xilinx-versal/abi.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/spec/build/bsps/aarch64/xilinx-versal/abi.yml
b/spec/build/bsps/aarch64/xilinx-versal/abi.yml
index 67340ffa08..90bfca9f76 100644
--- a/spec/build/bsps/aarch64/xilinx-versal/abi.yml
+++
On Tue, Jun 29, 2021 at 7:50 AM Sebastian Huber
wrote:
>
> On 29/06/2021 15:48, Kinsey Moore wrote:
> > On 6/9/2021 00:27, Sebastian Huber wrote:
> >> On 08/06/2021 22:18, Kinsey Moore wrote:
> >>> Provide the options necessary to enable any combination of CGEM ethernet
> >>> interfaces in
---
cpukit/include/rtems/posix/timer.h| 1 +
cpukit/posix/src/psxtimercreate.c | 5 +-
cpukit/posix/src/timergettime.c | 79 ---
testsuites/psxtests/psxtimer02/psxtimer.c | 38 ++-
4 files changed, 98 insertions(+), 25 deletions(-)
diff
I'll look at this, I didn't spend much time on the ilp32 or the vck190
targets. There's probably something that isn't supported in ilp32 mode
in the boot code.
On Mon, Jun 28, 2021 at 2:53 AM Sebastian Huber
wrote:
>
> Hello Gedare,
>
> I get linker errors with this BSP:
>
> Waf: Entering
On 29/06/2021 15:48, Kinsey Moore wrote:
On 6/9/2021 00:27, Sebastian Huber wrote:
On 08/06/2021 22:18, Kinsey Moore wrote:
Provide the options necessary to enable any combination of CGEM ethernet
interfaces in LibBSD. The default is still CGEM3, so this should
continue to operate as expected
On 6/9/2021 00:27, Sebastian Huber wrote:
On 08/06/2021 22:18, Kinsey Moore wrote:
Provide the options necessary to enable any combination of CGEM ethernet
interfaces in LibBSD. The default is still CGEM3, so this should
continue to operate as expected on typical Zynq Ultrascale+ MPSoC
On Tue, Jun 29, 2021 at 8:07 AM Matthew Joyce wrote:
>
> Dear Mentors,
>
> I've implemented my new functions locally in Newlib, created basic
> tests in RTEMS, now I want to add the patches to the RSB to test them.
>
> I think I understand that once I submit these patches to newlib, they
> should
Dear Mentors,
I've implemented my new functions locally in Newlib, created basic
tests in RTEMS, now I want to add the patches to the RSB to test them.
I think I understand that once I submit these patches to newlib, they
should only include the added lines of source code...not any changes
I suppose this could have been configured in libbsd (possibly
config.inc) instead if that's what you're getting at. The overall goal
is to be able to run the tests that use the network on any one of the
board variants that this BSP supports. My takeaway from the earlier
conversation on the
These are the last few lines of the report:
bindir=/home/richi/quick-start/LatestMaster/rtems/6/bin
--exec-prefix=/home/richi/quick-start/LatestMaster/rtems/6
--includedir=/home/richi/quick-start/LatestMaster/rtems/6/include
--libdir=/home/richi/quick-start/LatestMaster/rtems/6/lib
Hi,
I ran the following commands
mkdir -p $HOME/quick-start/LatestMaster/src
cd $HOME/quick-start/LatestMaster/src
git clone https://github.com/RTEMS/rtems-source-builder.git rsb
git clone https://github.com/RTEMS/rtems.git
cd $HOME/quick-start/LatestMaster/src/rsb/rtems
I have sent a new patch, hopefully, it captures the important things we
need to check for Strong APA working properly. (Task shifting during task
arrival/departure).
On Fri, Jun 25, 2021 at 6:44 PM Sebastian Huber <
sebastian.hu...@embedded-brains.de> wrote:
> On 25/06/2021 14:19, Richi Dubey
Update smpstrongapa01 to account for task shifting.
---
testsuites/smptests/smpstrongapa01/init.c | 66 ---
1 file changed, 47 insertions(+), 19 deletions(-)
diff --git a/testsuites/smptests/smpstrongapa01/init.c
b/testsuites/smptests/smpstrongapa01/init.c
index
68 matches
Mail list logo