tock / tock-stm32

Ports of Tock for STM32 chips and discovery boards
Other
22 stars 5 forks source link

Support for STM32F1 #1

Closed george-hopkins closed 6 years ago

george-hopkins commented 6 years ago

Pull Request Overview

Original Submission: helena-project/tock#686

This pull request attempts to bring Tock OS to the STM32F1 series. At the moment it includes all the necessary work to setup a working build infrastructure on Travis.

To avoid one big chunk of new code, this patch attempts to bring STM32F1 to tier III. The following features have been implemented so far:

With these features, the example application blink runs successfully. In addition, pre-compiled binaries for Cortex-M3 are available in userland/ to run Lua applications (they need to be copied to the kernel directory for the time being, though).

Testing Strategy

The pull request was tested on a custom board based on a STM32F103ZE. The board runs at 72 MHz with an external oscillator. A board layer for the official development board, Nucleo-F103RB is included.

TODO or Help Wanted

¹ This can be resolved later on after helena-project/tock#658 is merged

Formatting

alevy commented 6 years ago

@george-hopkins I'm going to add you as a contributor to this repository, if that's ok, so you can push directly to master, merge PRs, etc. As development starts getting going more we can figure out something if something more formal is needed, but I figure just the first few people that are contributing can get "commit rights" and we can go from there.

I likely won't be able to test code changes for the most part

george-hopkins commented 6 years ago

@alevy Thank you very much for your support! I'm honored to be considered as a collaborator. I will likely keep submitting changes through PRs rather than commiting directly such that others can comment and review them.

Please feel free to reach out to me if something comes up (my email is linked on my profile).

eddyp commented 5 years ago

Is there a reason why there is a tock-stm32 repo and the code is not merged into tock main repo? I saw the main discussion in https://github.com/tock/tock/pull/686 revolved around STM32 chips without an MPU, but after looking into the STM32F103ZG data sheet I saw it has an MPU, so I am confused.

Having in tockos main repo support for STM32 micros with MPU and having a tock-nonmpu repo seems to be a solution more in line with the decision around creating this repo.

alevy commented 5 years ago

@eddyp yes, you're right. The reason is purely historical---support for an STM32 with an MPU was just merged into the main repo. And I agree that we should focus any effort for STM32s with an MPU in that repo from now on.

eddyp commented 5 years ago

@alevy so the info on the site its not 100% accurate? I mean stm32 is not mentioned on the site, even if as a tier 2(?), this mislead me to believe I can experiment with Tock.

Maybe this info should be updated and this repo be renamed tock-nompu or at least the readme should say this is a fork aiming to support nonmpu stms and very immature stm32 ports?