-
-
Notifications
You must be signed in to change notification settings - Fork 0
CAN ISR #248
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
Draft
VihanJ
wants to merge
58
commits into
main
Choose a base branch
from
CAN_ISR
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,780
−611
Draft
CAN ISR #248
Changes from all commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
da4a67e
added id to CAN RX callback
ahoysal 05214b4
Added enum as CAN identifier in RX
ahoysal 9261965
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] f26d44f
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] d36a3ba
Changed CAN_RX_Callback to take in ID, data, size
VihanJ 9e1af22
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] 499d15c
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] 4337a58
added formatting for uint32_t id in can_tests.c
VihanJ 2b00473
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] 98923cf
cleand up can.c
VihanJ 82e7709
added basepri masking, also tried to prevent spurious NVIC interrupts…
VihanJ 6d6d9ff
added basepri masking, also tried to prevent spurious NVIC interrupts…
VihanJ 18af90b
Refactored to use statically sized spsc buffer
VihanJ ebc37b8
passed human testing
VihanJ 7245169
added notes in readme
VihanJ 28b8e14
Merge remote-tracking branch 'origin/main' into CAN_ISR
VihanJ 1b2adde
tried to fix CAN data baud rate in ECU/Core/Src/main.c
VihanJ 622bf6c
Merge branch 'main' into CAN_ISR
VihanJ 6afe33a
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] 837d821
Merge branch 'main' into CAN_ISR
dchansen06 7c779ca
Merge branch 'main' into CAN_ISR
dchansen06 f8c16e5
Merge branch 'main' into CAN_ISR
dchansen06 0442dec
Merge branch 'main' into CAN_ISR
dchansen06 fa981aa
Merge branch 'main' into CAN_ISR
dchansen06 7d90ef3
Merge branch 'main' into CAN_ISR
dchansen06 4125cc0
Merge branch 'main' into CAN_ISR
dchansen06 d60e880
Merge branch 'main' into CAN_ISR
dchansen06 c4c57b8
Merge branch 'main' into CAN_ISR
dchansen06 1bcc559
Merge branch 'main' into CAN_ISR
dchansen06 9581d92
Co-authored-by: Aagrim Hoysal <ahoysal@users.noreply.github.com>
VihanJ e91fba1
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] 2e2b115
Merge branch 'main' into CAN_ISR
dchansen06 0017a46
Merge branch 'main' into CAN_ISR
dchansen06 a1b248a
Merge branch 'main' into CAN_ISR
dchansen06 5e91ea8
new tests
VihanJ 42198b1
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] 5684d41
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] c02a207
Some working version of some testing maybe
dchansen06 2b60662
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] e7c8328
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] e3dd937
Everything builds
dchansen06 00f4d3d
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] cbf4520
Merge branch 'main' into CAN_ISR
dchansen06 3b27e30
changed tests
VihanJ 4700904
running through tests
VihanJ 4a44cfd
tested can_external_test.c can_internal_test.c can_stress_test.c, can…
VihanJ 9b2ebc3
deleted unused folders
VihanJ 432295c
Automatic Json Format: Standardized formatting automatically
github-actions[bot] 92d882e
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] 9b062ac
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] c823f4e
fixed for Release and Debug
VihanJ 058270b
Merge branch 'main' into CAN_ISR
dchansen06 f2e9b20
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] c759e43
added documentation with my good friend chat gibidi
VihanJ 39616ea
Automatic CMake Format: Standardized formatting automatically
github-actions[bot] 65d0e3d
Automatic Clang-Format: Standardized formatting automatically
github-actions[bot] 67e6887
Merge branch 'main' into CAN_ISR
dchansen06 5bbfa7a
Merge branch 'main' into CAN_ISR
dchansen06 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -81,7 +81,18 @@ add_gr_project(STM32G474xE TireTemp) | |||||||||||||||||||||||||||||||||||
| # Development | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4ADCTESTING) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4PERTESTING) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4CANTESTING) | ||||||||||||||||||||||||||||||||||||
| #add_gr_project(STM32G474xE G4CANTESTING) | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| #CAN Peripheral Testing | ||||||||||||||||||||||||||||||||||||
| #add_gr_project(STM32G474xE G4CANTESTING FILTER_TEST) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4CANTESTING CAN_EXTERNAL_TEST) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4CANTESTING CAN_INTERNAL_TEST) | ||||||||||||||||||||||||||||||||||||
| #add_gr_project(STM32G474xE G4CANTESTING INIT_TEST) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4CANTESTING CAN_RELEASE_TEST) | ||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4CANTESTING CAN_STRESS_TEST) | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| #add_gr_project(STM32G474xE G4CANTESTING) | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
|
Comment on lines
+84
to
+95
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||||||||||
| add_gr_project(STM32G474xE G4NEOTESTING) | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| # BLINKY Demos | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| cmake_minimum_required(VERSION 3.25) | ||
|
|
||
| # Setup compiler settings | ||
| set(CMAKE_C_STANDARD 11) | ||
| set(CMAKE_C_STANDARD_REQUIRED ON) | ||
| set(CMAKE_C_EXTENSIONS ON) | ||
|
|
||
| # Define the build type | ||
| if(NOT CMAKE_BUILD_TYPE) | ||
| set(CMAKE_BUILD_TYPE "Debug") | ||
| endif() | ||
|
|
||
| # Enable compile command to ease indexing with e.g. clangd | ||
| set(CMAKE_EXPORT_COMPILE_COMMANDS TRUE) | ||
|
|
||
| # Enable CMake support for ASM and C languages | ||
| enable_language( | ||
| C | ||
| ASM | ||
| ) | ||
|
|
||
| # Core project settings | ||
| project(${CMAKE_PROJECT_NAME}) | ||
|
|
||
| # what, does in fact not get the filename of somthing but rather the name of the project from the path | ||
| get_filename_component(PROJECT_NAME ${CMAKE_CURRENT_SOURCE_DIR} NAME) | ||
|
|
||
| add_library(${PROJECT_NAME}_USER_CODE INTERFACE) | ||
|
|
||
| target_sources( | ||
| ${PROJECT_NAME}_USER_CODE | ||
| INTERFACE | ||
| ../Core/Src/adc.c | ||
| ../Core/Src/crc.c | ||
| ../Core/Src/dma.c | ||
| ../Core/Src/gpio.c | ||
| ../Core/Src/i2c.c | ||
| main.c | ||
| ../Core/Src/spi.c | ||
| ../Core/Src/stm32g4xx_hal_msp.c | ||
| ../Core/Src/stm32g4xx_it.c | ||
| ../Core/Src/tim.c | ||
| ../Core/Src/usart.c | ||
| ) | ||
|
|
||
| set_target_properties( | ||
| ${NAME} | ||
| PROPERTIES | ||
| OUTPUT_NAME | ||
| "CAN_${NAME}" | ||
| ) | ||
|
|
||
| target_link_libraries(${PROJECT_NAME}_USER_CODE INTERFACE PERIPHERAL_CAN_LIB) | ||
|
|
||
| target_include_directories(${PROJECT_NAME}_USER_CODE INTERFACE ../Core/Inc) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
|
|
||
| #ifndef CAN_CONF_H | ||
| #define CAN_CONF_H | ||
|
|
||
| #define USECAN1 | ||
| #define TX_BUFFER_1_SIZE 10 | ||
|
|
||
| #define USECAN2 | ||
| #define TX_BUFFER_2_SIZE 10 | ||
|
|
||
| #endif |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I am not sure why this was changed, probably a merge issue. Right below is
G4CANTESTING