Skip to content
This repository has been archived by the owner on Jul 6, 2019. It is now read-only.

Basic support for the STM32F429I-DISCO board #81

Closed
wants to merge 10 commits into from
Closed

Conversation

ah-
Copy link

@ah- ah- commented Jul 3, 2014

This updates some of the existing stm32f4 support so it compiles again, and adds a very simple blink demo app.

To compile, run rake PLATFORM=stm32f4 build_blink_stm.

It doesn't yet use the platformtree stuff, as that depends on the newer pin.rs.

@bharrisau
Copy link
Contributor

Thanks for this. Can you update the travis.yml file to look something like this.

...
script:
    - rake $TARGS test
...
env:
  matrix:
    - PLATFORM=lpc17xx TARGS="build_all"
    - PLATFORM=stm32f4 TARGS="build_blink_stm"

And remove the allow failures stuff.

@bharrisau
Copy link
Contributor

cc #66

@bharrisau
Copy link
Contributor

If the above changes to travis dont work, might need to make it into an 'include' type http://docs.travis-ci.com/user/build-configuration/#Explicit-inclusion-of-jobs-into-the-build-matrix

@bharrisau
Copy link
Contributor

I've edited my first comment with the required changes.

@ah-
Copy link
Author

ah- commented Jul 3, 2014

Thanks, I've updated travis.yml. I suppose it'll still fail because of the int things with the newest rustc, but it should be better now.

Do we have any plans for supporting different boards? I have a NUCLEO-F401RE, and a very similar blink demo works fine on that as well, it just needs a different __STACK_BASE etc. in layout.ld and different pins.

@bharrisau
Copy link
Contributor

Nope, it builds successfully. Well done. Just need to rebase it once the lpcxx build is fixed. I haven't had a chance to review the code yet.

@bharrisau
Copy link
Contributor

As for supporting other boards, that is definitely the goal. I think the apps are mostly meant as an example/for zinc development. I'm not sure yet the best way to do your own project, ideally the PT stuff (or the zinc.rlib) would generate the linker files so you aren't tied to the zinc repository.

@farcaller
Copy link
Member

broken lpc part is being fixed over here: #80. I intend to merge this after that. The code should be ported over to PT, though, but it's better to have at least something working in the meantime.

Unfortunately PT is still quite fragile, there's a second chunk of big changes: #83, and I'm quite sure it's not the last, as I got problems with materialising templated structs (like DHT22<'a, S, P>).

@ah-
Copy link
Author

ah- commented Jul 3, 2014

Yes, I'm intending to get it onto PT, but that's more of a second step once the basics are working and up to what's currently in the lpc platform. Also it seems that PT is still very much in flux, so it's probably easier if I just work on something else that won't conflict all the time.

@farcaller
Copy link
Member

Please rebase on HEAD (#84 fixes tests).

@bharrisau
Copy link
Contributor

Looks like it will merge without a rebase (I just restarted the tests), but I'd like to see some of the commits squashed anyway.

@bharrisau
Copy link
Contributor

Bitrot has gotten to this - can you please rebase when you get a chance. Squash the commits a little at the same time.

errordeveloper pushed a commit to errordeveloper/zinc that referenced this pull request Jul 13, 2014
Signed-off-by: Ilya Dmitrichenko <[email protected]>
@errordeveloper errordeveloper mentioned this pull request Jul 13, 2014
errordeveloper pushed a commit to errordeveloper/zinc that referenced this pull request Jul 13, 2014
errordeveloper pushed a commit to errordeveloper/zinc that referenced this pull request Jul 13, 2014
errordeveloper added a commit to errordeveloper/zinc that referenced this pull request Jul 13, 2014
errordeveloper added a commit to errordeveloper/zinc that referenced this pull request Jul 13, 2014
@hacknbot hacknbot closed this in #97 Jul 14, 2014
@farcaller
Copy link
Member

Merged in #97, @ah- and @errordeveloper, thanks for your work!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants