[PATCH 3/4] dt-bindings: Add tids in bindings and wsen as vendor

Thomas Marangoni posted 4 patches 2 weeks ago
[PATCH 3/4] dt-bindings: Add tids in bindings and wsen as vendor
Posted by Thomas Marangoni 2 weeks ago
This patch adds the tids driver to the bindings as trivial-devices
and adds the WSEN manufacturer to the vendor-prefixes.
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
 Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index 2eff6f274302..e120767ce119 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -488,6 +488,8 @@ properties:
           - ti,tsc2003
             # Winbond/Nuvoton H/W Monitor
           - winbond,w83793
+            # Temperature sensor with i2c interface
+          - wsen,tids-2521020222501
 
 required:
   - compatible
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
index 42d2bc0ce027..2cf753fdf5a7 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1823,6 +1823,8 @@ patternProperties:
     description: Wobo
   "^wolfvision,.*":
     description: WolfVision GmbH
+  "^wsen,.*":
+    description: Würth Elektronik eiSos GmbH & Co. KG
   "^x-powers,.*":
     description: X-Powers
   "^xen,.*":
-- 
2.51.1

Re: [PATCH 3/4] dt-bindings: Add tids in bindings and wsen as vendor
Posted by Krzysztof Kozlowski 2 weeks ago
On 17/11/2025 13:38, Thomas Marangoni wrote:
> This patch adds the tids driver to the bindings as trivial-devices
> and adds the WSEN manufacturer to the vendor-prefixes.
> ---

You need to run checkpatch...

Please do not use "This commit/patch/change", but imperative mood. See
longer explanation here:
https://elixir.bootlin.com/linux/v6.16/source/Documentation/process/submitting-patches.rst#L94

Also, do not reference driver. Explain here hardware, why this is a
trivial device.

Best regards,
Krzysztof
Re: [PATCH 3/4] dt-bindings: Add tids in bindings and wsen as vendor
Posted by Guenter Roeck 2 weeks ago
On 11/17/25 04:38, Thomas Marangoni wrote:
> This patch adds the tids driver to the bindings as trivial-devices
> and adds the WSEN manufacturer to the vendor-prefixes.

This patch should be the first patch in the series.

> ---
>   Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
>   Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
>   2 files changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index 2eff6f274302..e120767ce119 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -488,6 +488,8 @@ properties:
>             - ti,tsc2003
>               # Winbond/Nuvoton H/W Monitor
>             - winbond,w83793
> +            # Temperature sensor with i2c interface
> +          - wsen,tids-2521020222501

I think this should just be "wsen,tids".

>   
>   required:
>     - compatible
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> index 42d2bc0ce027..2cf753fdf5a7 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
> @@ -1823,6 +1823,8 @@ patternProperties:
>       description: Wobo
>     "^wolfvision,.*":
>       description: WolfVision GmbH
> +  "^wsen,.*":
> +    description: Würth Elektronik eiSos GmbH & Co. KG
>     "^x-powers,.*":
>       description: X-Powers
>     "^xen,.*":