Skip to content

STM32 code cleaning for MBED-OS6 #12960

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 19 commits into from
May 14, 2020
Merged

Conversation

jeromecoutant
Copy link
Collaborator

@jeromecoutant jeromecoutant commented May 12, 2020

Summary of changes

This is following discussions with @MarceloSalazar

All ST targets are:

  • either keep as full MBED-OS6 target => these targets inherit then from "MCU_STM32"
  • either keep as constrained target => these targets inherit then from "MCU_STM32_BAREMETAL"
  • either removed (board is too constrained, no more available, or was not officially supported)

@ARMmbed/team-st-mcd

Impact of changes

Several targets are no more available in MBED-OS6

If you are still using it,
please use mbed-os-5.15 branch

Migration actions required

Documentation

STM32F0:
NUCLEO_F030R8 => removed
NUCLEO_F031K6 => removed
NUCLEO_F042K6 => removed
DISCO_F051R8 => removed
NUCLEO_F070RB => MCU_STM32_BAREMETAL
NUCLEO_F072RB => MCU_STM32_BAREMETAL
NUCLEO_F091RC => MCU_STM32_BAREMETAL

STM32F1:
DISCO_F100RB => removed
NUCLEO_F103RB => MCU_STM32_BAREMETAL

STM32F2:
NUCLEO_F207ZG => MCU_STM32

STM32F3:
NUCLEO_F302R8 => removed
NUCLEO_F303K8 => MCU_STM32_BAREMETAL
NUCLEO_F303RE => MCU_STM32_BAREMETAL
DISCO_F303VC => removed
NUCLEO_F303ZE => MCU_STM32
DISCO_F334C8 => removed
NUCLEO_F334R8 => removed

STM32F4:
NUCLEO_F401RE => MCU_STM32
DISCO_F401VC => removed
STEVAL_3DP001V1 => removed
ARCH_MAX => supported by Seeed Studio
DISCO_F407VG => removed
NUCLEO_F410RB => removed
NUCLEO_F411RE => MCU_STM32
MTS_DRAGONFLY_F411RE => supported by Multitech
MTS_MDOT_F411RE => supported by Multitech
NUCLEO_F412ZG => MCU_STM32
WIO_EMW3166 => supported by Seeed Studio
MTS_DRAGONFLY_F413RH => supported by Multitech
DISCO_F413ZH => MCU_STM32
NUCLEO_F413ZH => MCU_STM32
NUCLEO_F429ZI => MCU_STM32
DISCO_F429ZI => MCU_STM32
WIO_3G => supported by Seeed Studio
WIO_BG96 => supported by Seeed Studio
NUCLEO_F439ZI => MCU_STM32
NUCLEO_F446RE => MCU_STM32
B96B_F446VE => removed
NUCLEO_F446ZE => MCU_STM32
DISCO_F469NI => MCU_STM32
SDP_K1 => supported by Analog Devices

STM32F7:
DISCO_F746NG => MCU_STM32
NUCLEO_F746ZG => MCU_STM32
NUCLEO_F756ZG => MCU_STM32
UHURU_RAVEN => supported by Uhuru
NUCLEO_F767ZI => MCU_STM32
DISCO_F769NI => MCU_STM32

STM32G0:
NUCLEO_G071RB => MCU_STM32_BAREMETAL

STM32H7:
NUCLEO_H743ZI => removed
NUCLEO_H743ZI2 => MCU_STM32
GENERIC_H745I_CM4 => MCU_STM32
GENERIC_H745I_CM7 => MCU_STM32
DISCO_H747I => MCU_STM32
DISCO_H747I_CM7 => MCU_STM32
DISCO_H747I_CM4 => MCU_STM32

STM32L0:
NUCLEO_L011K4 => removed
NUCLEO_L031K6 => removed
DISCO_L053C8 => removed
NUCLEO_L053R8 => removed
DISCO_L072CZ_LRWAN1 => MCU_STM32
NUCLEO_L073RZ => MCU_STM32_BAREMETAL

STM32L1:
XDOT_L151CC => supported by Multitech
FF1705_L151CC => supported by Ltek
MOTE_L152RC => supported by Semtech
NUCLEO_L152RE => MCU_STM32

STM32L4:
NUCLEO_L432KC => MCU_STM32_BAREMETAL
NUCLEO_L433RC_P => MCU_STM32
ADV_WISE_1510 => supported by Advantech
NUCLEO_L452RE_P => MCU_STM32
MTS_DRAGONFLY_L471QG => supported by Multitech
DISCO_L475VG_IOT01A => MCU_STM32
NUCLEO_L476RG => MCU_STM32
DISCO_L476VG => MCU_STM32
RHOMBIO_L476DMW1K => supported by Rhombio
NUCLEO_L486RG => MCU_STM32
ADV_WISE_1570 => supported by Advantech
DISCO_L496AG => MCU_STM32
NUCLEO_L496ZG => MCU_STM32
NUCLEO_L496ZG_P => MCU_STM32
NUCLEO_L4R5ZI => MCU_STM32
NUCLEO_L4R5ZI_P => MCU_STM32
DISCO_L4R9I => MCU_STM32

STM32L5:
NUCLEO_L552ZE_Q => MCU_STM32
DISCO_L562QE => MCU_STM32

STM32WB:
NUCLEO_WB55RG => MCU_STM32


Pull request type

[] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[x] 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


@mergify mergify bot added the needs: work label May 12, 2020
@ciarmcom ciarmcom requested review from a team May 12, 2020 17:00
@ciarmcom
Copy link
Member

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

MarceloSalazar
MarceloSalazar previously approved these changes May 12, 2020
Copy link

@MarceloSalazar MarceloSalazar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jeromecoutant - the list of targets to be updated is good
Pending resolution on PSA issue

@0xc0170
Copy link
Contributor

0xc0170 commented May 13, 2020

Ci started

Although still needs psa fix for Travis

@mbed-ci
Copy link

mbed-ci commented May 13, 2020

Test run: SUCCESS

Summary: 7 of 7 test jobs passed
Build number : 1
Build artifacts

NUCLEO_F030R8 => removed
NUCLEO_F031K6 => removed
NUCLEO_F042K6 => removed
DISCO_F051R8  => removed
NUCLEO_F070RB => MCU_STM32_BAREMETAL
NUCLEO_F072RB => MCU_STM32_BAREMETAL
NUCLEO_F091RC => MCU_STM32_BAREMETAL
DISCO_F100RB  => removed
NUCLEO_F103RB => MCU_STM32_BAREMETAL
NUCLEO_F207ZG => MCU_STM32
NUCLEO_F302R8 => removed
NUCLEO_F303K8 => MCU_STM32_BAREMETAL
NUCLEO_F303RE => MCU_STM32_BAREMETAL
DISCO_F303VC  => removed
NUCLEO_F303ZE => MCU_STM32
DISCO_F334C8  => removed
NUCLEO_F334R8 => removed
NUCLEO_F401RE        => MCU_STM32
DISCO_F401VC         => removed
STEVAL_3DP001V1      => removed
DISCO_F407VG         => removed
NUCLEO_F410RB        => removed
NUCLEO_F411RE        => MCU_STM32
NUCLEO_F412ZG        => MCU_STM32
DISCO_F413ZH         => MCU_STM32
NUCLEO_F413ZH        => MCU_STM32
NUCLEO_F429ZI        => MCU_STM32
DISCO_F429ZI         => MCU_STM32
NUCLEO_F439ZI        => MCU_STM32
NUCLEO_F446RE        => MCU_STM32
B96B_F446VE          => removed
NUCLEO_F446ZE        => MCU_STM32
DISCO_F469NI         => MCU_STM32
DISCO_F746NG  => MCU_STM32
NUCLEO_F746ZG => MCU_STM32
NUCLEO_F756ZG => MCU_STM32
NUCLEO_F767ZI => MCU_STM32
DISCO_F769NI  => MCU_STM32
NUCLEO_G071RB => MCU_STM32_BAREMETAL
NUCLEO_H743ZI     => removed
NUCLEO_H743ZI2    => MCU_STM32
GENERIC_H745I_CM4 => MCU_STM32
GENERIC_H745I_CM7 => MCU_STM32
DISCO_H747I       => MCU_STM32
DISCO_H747I_CM7   => MCU_STM32
DISCO_H747I_CM4   => MCU_STM32
NUCLEO_L011K4       => removed
NUCLEO_L031K6       => removed
DISCO_L053C8        => removed
NUCLEO_L053R8       => removed
DISCO_L072CZ_LRWAN1 => MCU_STM32
NUCLEO_L073RZ       => MCU_STM32_BAREMETAL
NUCLEO_L152RE => MCU_STM32
NUCLEO_L432KC        => MCU_STM32_BAREMETAL
NUCLEO_L433RC_P      => MCU_STM32
NUCLEO_L452RE_P      => MCU_STM32
DISCO_L475VG_IOT01A  => MCU_STM32
NUCLEO_L476RG        => MCU_STM32
DISCO_L476VG         => MCU_STM32
NUCLEO_L486RG        => MCU_STM32
DISCO_L496AG         => MCU_STM32
NUCLEO_L496ZG        => MCU_STM32
NUCLEO_L496ZG_P      => MCU_STM32
NUCLEO_L4R5ZI        => MCU_STM32
NUCLEO_L4R5ZI_P      => MCU_STM32
DISCO_L4R9I          => MCU_STM32
NUCLEO_L552ZE_Q => MCU_STM32
DISCO_L562QE    => MCU_STM32
NUCLEO_WB55RG => MCU_STM32
@jeromecoutant
Copy link
Collaborator Author

Although still needs psa fix for Travis

Done

@mergify mergify bot dismissed MarceloSalazar’s stale review May 14, 2020 11:56

Pull request has been modified.

@0xc0170
Copy link
Contributor

0xc0170 commented May 14, 2020

CI started

@mergify mergify bot added needs: CI and removed needs: review labels May 14, 2020
@mbed-ci
Copy link

mbed-ci commented May 14, 2020

Test run: SUCCESS

Summary: 7 of 7 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 removed request for a team May 14, 2020 16:06
@0xc0170 0xc0170 merged commit 06ff4cf into ARMmbed:master May 14, 2020
@mergify mergify bot removed the ready for merge label May 14, 2020
@jeromecoutant jeromecoutant deleted the PR_REMOVE_TARGETS branch May 15, 2020 07:02
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.

5 participants