Skip to content
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

soc: samd5x: xosc32 initialization - 8 seconds start-up time #79949

Open
nandojve opened this issue Oct 16, 2024 Discussed in #79883 · 0 comments · May be fixed by #79950
Open

soc: samd5x: xosc32 initialization - 8 seconds start-up time #79949

nandojve opened this issue Oct 16, 2024 Discussed in #79883 · 0 comments · May be fixed by #79950
Assignees
Labels
platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM)

Comments

@nandojve
Copy link
Member

Discussed in #79883

Originally posted by wag-thom October 11, 2024
The commit 58a54e3 (@nandojve) changed in soc_samd5x.c in the osc32k_init() function the register configuration for the external 32K crystal.

| OSC32KCTRL_XOSC32K_STARTUP(6) | OSC32KCTRL_XOSC32K_CGM_XT;

Previously, the start-up value was set to 7 - which was a reserved according to the datasheet.
Now it is set to 6 - which translates to a start-up time of 8 seconds. I don't think this is a sensible value for most users.

image

I would suggest to either:

  • change the value to 0 (Which will result in the same behavior as before the change - I get the same start-up time with 0 or 7)
  • or let the user set the value with some Kconfig option
@nandojve nandojve added the platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) label Oct 16, 2024
@nandojve nandojve self-assigned this Oct 16, 2024
nandojve added a commit to nandojve/zephyr that referenced this issue Oct 16, 2024
The current selected start-up time takes 8 seconds to initialize.
When xosc32 is used as main clock reference it blocks the whole
initializarion of the system by that amount of time. This patch
relax that condition setting the initialization time to 62ms.

Fixes zephyrproject-rtos#79949

Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve linked a pull request Oct 16, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant