STM32 H7: fix USB in sleep mode

[JojoS62]

Problem as descriibed in #13641

USBDevice was not working with STM32H7 targets.

in sleep mode, USB interrupts were disabled
and USB Device was not working. Sleep mode is default in Mbed-os
e.g. in a call to sleep_for()

Summary of changes

Disabling ULPI clock in sleep mode is required for H7 targets. Added conditional disabling when macro exists.

Fix is necessary to make USBDevice working on STM32H7 targets.

Impact of changes

Tests were performed on STM32H743VI and Nucleo STM32H7A3ZIT6Q.
Targets without ULPI should not be affected. STM32F401 ompiled and tested positiv.

Migration actions required

Targets with H7 MCU can be checked and USBDevice can be enabled

Documentation

not affected

---

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)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

---

Reviewers

---

[ciarmcom]

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

[0xc0170]

@ARMmbed/mbed-os-maintainers reviewers bot is not working as reported the last week? Please review

[jeromecoutant]

Got issues with USB tests and NUCLEO_H743ZI2...
Seems there is in USB_CoreReset...???

[JojoS62]

That maybe an issue with the clock setting. There is an errata that the clock recovery system is not working with HSI48, so I used PLL3Q for my custom board.

could you please use this init in SetSystemClock:
https://github.com/JojoS62/custom_targets/blob/bfd51a0af6535af534cd7af54393975d84187780/TARGET_STM/TARGET_STM32H7/TARGET_STM32H743xI/TARGET_DEVEBOX_H743VI/system_clock.c#L287-L304

The multiplier and dividers need to be adjusted for a 8 MHz xtal.

    PeriphClkInitStruct.PLL3.PLL3M = 2;                              // 8 MHz / 2 = 4MHz
    PeriphClkInitStruct.PLL3.PLL3N = 48;                             // * 48 = 192 MHz
    PeriphClkInitStruct.PLL3.PLL3P = 2;                               // 192 /  2 = 96 MHz (unused)
    PeriphClkInitStruct.PLL3.PLL3Q = 4;                               // 192 / 4 = 48 MHz (USB)

This setting maybe also necessary for the H743ZI2:

        "config": {
            "usb_speed": {
                "help": "USE_USB_OTG_FS or USE_USB_OTG_HS or USE_USB_HS_IN_FS",
                "value": "USE_USB_OTG_FS"
            },

[jeromecoutant]

Let's merge this update.
I will enable USB for NUCLEO-H743 after more tests

[jeromecoutant]

@0xc0170 Let's start CI ?
Thx

[mbed-ci]

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