Commit a9a9e857 authored by Alex Elder's avatar Alex Elder Committed by Bjorn Andersson

arm64: dts: qcom: use qcom,gsi-loader for IPA

Depending on the platform, either the modem or the AP must load GSI
firmware for IPA before it can be used.  To date, this has been
indicated by the presence or absence of a "modem-init" property.

That mechanism has been deprecated.  Instead, we indicate how GSI
firmware should be loaded by the value of the "qcom,gsi-loader"
property.

Update all arm64 platforms that use IPA to use the "qcom,gsi-loader"
property to specify how the GSI firmware is loaded.

Update the affected nodes so the status property is last.
Signed-off-by: default avatarAlex Elder <elder@linaro.org>
Reviewed-by: default avatarKonrad Dybcio <konrad.dybcio@linaro.org>
[bjorn: Moved sc7280 change herobrine-lte-sku]
Signed-off-by: default avatarBjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20221231002716.2367375-3-elder@linaro.org
parent bf37b5bc
......@@ -19,12 +19,11 @@ &remoteproc_mpss {
};
&ipa {
status = "okay";
/*
* Trogdor doesn't have QHEE (Qualcomm's EL2 blob), so the
* modem needs to cover certain init steps (GSI init), and
* the AP needs to wait for it.
*/
modem-init;
qcom,gsi-loader = "modem";
status = "okay";
};
......@@ -21,7 +21,7 @@ mba_mem: memory@9c700000 {
};
&ipa {
modem-init;
qcom,gsi-loader = "modem";
status = "okay";
};
......
......@@ -756,8 +756,8 @@ &gpu {
};
&ipa {
qcom,gsi-loader = "modem";
status = "okay";
modem-init;
};
&lpasscc {
......
......@@ -474,8 +474,8 @@ zap-shader {
};
&ipa {
qcom,gsi-loader = "modem";
status = "okay";
modem-init;
};
&mss_pil {
......
......@@ -518,8 +518,9 @@ &i2c10 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
status = "okay";
};
&mdss {
......
......@@ -424,10 +424,10 @@ rmi4_f12: rmi4-f12@12 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
firmware-name = "qcom/sdm845/oneplus6/ipa_fws.mbn";
status = "okay";
};
&mdss {
......
......@@ -495,10 +495,10 @@ touchscreen@38 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
firmware-name = "qcom/sdm845/axolotl/ipa_fws.mbn";
status = "okay";
};
&mdss {
......
......@@ -299,9 +299,10 @@ &mss_pil {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
firmware-name = "qcom/sdm845/beryllium/ipa_fws.mbn";
status = "okay";
};
&pm8998_gpios {
......
......@@ -449,6 +449,7 @@ &ibb {
};
&ipa {
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
firmware-name = "qcom/sdm845/polaris/ipa_fws.mbn";
status = "okay";
......
......@@ -465,8 +465,9 @@ ecsh: hid@5c {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
status = "okay";
};
&mdss {
......
......@@ -383,9 +383,10 @@ digitizer@9 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&ipa_fw_mem>;
firmware-name = "qcom/sdm850/samsung/w737/ipa_fws.elf";
status = "okay";
};
/* No idea why it causes an SError when enabled */
......
......@@ -290,9 +290,9 @@ &i2c11 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&pil_ipa_fw_mem>;
status = "okay";
};
&mpss {
......
......@@ -376,7 +376,7 @@ &usb_2_qmpphy {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&pil_ipa_fw_mem>;
status = "okay";
};
......@@ -553,9 +553,10 @@ &i2c17 {
};
&ipa {
status = "okay";
qcom,gsi-loader = "self";
memory-region = <&pil_ipa_fw_mem>;
firmware-name = "qcom/sm8350/Sony/sagami/ipa_fws.mbn";
status = "okay";
};
&mpss {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment