STM32 usb: fixing missing header file

[0xc0170]

Summary of changes

Fixes #13838. The previous PR #13683 did miss these unfortunately.

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

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

---

Reviewers

---

[jeromecoutant]

| NUCLEO_F103RB-ARMC6 | NUCLEO_F103RB | drivers-tests-tests-usb_device-hid | OK | 22.18 | default |
| NUCLEO_F103RB-ARMC6 | NUCLEO_F103RB | drivers-tests-tests-usb_device-msd | OK | 18.64 | default |

[0xc0170]

CI started

[ciarmcom]

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

[mbed-ci]

Jenkins CI Test : ❌ FAILED

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-greentea-ARM	✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-cloud-example-ARM	✔️
jenkins-ci/mbed-os-ci_build-example-ARM	✔️
jenkins-ci/mbed-os-ci_dynamic-memory-usage	✔️
jenkins-ci/mbed-os-ci_greentea-test	✔️
jenkins-ci/mbed-os-ci_cloud-client-pytest	❌

[Allmoz]

Tested on STM32F103C8T6 and STM32F103RET6, it works for me.

Oh, so the include had to be in USBPhy_STM32.cpp

The use of mbed::DigitalOut usb_dp_pin(USB_DP, 0) ; is conditional, should the include be conditional too? that would be consistent with the spirit of #13683 of reducing the inclusion of libraries when not needed.

#if defined(TARGET_STM32F1)
void USB_reenumerate()
{
    // Force USB_DP pin (with external pull up) to 0
    mbed::DigitalOut usb_dp_pin(USB_DP, 0) ;
    wait_us(10000); // 10ms
}
#endif

[0xc0170]

Thank you both for testing !

@Allmoz yes that would be probably better. I moved it to implementation as that one needed it, so the scope already is reduced.

[0xc0170]

Moved the inclusion to where it is needed.

CI restarted

[mbed-ci]

Jenkins CI Test : ✔️ SUCCESS

Build Number: 3 | 🔒 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_build-greentea-ARM	✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM	✔️
jenkins-ci/mbed-os-ci_build-example-ARM	✔️
jenkins-ci/mbed-os-ci_dynamic-memory-usage	✔️
jenkins-ci/mbed-os-ci_greentea-test	✔️
jenkins-ci/mbed-os-ci_cloud-client-pytest	✔️

[0xc0170]

@ARMmbed/mbed-os-maintainers Please review

[harmut01]

LGTM