Skip to content
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

Analogout free implementation for F4/F7/H7 boards #14844

Merged
merged 3 commits into from
Jul 8, 2021

Conversation

AnishKumarHCL
Copy link
Contributor

Summary of changes

Implemented analogout_free for F4/F7/H7 boards.

Impact of changes

Migration actions required

Documentation


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[x] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers

@jeromecoutant


@ciarmcom ciarmcom added the release-type: patch Indentifies a PR as containing just a patch label Jun 30, 2021
@ciarmcom ciarmcom requested review from jeromecoutant and a team June 30, 2021 15:00
@ciarmcom
Copy link
Member

@AnishKumarHCL, thank you for your changes.
@jeromecoutant @ARMmbed/mbed-os-maintainers please review.

@jeromecoutant
Copy link
Collaborator

@0xc0170 seems there is some travis issues ?

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 1, 2021

It is, @AnishKumarHCL Can you just rebase on the recent master, 3.5 tools test should be gone from Travis

@AnishKumarHCL AnishKumarHCL force-pushed the analogout_free_implementation branch from b2e82d0 to 59ac864 Compare July 1, 2021 16:01
@AnishKumarHCL
Copy link
Contributor Author

done @0xc0170

Implemented analogout_free for STM32F4 family and added new
member "PinName pin" in dac_s structure.
Implemented analogout_free for STM32F7 Series.
analogout_free implemented with multiple DAC channel usage check.
@AnishKumarHCL AnishKumarHCL force-pushed the analogout_free_implementation branch from 59ac864 to 59ba97f Compare July 2, 2021 12:33
@mergify mergify bot added needs: CI and removed needs: work labels Jul 8, 2021
@0xc0170
Copy link
Contributor

0xc0170 commented Jul 8, 2021

CI started

@mbed-ci
Copy link

mbed-ci commented Jul 8, 2021

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-ARM ✔️
jenkins-ci/mbed-os-ci_cmake-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

@0xc0170 0xc0170 merged commit 3fcc834 into ARMmbed:master Jul 8, 2021
@mergify mergify bot removed the ready for merge label Jul 8, 2021
@mbedmain mbedmain added release-version: 6.13.0 Release-pending and removed release-type: patch Indentifies a PR as containing just a patch labels Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants