-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Added a link to the community supported demos repo in the demos readme.txt #987
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
FreeRTOS/Demo/readme.txt
Outdated
| 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 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
FreeRTOS/Demo/readme.txt
Outdated
| 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 |
There was a problem hiding this comment.
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.
FreeRTOS/Demo/readme.txt
Outdated
| 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 |
There was a problem hiding this comment.
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 <[email protected]>
AniruddhaKanhere
left a comment
There was a problem hiding this 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.
…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 <[email protected]> * Update and rename readme.txt to readme.md --------- Co-authored-by: Aniruddha Kanhere <[email protected]>
…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 <[email protected]> * Update and rename readme.txt to readme.md --------- Co-authored-by: Aniruddha Kanhere <[email protected]>
* 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 <[email protected]> Co-authored-by: Gaurav Aggarwal <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * 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 <[email protected]> Co-authored-by: kar-rahul-aws <[email protected]> * Add register tests to Nucleo-L152RE project (#982) * Add reg tests for CORTEX_MPU_M3_NUCLEO_L152RE Signed-off-by: Gaurav Aggarwal <[email protected]> Co-authored-by: kar-rahul-aws <[email protected]> * 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 <[email protected]> * Update and rename readme.txt to readme.md --------- Co-authored-by: Aniruddha Kanhere <[email protected]> * 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 <[email protected]> * Fix spell check Signed-off-by: Gaurav Aggarwal <[email protected]> --------- Signed-off-by: Gaurav Aggarwal <[email protected]> Co-authored-by: Marc-André Harvey <[email protected]> Co-authored-by: Gaurav-Aggarwal-AWS <[email protected]> Co-authored-by: Gaurav Aggarwal <[email protected]> * 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 <[email protected]> * 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 <[email protected]> * Increase timeout and correct config file path in the secure project Signed-off-by: Gaurav Aggarwal <[email protected]> * Remove auto-generated files Signed-off-by: Gaurav Aggarwal <[email protected]> --------- Signed-off-by: Devaraj Ranganna <[email protected]> Signed-off-by: Gaurav Aggarwal <[email protected]> Co-authored-by: Gaurav Aggarwal <[email protected]> Co-authored-by: Gaurav-Aggarwal-AWS <[email protected]> * 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 <[email protected]> * Fix hard fault because of main stack overflow Signed-off-by: Gaurav Aggarwal <[email protected]> --------- Signed-off-by: Gaurav Aggarwal <[email protected]> Co-authored-by: Gaurav Aggarwal <[email protected]> * Add reg tests to NuMaker M2351 project (#992) * Add reg tests to NuMaker M2351 project Signed-off-by: Gaurav Aggarwal <[email protected]> * Add reg tests for IAR Signed-off-by: Gaurav Aggarwal <[email protected]> * Fix formatting check Signed-off-by: Gaurav Aggarwal <[email protected]> * Fix header check Signed-off-by: Gaurav Aggarwal <[email protected]> --------- Signed-off-by: Gaurav Aggarwal <[email protected]> * Add reg tests to nRF9160 projects (#1001) Signed-off-by: Gaurav Aggarwal <[email protected]> * Fix incorrect file path in the project file (#1000) Signed-off-by: Gaurav Aggarwal <[email protected]> * Remove non-existent entry from the project file (#996) This removes a warning in the IDE. Signed-off-by: Gaurav Aggarwal <[email protected]> * Add non trustzone (NTZ) projects for M2351 (#1003) Signed-off-by: Gaurav Aggarwal <[email protected]> --------- Signed-off-by: Gaurav Aggarwal <[email protected]> Signed-off-by: Keith Packard <[email protected]> Signed-off-by: Devaraj Ranganna <[email protected]> Co-authored-by: kar-rahul-aws <[email protected]> Co-authored-by: Gaurav Aggarwal <[email protected]> Co-authored-by: Gaurav-Aggarwal-AWS <[email protected]> Co-authored-by: Darian <[email protected]> Co-authored-by: Paul Bartell <[email protected]> Co-authored-by: Keith Packard <[email protected]> Co-authored-by: kar-rahul-aws <[email protected]> Co-authored-by: Sergio Soares <[email protected]> Co-authored-by: Joseph Julicher <[email protected]> Co-authored-by: Aniruddha Kanhere <[email protected]> Co-authored-by: tabarnakos <[email protected]> Co-authored-by: Marc-André Harvey <[email protected]> Co-authored-by: Devaraj Ranganna <[email protected]>
…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 <[email protected]> * Update and rename readme.txt to readme.md --------- Co-authored-by: Aniruddha Kanhere <[email protected]>
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.