[PATCH v3 4/4] dt-bindings: trivial-devices: Add sht2x sensors

Kurt Borja posted 4 patches 1 day, 19 hours ago
There is a newer version of this series
[PATCH v3 4/4] dt-bindings: trivial-devices: Add sht2x sensors
Posted by Kurt Borja 1 day, 19 hours ago
Add sensirion,sht2x trivial sensors.

Cc: stable@vger.kernel.org
Signed-off-by: Kurt Borja <kuurtb@gmail.com>
---
 Documentation/devicetree/bindings/trivial-devices.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
index f3dd18681aa6f81255141bdda6daf8e45369a2c2..952244a7105591a0095b1ae57da7cb7345bdfc61 100644
--- a/Documentation/devicetree/bindings/trivial-devices.yaml
+++ b/Documentation/devicetree/bindings/trivial-devices.yaml
@@ -362,6 +362,9 @@ properties:
             # Sensirion low power multi-pixel gas sensor with I2C interface
           - sensirion,sgpc3
             # Sensirion temperature & humidity sensor with I2C interface
+          - sensirion,sht20
+          - sensirion,sht21
+          - sensirion,sht25
           - sensirion,sht4x
             # Sensortek 3 axis accelerometer
           - sensortek,stk8312

-- 
2.51.0
Re: [PATCH v3 4/4] dt-bindings: trivial-devices: Add sht2x sensors
Posted by Krzysztof Kozlowski 1 day, 14 hours ago
On 08/09/2025 03:33, Kurt Borja wrote:
> Add sensirion,sht2x trivial sensors.
> 
> Cc: stable@vger.kernel.org

No, drop. No bug to fix here.

Please organize the patch documenting compatible (DT bindings) before
their user.
See also:
https://elixir.bootlin.com/linux/v6.14-rc6/source/Documentation/devicetree/bindings/submitting-patches.rst#L46
> 


Best regards,
Krzysztof
Re: [PATCH v3 4/4] dt-bindings: trivial-devices: Add sht2x sensors
Posted by Kurt Borja 1 day, 10 hours ago
Hi Krzysztof,

On Mon Sep 8, 2025 at 2:02 AM -05, Krzysztof Kozlowski wrote:
> On 08/09/2025 03:33, Kurt Borja wrote:
>> Add sensirion,sht2x trivial sensors.
>> 
>> Cc: stable@vger.kernel.org
>
> No, drop. No bug to fix here.

I included it because stable is fine with device IDs [1]. Is this
avoided with dt-bindings related stuff?

>
> Please organize the patch documenting compatible (DT bindings) before
> their user.
> See also:
> https://elixir.bootlin.com/linux/v6.14-rc6/source/Documentation/devicetree/bindings/submitting-patches.rst#L46

I will reorder.

Thanks for the review!

>> 
>
>
> Best regards,
> Krzysztof

[1] https://elixir.bootlin.com/linux/v6.17-rc4/source/Documentation/process/stable-kernel-rules.rst#L15

-- 
 ~ Kurt
Re: [PATCH v3 4/4] dt-bindings: trivial-devices: Add sht2x sensors
Posted by Krzysztof Kozlowski 1 day, 7 hours ago
On 08/09/2025 13:28, Kurt Borja wrote:
> Hi Krzysztof,
> 
> On Mon Sep 8, 2025 at 2:02 AM -05, Krzysztof Kozlowski wrote:
>> On 08/09/2025 03:33, Kurt Borja wrote:
>>> Add sensirion,sht2x trivial sensors.
>>>
>>> Cc: stable@vger.kernel.org
>>
>> No, drop. No bug to fix here.
> 
> I included it because stable is fine with device IDs [1]. Is this
> avoided with dt-bindings related stuff?

You do not add quirks or new IDs. You add something completely new -
support for OF. That's a feature in this driver.

Best regards,
Krzysztof