* [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
@ 2026-02-27 16:58 Frank Li
2026-02-27 17:05 ` Fabio Estevam
2026-02-27 17:41 ` Andy Shevchenko
0 siblings, 2 replies; 7+ messages in thread
From: Frank Li @ 2026-02-27 16:58 UTC (permalink / raw)
To: Robin van der Gracht, Andy Shevchenko, Geert Uytterhoeven,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list
Cc: imx, Frank Li
Change additionalProperties to unevaluatedProperties because it refs to
/schemas/input/matrix-keymap.yaml.
Fix below CHECK_DTBS warnings:
arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Change in v2
- Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
- Add fixes tags
Resend include Rob's Ack tag:
Acked-by: Rob Herring (Arm) <robh@kernel.org>
Rob's Acked at
https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
---
.../devicetree/bindings/auxdisplay/holtek,ht16k33.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
index b90eec2077b4b..fe1272e86467e 100644
--- a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
+++ b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml
@@ -66,7 +66,7 @@ then:
required:
- refresh-rate-hz
-additionalProperties: false
+unevaluatedProperties: false
examples:
- |
--
2.43.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 16:58 [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning Frank Li
@ 2026-02-27 17:05 ` Fabio Estevam
2026-02-27 17:26 ` Conor Dooley
2026-02-27 17:41 ` Andy Shevchenko
1 sibling, 1 reply; 7+ messages in thread
From: Fabio Estevam @ 2026-02-27 17:05 UTC (permalink / raw)
To: Frank Li
Cc: Robin van der Gracht, Andy Shevchenko, Geert Uytterhoeven,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Fri, Feb 27, 2026 at 1:59 PM Frank Li <Frank.Li@nxp.com> wrote:
>
> Change additionalProperties to unevaluatedProperties because it refs to
> /schemas/input/matrix-keymap.yaml.
>
> Fix below CHECK_DTBS warnings:
> arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
>
> Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Change in v2
> - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> - Add fixes tags
>
> Resend include Rob's Ack tag:
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> Rob's Acked at
> https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
Rob's Ack is still missing.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 17:05 ` Fabio Estevam
@ 2026-02-27 17:26 ` Conor Dooley
2026-02-27 17:27 ` Conor Dooley
0 siblings, 1 reply; 7+ messages in thread
From: Conor Dooley @ 2026-02-27 17:26 UTC (permalink / raw)
To: Fabio Estevam
Cc: Frank Li, Robin van der Gracht, Andy Shevchenko,
Geert Uytterhoeven, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
[-- Attachment #1: Type: text/plain, Size: 1150 bytes --]
On Fri, Feb 27, 2026 at 02:05:23PM -0300, Fabio Estevam wrote:
> On Fri, Feb 27, 2026 at 1:59 PM Frank Li <Frank.Li@nxp.com> wrote:
> >
> > Change additionalProperties to unevaluatedProperties because it refs to
> > /schemas/input/matrix-keymap.yaml.
> >
> > Fix below CHECK_DTBS warnings:
> > arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> > from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
> >
> > Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> > Change in v2
> > - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > - Add fixes tags
> >
> > Resend include Rob's Ack tag:
> > Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > Rob's Acked at
> > https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
>
> Rob's Ack is still missing.
How do you get this wrong multiple times? Just use b4 to get your tags
Frank.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 17:26 ` Conor Dooley
@ 2026-02-27 17:27 ` Conor Dooley
0 siblings, 0 replies; 7+ messages in thread
From: Conor Dooley @ 2026-02-27 17:27 UTC (permalink / raw)
To: Fabio Estevam
Cc: Frank Li, Robin van der Gracht, Andy Shevchenko,
Geert Uytterhoeven, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
[-- Attachment #1: Type: text/plain, Size: 1397 bytes --]
On Fri, Feb 27, 2026 at 05:26:09PM +0000, Conor Dooley wrote:
> On Fri, Feb 27, 2026 at 02:05:23PM -0300, Fabio Estevam wrote:
> > On Fri, Feb 27, 2026 at 1:59 PM Frank Li <Frank.Li@nxp.com> wrote:
> > >
> > > Change additionalProperties to unevaluatedProperties because it refs to
> > > /schemas/input/matrix-keymap.yaml.
> > >
> > > Fix below CHECK_DTBS warnings:
> > > arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> > > from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
> > >
> > > Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> > > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > > ---
> > > Change in v2
> > > - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > > - Add fixes tags
> > >
> > > Resend include Rob's Ack tag:
> > > Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > > Rob's Acked at
> > > https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
> >
> > Rob's Ack is still missing.
>
> How do you get this wrong multiple times? Just use b4 to get your tags
> Frank.
Also, I complained about the subject not citing the specific binding
IIRC on the previous version and that wasn't fixed either.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 16:58 [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning Frank Li
2026-02-27 17:05 ` Fabio Estevam
@ 2026-02-27 17:41 ` Andy Shevchenko
2026-02-27 22:43 ` Frank Li
1 sibling, 1 reply; 7+ messages in thread
From: Andy Shevchenko @ 2026-02-27 17:41 UTC (permalink / raw)
To: Frank Li
Cc: Robin van der Gracht, Andy Shevchenko, Geert Uytterhoeven,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Fri, Feb 27, 2026 at 11:58:34AM -0500, Frank Li wrote:
> Change additionalProperties to unevaluatedProperties because it refs to
> /schemas/input/matrix-keymap.yaml.
>
> Fix below CHECK_DTBS warnings:
> arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
>
> Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Change in v2
> - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> - Add fixes tags
>
> Resend include Rob's Ack tag:
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
Independently on `b4` and its functionality this is incorrect way to put tags.
This has to be done in the same way as SoB above, for example.
I've just explained this already second time.
> Rob's Acked at
> https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
No need to resend _this_ this time.
Thanks!
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 17:41 ` Andy Shevchenko
@ 2026-02-27 22:43 ` Frank Li
2026-02-28 10:55 ` Andy Shevchenko
0 siblings, 1 reply; 7+ messages in thread
From: Frank Li @ 2026-02-27 22:43 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Robin van der Gracht, Andy Shevchenko, Geert Uytterhoeven,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Fri, Feb 27, 2026 at 07:41:58PM +0200, Andy Shevchenko wrote:
> On Fri, Feb 27, 2026 at 11:58:34AM -0500, Frank Li wrote:
> > Change additionalProperties to unevaluatedProperties because it refs to
> > /schemas/input/matrix-keymap.yaml.
> >
> > Fix below CHECK_DTBS warnings:
> > arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> > from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
> >
> > Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> > Change in v2
> > - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > - Add fixes tags
> >
> > Resend include Rob's Ack tag:
> > Acked-by: Rob Herring (Arm) <robh@kernel.org>
>
> Independently on `b4` and its functionality this is incorrect way to put tags.
> This has to be done in the same way as SoB above, for example.
> I've just explained this already second time.
Sorry mess up during debug something.
Frank
>
> > Rob's Acked at
> > https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
>
> No need to resend _this_ this time.
> Thanks!
>
> --
> With Best Regards,
> Andy Shevchenko
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning
2026-02-27 22:43 ` Frank Li
@ 2026-02-28 10:55 ` Andy Shevchenko
0 siblings, 0 replies; 7+ messages in thread
From: Andy Shevchenko @ 2026-02-28 10:55 UTC (permalink / raw)
To: Frank Li
Cc: Robin van der Gracht, Andy Shevchenko, Geert Uytterhoeven,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Miguel Ojeda,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
open list, imx
On Fri, Feb 27, 2026 at 05:43:31PM -0500, Frank Li wrote:
> On Fri, Feb 27, 2026 at 07:41:58PM +0200, Andy Shevchenko wrote:
> > On Fri, Feb 27, 2026 at 11:58:34AM -0500, Frank Li wrote:
> > > Change additionalProperties to unevaluatedProperties because it refs to
> > > /schemas/input/matrix-keymap.yaml.
> > >
> > > Fix below CHECK_DTBS warnings:
> > > arch/arm/boot/dts/nxp/imx/imx6dl-victgo.dtb: keypad@70 (holtek,ht16k33): 'keypad,num-columns', 'keypad,num-rows' do not match any of the regexes: '^pinctrl-[0-9]+$'
> > > from schema $id: http://devicetree.org/schemas/auxdisplay/holtek,ht16k33.yaml#
> > >
> > > Fixes: f12b457c6b25c ("dt-bindings: auxdisplay: ht16k33: Convert to json-schema")
> > > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > > ---
> > > Change in v2
> > > - Add Acked-by: Rob Herring (Arm) <robh@kernel.org>
> > > - Add fixes tags
> > >
> > > Resend include Rob's Ack tag:
> > > Acked-by: Rob Herring (Arm) <robh@kernel.org>
> >
> > Independently on `b4` and its functionality this is incorrect way to put tags.
> > This has to be done in the same way as SoB above, for example.
> > I've just explained this already second time.
>
> Sorry mess up during debug something.
> > > Rob's Acked at
> > > https://lore.kernel.org/imx/175796984570.3379405.7210704746413204244.robh@kernel.org/
> >
> > No need to resend _this_ this time.
I was under impression that the only thing is missing is Rob's Ack, but it
appears that something else should also be addressed. I think it warrants
the v3.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2026-02-28 10:55 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-27 16:58 [PATCH v2 1/1] dt-bindings: auxdisplay: Use unevaluatedProperties to fix common property warning Frank Li
2026-02-27 17:05 ` Fabio Estevam
2026-02-27 17:26 ` Conor Dooley
2026-02-27 17:27 ` Conor Dooley
2026-02-27 17:41 ` Andy Shevchenko
2026-02-27 22:43 ` Frank Li
2026-02-28 10:55 ` Andy Shevchenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox