Update MCXA metapac to work with newer toolchain#77
Update MCXA metapac to work with newer toolchain#77Wassasin merged 46 commits intoembassy-rs:mainfrom
Conversation
| ChipDescription { chip: "LPC55S16", metadata: Some("LPC55S16"), cores: &["LPC55S16"], metapac: false }, | ||
| ChipDescription { chip: "LPC55S69", metadata: Some("LPC55S6x"), cores: &["LPC55S69_cm33_core0", "LPC55S69_cm33_core1"], metapac: false }, | ||
| ChipDescription { chip: "MCXN947", metadata: None, cores: &["MCXN947_cm33_core0", "MCXN947_cm33_core1"], metapac: false }, | ||
| ChipDescription { chip: "MCXA256", metadata: Some("MCXA2xx"), cores: &["MCXA256"], metapac: true }, |
There was a problem hiding this comment.
I've been thinking about this. Technically, MCXA256 is one of the smaller variants. Should we move to the 266 SVD instead? According to the datasheet , the 266 has more uarts, more CAN, has the Segment LCD controller, and can lock the CPU at 240MHz.
It looks like, at some point, the PAC should learn about more variants of the MCXA family. Maybe not worth looking at it right now, but we should probably revisit this in the future.
There was a problem hiding this comment.
Doesn't matter that much in the end as long as the metadata is correct.
The metadata can work for multiple chips, but that's not really used yet.
Every peripheral can have an only_in field with regex, so it only applies to a subset of the chips.
But that's unused currently
| ## Updating | ||
|
|
||
| The chips supported by this project are defined in [`lib.rs`](/generator/src/lib.rs). | ||
| The SVDs can be selected from the submodule in `data/mcux-soc-svd`. |
There was a problem hiding this comment.
I think, at some point, it might also be worth removing the submodule and teaching the generator to download the latest release ZIP archive.
We would probably want to also (originally suggested by @diondokter) split the generated crate out to its own repository.
There was a problem hiding this comment.
I would at least want to pin the SVD to a specific version.
Wassasin
force-pushed
the
mcxa-metapac-update
branch
2 times, most recently
from
739b82d to
f188932
Compare
Wassasin
force-pushed
the
mcxa-metapac-update
branch
from
1b32c72 to
aa1f6fa
Compare
3 participants
Requires #69 to be merged first.
The transforms have been been kept up to date with the manual changes to the metapac. Furthermore, some metapac files do not have the correct casing for the various fields. This PR intends to remedy that.
Will cause breaking changes necessitating HAL fixes. Most notably enum variants are now PascalCase.
TODO
generator -- generate