Re: [PATCH rtems] bsps/imxrt: Add attribution in file headers

2021-07-20 Thread Sebastian Huber
On 21/07/2021 02:46, Chris Johns wrote: On 21/7/21 6:47 am, Gedare Bloom wrote: This seems fine to me. We don't have any standard way to document this kind of attribution. However, we have had individual authors add their names below their company's copyright. It might make sense to put the

Re: [PATCH] bsp: Remove fatal from exit(0). Add extended heap error output

2021-07-20 Thread Sebastian Huber
Hello Chris, thanks, this is a nice improvement. On 21/07/2021 07:17, chr...@rtems.org wrote: --- a/bsps/shared/start/bspfatal-default.c +++ b/bsps/shared/start/bspfatal-default.c @@ -22,15 +22,24 @@ void bsp_fatal_extension( { #if BSP_VERBOSE_FATAL_EXTENSION Thread_Control

[PATCH] bsp: Remove fatal from exit(0). Add extended heap error output

2021-07-20 Thread chrisj
From: Chris Johns --- bsps/shared/start/bspfatal-default.c | 53 +--- 1 file changed, 49 insertions(+), 4 deletions(-) diff --git a/bsps/shared/start/bspfatal-default.c b/bsps/shared/start/bspfatal-default.c index 0289dbda63..5e93b21b04 100644 ---

Re: LwIP for RTEMS state querry Was: [PATCH rtems-lwip v2] STM32 lwIP addition

2021-07-20 Thread Vijay Kumar Banerjee
Hi Pavel, On Tue, Jul 20, 2021 at 1:13 PM Pavel Pisa wrote: > > Hello everybody, > > I have no technical stuff to share for now, > but I am happy that work on LwIP alternative > stack for RTEMS continues on. > > I have question which repo is considered as > mainline (integration point) for this

Re: [PATCH v2 1/5] rtems-utils.h: Create ostream_guard

2021-07-20 Thread Chris Johns
Close, some minor formatting issues On 21/7/21 1:06 am, Ryan Long wrote: > --- > rtemstoolkit/rtems-utils.h | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/rtemstoolkit/rtems-utils.h b/rtemstoolkit/rtems-utils.h > index 4ce9c68..c4a262e 100644 > ---

Re: [PATCH] build: Add the BSP family to the enable set

2021-07-20 Thread Chris Johns
On 20/7/21 8:25 pm, Sebastian Huber wrote: > On 15/07/2021 08:19, Sebastian Huber wrote: >> This makes it possible to use the BSP family in expressions of the enabled-by >> attribute. >> --- >>   wscript | 7 ++- >>   1 file changed, 6 insertions(+), 1 deletion(-) >> >> diff --git a/wscript

Re: [PATCH rtems] bsps/imxrt: Add attribution in file headers

2021-07-20 Thread Chris Johns
On 21/7/21 6:47 am, Gedare Bloom wrote: > This seems fine to me. We don't have any standard way to document this > kind of attribution. However, we have had individual authors add their > names below their company's copyright. It might make sense to put the > sponsorship part beneath the copyright

[PATCH] linker_set.h: Add alignof implementation for when not C11 or C++11

2021-07-20 Thread Joel Sherrill
The default implementation was completely broken. Use the GCC specific __alignof__ if compiling for C99 or C++03. If not C++11, C11, or GCC, then it is an error. --- freebsd/sys/sys/linker_set.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/freebsd/sys/sys/linker_set.h

Re: [PATCH rtems] bsps/imxrt: Add attribution in file headers

2021-07-20 Thread Gedare Bloom
This seems fine to me. We don't have any standard way to document this kind of attribution. However, we have had individual authors add their names below their company's copyright. It might make sense to put the sponsorship part beneath the copyright of the sponsored company/person? Not a big

RE: Help with RTEMS Tester Results for uC5282

2021-07-20 Thread gerberhe11
If you’d like I can send over the Git repo that has the Qemu that I’ve been working on that includes the current state of the uC5282 emulation! Let me know if this would be good, and I’ll make sure it is completely up to date with what I have working locally. I’ll continue to look into some of

Re: LwIP for RTEMS state querry Was: [PATCH rtems-lwip v2] STM32 lwIP addition

2021-07-20 Thread Pavel Pisa
Hello everybody, I have no technical stuff to share for now, but I am happy that work on LwIP alternative stack for RTEMS continues on. I have question which repo is considered as mainline (integration point) for this work. Is it devel branch of Vijay Kumar Banerjee's repo

Re: Help with RTEMS Tester Results for uC5282

2021-07-20 Thread Joel Sherrill
On Tue, Jul 20, 2021, 12:22 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > Hello, > > On 20/07/2021 00:43, gerberh...@gmail.com wrote: > > Dr. Bloom mentioned that some of these results could likely be due to > > flawed TLS implementation of m68k. > > this could be also an

[PATCH v2 2/5] rtems-exeinfo.cpp: Restore ostream format

2021-07-20 Thread Ryan Long
CID 1503006: Not restoring ostream format CID 1503007: Not restoring ostream format Used a variable to store the format of the ostream before any changes, and copied what was originally there back into the stream before returning from the function. Closes #4469 --- linkers/rtems-exeinfo.cpp | 7

[PATCH v2 5/5] ReportsText.cc: Restore ostream format

2021-07-20 Thread Ryan Long
CID 1505940: Not restoring ostream format Save format of stream before changing it, and change it back before returning. Closes #4472 --- tester/covoar/ReportsText.cc | 5 + 1 file changed, 5 insertions(+) diff --git a/tester/covoar/ReportsText.cc b/tester/covoar/ReportsText.cc index

[PATCH v2 4/5] ReportsHtml.cc: Restore ostream format

2021-07-20 Thread Ryan Long
CID 1505939: Not restoring ofstream format Save format of stream before changing it, and change it back before returning. Closes #4471 --- tester/covoar/ReportsHtml.cc | 5 + 1 file changed, 5 insertions(+) diff --git a/tester/covoar/ReportsHtml.cc b/tester/covoar/ReportsHtml.cc index

[PATCH v2 3/5] CoverageMapBase.cc: Restore ostream format

2021-07-20 Thread Ryan Long
CID 1503022: Not restoring ostream format Save format of stream before changing it, and change it back before returning. Closes #4470 --- tester/covoar/CoverageMapBase.cc | 6 ++ 1 file changed, 6 insertions(+) diff --git a/tester/covoar/CoverageMapBase.cc

[PATCH v2 0/5] Fix ostream format Coverity issues

2021-07-20 Thread Ryan Long
Hi, This patchset has Chris's recommended fix for these issues. Thanks, Ryan Ryan Long (5): rtems-utils.h: Create ostream_guard rtems-exeinfo.cpp: Restore ostream format CoverageMapBase.cc: Restore ostream format ReportsHtml.cc: Restore ostream format ReportsText.cc: Restore ostream

[PATCH v2 1/5] rtems-utils.h: Create ostream_guard

2021-07-20 Thread Ryan Long
--- rtemstoolkit/rtems-utils.h | 18 ++ 1 file changed, 18 insertions(+) diff --git a/rtemstoolkit/rtems-utils.h b/rtemstoolkit/rtems-utils.h index 4ce9c68..c4a262e 100644 --- a/rtemstoolkit/rtems-utils.h +++ b/rtemstoolkit/rtems-utils.h @@ -47,6 +47,24 @@ namespace rtems

[PATCH rtems] bsps/imxrt: Add attribution in file headers

2021-07-20 Thread Christian Mauderer
Onto Innovations Incorporated originally sponsored the development of this BSP. This patch adds the attribution for it. The patch also fixes an old license header in bspstarthooks.c that was accidentally copied into that file. --- bsps/arm/imxrt/console/console.c | 1 +

AW: Building a library for use with RTEMS via CMake

2021-07-20 Thread Andre.Nahrwold
Hey guys, that was so far very helpful in understanding the problem in more depth. Putting in the right flags at least reduced the linker errors by nearly 99% or so, Thanks. Maybe I am on the right path now and just need to tweak it a bit to make it work. The rtems-cmake repository looks also

Re: [PATCH] Fixes for TMS570 BSP

2021-07-20 Thread Sebastian Huber
On 16/07/2021 13:59, Robin Müller wrote:  I think this patch was forgotten. Pushing it up :-) Thanks, I checked it in. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94

Re: [PATCH] STM32H7 ethernet pin corrections

2021-07-20 Thread Sebastian Huber
On 16/07/2021 14:32, Robin Mueller wrote: These patches were submitted a few months ago, but it was found out that the default-by-family: [] were missing in the GPIO .yml lines. This was fixed in this patch. This patch accounts for different pins for the ETH peripheral on STM32H7 devices. For

Re: [PATCH] build: Add the BSP family to the enable set

2021-07-20 Thread Sebastian Huber
On 15/07/2021 08:19, Sebastian Huber wrote: This makes it possible to use the BSP family in expressions of the enabled-by attribute. --- wscript | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/wscript b/wscript index 487951fdba..1331ae149e 100755 --- a/wscript +++

Re: Building a library for use with RTEMS via CMake

2021-07-20 Thread Robin Müller
I managed to compile our project with CMake, using this repository: https://github.com/spacefisch/rtems-cmake It uses the pkg-config files to set up the cross compiler given the BSP and RTEMS prefix information. Maybe this can help you Kind Regards Robin On Tue, 20 Jul 2021 at 00:50, Michael