Skip to content

ARM: dts: bcm2835: Switch HSM clock to firmware #5006

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

Merged

Conversation

mripard
Copy link
Contributor

@mripard mripard commented Apr 25, 2022

This is a PR to fix a stall on the Pi3 when it boots without an HDMI display connected.

It was reported in the PR #4940

When the bootloader doesn't initialize boots without a display connected
to the HDMI connector, it won't initialize the HSM clock.

Then, when Linux boots, the default parent fetched by the clock
framework will be the first parent for the HSM clock, which is "gnd".

That clock is never registered, leading to the HSM clock being orphaned,
which leads to other issues with the recent clock range handling.

It's not clear what the proper solution would be, but the firmware
clocks driver doesn't have that parenting issue and the clock rate is
properly initialized for this clock whenever it hasn't been set by the
bootloader.

Signed-off-by: Maxime Ripard <[email protected]>
@pelwell
Copy link
Contributor

pelwell commented Apr 25, 2022

The first sentence of the commit message is hard to follow, but the patch looks OK.

@popcornmix
Copy link
Collaborator

I can confirm this fixes the hang on boot on Pi3+ with hdmi unplugged.

@pelwell pelwell merged commit 82f448e into raspberrypi:rpi-5.15.y Apr 25, 2022
@pelwell
Copy link
Contributor

pelwell commented Apr 25, 2022

Merged with a slightly tweaked commit message.

popcornmix added a commit to raspberrypi/firmware that referenced this pull request Apr 25, 2022
See: raspberrypi/linux#5006

kernel: overlays: Add drm parameter to pitft28-resistive
See: raspberrypi/linux#5004

kernel: Make IMA able to wait and correctly detect the TPM device
See: raspberrypi/linux#5003
popcornmix added a commit to raspberrypi/rpi-firmware that referenced this pull request Apr 25, 2022
See: raspberrypi/linux#5006

kernel: overlays: Add drm parameter to pitft28-resistive
See: raspberrypi/linux#5004

kernel: Make IMA able to wait and correctly detect the TPM device
See: raspberrypi/linux#5003
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants