[PATCH v4 04/11] kern_tc.c: Replace FreeBSD event mechanism by adding pointers to function

--- cpukit/include/sys/timepps.h | 24 + cpukit/score/src/kern_tc.c | 41 2 files changed, 65 insertions(+) diff --git a/cpukit/include/sys/timepps.h b/cpukit/include/sys/timepps.h index 5703381ffa..2513298557 100644 ---

[PATCH v4 03/11] kern_tc.c: Add atomic dependencies required by the PPS API

--- cpukit/include/sys/timepps.h | 7 +++ cpukit/score/src/kern_tc.c | 7 +++ 2 files changed, 14 insertions(+) diff --git a/cpukit/include/sys/timepps.h b/cpukit/include/sys/timepps.h index 01212f0b43..5703381ffa 100644 --- a/cpukit/include/sys/timepps.h +++

[PATCH v4 05/11] timecounter.h: Rename tc_getfrequency() to _Timecounter_Get_frequency()

--- cpukit/include/rtems/score/timecounter.h | 8 cpukit/include/sys/timetc.h | 3 +++ cpukit/score/src/kern_tc.c | 2 +- 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/cpukit/include/rtems/score/timecounter.h

[PATCH v4 00/11] ENABLE PPS in RTEMS6

This is the 4th version of the patches for enabling the PPS API in RTEMS6. Gabriel Moyano (11): kern_ntptime.c: Disable freebsd features kern_ntptime.c: Add lmax() qmin() definitions kern_tc.c: Add atomic dependencies required by the PPS API kern_tc.c: Replace FreeBSD event mechanism by

[PATCH v4 02/11] kern_ntptime.c: Add lmax() qmin() definitions

--- cpukit/score/src/kern_ntptime.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cpukit/score/src/kern_ntptime.c b/cpukit/score/src/kern_ntptime.c index 610386833c..da6b425064 100644 --- a/cpukit/score/src/kern_ntptime.c +++ b/cpukit/score/src/kern_ntptime.c @@ -71,6 +71,8 @@

[PATCH v4 01/11] kern_ntptime.c: Disable freebsd features

--- cpukit/score/src/kern_ntptime.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cpukit/score/src/kern_ntptime.c b/cpukit/score/src/kern_ntptime.c index cb39133408..610386833c 100644 --- a/cpukit/score/src/kern_ntptime.c +++ b/cpukit/score/src/kern_ntptime.c @@ -73,9

[PATCH] bsps/stm32h7: clarify license of stm32h7b3i-dk BSP variant

The system_stm32h7xx.c file provided in the boards/stm/stm32h7b3i-dk subdirectory needs a bit of clarification as it references "root directory" in its license comment and it's not clear where this points out. Let's add clarification comment about it and also based on it and resulting license

[PATCH rtems-docs] c-user: Update references to --enable-* configure options.

Also updated list of SMP architectures and added enough lead in to let users know the source code was the definitive answer. --- c-user/config/general.rst | 6 +++--- c-user/config/mpci.rst| 17 c-user/config/posix-api.rst

[PATCH v4 09/11] timepps.h: PPS_SYNC defined by default

--- cpukit/include/sys/timepps.h | 1 + cpukit/score/src/kern_ntptime.c | 10 -- testsuites/sptests/spntp01/init.c | 2 +- 3 files changed, 2 insertions(+), 11 deletions(-) diff --git a/cpukit/include/sys/timepps.h b/cpukit/include/sys/timepps.h index 2513298557..30bca70c8d

[PATCH v4 06/11] kern_tc.c: Add definitions required by PPS API

--- cpukit/score/src/kern_tc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/cpukit/score/src/kern_tc.c b/cpukit/score/src/kern_tc.c index 2e2ffd8ef1..25d0837b63 100644 --- a/cpukit/score/src/kern_tc.c +++ b/cpukit/score/src/kern_tc.c @@ -56,6 +56,11 @@ #definetimecounter

[PATCH v4 08/11] kern_ntptime.c: Add define in order to remove warning

--- cpukit/score/src/kern_ntptime.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/cpukit/score/src/kern_ntptime.c b/cpukit/score/src/kern_ntptime.c index da6b425064..c1b4013e9b 100644 --- a/cpukit/score/src/kern_ntptime.c +++ b/cpukit/score/src/kern_ntptime.c @@ -58,6 +58,9 @@

[PATCH v4 11/11] testsuites/sptests: Add sppps01 test

--- spec/build/testsuites/sptests/grp.yml | 2 + spec/build/testsuites/sptests/sppps01.yml | 19 +++ testsuites/sptests/sppps01/init.c | 191 ++ 3 files changed, 212 insertions(+) create mode 100644 spec/build/testsuites/sptests/sppps01.yml create mode 100644

[PATCH v4 07/11] kern_tc.c: Enable PPS API support

--- cpukit/score/src/kern_tc.c | 4 1 file changed, 4 deletions(-) diff --git a/cpukit/score/src/kern_tc.c b/cpukit/score/src/kern_tc.c index 25d0837b63..80b1be364c 100644 --- a/cpukit/score/src/kern_tc.c +++ b/cpukit/score/src/kern_tc.c @@ -1903,7 +1903,6 @@ SYSCTL_PROC(_kern_timecounter,

[PATCH v4 10/11] timecounter.h: Add _Timecounter_Discipline()

--- cpukit/include/rtems/score/timecounter.h | 17 + cpukit/include/sys/timepps.h | 3 +++ 2 files changed, 20 insertions(+) diff --git a/cpukit/include/rtems/score/timecounter.h b/cpukit/include/rtems/score/timecounter.h index fdade06128..95e0839cff 100644 ---

Re: [PATCH 1/7] bsps/stm32h7: add board C files for stm32h757i-eval BSP

On 19/05/2022 09:03, Karel Gardas wrote: On 5/19/22 08:55, Sebastian Huber wrote: On 16/05/2022 14:12, Karel Gardas wrote: + ** +  * @attention +  * +  * Copyright (c) 2019 STMicroelectronics. +  * All rights

Re: [PATCH] bsp/arm: allocate .rtemsstack on REGION_STACK instead of on REGION_WORK

On 19/05/2022 08:59, Karel Gardas wrote: On 5/19/22 08:53, Sebastian Huber wrote: On 07/04/2022 21:15, Karel Gardas wrote: REGION_WORK may be backed by external RAM which may not be initialized in a time we need stack to work well. E.g. code loaded in flash, stack allocated on in-cpu SRAM and

Re: [PATCH 1/7] bsps/stm32h7: add board C files for stm32h757i-eval BSP

On 16/05/2022 14:12, Karel Gardas wrote: + ** + * @attention + * + * Copyright (c) 2019 STMicroelectronics. + * All rights reserved. + * + * This software is licensed under terms that can be found in the LICENSE

Re: [PATCH 7/7] bsps/stm32h7: set default SDRAM x sizes on stm32h757i-eval BSP

Apart from the two comments, the patch set is fine. With respect to the license text, maybe just copy the content of the LICENSE file into the source file. Maybe STM fixes the Github issues and provides SPDX License Identifiers in their next release. -- embedded brains GmbH Herr Sebastian

Re: [PATCH 5/7] bsps/stm32h7: provide linkcmds for SRAM, FLASH+SDRAM and SRAM+SDRAM linking

On 16/05/2022 14:12, Karel Gardas wrote: + bld.install_as("${BSP_LIBDIR}/linkcmds.flash+sdram", "linkcmds.flash+sdram") I would not use '+' in file names. -- embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.hu...@embedded-brains.de phone:

Re: [PATCH 1/7] bsps/stm32h7: add board C files for stm32h757i-eval BSP

On 19/05/2022 09:18, Karel Gardas wrote: On 5/19/22 09:06, Sebastian Huber wrote: On 19/05/2022 09:03, Karel Gardas wrote: On 5/19/22 08:55, Sebastian Huber wrote: On 16/05/2022 14:12, Karel Gardas wrote: + ** + 

Re: [PATCH] bsp/arm: allocate .rtemsstack on REGION_STACK instead of on REGION_WORK

On 07/04/2022 21:15, Karel Gardas wrote: REGION_WORK may be backed by external RAM which may not be initialized in a time we need stack to work well. E.g. code loaded in flash, stack allocated on in-cpu SRAM and data (REGION_WORK) on external SDRAM. Sponsored-By: Precidata This change could

Re: [PATCH 5/7] bsps/stm32h7: provide linkcmds for SRAM, FLASH+SDRAM and SRAM+SDRAM linking

On 19/05/2022 09:06, Karel Gardas wrote: On 5/19/22 08:58, Sebastian Huber wrote: On 16/05/2022 14:12, Karel Gardas wrote: +  bld.install_as("${BSP_LIBDIR}/linkcmds.flash+sdram", "linkcmds.flash+sdram") I would not use '+' in file names. Indeed, then there are two obvious choices: -

AW: [PATCH v3 04/11] kern_tc.c: Replace FREEBSD event mechanism by adding pointers to function

Thx, I'll send a new version of the commits today. The issue in the NTP test will also be resolved. > FREEBSD should be FreeBSD. > > On 04/05/2022 14:12, Gabriel Moyano wrote: > > --- > > cpukit/include/sys/timepps.h | 21 > > cpukit/score/src/kern_tc.c | 38