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

boards: others: add esp32c3_supermini board support #78718

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rlneumiller
Copy link
Contributor

Adding the board under vendor "others" having been unable to determine
the original board designer\vendor with any confidence

Tested samples:
hello_world
basic/blinky

Signed-off-by: <Arrel Neumiller [email protected]>

sylvioalves
sylvioalves previously approved these changes Sep 19, 2024
kartben
kartben previously approved these changes Sep 19, 2024
sylvioalves
sylvioalves previously approved these changes Sep 20, 2024
Copy link

@marekmatej marekmatej left a comment

Choose a reason for hiding this comment

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

Besides the vendor and copyright messages, LGTM

boards/others/esp32c3_supermini/Kconfig.defconfig Outdated Show resolved Hide resolved
boards/others/esp32c3_supermini/Kconfig.defconfig Outdated Show resolved Hide resolved
boards/others/esp32c3_supermini/board.yml Outdated Show resolved Hide resolved
boards/others/esp32c3_supermini/esp32c3_supermini.yaml Outdated Show resolved Hide resolved
@marekmatej
Copy link

Forget to ask for the board picture in the review :)

@marekmatej
Copy link

Please rephrase your commit message to have a general description of adding the new board. The message you put is more suitable for the PR description.

@marekmatej
Copy link

I really don't like the picture quality (and size). Could you maybe use this existing image of the board instead? https://templates.blakadder.com/assets/device_images/SuperMini-ESP32-C3.webp

@rlneumiller rlneumiller force-pushed the add_board_esp32c3_supermini branch 2 times, most recently from d2f2ce7 to 7f89d82 Compare October 1, 2024 15:50
@marekmatej
Copy link

marekmatej commented Oct 14, 2024

@rlneumiller would you please update the board image file name according to the latest standard outlined here #78718

@rlneumiller rlneumiller deleted the add_board_esp32c3_supermini branch October 15, 2024 15:27
@rlneumiller rlneumiller restored the add_board_esp32c3_supermini branch October 15, 2024 19:40
@rlneumiller rlneumiller reopened this Oct 15, 2024
marekmatej
marekmatej previously approved these changes Oct 16, 2024
@marekmatej
Copy link

@nordicjm can you please revisit?

Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

some minor documnetation comments - thanks for the PR!

boards/others/esp32c3_supermini/doc/index.rst Outdated Show resolved Hide resolved
boards/others/esp32c3_supermini/board.yml Show resolved Hide resolved
boards/others/esp32c3_supermini/doc/index.rst Show resolved Hide resolved
Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

some minor documnetation comments - thanks for the PR!

Add support for ESP32-C3 Super Mini board

Signed-off-by: Arrel Neumiller <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants