Skip to content

Update of master #2

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 6 commits into from
Sep 13, 2019
Merged

Conversation

sabas1080
Copy link
Member

Pull Request template

Please, Make sure that your PR is not a duplicate.
Search among the Pull request before creating one.

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Thanks for submitting a pull request.
Please provide enough information so that others can review your pull request:

Summary

This PR fixes/implements the following bugs/features

  • Bug 1
  • Bug 2
  • Feature 1
  • Feature 2
  • Breaking changes

Explain the motivation for making this change. What existing problem does the pull request solve?

Validation

  • Ensure Travis CI build is passed.
  • Demonstrate the code is solid. [e.g. Provide a sketch]

Code formatting

  • Ensure AStyle check is passed thanks Travis CI (soon)

Closing issues

Fixes #xxx

ABOSTM and others added 6 commits September 11, 2019 20:27
Signed-off-by: Frederic Pillon <[email protected]>
If HAL_ADC_MODULE_ENABLED and HAL_DAC_MODULE_ENABLED are not defined
it is not used.
No more used if HAL_TIM_MODULE_ENABLED since HardwareTimer.
Signed-off-by: Frederic Pillon <[email protected]>
If no analog pin required then `NUM_ANALOG_INPUTS` and `NUM_ANALOG_FIRST`
could be not defined or set to respectively `0` and `NUM_DIGITAL_PINS`

Nevertheless, this kept ADC internal channels functional if
`HAL_ADC_MODULE_ENABLED` still defined.

Signed-off-by: Frederic Pillon <[email protected]>
@sabas1080 sabas1080 merged commit 4aa7306 into ElectronicCats:port_stm32l051c8t6 Sep 13, 2019
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.

4 participants