2022-05-23 05:52:09

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH 12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible

Add qcom,msm8998 SoC fallback to the board compatible.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm64/boot/dts/qcom/msm8998-mtp.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
index 66540d2ca13b..db240a3207dc 100644
--- a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
+++ b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
@@ -7,7 +7,7 @@

/ {
model = "Qualcomm Technologies, Inc. MSM8998 v1 MTP";
- compatible = "qcom,msm8998-mtp";
+ compatible = "qcom,msm8998-mtp", "qcom,msm8998";

qcom,board-id = <8 0>;
};
--
2.32.0



2022-05-23 08:10:10

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible

On 22/05/2022 11:11, Konrad Dybcio wrote:
>
>
> On 21.05.2022 18:45, Krzysztof Kozlowski wrote:
>> Add qcom,msm8998 SoC fallback to the board compatible.
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>> ---
>> arch/arm64/boot/dts/qcom/msm8998-mtp.dts | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
>> index 66540d2ca13b..db240a3207dc 100644
>> --- a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
>> +++ b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
>> @@ -7,7 +7,7 @@
>>
>> / {
>> model = "Qualcomm Technologies, Inc. MSM8998 v1 MTP";
> A bit off-topic, but should this board still be considered supported? If it really is msm8998v1, it should not work in the current state of mainline, as there are many differences between the pre-production v1 and the final v2.1, even as deep as in the clock tree.

Unless you are sure there are no users or it is not working, there is
not much benefit to remove such board. Specially that it's DTSI is
re-used by one more - fxtec-pro1.


Best regards,
Krzysztof

2022-05-23 08:11:23

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH 12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible



On 21.05.2022 18:45, Krzysztof Kozlowski wrote:
> Add qcom,msm8998 SoC fallback to the board compatible.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/msm8998-mtp.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
> index 66540d2ca13b..db240a3207dc 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8998-mtp.dts
> @@ -7,7 +7,7 @@
>
> / {
> model = "Qualcomm Technologies, Inc. MSM8998 v1 MTP";
A bit off-topic, but should this board still be considered supported? If it really is msm8998v1, it should not work in the current state of mainline, as there are many differences between the pre-production v1 and the final v2.1, even as deep as in the clock tree.

Konrad
> - compatible = "qcom,msm8998-mtp";
> + compatible = "qcom,msm8998-mtp", "qcom,msm8998";
>
> qcom,board-id = <8 0>;
> };
>

2022-06-22 08:37:51

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: (subset) [PATCH 12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible

On Sat, 21 May 2022 18:45:50 +0200, Krzysztof Kozlowski wrote:
> Add qcom,msm8998 SoC fallback to the board compatible.
>
>

Applied, thanks!

[12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible
https://git.kernel.org/krzk/linux/c/4fb4a39fdbc8bd0aa35de0970d26cbc6c5abf946

Best regards,
--
Krzysztof Kozlowski <[email protected]>

2022-07-03 04:45:52

by Bjorn Andersson

[permalink] [raw]
Subject: Re: (subset) [PATCH 12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible

On Sat, 21 May 2022 18:45:50 +0200, Krzysztof Kozlowski wrote:
> Add qcom,msm8998 SoC fallback to the board compatible.
>
>

Applied, thanks!

[12/12] arm64: dts: qcom: msm8998-mtp: correct board compatible
commit: 6dfe4e195271a59f396d414c5fae299eefa5d3da

Best regards,
--
Bjorn Andersson <[email protected]>