-
Notifications
You must be signed in to change notification settings - Fork 0
Refactor/spi #554
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
FoniksFox
wants to merge
104
commits into
development
Choose a base branch
from
refactor/SPI
base: development
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.
Draft
Refactor/spi #554
Changes from all commits
Commits
Show all changes
104 commits
Select commit
Hold shift + click to select a range
09de7c7
It compiles the old version with constexpr
Cantonplas 8962c95
Draft of the new dma class
Cantonplas 1d158c0
Minor change on where is the instance declared on dma.hpp
Cantonplas ea78eb3
limpieza de funciones, enums y structs
jorgecanut b8df87e
feat: distintos inscribe definidos, no creo que sea lo mejor que se p…
jorgecanut 85db060
feat: mejor enfoque, pero no se si el correcto
jorgecanut 52ae1a5
feat: comprueba el numero de perifericos, cambios de nombre a algunas…
jorgecanut 3dd6ae4
feat: struct constructor, better is... logic, refactor inscribe (tech…
jorgecanut 8ca78ee
algunas cosillas para que compile. falta conseguir pasarle las variab…
jorgecanut 3c44a60
feat: agregar static a las variables constexpr
jorgecanut d2005f2
me lo quiero pasar al PC solamente, luego hago bien el commit
jorgecanut 9d2a930
feat: agregar funciones que comprueban si el stream y la instancias y…
jorgecanut 38fd08b
fuck constexpr, todo en runtime. feat: inscribe funciona, solo quedar…
jorgecanut 4d0e6be
repartir entre archivos las funciones y hacer el start
jorgecanut cca62c8
feat: start completado, mejor estructura, todos los perifericos confi…
jorgecanut 5212ce9
hacer funciones inline para que pueda compilar
jorgecanut 9655dea
adc inscribe_dma now working, moved dma implementations from cpp to h…
jorgecanut e2af8c5
Initial structure
jorgesg82 3eec305
no need to have an id for instance
jorgesg82 fa3498b
erased no longer needed code
jorgesg82 c338bf8
Juntar estructura propuesta de compiletime con la clase DMA
jorgecanut 0440eb4
cambio de nombre al antiguo dma y empiezo con la nueva infraestructur…
jorgecanut 98a34fb
añadir funciones de configuracion del handle DMA
jorgecanut c1966cf
Added DigitalInput and DigitalOutput Services, and added support for …
jorgesg82 b245442
todo lo facil hecho, queda lo dificil (handle y global_handle)
jorgecanut ba25865
entry, build e instances_ ahora se rigen por stream y no instancia (l…
jorgecanut 968eee6
se me habia olvidado desreferenciar
jorgecanut 0826a02
modificar STLIB.hpp para meter el contexto de la DMA
jorgecanut 4c8a7f3
throws comentados, una dma menos
jorgecanut fb86f46
It compiles the old version with constexpr
Cantonplas ac4df89
Draft of the new dma class
Cantonplas 4b9d4ba
Minor change on where is the instance declared on dma.hpp
Cantonplas d085938
limpieza de funciones, enums y structs
jorgecanut 2e9252c
feat: distintos inscribe definidos, no creo que sea lo mejor que se p…
jorgecanut 5ad8632
feat: mejor enfoque, pero no se si el correcto
jorgecanut d832eec
feat: comprueba el numero de perifericos, cambios de nombre a algunas…
jorgecanut e39e204
feat: struct constructor, better is... logic, refactor inscribe (tech…
jorgecanut 1634bdb
algunas cosillas para que compile. falta conseguir pasarle las variab…
jorgecanut 81fb468
feat: agregar static a las variables constexpr
jorgecanut 5aa3106
me lo quiero pasar al PC solamente, luego hago bien el commit
jorgecanut d70ab88
feat: agregar funciones que comprueban si el stream y la instancias y…
jorgecanut 6d8552f
fuck constexpr, todo en runtime. feat: inscribe funciona, solo quedar…
jorgecanut dc97111
repartir entre archivos las funciones y hacer el start
jorgecanut d70d07f
feat: start completado, mejor estructura, todos los perifericos confi…
jorgecanut 8f496ae
hacer funciones inline para que pueda compilar
jorgecanut 2651dc6
adc inscribe_dma now working, moved dma implementations from cpp to h…
jorgecanut 3996ec0
Initial structure
jorgesg82 d7e562c
cambio de nombre al antiguo dma y empiezo con la nueva infraestructur…
jorgecanut c152e48
añadir funciones de configuracion del handle DMA
jorgecanut 61d093c
Added DigitalInput and DigitalOutput Services, and added support for …
jorgesg82 db10f57
todo lo facil hecho, queda lo dificil (handle y global_handle)
jorgecanut 9a029ec
entry, build e instances_ ahora se rigen por stream y no instancia (l…
jorgecanut 2f65b23
se me habia olvidado desreferenciar
jorgecanut 04fb46b
modificar STLIB.hpp para meter el contexto de la DMA
jorgecanut 1503056
eliminar que reciba el handle
jorgecanut 7974f64
eliminar que reciba el handle
jorgecanut 38778af
sacar streams fuera del struct
jorgecanut 54e9082
conflictos que no deberian de salir
jorgecanut 36e5d71
el commit que se me olvido hacer para que poder testear
jorgecanut c61c735
resolver conflictos
jorgecanut 10390cf
cambios para poder utilizar none y algunos errores corregidos, queda …
jorgecanut f908ab5
ni me acuerdo, mañana sigo
jorgecanut 2631a37
debugging validacion
jorgecanut b6b2c30
cmake cambiado para que se incluya la dma2.cpp
jorgecanut 9e4fb9e
cambios finales, prueba funciona
jorgecanut 0cb7a11
ahora de verdad funciona
jorgecanut 33b4942
limpieza de rama p1
jorgecanut 3f5b436
limpieza de rama p2
jorgecanut 550fde1
cambiar gpio para que el static assert me deje compilar
jorgecanut d3432a1
feat(SPI): First implementation with most of the infrastructure and h…
FoniksFox f5cb029
feat(SPI): Baudrate prescaler calculation
FoniksFox efbf9c0
feat(SPI): Add IRQ handlers and HAL callbacks
FoniksFox 2c95ba0
fix(SPI): Fix error spi number
FoniksFox a2f0639
fix(SPI): Callback handling
FoniksFox 8ca8bde
feat(SPI): Implement API for both Master and Slave
FoniksFox baaf149
doc(SPI): Add tooltip comments
FoniksFox b0e1516
fix(SPI): Typo in ErrorHandler function name
FoniksFox d6595a3
fix(SPI): Move spi_instances declaration
FoniksFox 7f0e3d8
fix(SPI): General syntax errror fixes
FoniksFox c090b4b
feat(SPI): Add SPI domain to ST-LIB
FoniksFox 288e5c0
fix(SPI): Remove forgotten line
FoniksFox 4d6f8b4
fix(SPI): General fixes to make it compile
FoniksFox 3af6613
fix(SPI): Forgot to set up max baudrate
FoniksFox b86ca78
Merge remote-tracking branch 'origin/feat/DMA-updated' into refactor/SPI
FoniksFox 101d022
Merge branch 'development' into feat/DMA-updated
FoniksFox 1bd4fc2
feat(DMA): udpate inscribe method to return indices
FoniksFox 2f20354
Merge branch 'feat/DMA-updated' into refactor/SPI
FoniksFox eb72bf6
feat(SPI): Add DMA linking
FoniksFox 980fa9c
feat(SPI): Implement configuration API + some cleanup
FoniksFox ba15465
fix(SPI): Update SPI in ST-LIB.hpp
FoniksFox a6402af
feat(SPI): Make inscribe return size_t
FoniksFox 5cce7e4
fix(SPI): Fix compile_error
FoniksFox 4ce57b8
Merge remote-tracking branch 'origin/development' into refactor/SPI
FoniksFox 72f165b
fix(SPI): Fix inscribe
FoniksFox 8404700
fix(SPI): Fix things
FoniksFox 2a92b34
Merge remote-tracking branch 'origin/development' into refactor/SPI
FoniksFox 6538aff
fix(SPI): Pin A15 AF were incorrect
FoniksFox 94ae350
Merge remote-tracking branch 'origin/development' into refactor/SPI
FoniksFox a6cc1e3
fix(SPI): Fix instances array
FoniksFox ff3cf48
Merge remote-tracking branch 'origin/development' into refactor/SPI
FoniksFox 6d49eb3
feat(SPI): Add templated functions to acccept anything
FoniksFox 770f323
fix(SPI): Alignments and correct frame calculations
FoniksFox 1731518
feat(SPI): Add slave software nss interface
FoniksFox a559017
fix(GH_Actions): Don't upload build artifacts
FoniksFox 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
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 |
|---|---|---|
|
|
@@ -39,4 +39,4 @@ class DMA { | |
| private: | ||
| static vector<Stream> available_streams; | ||
| static vector<Stream> inscribed_streams; | ||
| }; | ||
| }; | ||
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.
The library type was changed from STATIC to OBJECT without explanation. OBJECT libraries behave differently than STATIC libraries - they don't create an archive file and their object files are directly linked into targets that use them. This change can affect link-time optimization, symbol visibility, and how the library is consumed by other targets. Please document why this change was made and verify it doesn't break existing build configurations or downstream projects that depend on this library.