2023-05-12 20:58:57

by Dario Binacchi

[permalink] [raw]
Subject: [PATCH] arm64: dts: imx8mq-mnt-reform2: drop simple-panel compatible

The "simple-panel" compatible is not documented and nothing in Linux
kernel binds to it.

Signed-off-by: Dario Binacchi <[email protected]>
---

arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
index 200268660518..3ae3824be027 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts
@@ -26,7 +26,7 @@ backlight: backlight {
};

panel {
- compatible = "innolux,n125hce-gn1", "simple-panel";
+ compatible = "innolux,n125hce-gn1";
power-supply = <&reg_main_3v3>;
backlight = <&backlight>;
no-hpd;
--
2.32.0



2023-05-13 09:31:43

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mq-mnt-reform2: drop simple-panel compatible

On 13/05/2023 11:22, Krzysztof Kozlowski wrote:
> On 12/05/2023 22:46, Dario Binacchi wrote:
>> The "simple-panel" compatible is not documented and nothing in Linux
>> kernel binds to it.
>>
>> Signed-off-by: Dario Binacchi <[email protected]>
>> ---
>>
>> arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 2 +-
>
> I already sent it:
>
> https://lore.kernel.org/all/[email protected]/

And makes me wonder, why did you use exactly the same words as my patch...

Best regards,
Krzysztof


2023-05-13 09:31:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: imx8mq-mnt-reform2: drop simple-panel compatible

On 12/05/2023 22:46, Dario Binacchi wrote:
> The "simple-panel" compatible is not documented and nothing in Linux
> kernel binds to it.
>
> Signed-off-by: Dario Binacchi <[email protected]>
> ---
>
> arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 2 +-

I already sent it:

https://lore.kernel.org/all/[email protected]/


Best regards,
Krzysztof