-
Notifications
You must be signed in to change notification settings - Fork 7.6k
[draft] driver: timer: mspm0: Add timer as sysclock #91490
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?
Conversation
Add a support for timer as sysclock Signed-off-by: Saravanan Sekar <saravanan@linumiz.com>
|
|
||
compatible: "ti,mspm0-timer-sysclock" | ||
|
||
include: base.yaml |
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.
add a required: true for interrupt and reg attributes.
select SYSTEM_TIMER_HAS_DISABLE_SUPPORT | ||
help | ||
This module implements a kernel device driver for the TI MSPM0 | ||
Timer and provides the standard "system clock driver" |
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.
could you detail what could be the usage over the default cortex_m's systick driver? (any pros/cons etc..)
|
||
config TI_MSPM0_TIMER | ||
bool "TI MSPM0 timer" | ||
default n |
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.
you don't need to precise the default for a boolean if it is not about enabling it.
@tbursztyka thanks for review.. PR is not ready for review yet |
Thus the draft label, but it does not harm to do a pre-review |
Add a support for timer used as sysclock