Skip to content

Esp32c3#20

Open
faldor20 wants to merge 1 commit intoelcritch:develfrom
faldor20:esp32c3
Open

Esp32c3#20
faldor20 wants to merge 1 commit intoelcritch:develfrom
faldor20:esp32c3

Conversation

@faldor20
Copy link

@faldor20 faldor20 commented Apr 3, 2023

Added support for esp32c3 board

fix issue with incorrect pin
@faldor20 faldor20 changed the base branch from master to devel April 3, 2023 12:51
###
# ESP32 specific io_mux_reg.h port
##
arsoienieonrstg
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo?

@@ -0,0 +1,551 @@
/*
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this file?


const hdr = "soc/io_mux_reg.h"

var SLP_OE* {.importcpp: "SLP_OE", header: hdr.}: int
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe there's a new option which is to use let. Also, these should use importc?

Suggested change
var SLP_OE* {.importcpp: "SLP_OE", header: hdr.}: int
let SLP_OE* {.importc: "SLP_OE", header: hdr.}: int

when defined(esp32s3):
include ./esp32s3/io_mux_reg
elif defined(esp32c3):
include ./esp32c3/io_mux_reg
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, this pattern appears to be working well.

@elcritch
Copy link
Owner

@faldor20 heya I reviewed this last year but forgot to submit the review. This still looks alright to me, pending my comments.

@faldor20
Copy link
Author

Hey, I'm probably never going to get to this, I don't actually use nim for anything these days and it's been well over a year since I last even had it installed. You're welcome to take it over and merge it though :)

@elcritch
Copy link
Owner

Hey, I'm probably never going to get to this, I don't actually use nim for anything these days and it's been well over a year since I last even had it installed. You're welcome to take it over and merge it though :)

That's too bad, but thanks for the heads up!

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.

2 participants