[PATCH v3 2/3] dt-bindings: iio: light: opt3001: add compatible for opt3002

Emil Gedenryd posted 3 patches 2 months, 2 weeks ago
There is a newer version of this series
[PATCH v3 2/3] dt-bindings: iio: light: opt3001: add compatible for opt3002
Posted by Emil Gedenryd 2 months, 2 weeks ago
The opt3002 is a Light-to-Digital Sensor by TI with support for wide-range
spectrum light. It shares most properties with their opt3001 model with
the exception of having a wide spectral bandwidth, ranging from 300 nm
to 1000 nm.

Add the compatible string of opt3002.

Signed-off-by: Emil Gedenryd <emil.gedenryd@axis.com>
---
 Documentation/devicetree/bindings/iio/light/ti,opt3001.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/light/ti,opt3001.yaml b/Documentation/devicetree/bindings/iio/light/ti,opt3001.yaml
index 441e9343fc97..67ca8d08256a 100644
--- a/Documentation/devicetree/bindings/iio/light/ti,opt3001.yaml
+++ b/Documentation/devicetree/bindings/iio/light/ti,opt3001.yaml
@@ -15,7 +15,9 @@ description: |
 
 properties:
   compatible:
-    const: ti,opt3001
+    enum:
+      - ti,opt3001
+      - ti,opt3002
 
   reg:
     maxItems: 1

-- 
2.39.2
Re: [PATCH v3 2/3] dt-bindings: iio: light: opt3001: add compatible for opt3002
Posted by Conor Dooley 2 months, 2 weeks ago
On Mon, Sep 16, 2024 at 04:56:38PM +0200, Emil Gedenryd wrote:
> The opt3002 is a Light-to-Digital Sensor by TI with support for wide-range
> spectrum light. It shares most properties with their opt3001 model with
> the exception of having a wide spectral bandwidth, ranging from 300 nm
> to 1000 nm.
> 
> Add the compatible string of opt3002.
> 
> Signed-off-by: Emil Gedenryd <emil.gedenryd@axis.com>

Pretty sure I gave you an ack for this already, provided you added the
a description of the differences, which you have.

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.
Re: [PATCH v3 2/3] dt-bindings: iio: light: opt3001: add compatible for opt3002
Posted by Emil Gedenryd 2 months, 1 week ago
On Mon, 2024-09-16 at 16:24 +0100, Conor Dooley wrote:
> On Mon, Sep 16, 2024 at 04:56:38PM +0200, Emil Gedenryd wrote:
> > The opt3002 is a Light-to-Digital Sensor by TI with support for wide-range
> > spectrum light. It shares most properties with their opt3001 model with
> > the exception of having a wide spectral bandwidth, ranging from 300 nm
> > to 1000 nm.
> > 
> > Add the compatible string of opt3002.
> > 
> > Signed-off-by: Emil Gedenryd <emil.gedenryd@axis.com>
> 
> Pretty sure I gave you an ack for this already, provided you added the
> a description of the differences, which you have.
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Cheers,
> Conor.

Thank you!

Regards,
Emil