Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Mbed CE Build System Refactor #1

Merged
merged 44 commits into from
May 8, 2022
Merged

Mbed CE Build System Refactor #1

merged 44 commits into from
May 8, 2022

Conversation

multiplemonomials
Copy link
Collaborator

@multiplemonomials multiplemonomials commented Apr 9, 2022 •
edited
Loading

Summary of changes

  • Make Mbed OS an OBJECT library instead of an INTERFACE library. This removes the problem of Mbed getting built multiple times when there are multiple executables in a project.
  • Clean up and refactor logic in top-level build scripts

Impact of changes

  • It now works properly to link mbed-os to multiple different exectuables in a project, as well as to STATIC library targets

Migration actions required

Documentation

None

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

Sadik.Ozer and others added 30 commits April 8, 2022 10:40
Add initial version of files
028a85c 
Signed-off-by: Sadik.Ozer <[email protected]>
Update cmsis files
fc90e9a 
Signed-off-by: Sadik.Ozer <[email protected]>
Remove __R because of confliction
f40887a 
Signed-off-by: Sadik.Ozer <[email protected]>
Add peripheral interface files
704bfc7 
Signed-off-by: Sadik.Ozer <[email protected]>
Add peripheral driver files
8a2681b 
Signed-off-by: Sadik.Ozer <[email protected]>
Fix rtc read issue
65e6595 
Signed-off-by: Sadik.Ozer <[email protected]>
Provide all uart mapping
f1bce73 
Signed-off-by: Sadik.Ozer <[email protected]>
Provide option to user to drive SS pin too
24f4738 
Signed-off-by: Sadik.Ozer <[email protected]>
Leave last page of flash
9298365 
It is used by rom bootloader

Signed-off-by: Sadik.Ozer <[email protected]>
Update LP API
7826582 
Signed-off-by: Sadik.Ozer <[email protected]>
Update system files and mbed wrappers
2f813fc 
Signed-off-by: Sadik.Ozer <[email protected]>
Fix GCC_ARM warnings
8323e9a 
Signed-off-by: Sadik.Ozer <[email protected]>
utf-8 check
4dd0144 
Signed-off-by: Sadik.Ozer <[email protected]>
@0xc0170
Merge pull request ARMmbed#15263 from MaximIntegrated/add-MAX32670
47a32a0 
Add MAX32670
@0xc0170
github action: update checkout to v3
8a59067 
See actions/checkout#762
Our tests are failing as checkout is not working (no git repository found)
@0xc0170
github action: add git safe directory
ad15f0a 
Where we use git commands directly, we need to add safe directory. This is because
the recent git changes introduced, for details visit:
actions/checkout#766
@0xc0170
Merge pull request ARMmbed#15270 from 0xc0170/fix-test-github-action-…
decc6d0 
…git-failure

Github actions: use safe.directory for actions invoking git
@jeromecoutant
STM32L0 : I2C2 was missing
271ed68
@hifoolno
tcp_out: fix tcp_output_fill_options() arguments
173c070
@jeromecoutant
STM32G4 : add UART5 in IRQ init
9b1d4ee
@jeromecoutant
STM32G4 : enable SERIAL_ASYNCH in default configuration
1985e77
@caspermeijn
semihosting: Fix typo in semihost_rename
904f867 
Fix typo in semihost_rename. Now it actually sends the `new_name` as well as the `old_name`.
@0xc0170
Merge pull request ARMmbed#15268 from jeromecoutant/PR_L0_I2C2
2d652c9 
STM32L0 : I2C2 was missing
@0xc0170
Merge pull request ARMmbed#15274 from hifoolno/bug4
d17beb7 
tcp_out: fix tcp_output_fill_options() arguments
@0xc0170
Merge pull request ARMmbed#15269 from jeromecoutant/PR_G4_UART_ASYNC
88b6bb0 
STM32G4 : enable UART ASYNC
@0xc0170
Merge pull request ARMmbed#15271 from caspermeijn/patch-1
9f326aa 
semihosting: Fix typo in semihost_rename
@jeromecoutant
STM32L0: add MCU_STM32L071xB support
6a8a52a
@0xc0170
Merge pull request ARMmbed#15277 from jeromecoutant/PR_L071KB
2026861 
STM32L0: add MCU_STM32L071xB support
@jeromecoutant
STM32F334xx wrong RAM size
de4ea6e
@0xc0170
Merge pull request ARMmbed#15279 from jeromecoutant/PR_F334
f2c9c60 
STM32F334xx wrong RAM size
@multiplemonomials
Switch mbed-baremetal and mbed-os to be OBJECT libraries. Can reuse t…
0ba7650 
…he existing create_distro function for this.
@multiplemonomials
Write defines into a disk file instead of passing 100s of them on the…
7b5e25c 
… command line directly. Fixes some issues with build commands on my machine, and makes it a lot easier to view the make VERBOSE=1 output.
@multiplemonomials
Clean up top-level build file and build options, fix build issues on …
a39d51b 
…MinGW
@multiplemonomials
- Split mbed-core and mbed-rtos into -sources and -flags targets
a341c9c 
- Fix some logic errors in top level cmakelists related to building greentea tests
- Fix mbed_create_distro() for OBJECT libraries
@multiplemonomials
Fix PSA and MUSCA build
5990640
@multiplemonomials
Enable running host tests in Github Actions
189ecd2
@multiplemonomials
Enable reset_reason HAL test
da83281
@multiplemonomials
Fix a few more HAL tests
6f6eeec
@multiplemonomials
Convert the rest of the hal tests, convert mbedtls to an OBJECT library
bd23d10
@multiplemonomials
Fix typo preventing unification test from running
c43d248
@multiplemonomials multiplemonomials force-pushed the mbed-ce-build-system-refactor branch from b39684a to c43d248 Compare May 8, 2022 00:52
@multiplemonomials multiplemonomials merged commit b3597c7 into master May 8, 2022
@multiplemonomials multiplemonomials deleted the mbed-ce-build-system-refactor branch May 8, 2022 18:25
@multiplemonomials multiplemonomials mentioned this pull request Feb 4, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@multiplemonomials @0xc0170 @jeromecoutant @hifoolno @caspermeijn