metro: Hello, world! - #2676
metro: Hello, world!#2676
Conversation
labbott
left a comment
labbott
left a comment
There was a problem hiding this comment.
A few notes on what order we might want to prioritize adding things just so that it Does Something
| // initialized, the peripherals won't work, but the result should not be | ||
| // _fatal._ | ||
|
|
||
| // Pin mapping: |
There was a problem hiding this comment.
Spot check shows these still match
| // NOTE: this implementation only brings up a 20-bit address on the FMC bus, | ||
| // because that's what worked on Grapefruit. The hardware supports a 24-bit | ||
| // address! |
There was a problem hiding this comment.
Still a relevant comment ?
| # task-slots = ["i2c_driver", "sensor", "metro_seq"] | ||
| # notifications = ["timer"] | ||
|
|
||
| # [tasks.hiffy] |
There was a problem hiding this comment.
I'd vote to get hiffy added ASAP since we may want to use it to hit things with hammers
| # stacksize = 4000 | ||
| # notifications = ["timer"] | ||
|
|
||
| # [tasks.update_server] |
There was a problem hiding this comment.
This should be safe to bring in since it's pure STM32H7
| # [tasks.sys] | ||
| # name = "drv-stm32xx-sys" | ||
| # features = ["h753", "exti", "no-panic"] | ||
| # priority = 1 | ||
| # uses = ["rcc", "gpios", "system_flash", "syscfg", "exti"] | ||
| # start = true | ||
| # task-slots = ["jefe"] | ||
| # notifications = ["exti-wildcard-irq"] | ||
|
|
||
| # [tasks.sys.interrupts] | ||
| # "exti.exti0" = "exti-wildcard-irq" | ||
| # "exti.exti1" = "exti-wildcard-irq" | ||
| # "exti.exti2" = "exti-wildcard-irq" | ||
| # "exti.exti3" = "exti-wildcard-irq" | ||
| # "exti.exti4" = "exti-wildcard-irq" | ||
| # "exti.exti9_5" = "exti-wildcard-irq" | ||
| # "exti.exti15_10" = "exti-wildcard-irq" |
There was a problem hiding this comment.
I'd vote to review/enable this first since this should mostly be STM32H7 specific things
| # task-slots = ["sys", "i2c_driver"] | ||
| # stacksize = 800 | ||
|
|
||
| # [tasks.user_leds] |
There was a problem hiding this comment.
This is also a useful one to enable quickly
|
A general comment is that metro will have a new sequencer FPGA image that is not common with cosmos, though for most purposes they'll look the same barring some new features around programming the Versal etc. I'm working on getting preliminary bit files but we can start the plumbing by renaming a cosmo seq image if that is helpful |
|
Opened up a tracking issue for follow-up work for this PR: #2678, might be better to move discussions/notes there instead of this PR which I hope to merge Soon™️. |
3 participants
Adds a new, mostly commented out, definition for "Metro" compute sled hardware.
This PR is basically:
cosmo/rev-b.tomlchanges applied tometro/base.tomlapp/metrofolderThe intent is to start with everything commented out (and CI being green), and work with @labbott over the next bit to review and uncomment one block at a time as necessary.
We may not want to squash-merge this PR, but I'm open to thoughts on that.