Skip to content

soc/da1469x: Fix CONFIG_SYS_CLOCK_*_PER_SEC defines #73336

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

Conversation

ydamigos
Copy link
Collaborator

@ydamigos ydamigos commented May 27, 2024

Needs PR #73403 for function "dt_node_ph_prop_path"

The function is used so the following configuration options for DA1469x:

SYS_CLOCK_HW_CYCLES_PER_SEC
SYS_CLOCK_TICKS_PER_SEC

could get their values according to lp_clock node's clock-src property.

@zephyrbot zephyrbot added area: Kconfig platform: Renesas SmartBond Renesas Electronics Corporation, SmartBond area: Devicetree labels May 27, 2024
@ydamigos ydamigos changed the title Da1469x fix ticks per sec kconfig macros soc/da1469x: Fix CONFIG_SYS_CLOCK_*_PER_SEC defines May 27, 2024
@ydamigos
Copy link
Collaborator Author

"Compliance Checks" failure seems unrelated to PR changes.

Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be a separate PR

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Opened PR #73403

@ydamigos ydamigos added the DNM This PR should not be merged (Do Not Merge) label May 28, 2024
@ydamigos ydamigos force-pushed the da1469x-fix-ticks-per-sec-kconfig-macros branch from 99f2102 to c995ac7 Compare May 28, 2024 12:56
@decsny decsny removed their request for review June 4, 2024 13:04
@ydamigos ydamigos force-pushed the da1469x-fix-ticks-per-sec-kconfig-macros branch from c995ac7 to fe77741 Compare June 6, 2024 08:14
@ydamigos
Copy link
Collaborator Author

ydamigos commented Jun 6, 2024

PR #73403 was merged. Removing DNM label.

@ydamigos ydamigos removed the DNM This PR should not be merged (Do Not Merge) label Jun 6, 2024
The following configuration options:

SYS_CLOCK_HW_CYCLES_PER_SEC
SYS_CLOCK_TICKS_PER_SEC

should get their values according to lp_clock node's
clock-src property.

Signed-off-by: Ioannis Damigos <[email protected]>
@ydamigos ydamigos force-pushed the da1469x-fix-ticks-per-sec-kconfig-macros branch from fe77741 to f0b9e31 Compare June 6, 2024 13:34
@aescolar aescolar merged commit b7fbd91 into zephyrproject-rtos:main Jun 7, 2024
22 checks passed
@ydamigos ydamigos deleted the da1469x-fix-ticks-per-sec-kconfig-macros branch June 7, 2024 08:20
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.

7 participants