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

NUCLEO_F446ZE: Add missing AnalogIn pins on PF_3, PF_5 and PF_10. #3879

Merged
merged 1 commit into from Mar 9, 2017

Conversation

Projects
None yet
4 participants
@bcostm
Contributor

bcostm commented Mar 3, 2017

Description

During ci-tests, I discovered that AnalogIn was missing on these pins (Arduino A3, A4, A5). Don't know why ?

Tested OK with ci-test shield:

+-----------------------+---------------+-----------------------+------------------------------------+--------+--------+--------+--------------------+
| target                | platform_name | test suite            | test case                          | passed | failed | result | elapsed_time (sec) |
+-----------------------+---------------+-----------------------+------------------------------------+--------+--------+--------+--------------------+
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_0              | 1      | 0      | OK     | 0.05               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_1              | 1      | 0      | OK     | 0.05               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_2              | 1      | 0      | OK     | 0.04               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_3              | 1      | 0      | OK     | 0.07               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_4              | 1      | 0      | OK     | 0.04               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogin    | Analog Input on AIN_5              | 1      | 0      | OK     | 0.05               |
| NUCLEO_F446ZE-GCC_ARM | NUCLEO_F446ZE | tests-api-analogout   | AnalogOut on AOUT                  | 1      | 0      | OK     | 0.04               |

Status

READY

Migrations

NO

Add AnalogIn pins on PF3, PF5 and PF10.
Don't know why it was missing ?
@0xc0170

0xc0170 approved these changes Mar 6, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 6, 2017

/morph test

@0xc0170 0xc0170 added the needs: CI label Mar 6, 2017

@mbed-bot

This comment has been minimized.

mbed-bot commented Mar 6, 2017

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 6, 2017

@bridadan Please have a look at the failure

@bridadan

This comment has been minimized.

Contributor

bridadan commented Mar 6, 2017

Issue should be resolved now, thanks @0xc0170

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Mar 6, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1638

All builds and test passed!

@0xc0170 0xc0170 merged commit 45c99e6 into ARMmbed:master Mar 9, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has started
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@bcostm bcostm deleted the bcostm:fix_nucleo-f446ze_adc_pins branch Mar 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment