Skip to content

[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 11 commits into from
Apr 9, 2015
Merged

[TARGET_STMF4] reorg hal #1016

merged 11 commits into from
Apr 9, 2015

Conversation

ohagendorf
Copy link
Contributor

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.

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 9, 2015

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
@ohagendorf
Copy link
Contributor Author

I rebased the PR.

@bcostm
Copy link
Contributor

bcostm commented Apr 9, 2015

I like it. It will be easier now to initialize the peripherals.
Thanks!

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 9, 2015

+1

0xc0170 added a commit that referenced this pull request Apr 9, 2015
@0xc0170 0xc0170 merged commit 5a81e89 into ARMmbed:master Apr 9, 2015
@ohagendorf
Copy link
Contributor Author

When everybody likes this I'll do the same with the other hal folders.

@ohagendorf ohagendorf deleted the DISCO_F407_pins branch April 9, 2015 11:30
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.

3 participants