arm64: dts: imx8mq-mnt-reform2: drop invalid simple-panel compatible

Message ID 20230326204502.80794-1-krzysztof.kozlowski@linaro.org
State New
Headers
Series arm64: dts: imx8mq-mnt-reform2: drop invalid simple-panel compatible |

Commit Message

Krzysztof Kozlowski March 26, 2023, 8:45 p.m. UTC
  "simple-panel" compatible is not documented and nothing in Linux kernel
binds to it:

  imx8mq-mnt-reform2.dtb: panel: compatible: ['innolux,n125hce-gn1', 'simple-panel'] is too long

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Krzysztof Kozlowski May 13, 2023, 9:22 a.m. UTC | #1
On 26/03/2023 22:45, Krzysztof Kozlowski wrote:
> "simple-panel" compatible is not documented and nothing in Linux kernel
> binds to it:
> 
>   imx8mq-mnt-reform2.dtb: panel: compatible: ['innolux,n125hce-gn1', 'simple-panel'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  arch/arm64/boot/dts/freescale/imx8mq-mnt-reform2.dts | 2 +-


Shawn,

Can you pick up the patch?

Best regards,
Krzysztof
  
Shawn Guo May 14, 2023, 1:41 a.m. UTC | #2
On Sun, Mar 26, 2023 at 10:45:02PM +0200, Krzysztof Kozlowski wrote:
> "simple-panel" compatible is not documented and nothing in Linux kernel
> binds to it:
> 
>   imx8mq-mnt-reform2.dtb: panel: compatible: ['innolux,n125hce-gn1', 'simple-panel'] is too long
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Applied, thanks!
  

Patch

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;