Skip to contentSkip to conversation

arm64: dts: qcom: sm8350-mtp: Use mdt files for firmware

PATCH In review Vinod Koul 2 participants 2 messages Jun 15, 2021 – Jun 16, 2021

Subsystems (3)
lore
Share & tools
lore

Vinod Koul

author
#0 lore raw

PATCH v2 1/1 arm64: dts: qcom: sm8350-mtp: Use mdt files for firmware

As discussed in [1], we should keep one of the file formats for firmware and not change.

So to simplify we chose to use mdt for firmware file. This would enable folks to work with upstream linux-firmware as well as downstream firmwares.

So switch it for SM8350 which is a new platform, so switch can be done safely.

[1]: http://lore.kernel.org/r/CALAqxLXn6wFBAxRkThxWg5RvTuFEX80kHPt8BVja1CpAB-qzGA@mail.gmail.com

---

Changes in v2:
- Add more details about the switch in changelog

 arch/arm64/boot/dts/qcom/sm8350-mtp.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
arch/arm64/boot/dts/qcom/sm8350-mtp.dts +4 -4
index 93740444dd1e..d859305f1f75 100644
@@ -40,7 +40,7 @@ vph_pwr: vph-pwr-regulator {
 
 &adsp {
 	status = "okay";
-	firmware-name = "qcom/sm8350/adsp.mbn";
+	irmware-name = "qcom/sm8350/adsp.mdt";
 };
 
 &apps_rsc {
@@ -278,12 +278,12 @@ vreg_l13c_3p0: ldo13 {
 
 &cdsp {
 	status = "okay";
-	firmware-name = "qcom/sm8350/cdsp.mbn";
+	firmware-name = "qcom/sm8350/cdsp.mdt";
 };
 
 &mpss {
 	status = "okay";
-	firmware-name = "qcom/sm8350/modem.mbn";
+	firmware-name = "qcom/sm8350/modem.mdt";
 };
 
 &qupv3_id_1 {
@@ -292,7 +292,7 @@ &qupv3_id_1 {
 
 &slpi {
 	status = "okay";
-	firmware-name = "qcom/sm8350/slpi.mbn";
+	firmware-name = "qcom/sm8350/slpi.mdt";
 };
 
 &tlmm {
Signature · 1 line
2.31.1
#1 lore raw
Quoted 12 lines · Vinod Koul

As discussed in [1], we should keep one of the file formats for firmware and not change.

So to simplify we chose to use mdt for firmware file. This would enable folks to work with upstream linux-firmware as well as downstream firmwares.

So switch it for SM8350 which is a new platform, so switch can be done safely.

[1]: http://lore.kernel.org/r/CALAqxLXn6wFBAxRkThxWg5RvTuFEX80kHPt8BVja1CpAB-qzGA@mail.gmail.com

As mentioned elsewhere, we used to run exclusively with mdt + bNN files
- as that's what we typically get with Android - and I kept receiving reports about people missing individual bNN files, or managing to upgrade some but not all of the files etc. Some of these errors tells you which file you're missing, others just tells you that the hashes didn't match whatever was loaded.

As such, the move to .mbn files has saved us quite a bit of head ache.

That said, the code doesn't care about the suffix per say, it starts loading the firmware and detects if it's a combined file or not. So you can simply copy or symlink the .mdt to match the given .mbn path.

Quoted 14 lines

---

Changes in v2:
- Add more details about the switch in changelog

 arch/arm64/boot/dts/qcom/sm8350-mtp.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
arch/arm64/boot/dts/qcom/sm8350-mtp.dts +0 -0
index 93740444dd1e..d859305f1f75 100644

arch/arm64/boot/dts/qcom/sm8350-mtp.dts vph_pwr: vph-pwr-regulator { · line 43

Quoted 5 lines
arch/arm64/boot/dts/qcom/sm8350-mtp.dts +1 -1
@@ -40,7 +40,7 @@ vph_pwr: vph-pwr-regulator {
 
 &adsp {
 	status = "okay";
-	firmware-name = "qcom/sm8350/adsp.mbn";
+	irmware-name = "qcom/sm8350/adsp.mdt";

And you're already doing something "non-standard" if your MTP has adsp.mdt in this path :)

Regards,
Bjorn

Quoted 30 lines
arch/arm64/boot/dts/qcom/sm8350-mtp.dts +3 -3
 };
 
 &apps_rsc {
@@ -278,12 +278,12 @@ vreg_l13c_3p0: ldo13 {
 
 &cdsp {
 	status = "okay";
-	firmware-name = "qcom/sm8350/cdsp.mbn";
+	firmware-name = "qcom/sm8350/cdsp.mdt";
 };
 
 &mpss {
 	status = "okay";
-	firmware-name = "qcom/sm8350/modem.mbn";
+	firmware-name = "qcom/sm8350/modem.mdt";
 };
 
 &qupv3_id_1 {
@@ -292,7 +292,7 @@ &qupv3_id_1 {
 
 &slpi {
 	status = "okay";
-	firmware-name = "qcom/sm8350/slpi.mbn";
+	firmware-name = "qcom/sm8350/slpi.mdt";
 };
 
 &tlmm {
Signature · 1 line
2.31.1