devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] dt-bindings: Drop 'db' suffix duplicating dtschema
@ 2025-06-27  8:01 Krzysztof Kozlowski
  2025-06-27  8:01 ` [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: " Krzysztof Kozlowski
  2025-06-27  8:01 ` [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: " Krzysztof Kozlowski
  0 siblings, 2 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-06-27  8:01 UTC (permalink / raw)
  To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Vincent Knecht, Vinod Koul, Kishon Vijay Abraham I,
	Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Li Jun
  Cc: linux-leds, devicetree, linux-kernel, linux-phy, imx,
	linux-arm-kernel, Krzysztof Kozlowski

RFC because this depends on dtschema changes and should be accepted
after new dtschema is released with this merged:
https://github.com/devicetree-org/dt-schema/pull/166

Best regards,
Krzysztof

---
Krzysztof Kozlowski (2):
      dt-bindings: leds: issi,is31fl319x: Drop 'db' suffix duplicating dtschema
      dt-bindings: phy: fsl,imx8mq-usb: Drop 'db' suffix duplicating dtschema

 Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml   | 1 -
 Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml | 1 -
 2 files changed, 2 deletions(-)
---
base-commit: ecb259c4f70dd5c83907809f45bf4dc6869961d7
change-id: 20250627-dt-bindings-db-38eaa005ccd0

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


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

* [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 [PATCH 0/2] dt-bindings: Drop 'db' suffix duplicating dtschema Krzysztof Kozlowski
@ 2025-06-27  8:01 ` Krzysztof Kozlowski
  2025-06-27  8:14   ` Krzysztof Kozlowski
  2025-06-27  9:31   ` Rob Herring (Arm)
  2025-06-27  8:01 ` [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: " Krzysztof Kozlowski
  1 sibling, 2 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-06-27  8:01 UTC (permalink / raw)
  To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Vincent Knecht, Vinod Koul, Kishon Vijay Abraham I,
	Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Li Jun
  Cc: linux-leds, devicetree, linux-kernel, linux-phy, imx,
	linux-arm-kernel, Krzysztof Kozlowski

A common property unit suffix '-db' was added to dtschema, thus
in-kernel bindings should not reference the type.

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

---

RFC because this depends on dtschema changes and should be accepted
after new dtschema is released with this merged:
https://github.com/devicetree-org/dt-schema/pull/166
---
 Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml b/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
index 3c0431c51159e549920b17cf2ddfd1b85ecde810..906735acfbaf94fa08244f771139df207dd6e4da 100644
--- a/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
+++ b/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
@@ -42,7 +42,6 @@ properties:
     description: GPIO attached to the SDB pin.
 
   audio-gain-db:
-    $ref: /schemas/types.yaml#/definitions/uint32
     default: 0
     description: Audio gain selection for external analog modulation input.
     enum: [0, 3, 6, 9, 12, 15, 18, 21]

-- 
2.43.0


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

* [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 [PATCH 0/2] dt-bindings: Drop 'db' suffix duplicating dtschema Krzysztof Kozlowski
  2025-06-27  8:01 ` [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: " Krzysztof Kozlowski
@ 2025-06-27  8:01 ` Krzysztof Kozlowski
  2025-06-27  8:14   ` Krzysztof Kozlowski
  2025-06-27  9:31   ` Rob Herring (Arm)
  1 sibling, 2 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-06-27  8:01 UTC (permalink / raw)
  To: Lee Jones, Pavel Machek, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Vincent Knecht, Vinod Koul, Kishon Vijay Abraham I,
	Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Li Jun
  Cc: linux-leds, devicetree, linux-kernel, linux-phy, imx,
	linux-arm-kernel, Krzysztof Kozlowski

A common property unit suffix '-db' was added to dtschema, thus
in-kernel bindings should not reference the type.

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

---

RFC because this depends on dtschema changes and should be accepted
after new dtschema is released with this merged:
https://github.com/devicetree-org/dt-schema/pull/166
---
 Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml b/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml
index 22dd91591a09428214afaa4c9c8e37aae9bd8aba..6a47e08e0e97b286538798190225ca2966a7ab34 100644
--- a/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml
@@ -76,7 +76,6 @@ properties:
     description:
       Adjust TX de-emphasis attenuation in dB at nominal
       3.5dB point as per USB specification
-    $ref: /schemas/types.yaml#/definitions/uint32
     minimum: 0
     maximum: 36
 

-- 
2.43.0


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

* Re: [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 ` [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: " Krzysztof Kozlowski
@ 2025-06-27  8:14   ` Krzysztof Kozlowski
  2025-06-27  9:31   ` Rob Herring (Arm)
  1 sibling, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-06-27  8:14 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Lee Jones, Pavel Machek, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vincent Knecht, Vinod Koul,
	Kishon Vijay Abraham I, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Li Jun
  Cc: linux-leds, devicetree, linux-kernel, linux-phy, imx,
	linux-arm-kernel

On 27/06/2025 10:01, Krzysztof Kozlowski wrote:
> A common property unit suffix '-db' was added to dtschema, thus
> in-kernel bindings should not reference the type.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> RFC because this depends on dtschema changes and should be accepted
> after new dtschema is released with this merged:
> https://github.com/devicetree-org/dt-schema/pull/166
... and I forgot `b4 prep --add-prefixes RFC`. Please treat as RFC.

Best regards,
Krzysztof

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

* Re: [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 ` [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: " Krzysztof Kozlowski
@ 2025-06-27  8:14   ` Krzysztof Kozlowski
  2025-06-27  9:31   ` Rob Herring (Arm)
  1 sibling, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-06-27  8:14 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Lee Jones, Pavel Machek, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vincent Knecht, Vinod Koul,
	Kishon Vijay Abraham I, Shawn Guo, Sascha Hauer,
	Pengutronix Kernel Team, Fabio Estevam, Li Jun
  Cc: linux-leds, devicetree, linux-kernel, linux-phy, imx,
	linux-arm-kernel

On 27/06/2025 10:01, Krzysztof Kozlowski wrote:
> A common property unit suffix '-db' was added to dtschema, thus
> in-kernel bindings should not reference the type.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> RFC because this depends on dtschema changes and should be accepted
> after new dtschema is released with this merged:
> https://github.com/devicetree-org/dt-schema/pull/166
> ---
... and I forgot `b4 prep --add-prefixes RFC`. Please treat as RFC.

Best regards,
Krzysztof

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

* Re: [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 ` [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: " Krzysztof Kozlowski
  2025-06-27  8:14   ` Krzysztof Kozlowski
@ 2025-06-27  9:31   ` Rob Herring (Arm)
  1 sibling, 0 replies; 7+ messages in thread
From: Rob Herring (Arm) @ 2025-06-27  9:31 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-leds, Vinod Koul, Sascha Hauer, linux-kernel,
	Kishon Vijay Abraham I, linux-phy, Vincent Knecht, Conor Dooley,
	devicetree, Shawn Guo, Li Jun, imx, Lee Jones,
	Pengutronix Kernel Team, linux-arm-kernel, Pavel Machek,
	Fabio Estevam, Krzysztof Kozlowski


On Fri, 27 Jun 2025 10:01:51 +0200, Krzysztof Kozlowski wrote:
> A common property unit suffix '-db' was added to dtschema, thus
> in-kernel bindings should not reference the type.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> RFC because this depends on dtschema changes and should be accepted
> after new dtschema is released with this merged:
> https://github.com/devicetree-org/dt-schema/pull/166
> ---
>  Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml: audio-gain-db: missing type definition

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250627-dt-bindings-db-v1-1-d5c7072acbee@linaro.org

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.


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

* Re: [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: Drop 'db' suffix duplicating dtschema
  2025-06-27  8:01 ` [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: " Krzysztof Kozlowski
  2025-06-27  8:14   ` Krzysztof Kozlowski
@ 2025-06-27  9:31   ` Rob Herring (Arm)
  1 sibling, 0 replies; 7+ messages in thread
From: Rob Herring (Arm) @ 2025-06-27  9:31 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: linux-leds, Krzysztof Kozlowski, Vinod Koul,
	Kishon Vijay Abraham I, Vincent Knecht, Lee Jones, Shawn Guo,
	linux-kernel, Sascha Hauer, Pengutronix Kernel Team,
	linux-arm-kernel, linux-phy, imx, devicetree, Conor Dooley,
	Pavel Machek, Li Jun, Fabio Estevam


On Fri, 27 Jun 2025 10:01:52 +0200, Krzysztof Kozlowski wrote:
> A common property unit suffix '-db' was added to dtschema, thus
> in-kernel bindings should not reference the type.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> ---
> 
> RFC because this depends on dtschema changes and should be accepted
> after new dtschema is released with this merged:
> https://github.com/devicetree-org/dt-schema/pull/166
> ---
>  Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml | 1 -
>  1 file changed, 1 deletion(-)
> 

My bot found errors running 'make dt_binding_check' on your patch:

yamllint warnings/errors:

dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/phy/fsl,imx8mq-usb-phy.yaml: fsl,phy-pcs-tx-deemph-3p5db-attenuation-db: missing type definition

doc reference errors (make refcheckdocs):

See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20250627-dt-bindings-db-v1-2-d5c7072acbee@linaro.org

The base for the series is generally the latest rc1. A different dependency
should be noted in *this* patch.

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:

pip3 install dtschema --upgrade

Please check and re-submit after running the above command yourself. Note
that DT_SCHEMA_FILES can be set to your schema file to speed up checking
your schema. However, it must be unset to test all examples with your schema.


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

end of thread, other threads:[~2025-06-27  9:31 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-27  8:01 [PATCH 0/2] dt-bindings: Drop 'db' suffix duplicating dtschema Krzysztof Kozlowski
2025-06-27  8:01 ` [PATCH 1/2] dt-bindings: leds: issi,is31fl319x: " Krzysztof Kozlowski
2025-06-27  8:14   ` Krzysztof Kozlowski
2025-06-27  9:31   ` Rob Herring (Arm)
2025-06-27  8:01 ` [PATCH 2/2] dt-bindings: phy: fsl,imx8mq-usb: " Krzysztof Kozlowski
2025-06-27  8:14   ` Krzysztof Kozlowski
2025-06-27  9:31   ` Rob Herring (Arm)

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).