Re: [PATCH v2 1/2] dt-bindings: imx335: Mention reset-gpio polarity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

On 29/07/24 4:40 pm, Laurent Pinchart wrote:
Hi Umang,

Thank you for the patch.

On Mon, Jul 29, 2024 at 04:34:36PM +0530, Umang Jain wrote:
Mention the reset-gpio polarity in the device tree bindings.
It is GPIO_ACTIVE_LOW according to the datasheet.

Signed-off-by: Umang Jain <umang.jain@xxxxxxxxxxxxxxxx>
---
  Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
index 106c36ee966d..fb4c9d42ed1c 100644
--- a/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/sony,imx335.yaml
@@ -92,6 +92,8 @@ examples:
              ovdd-supply = <&camera_vddo_1v8>;
              dvdd-supply = <&camera_vddd_1v2>;
+ reset-gpios = <&gpio 50 GPIO_ACTIVE_LOW>;
+
I think it's good to include this in the example, but it doesn't match
the commit message. I was expecting to see a change to the binding
rules, not to the example.

Currently the binding already states reset-gpio as

```
  reset-gpios:
    description: Reference to the GPIO connected to the XCLR pin, if any.
    maxItems: 1
```

Pardon my limited knowledge here, do you mean something like :

```
  reset-gpios:
    description: Reference to the GPIO connected to the XCLR pin (active LOW), if any.
    maxItems: 1
```

or something else?

              port {
                  imx335: endpoint {
                      remote-endpoint = <&cam>;





[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux