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

2021-09-27 Thread Christian MAUDERER
Hello, if I didn't miss a documentation patch regarding this, the topic is still open. I delayed it a bit because I didn't had the time back then and I think Chris was quite busy too at that time. I hope now is the right time to re-start the discussion. Am 21.07.21 um 08:58 schrieb Chris Joh

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

2021-09-27 Thread Chris Johns
On 27/9/21 6:17 pm, Christian MAUDERER wrote: > Hello, > > if I didn't miss a documentation patch regarding this, the topic is still > open. > I delayed it a bit because I didn't had the time back then and I think Chris > was > quite busy too at that time. I hope now is the right time to re-star

Re: [PATCHES] Fix PPP in libbsd and optimize ATSAM console

2021-09-27 Thread Christian MAUDERER
Hello, same like for another patch set: There is a pending discussion for this one. Chris: Back when I posted the patch set you mentioned that you are not entirely happy about it in a discord chat (about 17.08.2021 6:50 in UTC time zone). Is there still an open point left of can I push the pat

Re: [PATCHES] Fix PPP in libbsd and optimize ATSAM console

2021-09-27 Thread Chris Johns
On 27/9/21 6:24 pm, Christian MAUDERER wrote: > same like for another patch set: There is a pending discussion for this one. Yeap and so we need to resolve it. I have reordered your original post so the chat follows. I hope that is OK? > Is there still an open point left of can I push the patche

Re: [PATCHES] Fix PPP in libbsd and optimize ATSAM console

2021-09-27 Thread Christian MAUDERER
Hello Chris, Am 27.09.21 um 10:51 schrieb Chris Johns: On 27/9/21 6:24 pm, Christian MAUDERER wrote: same like for another patch set: There is a pending discussion for this one. Yeap and so we need to resolve it. I have reordered your original post so the chat follows. I hope that is OK? O

[PATCH rtems-tools v3 1/7] CoverageRanges.cc: Fix formatting

2021-09-27 Thread Ryan Long
--- tester/covoar/CoverageRanges.cc | 13 + tester/covoar/CoverageRanges.h | 13 +++-- 2 files changed, 12 insertions(+), 14 deletions(-) diff --git a/tester/covoar/CoverageRanges.cc b/tester/covoar/CoverageRanges.cc index 159c556..cfd58df 100644 --- a/tester/covoar/CoverageR

[PATCH rtems-tools v3 0/7] Convert and reformat pt.3

2021-09-27 Thread Ryan Long
Hi, For this series of patches, I - Removed #define for size of the discardBuff - Fixed code preventing patchset from building Thanks, Ryan Ryan Long (7): CoverageRanges.cc: Fix formatting TraceReader: Convert to C++ TraceWriter: Convert to C++ TraceReaderLogQEMU.cc: Fix formatting Tr

[PATCH rtems-tools v3 2/7] TraceReader: Convert to C++

2021-09-27 Thread Ryan Long
--- tester/covoar/ObjdumpProcessor.cc | 4 +- tester/covoar/ObjdumpProcessor.h| 2 +- tester/covoar/TargetBase.cc | 6 +- tester/covoar/TargetBase.h | 4 +- tester/covoar/TraceReaderBase.h | 4 +- tester/covoar/TraceReaderLogQEMU.cc | 128 --

[PATCH rtems-tools v3 4/7] TraceReaderLogQEMU.cc: Fix formatting

2021-09-27 Thread Ryan Long
--- tester/covoar/TraceReaderLogQEMU.cc | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/tester/covoar/TraceReaderLogQEMU.cc b/tester/covoar/TraceReaderLogQEMU.cc index d02b555..771f808 100644 --- a/tester/covoar/TraceReaderLogQEMU.cc +++ b/tester/covoar/TraceRe

[PATCH rtems-tools v3 3/7] TraceWriter: Convert to C++

2021-09-27 Thread Ryan Long
--- tester/covoar/TraceWriterBase.h | 3 ++- tester/covoar/TraceWriterQEMU.cc | 38 +- tester/covoar/TraceWriterQEMU.h | 2 +- 3 files changed, 16 insertions(+), 27 deletions(-) diff --git a/tester/covoar/TraceWriterBase.h b/tester/covoar/TraceWriterBase.h

[PATCH rtems-tools v3 5/7] TraceWriterQEMU.cc: Fix formatting

2021-09-27 Thread Ryan Long
--- tester/covoar/TraceWriterQEMU.cc | 46 +++- 1 file changed, 26 insertions(+), 20 deletions(-) diff --git a/tester/covoar/TraceWriterQEMU.cc b/tester/covoar/TraceWriterQEMU.cc index 58c177a..0df9617 100644 --- a/tester/covoar/TraceWriterQEMU.cc +++ b/tester/

[PATCH rtems-tools v3 6/7] TraceReaderLogQEMU.h: Fix formatting

2021-09-27 Thread Ryan Long
--- tester/covoar/TraceReaderLogQEMU.h | 1 - 1 file changed, 1 deletion(-) diff --git a/tester/covoar/TraceReaderLogQEMU.h b/tester/covoar/TraceReaderLogQEMU.h index 59b5d23..636df3a 100644 --- a/tester/covoar/TraceReaderLogQEMU.h +++ b/tester/covoar/TraceReaderLogQEMU.h @@ -22,7 +22,6 @@ names

[PATCH rtems-tools v3 7/7] TraceWriterQEMU.h: Fix formatting

2021-09-27 Thread Ryan Long
--- tester/covoar/TraceWriterQEMU.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tester/covoar/TraceWriterQEMU.h b/tester/covoar/TraceWriterQEMU.h index aa89860..7078837 100644 --- a/tester/covoar/TraceWriterQEMU.h +++ b/tester/covoar/TraceWriterQEMU.h @@ -44,9 +44,9 @

[PATCH rtems-docs] eng: Add rules for attribution

2021-09-27 Thread Christian Mauderer
This adds some rules how an attribution for sponsored development should look like. --- Note: This patch is more or less an early draft. See the following discussion for more background: https://lists.rtems.org/pipermail/devel/2021-September/069465.html Points that need discussion: * Are there

[PATCH rtems-tools] rtems-bin2c.c: Check length of buffer to be copied

2021-09-27 Thread Ryan Long
CID 1063892: Copy into fixed size buffer in process(). Closes #4424 --- misc/bin2c/rtems-bin2c.c | 9 + 1 file changed, 9 insertions(+) diff --git a/misc/bin2c/rtems-bin2c.c b/misc/bin2c/rtems-bin2c.c index 8e32d25..1668063 100644 --- a/misc/bin2c/rtems-bin2c.c +++ b/misc/bin2c/rtems-bi

Re: [PATCH rtems-docs] eng: Add rules for attribution

2021-09-27 Thread Joel Sherrill
The Microblaze port is interesting for attribution. I did initial work on it. Hesham added to that and got Hello on a board. Alex is close to submitting the port in a nice state. This is almost seven years across three developers.. The original work predates source code reorganisation. Alex delete

Re: [PATCH rtems-tools] rtems-bin2c.c: Check length of buffer to be copied

2021-09-27 Thread Chris Johns
On 28/9/21 7:28 am, Ryan Long wrote: > CID 1063892: Copy into fixed size buffer in process(). > > Closes #4424 > --- > misc/bin2c/rtems-bin2c.c | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/misc/bin2c/rtems-bin2c.c b/misc/bin2c/rtems-bin2c.c > index 8e32d25..1668063 100644 >

Re: [PATCH rtems-tools v3 0/7] Convert and reformat pt.3

2021-09-27 Thread Chris Johns
OK Thanks Chris On 27/9/21 10:49 pm, Ryan Long wrote: > Hi, > > For this series of patches, I > > - Removed #define for size of the discardBuff > - Fixed code preventing patchset from building > > Thanks, > Ryan > > Ryan Long (7): > CoverageRanges.cc: Fix formatting > TraceReader: Convert

Re: [PATCH rtems-docs] eng: Add rules for attribution

2021-09-27 Thread Christian MAUDERER
Hello Joel, Am 28.09.21 um 01:12 schrieb Joel Sherrill: The Microblaze port is interesting for attribution. I did initial work on it. Hesham added to that and got Hello on a board. Alex is close to submitting the port in a nice state. This is almost seven years across three developers.. The o