mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] Tighten compatible matching for himax,hx83112{a,b}
@ 2026-10-02 14:49 Luca Weiss
  2026-10-02 14:49 ` [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching Luca Weiss
  2026-10-02 14:49 ` [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: " Luca Weiss
  0 siblings, 2 replies; 8+ messages in thread
From: Luca Weiss @ 2026-10-02 14:49 UTC (permalink / raw)
  To: Neil Armstrong, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel, Luca Weiss

As pointed out in the nt37705 series, fix the match for compatible in
the himax,hx83112a & himax,hx83112b schemas.

Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
Luca Weiss (2):
      dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching
      dt-bindings: display: panel: himax,hx83112b: Tighten compatible matching

 Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 3 +--
 Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)
---
base-commit: 72d3fcf802c45d00b300f25b848a93c3a2bd7c7e
change-id: 20261002-hx83112-compatible-contains-91f8ce8ec3cd

Best regards,
--  
Luca Weiss <luca.weiss@fairphone.com>


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching
  2026-10-02 14:49 [PATCH 0/2] Tighten compatible matching for himax,hx83112{a,b} Luca Weiss
@ 2026-10-02 14:49 ` Luca Weiss
  2026-10-05  9:27   ` Neil Armstrong
  2026-10-02 14:49 ` [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: " Luca Weiss
  1 sibling, 1 reply; 8+ messages in thread
From: Luca Weiss @ 2026-10-02 14:49 UTC (permalink / raw)
  To: Neil Armstrong, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel, Luca Weiss

The compatible was defined too wide, we just want "djn,9a-3r063-1102b"
and nothing else.

Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
 Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
index 2c60d0cd704e..3dd9b25ff13f 100644
--- a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
+++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
@@ -18,8 +18,7 @@ allOf:
 
 properties:
   compatible:
-    contains:
-      const: djn,9a-3r063-1102b
+    const: djn,9a-3r063-1102b
 
   reg:
     maxItems: 1

-- 
2.56.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: Tighten compatible matching
  2026-10-02 14:49 [PATCH 0/2] Tighten compatible matching for himax,hx83112{a,b} Luca Weiss
  2026-10-02 14:49 ` [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching Luca Weiss
@ 2026-10-02 14:49 ` Luca Weiss
  2026-10-05  9:36   ` Luca Weiss
  1 sibling, 1 reply; 8+ messages in thread
From: Luca Weiss @ 2026-10-02 14:49 UTC (permalink / raw)
  To: Neil Armstrong, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel, Luca Weiss

The compatible was defined too wide, we just want "djn,98-03057-6598b-i"
and nothing else.

Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
---
 Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
index e58bb3d45331..34e97003d293 100644
--- a/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
+++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
@@ -18,8 +18,7 @@ allOf:
 
 properties:
   compatible:
-    contains:
-      const: djn,98-03057-6598b-i
+    const: djn,98-03057-6598b-i
 
   reg:
     maxItems: 1

-- 
2.56.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching
  2026-10-02 14:49 ` [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching Luca Weiss
@ 2026-10-05  9:27   ` Neil Armstrong
  2026-10-05  9:35     ` Luca Weiss
  2026-10-05 10:54     ` Conor Dooley
  0 siblings, 2 replies; 8+ messages in thread
From: Neil Armstrong @ 2026-10-05  9:27 UTC (permalink / raw)
  To: Luca Weiss, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel

Hi,

On 10/2/26 16:49, Luca Weiss wrote:
> The compatible was defined too wide, we just want "djn,9a-3r063-1102b"
> and nothing else.

Can you add Fixes tags ?

Thanks,
Neil

> 
> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
>   Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
> index 2c60d0cd704e..3dd9b25ff13f 100644
> --- a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
> @@ -18,8 +18,7 @@ allOf:
>   
>   properties:
>     compatible:
> -    contains:
> -      const: djn,9a-3r063-1102b
> +    const: djn,9a-3r063-1102b
>   
>     reg:
>       maxItems: 1
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching
  2026-10-05  9:27   ` Neil Armstrong
@ 2026-10-05  9:35     ` Luca Weiss
  2026-10-05 10:54     ` Conor Dooley
  1 sibling, 0 replies; 8+ messages in thread
From: Luca Weiss @ 2026-10-05  9:35 UTC (permalink / raw)
  To: Neil Armstrong, Luca Weiss, Jessica Zhang, Maarten Lankhorst,
	Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel

On Mon Oct 5, 2026 at 11:27 AM CEST, Neil Armstrong wrote:
> Hi,
>
> On 10/2/26 16:49, Luca Weiss wrote:
>> The compatible was defined too wide, we just want "djn,9a-3r063-1102b"
>> and nothing else.
>
> Can you add Fixes tags ?

Sure, I can do that. Wasn't sure if that's a thing on dt-bindings
patches, but it seems to be used regularly.

Fixes: 3b59787a5170 ("dt-bindings: display: panel: Add Himax HX83112A")

Regards
Luca

>
> Thanks,
> Neil
>
>> 
>> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
>> ---
>>   Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml | 3 +--
>>   1 file changed, 1 insertion(+), 2 deletions(-)
>> 
>> diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
>> index 2c60d0cd704e..3dd9b25ff13f 100644
>> --- a/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
>> +++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112a.yaml
>> @@ -18,8 +18,7 @@ allOf:
>>   
>>   properties:
>>     compatible:
>> -    contains:
>> -      const: djn,9a-3r063-1102b
>> +    const: djn,9a-3r063-1102b
>>   
>>     reg:
>>       maxItems: 1
>> 



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: Tighten compatible matching
  2026-10-02 14:49 ` [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: " Luca Weiss
@ 2026-10-05  9:36   ` Luca Weiss
  2026-10-05 10:55     ` Conor Dooley
  0 siblings, 1 reply; 8+ messages in thread
From: Luca Weiss @ 2026-10-05  9:36 UTC (permalink / raw)
  To: Luca Weiss, Neil Armstrong, Jessica Zhang, Maarten Lankhorst,
	Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Luca Weiss
  Cc: ~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel

On Fri Oct 2, 2026 at 4:49 PM CEST, Luca Weiss wrote:
> The compatible was defined too wide, we just want "djn,98-03057-6598b-i"
> and nothing else.

Fixes: 54bd1390e984 ("dt-bindings: display: panel: Add Himax HX83112B")

> Signed-off-by: Luca Weiss <luca.weiss@fairphone.com>
> ---
>  Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml b/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
> index e58bb3d45331..34e97003d293 100644
> --- a/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/himax,hx83112b.yaml
> @@ -18,8 +18,7 @@ allOf:
>  
>  properties:
>    compatible:
> -    contains:
> -      const: djn,98-03057-6598b-i
> +    const: djn,98-03057-6598b-i
>  
>    reg:
>      maxItems: 1



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching
  2026-10-05  9:27   ` Neil Armstrong
  2026-10-05  9:35     ` Luca Weiss
@ 2026-10-05 10:54     ` Conor Dooley
  1 sibling, 0 replies; 8+ messages in thread
From: Conor Dooley @ 2026-10-05 10:54 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: Luca Weiss, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss,
	~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 75 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: Tighten compatible matching
  2026-10-05  9:36   ` Luca Weiss
@ 2026-10-05 10:55     ` Conor Dooley
  0 siblings, 0 replies; 8+ messages in thread
From: Conor Dooley @ 2026-10-05 10:55 UTC (permalink / raw)
  To: Luca Weiss
  Cc: Neil Armstrong, Jessica Zhang, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Luca Weiss,
	~postmarketos/upstreaming, phone-devel, dri-devel, devicetree,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 75 bytes --]

Acked-by: Conor Dooley <conor.dooley@microchip.com>
pw-bot: not-applicable

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2026-10-05 10:55 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-02 14:49 [PATCH 0/2] Tighten compatible matching for himax,hx83112{a,b} Luca Weiss
2026-10-02 14:49 ` [PATCH 1/2] dt-bindings: display: panel: himax,hx83112a: Tighten compatible matching Luca Weiss
2026-10-05  9:27   ` Neil Armstrong
2026-10-05  9:35     ` Luca Weiss
2026-10-05 10:54     ` Conor Dooley
2026-10-02 14:49 ` [PATCH 2/2] dt-bindings: display: panel: himax,hx83112b: " Luca Weiss
2026-10-05  9:36   ` Luca Weiss
2026-10-05 10:55     ` Conor Dooley

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®