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.

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

[WIP] STM32F411CE (blackpill) port #159

Closed
wants to merge 3 commits into from
Closed

Conversation

sarg
Copy link
Contributor

@sarg sarg commented Jan 29, 2024

Hi there!
I've got myself a Blackpill board which is based on the same STM32F4-family MCU. Given it's the same architecture, I thought it'll be easy to port hydrafw to this board. F411CE has lower speed (100Mhz) and less peripherals, so I had to adjust the code a bit to make it work. I've tested the firmware on the device and it works fine. I wasn't able to verify all modes, but UART, ADC, GPIO seem to work. flashrom recognizes the device as well, but I haven't yet read a flash chip over SPI, probably due to bad contact when using a test clip. debug timing command hangs the MCU, not sure what to make of it.

If you're willing to accept such addition to your project, we can work on cleaning this WIP branch and making the code open to run on other boards.

@bvernoux
Copy link
Member

Thank you for your contribution, However, these modifications will cause issues with the HydraBus HW (lot of things are broken and will requires a better abstraction layer).
Unfortunately, supporting BlackPill requires ownership and time investment, which I do not currently plan on doing due to its limitations compared to the dedicated HydraBus platform designed specifically for HydraFW.

@sarg
Copy link
Contributor Author

sarg commented Jan 30, 2024

Fair enough.

Check this commit though, it's just a small clean up of the existing code: d17cff9

@sarg sarg closed this Jan 30, 2024
@bvernoux
Copy link
Member

bvernoux commented Jan 30, 2024

Your project is very interesting anyway, if you want to maintain your own fork it is probably the best we could also add a link to it on the Wiki if you want...
Other contribution to hydrafw are welcome of course

@bvernoux
Copy link
Member

Fair enough.

Check this commit though, it's just a small clean up of the existing code: d17cff9

You are welcome to do a new PR with that cleanup to be merged

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

Successfully merging this pull request may close these issues.

None yet

2 participants