Skip to content

qt-py m0 definition#8

Draft
hierophect wants to merge 1 commit intoadafruit:mainfrom
hierophect:qtpy-samd21
Draft

qt-py m0 definition#8
hierophect wants to merge 1 commit intoadafruit:mainfrom
hierophect:qtpy-samd21

Conversation

@hierophect
Copy link
Copy Markdown
Collaborator

Definition for the Adafruit QT Py M0 (samd21). Drafted while I figure out why dts is acting up again:

devicetree error: /Users/hierophect/hiero_projects/zephyrtesting/adafruit-zephyr-support/boards/qt_py_m0/qt_py_m0-pinctrl.dtsi:25 (column 14): parse error: expected number or parenthesized expression

I'm fairly confident there's nothing wrong at that actual line, as it's functionally identical to other boards of this type, and is likely originating from some other part of the devicetree.

Copy link
Copy Markdown
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

One request on the partitions. Thanks!

Comment on lines +83 to +91
/*
* The final 16 KiB is reserved for the application.
* Storage partition will be used by FCB/LittleFS/NVS
* if enabled.
*/
storage_partition: partition@3c000 {
label = "storage";
reg = <0x0003c000 DT_SIZE_K(16)>;
};
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you need this for every board or can it be used for anything with a xiao_i2c?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good question. There's no reason it shouldn't based on the contents, just have to look at how to get the build system to recognize it.

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