* [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
@ 2023-04-12 19:47 Konrad Dybcio
2023-04-12 20:20 ` Marijn Suijten
0 siblings, 1 reply; 6+ messages in thread
From: Konrad Dybcio @ 2023-04-12 19:47 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu
Cc: Marijn Suijten, linux-arm-msm, devicetree, linux-kernel, Konrad Dybcio
The fallback compatible was missing, resulting in dtbs_check failing.
Fix it.
Fixes: 51c4c2bd6f31 ("arm64: dts: qcom: sm8250-xiaomi-elish-boe: Add mdss and dsi panel")
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
---
foo b4 requires i put something here
---
arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts | 2 +-
arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
index 8b2ae39950ff..de6101ddebe7 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
+++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
@@ -13,6 +13,6 @@ / {
};
&display_panel {
- compatible = "xiaomi,elish-boe-nt36523";
+ compatible = "xiaomi,elish-boe-nt36523", "novatek,nt36523";
status = "okay";
};
diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
index a4d5341495cf..4cffe9c703df 100644
--- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
+++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
@@ -13,6 +13,6 @@ / {
};
&display_panel {
- compatible = "xiaomi,elish-csot-nt36523";
+ compatible = "xiaomi,elish-csot-nt36523", "novatek,nt36523";
status = "okay";
};
---
base-commit: 7d8214bba44c1aa6a75921a09a691945d26a8d43
change-id: 20230412-topic-elish_compat-73c4511957a5
Best regards,
--
Konrad Dybcio <konrad.dybcio@linaro.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
2023-04-12 19:47 [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles Konrad Dybcio
@ 2023-04-12 20:20 ` Marijn Suijten
2023-04-12 20:22 ` Konrad Dybcio
0 siblings, 1 reply; 6+ messages in thread
From: Marijn Suijten @ 2023-04-12 20:20 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu, linux-arm-msm, devicetree, linux-kernel
On 2023-04-12 21:47:20, Konrad Dybcio wrote:
> The fallback compatible was missing, resulting in dtbs_check failing.
> Fix it.
>
> Fixes: 51c4c2bd6f31 ("arm64: dts: qcom: sm8250-xiaomi-elish-boe: Add mdss and dsi panel")
> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
As usual: Krzysztof already sent these:
https://lore.kernel.org/linux-arm-msm/20230408130809.52319-2-krzysztof.kozlowski@linaro.org/T/#u
> ---
> foo b4 requires i put something here
What?
> ---
> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts | 2 +-
> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
> index 8b2ae39950ff..de6101ddebe7 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
> @@ -13,6 +13,6 @@ / {
> };
>
> &display_panel {
> - compatible = "xiaomi,elish-boe-nt36523";
> + compatible = "xiaomi,elish-boe-nt36523", "novatek,nt36523";
> status = "okay";
> };
> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
> index a4d5341495cf..4cffe9c703df 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
> @@ -13,6 +13,6 @@ / {
> };
>
> &display_panel {
> - compatible = "xiaomi,elish-csot-nt36523";
> + compatible = "xiaomi,elish-csot-nt36523", "novatek,nt36523";
> status = "okay";
> };
>
> ---
> base-commit: 7d8214bba44c1aa6a75921a09a691945d26a8d43
> change-id: 20230412-topic-elish_compat-73c4511957a5
>
> Best regards,
> --
> Konrad Dybcio <konrad.dybcio@linaro.org>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
2023-04-12 20:20 ` Marijn Suijten
@ 2023-04-12 20:22 ` Konrad Dybcio
2023-04-12 20:23 ` Konrad Dybcio
0 siblings, 1 reply; 6+ messages in thread
From: Konrad Dybcio @ 2023-04-12 20:22 UTC (permalink / raw)
To: Marijn Suijten
Cc: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu, linux-arm-msm, devicetree, linux-kernel
On 12.04.2023 22:20, Marijn Suijten wrote:
> On 2023-04-12 21:47:20, Konrad Dybcio wrote:
>> The fallback compatible was missing, resulting in dtbs_check failing.
>> Fix it.
>>
>> Fixes: 51c4c2bd6f31 ("arm64: dts: qcom: sm8250-xiaomi-elish-boe: Add mdss and dsi panel")
>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>
> As usual: Krzysztof already sent these:
>
> https://lore.kernel.org/linux-arm-msm/20230408130809.52319-2-krzysztof.kozlowski@linaro.org/T/#u
Right.
>
>> ---
>> foo b4 requires i put something here
>
> What?
git checkout linux-next/master
<craft a single patch>
b4 prep -e $(cat localversion-next| cut -c 2-)
b4 prep --edit-cover
/* you get a bunch of EDITMEs that b4 really insists you fill out */
Konrad
>
>> ---
>> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts | 2 +-
>> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>> index 8b2ae39950ff..de6101ddebe7 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>> @@ -13,6 +13,6 @@ / {
>> };
>>
>> &display_panel {
>> - compatible = "xiaomi,elish-boe-nt36523";
>> + compatible = "xiaomi,elish-boe-nt36523", "novatek,nt36523";
>> status = "okay";
>> };
>> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>> index a4d5341495cf..4cffe9c703df 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>> @@ -13,6 +13,6 @@ / {
>> };
>>
>> &display_panel {
>> - compatible = "xiaomi,elish-csot-nt36523";
>> + compatible = "xiaomi,elish-csot-nt36523", "novatek,nt36523";
>> status = "okay";
>> };
>>
>> ---
>> base-commit: 7d8214bba44c1aa6a75921a09a691945d26a8d43
>> change-id: 20230412-topic-elish_compat-73c4511957a5
>>
>> Best regards,
>> --
>> Konrad Dybcio <konrad.dybcio@linaro.org>
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
2023-04-12 20:22 ` Konrad Dybcio
@ 2023-04-12 20:23 ` Konrad Dybcio
2023-04-12 20:28 ` Marijn Suijten
0 siblings, 1 reply; 6+ messages in thread
From: Konrad Dybcio @ 2023-04-12 20:23 UTC (permalink / raw)
To: Marijn Suijten
Cc: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu, linux-arm-msm, devicetree, linux-kernel
On 12.04.2023 22:22, Konrad Dybcio wrote:
>
>
> On 12.04.2023 22:20, Marijn Suijten wrote:
>> On 2023-04-12 21:47:20, Konrad Dybcio wrote:
>>> The fallback compatible was missing, resulting in dtbs_check failing.
>>> Fix it.
>>>
>>> Fixes: 51c4c2bd6f31 ("arm64: dts: qcom: sm8250-xiaomi-elish-boe: Add mdss and dsi panel")
>>> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
>>
>> As usual: Krzysztof already sent these:
>>
>> https://lore.kernel.org/linux-arm-msm/20230408130809.52319-2-krzysztof.kozlowski@linaro.org/T/#u
> Right.
>
>>
>>> ---
>>> foo b4 requires i put something here
>>
>> What?
> git checkout linux-next/master
> <craft a single patch>
git checkout -b branchname
> b4 prep -e $(cat localversion-next| cut -c 2-)
> b4 prep --edit-cover
> /* you get a bunch of EDITMEs that b4 really insists you fill out */
>
Konrad
> Konrad
>>
>>> ---
>>> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts | 2 +-
>>> arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts | 2 +-
>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>>> index 8b2ae39950ff..de6101ddebe7 100644
>>> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>>> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-boe.dts
>>> @@ -13,6 +13,6 @@ / {
>>> };
>>>
>>> &display_panel {
>>> - compatible = "xiaomi,elish-boe-nt36523";
>>> + compatible = "xiaomi,elish-boe-nt36523", "novatek,nt36523";
>>> status = "okay";
>>> };
>>> diff --git a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>>> index a4d5341495cf..4cffe9c703df 100644
>>> --- a/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>>> +++ b/arch/arm64/boot/dts/qcom/sm8250-xiaomi-elish-csot.dts
>>> @@ -13,6 +13,6 @@ / {
>>> };
>>>
>>> &display_panel {
>>> - compatible = "xiaomi,elish-csot-nt36523";
>>> + compatible = "xiaomi,elish-csot-nt36523", "novatek,nt36523";
>>> status = "okay";
>>> };
>>>
>>> ---
>>> base-commit: 7d8214bba44c1aa6a75921a09a691945d26a8d43
>>> change-id: 20230412-topic-elish_compat-73c4511957a5
>>>
>>> Best regards,
>>> --
>>> Konrad Dybcio <konrad.dybcio@linaro.org>
>>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
2023-04-12 20:23 ` Konrad Dybcio
@ 2023-04-12 20:28 ` Marijn Suijten
2023-04-12 20:29 ` Konrad Dybcio
0 siblings, 1 reply; 6+ messages in thread
From: Marijn Suijten @ 2023-04-12 20:28 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu, linux-arm-msm, devicetree, linux-kernel
On 2023-04-12 22:23:25, Konrad Dybcio wrote:
[..]
> >>> foo b4 requires i put something here
> >>
> >> What?
> > git checkout linux-next/master
> > <craft a single patch>
> git checkout -b branchname
> > b4 prep -e $(cat localversion-next| cut -c 2-)
I think that could just be:
b4 prep -n branchname
(in favour of prep -e _and_ checkout -b)
Might also need to pass `-f $(cut -c 2- localversion-next)` to get the
forkpoint right though.
> > b4 prep --edit-cover
> > /* you get a bunch of EDITMEs that b4 really insists you fill out */
And what if you remove the line entirely?
- Marijn
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles
2023-04-12 20:28 ` Marijn Suijten
@ 2023-04-12 20:29 ` Konrad Dybcio
0 siblings, 0 replies; 6+ messages in thread
From: Konrad Dybcio @ 2023-04-12 20:29 UTC (permalink / raw)
To: Marijn Suijten
Cc: Andy Gross, Bjorn Andersson, Rob Herring, Krzysztof Kozlowski,
Jianhua Lu, linux-arm-msm, devicetree, linux-kernel
On 12.04.2023 22:28, Marijn Suijten wrote:
> On 2023-04-12 22:23:25, Konrad Dybcio wrote:
> [..]
>>>>> foo b4 requires i put something here
>>>>
>>>> What?
>>> git checkout linux-next/master
>>> <craft a single patch>
>> git checkout -b branchname
>>> b4 prep -e $(cat localversion-next| cut -c 2-)
>
> I think that could just be:
>
> b4 prep -n branchname
>
> (in favour of prep -e _and_ checkout -b)
Maybe. I have reverse-i-search + muscle memory for the more bloated
approach though..
>
> Might also need to pass `-f $(cut -c 2- localversion-next)` to get the
> forkpoint right though.
>
>>> b4 prep --edit-cover
>>> /* you get a bunch of EDITMEs that b4 really insists you fill out */
>
> And what if you remove the line entirely?
I just tried that after replying to you, on the Edo venus patch..
Apparently works as well..
Konrad
>
> - Marijn
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-04-12 20:29 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-12 19:47 [PATCH] arm64: dts: qcom: sm8250-elish-*: Fix panel compatibles Konrad Dybcio
2023-04-12 20:20 ` Marijn Suijten
2023-04-12 20:22 ` Konrad Dybcio
2023-04-12 20:23 ` Konrad Dybcio
2023-04-12 20:28 ` Marijn Suijten
2023-04-12 20:29 ` Konrad Dybcio
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®