Skip to content

Remove unsupported targets #12775

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 15 commits into from
Apr 14, 2020
Merged

Remove unsupported targets #12775

merged 15 commits into from
Apr 14, 2020

Conversation

MarceloSalazar
Copy link

@MarceloSalazar MarceloSalazar commented Apr 8, 2020

Summary of changes

This is the first of a series of PRs to remove targets that are not supported or retired as explained in recent blog post.

There are individual commits to remove targets from targets.json and files/folders for the corresponding targets.

Impact of changes

The following targets are being removed:

  • Thundersoft (all)
  • Realtek RTL8195A
  • Ublox ODIN and related (OKDO, MTB)
  • RDA (all)
  • Gigadevice GD32_E103VB
  • Atmel (all)
  • TI (all)
  • Etteplan Elmo
  • Wiznet (all)
  • Silica
  • Renesas VK_RZ_A1H
  • Rename MTB_MXCHIP_EMW3166 to WIO_EMW3166
  • VBLUNO51 and VBLUNO52

Migration actions required

  • The targets listed above continue to be available in the 5.15 branch.

Documentation

  • None.
    We'll update os.mbed.com/platforms with the list of supported platforms for Mbed 6

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

Expect regressions tests to be done by CI as part of this PR.

Reviewers

@ARMmbed/mbed-os-maintainers

0xc0170
0xc0170 previously approved these changes Apr 8, 2020
@0xc0170 0xc0170 requested a review from bulislaw April 8, 2020 10:37
@mergify mergify bot added the needs: work label Apr 8, 2020
@mergify mergify bot dismissed 0xc0170’s stale review April 8, 2020 11:29

Pull request has been modified.

@evedon
Copy link
Contributor

evedon commented Apr 8, 2020

Huge clean-up. Thanks Marcelo!

@mergify
Copy link

mergify bot commented Apr 9, 2020

This PR cannot be merged due to conflicts. Please rebase to resolve them.

@mergify mergify bot removed the needs: CI label Apr 9, 2020
@MarceloSalazar
Copy link
Author

Rebased

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 9, 2020

CI started

@mergify mergify bot added needs: work and removed needs: CI labels Apr 9, 2020
@mbed-ci
Copy link

mbed-ci commented Apr 9, 2020

Test run: FAILED

Summary: 2 of 3 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_build-ARM
  • jenkins-ci/mbed-os-ci_build-GCC_ARM

@MarceloSalazar
Copy link
Author

I've identified and fixed an issue on the WIO_EMW3166 target for the WICED driver.
However, I can't reproduce any of the other errors. It seems there is a parameter null but I don't know where it is coming from.
@0xc0170 perhaps you could have a look?

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 14, 2020

cc @ARMmbed/mbed-os-test looks like something in CI, please review

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 14, 2020

CI restarted

@mbed-ci
Copy link

mbed-ci commented Apr 14, 2020

Test run: SUCCESS

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

@0xc0170
Copy link
Contributor

0xc0170 commented Apr 14, 2020

Just to make sure, I'll cc here teams to be aware.

@ARMmbed/team-realtek @ThunderSoft123 @ARMmbed/team-ublox @ARMmbed/team-wiznet

I am doing final review and will merge.

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