Skip to content

Conversation

@n9wxu
Copy link
Contributor

@n9wxu n9wxu commented Apr 10, 2023

This change simply adds a short paragraph and a link to the community supported demos.

Test Steps

No code was affected

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@n9wxu n9wxu requested a review from a team as a code owner April 10, 2023 23:22
and the Demo/Common directories. If this is the only port you are
interested in then all the other directories can be ignored.

Community Supported Demos
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you consider adding this in the main readme itself? @n9wxu

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An off topic(?) point: I would have like this to be a .md file so that github could render the text in a more readable manner. The "community supported demos" line should be a highlighted (or a heading) section to attract user's attention to it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe not so off topic - I propose to make it an md file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@amazonKamath I did not consider the main readme because I figured a user would be in the demo folder looking for additional demos.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AniruddhaKanhere I also like the md file. I did not use an md file because the current file is txt and it would want additional formatting to make a pretty md.

rawalexe
rawalexe previously approved these changes Apr 11, 2023
kstribrnAmzn
kstribrnAmzn previously approved these changes Apr 11, 2023
and the Demo/Common directories. If this is the only port you are
interested in then all the other directories can be ignored.

Community Supported Demos
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An off topic(?) point: I would have like this to be a .md file so that github could render the text in a more readable manner. The "community supported demos" line should be a highlighted (or a heading) section to attract user's attention to it.

and the Demo/Common directories. If this is the only port you are
interested in then all the other directories can be ignored.

Community Supported Demos
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe not so off topic - I propose to make it an md file.

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>
@n9wxu n9wxu dismissed stale reviews from kstribrnAmzn and rawalexe via bef0ef6 April 11, 2023 18:28
Copy link
Member

@AniruddhaKanhere AniruddhaKanhere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @n9wxu for this PR!
I approve of this change.

