-
Notifications
You must be signed in to change notification settings - Fork 3k
[TARGET_STMF4] reorg hal #1016
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
Merged
Merged
[TARGET_STMF4] reorg hal #1016
Conversation
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
Can you resolve the conflict please? |
Removing pin configuration from analogin_api, analogout_api and pwmout_api (hal function, used by every STM32F4 target) to the target specific files.
same changes but for NUCLEO_F411RE as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG
same changes but for DISCO_F429ZI as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG
same changes but for NUCLEO_F401RE as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG
same changes but for ARCH_MAX as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG one bugfix for DISCO_F429 - wrong DAC channel numbers
same changes but for DISCO_F401VC as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG
same changes but for MTS_DRAGONFLY_F411RE as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG not tested because hardware not yet available
same changes but for MTS_MDOT_F405RG as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG not tested because hardware not yet available
same changes but for MTS_MDOT_F411RE as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG not tested because hardware not yet available
same changes but for UBLOX_C029 as in commit [TARGET_STMF4] reorg hal for DISCO_F407VG not tested because hardware not yet available
some test extensions
32dcbd3
to
dce050a
Compare
I rebased the PR. |
I like it. It will be easier now to initialize the peripherals. |
+1 |
When everybody likes this I'll do the same with the other hal folders. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Removing pin configurations from analogin_api, analogout_api and pwmout_api (hal function, used by every STM32F4 target) to the target specific files - that means some more target specific configurations are moved to the folder with the target specific config.
The following targets are tested:
DISCO_F407
DISCO_F429
NUCLEO_F401RE
NUCLEO_F411RE
ARCH_MAX
All singletests inkl. "digital_loop" and "analog_loop" are OK accept RTC tests for DISCO_F407, DISCO_429 and ARCH_MAX but they failed before too.
The other targets can be compiled but it seems the hardware is not yet available so they are not tested.