-
Notifications
You must be signed in to change notification settings - Fork 8.3k
boards: microchip: Add pic64gx_curiosity_kit support #99692
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
base: main
Are you sure you want to change the base?
boards: microchip: Add pic64gx_curiosity_kit support #99692
Conversation
Add minimal support for PIC64GX SoC and devicetree Signed-off-by: Pierre-Henry Moussay <[email protected]>
Add minimal support for PIC64GX curiosity kit Signed-off-by: Pierre-Henry Moussay <[email protected]>
Add pic64 SoC and pic64gx_curiosity_kit to Microchip RISCV board and SoC Signed-off-by: Pierre-Henry Moussay <[email protected]>
|
Hello @pmoussay-emdalo, and thank you very much for your first pull request to the Zephyr project! |
|
| - fkokosinski | ||
| - kgugala | ||
| - tgorochowik | ||
| - con-pax |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
way too many maintainers here, are all of those active? @fkokosinski @kgugala @tgorochowik are you still actively maintaining this platform?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, we're still maintaining Microchip RISC-V platforms. @pmoussay-emdalo - feel free to remove relegate @tgorochowik and @kgugala to collaborators
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to give a confirmation, we discussed this offline and we're okay with moving us
fkokosinski
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Convert this photo to webp and compress it using e.g. https://tinypng.com/
| @@ -0,0 +1,38 @@ | |||
| # SPDX-License-Identifier: GPL-2.0-or-later | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please consult this documentation page: https://docs.zephyrproject.org/latest/contribute/guidelines.html#licensing
|
|
||
| if BOARD_PIC64GX_CURIOSITY_KIT | ||
|
|
||
| endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this file needed?
| CONFIG_FPU=n | ||
| # GPIO driver options | ||
| CONFIG_GPIO=y | ||
| CONFIG_I2C=y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be selected by samples/tests/application that actually use I2C
| CONFIG_SYS_CLOCK_TICKS_PER_SEC=1000 | ||
| # GPIO driver options | ||
| CONFIG_GPIO=y | ||
| CONFIG_I2C=y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
| CONFIG_SYS_CLOCK_TICKS_PER_SEC=1000 | ||
| # GPIO driver options | ||
| CONFIG_GPIO=y | ||
| CONFIG_I2C=y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
| CONFIG_SYS_CLOCK_TICKS_PER_SEC=1000 | ||
| # GPIO driver options | ||
| CONFIG_GPIO=y | ||
| CONFIG_I2C=y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
| - fkokosinski | ||
| - kgugala | ||
| - tgorochowik | ||
| - con-pax |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, we're still maintaining Microchip RISC-V platforms. @pmoussay-emdalo - feel free to remove relegate @tgorochowik and @kgugala to collaborators
| payload-name: "zephyr" | ||
| } | ||
| Please refer to the following README.md for more information on playload generation: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s'/playload/payload/'



Add initial support for the Microchip PIC64GX Curiosity Kit development Board.
This work adds initial support for 'pic64gx_curiosity_kit' which has a five core micro-
processor subsystem with:
SiFive E51 Monitor core (1 x RV64IMAC)
SiFive U54 Application cores (4 x RV64GC)
Product page: https://www.microchip.com/en-us/development-tool/curiosity-pic64gx1000-kit