@n9wxu n9wxu merged commit 1309654 into FreeRTOS:main Apr 11, 2023
moninom1 pushed a commit to moninom1/FreeRTOS that referenced this pull request Apr 27, 2023
…e.txt (FreeRTOS#987)

* Added a link to the community supported demos repo in the demos readme.txt

* Update FreeRTOS/Demo/readme.txt

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>

* Update and rename readme.txt to readme.md

---------

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>
moninom1 pushed a commit to moninom1/FreeRTOS that referenced this pull request Apr 27, 2023
…e.txt (FreeRTOS#987)

* Added a link to the community supported demos repo in the demos readme.txt

* Update FreeRTOS/Demo/readme.txt

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>

* Update and rename readme.txt to readme.md

---------

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>
amazonKamath pushed a commit that referenced this pull request Apr 27, 2023
* Update Readme for the FreeRTOS_PLUS_TCP_ECHO_QEMU_msp2 (#939)

Update Readme for the FreeRTOS_PLUS_TCP_ECHO_QEMU_msp2

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>

* Update TCPEchoClient_SingleTasks.c (#968)

Bug fix for #967

* Remove cast from configMINIMAL_STACK_SIZE (#971)

The configMINIMAL_STACK_SIZE was defined to PTHREAD_STACK_MIN with cast
to unsigned short. This cast cab be too restrictive and result in
truncation on some platforms. This commit removes this unnecessary cast.

Related issue - #967

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add unit tests for GetStaticBuffer functions (#964)

Add unit tests for the various ...GetStaticBuffer() functions added in
FreeRTOS/FreeRTOS-Kernel#641.
---------

Co-authored-by: Paul Bartell <pbartell@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Prepare startup.c for use with --gc-sections

Ensure that the handler2_address_const location will be correctly
aligned when compiled to a separate section.

Add the 'used' attribute to isr_vector to ensure it will not be elided
from the output.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Fix use of -ffunction-sections -fdata-sections

For these options to be of any use, they must be passed to the
compiler when building object files and then the --gc-sections flag
must be passed to the linker.

Add -ffunction-sections -fdata-sections to CFLAGS and then add
-Wl,--gc-sections to LDFLAGS.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Enable picolibc TLS when needed

When building with Picolibc that has TLS support, enable the relevant
FreeRTOS code.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Provide picolibc syscall implementation

Instead of a set of POSIX-compatible APIs as needed by newlib,
picolibc needs a FILE struct allocated that references a function to
output a single character. Picolibc also doesn't need sbrk as it has
its own version

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Add TLS support to linker script

Allocate ROM for initialized thread local storage variables. Allocate
TLS offsets for all thread local storage variables.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Define symbols needed for picolibc sbrk implementation

Picolibc has an internal version of sbrk that uses slightly different
symbol names than the demo provides. Add these in the linker script.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/CORTEX_M3_MPS2_QEMU_GCC: Add option to build with picolibc

When built with PICOLIBC=1, selects picolibc as the C library by
removing the newlib-nano linker flags and adding
--specs=picolibc.specs to the compiler flags. This also selects the
integer-only printf variant provided within picolibc to reduce flash
use by adding -DPICOLIBC_INTEGER_PRINTF_SCANF.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/RISC-V_RV32_QEMU_VIRT_GCC: Enable configUSE_TRACE_FACILITY

I get a build error when this is not set as it (or
configGENERATE_RUN_TIME_STATS) are required when using
configUSE_STATS_FORMATTING_FUNCTIONS

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/RISC-V_RV32_QEMU_VIRT_GCC: Set -march=rv32imac_zicsr

Need to add _zicsr for more recent Risc-V toolchains which don't add
this extension to the default set anymore.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/RISC-V_RV32_QEMU_VIRT_GCC: Enable picolibc TLS when needed

When building with Picolibc that has TLS support, enable the relevant
FreeRTOS code.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/RISC-V_RV32_QEMU_VIRT_GCC: Add TLS support to linker script

Allocate ROM for initialized thread local storage variables. Allocate
TLS offsets for all thread local storage variables.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Demo/RISC-V_RV32_QEMU_VIRT_GCC: Add option to build with picolibc

When built with PICOLIBC=1, selects picolibc as the C library, uses
semihosting to display messages and uses picolibc stdio for output.

Signed-off-by: Keith Packard <keithpac@amazon.com>

* Improve vAssertCalled function to include filename / line number info.

* Enable newlib and picolibc thread local storage

* Increase minimal stack size config to 2048 words

Accomodates increased usage due to newlib thread local storage

* Add register tests to H743ZI2 demo project (#977)

Add register tests to H743ZI2 demo project.

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: kar-rahul-aws <karahulx@amazon.com>

* Add register tests to Nucleo-L152RE project (#982)

* Add reg tests for CORTEX_MPU_M3_NUCLEO_L152RE

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: kar-rahul-aws <karahulx@amazon.com>

* Set configMAX_SYSCALL_INTERRUPT_PRIORITY to 4 (#984)

Set configMAX_SYSCALL_INTERRUPT_PRIORITY to 4 instead of 5 to avoid hitting the assert in `Source/portable/GCC/ARM_CM3/port.c` (`configASSERT( ( configMAX_SYSCALL_INTERRUPT_PRIORITY & 0x1U ) == 0U ); `)

* Added a link to the community supported demos repo in the demos readme.txt (#987)

* Added a link to the community supported demos repo in the demos readme.txt

* Update FreeRTOS/Demo/readme.txt

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>

* Update and rename readme.txt to readme.md

---------

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>

* Added support for static memory allocation in FreeRTOS-Plus-CLI (#983)

* Added support for static memory allocation in FreeRTOS-Plus-CLI

* Removed relative include path

* removed whitespace changes

* Removed whitespace changes

* Code review suggestions

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Fix spell check

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

---------

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Marc-André Harvey <marc-andre.harvey@d-ta.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add reg tests to LPC55S69 project (#989)

* Update LPCXpresso55S69 SDK to 2.13.1

* Enable print from non-secure side

* Add register tests

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Update M33F simulator Keil example (#990)

* Update M33F simulator Keil example

The example is updated to use latest CMSIS 5.9.0 and Device startup
2.1.0.

Signed-off-by: Devaraj Ranganna <devaraj.ranganna@arm.com>

* Increase timeout and correct config file path in the secure project

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Remove auto-generated files

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

---------

Signed-off-by: Devaraj Ranganna <devaraj.ranganna@arm.com>
Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>

* Add Reg tests for CORTEX M33F Keil Simulator Project (#991)

* Add Reg tests for CORTEX M33F Keil Simulator Project

* Code review suggestions

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Fix hard fault because of main stack overflow

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

---------

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add reg tests to NuMaker M2351 project (#992)

* Add reg tests to NuMaker M2351 project

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add reg tests for IAR

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Fix formatting check

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Fix header check

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

---------

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add reg tests to nRF9160 projects (#1001)

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Fix incorrect file path in the project file (#1000)

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Remove non-existent entry from the project file (#996)

This removes a warning in the IDE.

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Add non trustzone (NTZ) projects for M2351 (#1003)

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

---------

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Signed-off-by: Keith Packard <keithpac@amazon.com>
Signed-off-by: Devaraj Ranganna <devaraj.ranganna@arm.com>
Co-authored-by: kar-rahul-aws <118818625+kar-rahul-aws@users.noreply.github.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>
Co-authored-by: Darian <32921628+Dazza0@users.noreply.github.com>
Co-authored-by: Paul Bartell <pbartell@amazon.com>
Co-authored-by: Keith Packard <keithpac@amazon.com>
Co-authored-by: kar-rahul-aws <karahulx@amazon.com>
Co-authored-by: Sergio Soares <eng.sergiosoares@gmail.com>
Co-authored-by: Joseph Julicher <jjulicher@mac.com>
Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>
Co-authored-by: tabarnakos <46621477+tabarnakos@users.noreply.github.com>
Co-authored-by: Marc-André Harvey <marc-andre.harvey@d-ta.com>
Co-authored-by: Devaraj Ranganna <devaraj.ranganna@arm.com>
every-breaking-wave pushed a commit to every-breaking-wave/FreeRTOS that referenced this pull request Nov 15, 2024
Zangetsu112 pushed a commit to Zangetsu112/FreeRTOS-evpp that referenced this pull request Aug 18, 2025
…e.txt (FreeRTOS#987)

* Added a link to the community supported demos repo in the demos readme.txt

* Update FreeRTOS/Demo/readme.txt

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>

* Update and rename readme.txt to readme.md

---------

Co-authored-by: Aniruddha Kanhere <60444055+AniruddhaKanhere@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants