Skip to content

Add Daisy Seed variant [replace #1063] #1070

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 2 commits into from
May 15, 2020
Merged

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented May 15, 2020

This PR replace the PR #1063 from @stephenhensley

Electrosmith Daisy Variant

Summary

Adds the Electrosmith Daisy as a variant

The Daisy is a small form factor, audio-centric SOM embedding an STM32H750

It features a high-quality,stereo audio codec, 64MB of external SDRAM, 8MB of external flash,

Support for the additional memories/audio has not been added yet (audio seems like it should be done in its own library). So, for now the variant has 512kB of internal SRAM, and 128kB of internal flash mapped as the default memories.

InitVariant() needs to be added to initialize the external memories, but all of the external pins, and several of their alternate functions have been tested (SPI, I2C, UART, ADC, DAC, PWM, etc.).

Signed-off-by: Stephen Hensley <stephen.p.hensley@gmail.com>
Co-Authored-By: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm added the new variant Add support of new bard label May 15, 2020
@fpistm fpistm added this to the 1.9.0 milestone May 15, 2020
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm merged commit 7bf9f34 into stm32duino:master May 15, 2020
@fpistm fpistm deleted the pr_1063_review branch May 15, 2020 13:15
@fpistm fpistm mentioned this pull request May 15, 2020
@stephenhensley
Copy link

@fpistm

Thanks! Sorry for not getting back fast enough.

I'll definitely keep up on it more in the future.

Once I've retested the hardware/peripherals with new clock tree settings I'll have an update that adds SDRAM init, and moves CPU freq to the maximum 480MHz.

Do you have a standard for handling alternate memories? (i.e. should I have a variant that uses external SDRAM as default bss, or just provide a macro for the appropriate section attribute?)

@fpistm
Copy link
Member Author

fpistm commented May 15, 2020

No worries,

anyway the 1.9.0 will be released today.

So update will be done on the next release.
For the alternate memories, I never do it anyway you can do all you want in the variant and linker script, you can also change the default startup file if needed.
IIRW Malyan variant do something like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new variant Add support of new bard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants