-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Add soc em32f967 #97843
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?
Add soc em32f967 #97843
Conversation
This is a SOC based on ARM. Signed-off-by: Johnny Chuang <[email protected]>
This is a development board for Elan em32f967 SOC. Signed-off-by: Johnny Chuang <[email protected]>
Hello @ElanJohnnyChuang, and thank you very much for your first pull request to the Zephyr project! |
|
reg = <0x20000000 DT_SIZE_K(448)>; | ||
}; | ||
|
||
clocks { |
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 file is not formatted correctly
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 draft PR update is mainly for me to get familiar with the workflow. I’m aware that the current code still has many aspects that don’t meet the requirements. I will address those based on your suggestions and the issues reported by the CI pipeline. After the necessary fixes and optimizations, I’ll force‑push the changes to this PR and then convert the draft PR into a regular PR for your review. Thank you.
default 64 | ||
|
||
config SYS_CLOCK_HW_CYCLES_PER_SEC | ||
default 60000000 |
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.
get from dt function from dts
HCLKF_GHM_M2D = 0x2b, | ||
PCLKG_SSP1 = 0x30, | ||
PCLKG_ALL = 0xffff | ||
} CLKGatingSwitch; |
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.
zephyr is snake case not camel case
#include "em32f967.h" | ||
#include "soc_967.h" | ||
|
||
extern const struct device __device_dts_ord_DT_CHOSEN_zephyr_cortex_m_idle_timer_ORD; |
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.
no
This is a Draft PR test for Elan em32f967 SOC.