mirrors/linux
2
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git synced 2026-05-09 17:17:04 -04:00
Code Issues Packages Projects Releases Wiki Activity

arm64: dts: qcom: sc7280-herobrine: Fix PCIe regulator glitch at bootup

Browse Source 
While scoping signals, we found that the PCIe signals weren't
compliant at bootup. Specifically, the bootloader was setting up PCIe
and leaving it configured, then jumping to the kernel. The kernel was
turning off the regulator while leaving the PCIe clock running, which
was a violation.

In the regulator bindings (and the Linux kernel driver that uses
them), there's currently no way to specify that a GPIO-controlled
regulator should keep its state at bootup. You've got to pick either
"on" or "off". Let's switch it so that the PCIe regulator defaults to
"on" instead of "off". This should be a much safer way to go and
avoids the timing violation. The regulator will still be turned off
later if there are no users.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/20220310130429.1.Id41fda1d7f5d9230bc45c1b85b06b0fb0ddd29af@changeid
This commit is contained in:
Douglas Anderson
2022-03-10 13:04:34 -08:00
committed by Bjorn Andersson
parent 6675ac9d00
commit 0d40497d05
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
View File

@@ -180,6 +180,13 @@ pp3300_ssd: pp3300-ssd-regulator {
pinctrl-names = "default";
pinctrl-0 = <&ssd_en>;
/*
* The bootloaer may have left PCIe configured. Powering this
* off while the PCIe clocks are still running isn't great,
* so it's better to default to this regulator being on.
*/
regulator-boot-on;
vin-supply = <&pp3300_z1>;
};