devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: Turn on undocumented compatible checks
@ 2023-11-28 22:10 Rob Herring
  2023-11-29  8:29 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 2+ messages in thread
From: Rob Herring @ 2023-11-28 22:10 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Conor Dooley; +Cc: devicetree, linux-kernel

The undocumented compatibles in the examples are down to just a few
left. Turn on the warning by default. The increased visibility should
get the remaining warnings fixed.

Signed-off-by: Rob Herring <robh@kernel.org>
---
After the last batch of fixes I've sent out, there's 6 warnings 
remaining:

 ['socionext,milbeaut-usio-uart']
 ['sprd,sc9863a-glbregs', 'syscon', 'simple-mfd']
 ['brcm,bcm2711-avs-monitor', 'syscon', 'simple-mfd']
 ['fsl,imx6q-anatop', 'syscon', 'simple-mfd']
 ['hisilicon,hi3798cv200-combphy']
 ['qca,ar9331-switch']

 Documentation/devicetree/bindings/Makefile | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/Makefile b/Documentation/devicetree/bindings/Makefile
index 3e886194b043..3e3e49d8a733 100644
--- a/Documentation/devicetree/bindings/Makefile
+++ b/Documentation/devicetree/bindings/Makefile
@@ -61,9 +61,6 @@ override DTC_FLAGS := \
 	-Wno-unique_unit_address \
 	-Wunique_unit_address_if_enabled
 
-# Disable undocumented compatible checks until warning free
-override DT_CHECKER_FLAGS ?=
-
 $(obj)/processed-schema.json: $(DT_DOCS) $(src)/.yamllint check_dtschema_version FORCE
 	$(call if_changed_rule,chkdt)
 
-- 
2.42.0


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

* Re: [PATCH] dt-bindings: Turn on undocumented compatible checks
  2023-11-28 22:10 [PATCH] dt-bindings: Turn on undocumented compatible checks Rob Herring
@ 2023-11-29  8:29 ` Krzysztof Kozlowski
  0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2023-11-29  8:29 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley; +Cc: devicetree, linux-kernel

On 28/11/2023 23:10, Rob Herring wrote:
> The undocumented compatibles in the examples are down to just a few
> left. Turn on the warning by default. The increased visibility should
> get the remaining warnings fixed.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> After the last batch of fixes I've sent out, there's 6 warnings 
> remaining:
> 
>  ['socionext,milbeaut-usio-uart']
>  ['sprd,sc9863a-glbregs', 'syscon', 'simple-mfd']
>  ['brcm,bcm2711-avs-monitor', 'syscon', 'simple-mfd']
>  ['fsl,imx6q-anatop', 'syscon', 'simple-mfd']
>  ['hisilicon,hi3798cv200-combphy']
>  ['qca,ar9331-switch']
> 

Yay!

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof


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

end of thread, other threads:[~2023-11-29  8:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-28 22:10 [PATCH] dt-bindings: Turn on undocumented compatible checks Rob Herring
2023-11-29  8:29 ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).