Re: [PATCH rtems v3 1/1] Add the Regulator Interface and test

2023-08-09 Thread Sebastian Huber
On 09.08.23 17:42, Joel Sherrill wrote: Updates #4924. The Regulator is an application support class which is used to deal with the scenario where there is a bursty input source which needs to be metered out to a destination sink. The maximum size of bursts needs to be known and the delivery

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-09 Thread Sebastian Huber
On 10.08.23 03:13, Chris Johns wrote: On 9/8/2023 7:12 pm, Sebastian Huber wrote: On 09.08.23 11:10, Cedric Berger wrote: On 09.08.23 10:51, Sebastian Huber wrote: We could add some text to the option description: # Defines the program prefix of tools (compiler, assembler, linker). # This

Re: [PATCH rtems v3 1/1] Add the Regulator Interface and test

2023-08-09 Thread Chris Johns
Hi Joel, This looks good. A minor nit relates to pointer checks. We have: https://docs.rtems.org/branches/master/eng/coding-conventions.html#language-and-compiler I do not know how well enforced the pointer check for NULL in `if` is? Chris On 10/8/2023 1:42 am, Joel Sherrill wrote: > Updates

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-09 Thread Chris Johns
On 9/8/2023 7:12 pm, Sebastian Huber wrote: > On 09.08.23 11:10, Cedric Berger wrote: >> On 09.08.23 10:51, Sebastian Huber wrote: >>> We could add some text to the option description: >>> >>> # Defines the program prefix of tools (compiler, assembler, linker). >>> # This option may be used to

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Chris Johns
On 10/8/2023 2:41 am, Joel Sherrill wrote: > Reading the EPICS discussion, I wonder if this should be added for all > motorola_powerpc BSP variants. > > Gedare/Chris/anyone with another board in the family? What do you think? Michael asked if the setting is present when building libc, libm etc

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Uchenna Ezeobi
On Wed, Aug 9, 2023 at 3:31 PM Vijay Kumar Banerjee wrote: > > On Wed, Aug 9, 2023 at 3:26 PM Vijay Kumar Banerjee wrote: > > > > On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill wrote: > > > > > > Vijay.. go ahead and push this if you don't mind. > > > > > Sure, I will push this. > > Pushed.

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 3:26 PM Vijay Kumar Banerjee wrote: > > On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill wrote: > > > > Vijay.. go ahead and push this if you don't mind. > > > Sure, I will push this. Pushed. Thanks. > > > I was just wondering if we could make a blanket statement that all >

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill wrote: > > Vijay.. go ahead and push this if you don't mind. > Sure, I will push this. > I was just wondering if we could make a blanket statement that all > motorola_powerpc variants need this flag. I do not have a test case other than the EPICS

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Joel Sherrill
Vijay.. go ahead and push this if you don't mind. I was just wondering if we could make a blanket statement that all motorola_powerpc variants need this flag. On Wed, Aug 9, 2023 at 12:01 PM Vijay Kumar Banerjee wrote: > On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill wrote: > > > > Reading the

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill wrote: > > Reading the EPICS discussion, I wonder if this should be added for all > motorola_powerpc BSP variants. > MVME2700 and the qemu variants already add it separately

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Joel Sherrill
Reading the EPICS discussion, I wonder if this should be added for all motorola_powerpc BSP variants. Gedare/Chris/anyone with another board in the family? What do you think? --joel On Wed, Aug 9, 2023 at 11:14 AM Uchenna Ezeobi wrote: > On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill wrote: >

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Uchenna Ezeobi
On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill wrote: > > I'm ok with this if there is a known case of an alignment exception that > causes this. > We were getting the error from the https://github.com/epics-base/epics-base/issues/29 while trying to build EPICS on RTEMS-MVME2100, doing this fixed

[PATCH rtems-examples v3] Add regulator example

2023-08-09 Thread Joel Sherrill
Updates #4924. --- misc/Makefile | 6 + misc/regulator/Makefile| 20 +++ misc/regulator/regulator_example.c | 207 + misc/regulator/rtems_config.c | 59 misc/regulator/wscript | 15 +++ misc/wscript

[PATCH rtems-docs v3] Add Regulator documentation based on V1 functionality

2023-08-09 Thread Joel Sherrill
--- README.txt | 19 +- c-user/index.rst| 1 + c-user/regulator/background.rst | 90 c-user/regulator/directives.rst | 549 c-user/regulator/index.rst | 19 +

[PATCH rtems v3 1/1] Add the Regulator Interface and test

2023-08-09 Thread Joel Sherrill
Updates #4924. The Regulator is an application support class which is used to deal with the scenario where there is a bursty input source which needs to be metered out to a destination sink. The maximum size of bursts needs to be known and the delivery method must be configured to deliver

[PATCH rtems v3 0/1] Regulator Patch Set

2023-08-09 Thread Joel Sherrill
Hi This adds the regulator interface. I have implemented every suggestion including adding a way to get some statistics and synchronizing with the Delivery thread as part of deleting the regulator. Subsequent patches posted will add an example to rtems-examples and adding a chapter to the

Re: [PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Joel Sherrill
I'm ok with this if there is a known case of an alignment exception that causes this. We didn't want to force it into every PowerPC BSP. Do you have a case that failed and was resolved by adding this? On Wed, Aug 9, 2023 at 7:36 AM Uchenna Ezeobi wrote: > Update #3767 > --- >

Re: [PATCH] samples/iostream: Produce proper begin/end message

2023-08-09 Thread Joel Sherrill
Looks ok to push. On Wed, Aug 9, 2023 at 3:37 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > Print the full test information. > --- > testsuites/samples/iostream/init.cc | 9 + > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git

Re: [PATCH] cpukit/riscv : RISCV multitasking with non SMP

2023-08-09 Thread Sebastian Huber
Hello Kevin, the patch itself is all right, except some formatting issues. However, I still don't understand why it is necessary. I did run the test suite on a non-SMP system and it worked well. Interrupts should be enabled in void _Thread_Handler( void ) { Thread_Control *executing;

Status of CAN API?

2023-08-09 Thread Philip Kirkpatrick
Hello, Some people on our team here at Reflex are preparing to implement some CAN drivers. Specifically for TMS570 and for ZynqMP-RPU (side note my latest patch for that on Jun 29th is still sitting there unreviewed). I was wondering what the current status of the CAN API is. I saw in August of

[PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

2023-08-09 Thread Uchenna Ezeobi
Update #3767 --- spec/build/bsps/powerpc/motorola_powerpc/abi.yml | 4 1 file changed, 4 insertions(+) diff --git a/spec/build/bsps/powerpc/motorola_powerpc/abi.yml b/spec/build/bsps/powerpc/motorola_powerpc/abi.yml index 6734e9babf..2438c30f1d 100644 ---

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-09 Thread Sebastian Huber
On 09.08.23 11:10, Cedric Berger wrote: On 09.08.23 10:51, Sebastian Huber wrote: We could add some text to the option description: # Defines the program prefix of tools (compiler, assembler, linker). # This option may be used to build RTEMS with a vendor tool suite. # Please note that using

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-09 Thread Cedric Berger
On 09.08.23 10:51, Sebastian Huber wrote: We could add some text to the option description: # Defines the program prefix of tools (compiler, assembler, linker). # This option may be used to build RTEMS with a vendor tool suite. # Please note that using tool suites not provided by the RTEMS

Re: [PATCH] build: Add PROGRAM_PREFIX option

2023-08-09 Thread Sebastian Huber
On 09.08.23 03:26, Chris Johns wrote: On 8/8/2023 11:14 pm, Sebastian Huber wrote: On 08.08.23 08:06, Chris Johns wrote: n 7/8/2023 4:06 pm, Sebastian Huber wrote: On 07.08.23 00:25, Chris Johns wrote: On 4/8/2023 4:39 pm, Sebastian Huber wrote: On 04.08.23 08:22, Chris Johns wrote: On

[PATCH] samples/iostream: Produce proper begin/end message

2023-08-09 Thread Sebastian Huber
Print the full test information. --- testsuites/samples/iostream/init.cc | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/testsuites/samples/iostream/init.cc b/testsuites/samples/iostream/init.cc index 22d973019b..ddd38a9d15 100644 ---

[PATCH] cpukit/riscv : RISCV multitasking with non SMP

2023-08-09 Thread Kevin Eyssartier
Modification of RISCV multitasking in case of non SMP configuration (single core). The previous RISC-V multitasking implementation was not functionnal in case of non-SMP configuration. This modification implement the _RISCV_Start_multitasking function even if the SMP flag is disabled. --- Here

[PATCH] arm: Add cache support for ARM926EJ-S

2023-08-09 Thread Sebastian Huber
The ARM926EJ-S is an ARMv5T architecture processor and lacks some features of ARMv6 processors such as the ARM1176JZF-S. --- bsps/arm/shared/cache/cache-cp15.c| 26 +++ .../score/cpu/arm/include/libcpu/arm-cp15.h | 26 --- 2 files changed, 48