| Message ID | 20230509190031.769298-1-niklas.soderlund+renesas@ragnatech.se (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] dt-bindings: i2c: maxim,max96712: Require setting bus-type property | expand |
On Tue, May 09, 2023 at 09:00:31PM +0200, Niklas Söderlund wrote: > The MAX96712 can support both a CSI-2 C-PHY and D-PHY bus. The initial > staging driver however only supported D-PHY and the bus-type property > was left optional. > > In preparation for adding C-PHY support to the staging driver make the > bus-type property mandatory as it is needed to select the correct PHY > mode. Without the bus-type property present, the driver falls-back to > D-PHY mode, so the change is functionally backward compatible with old > DTS files lacking the property. > > The only in-tree DTS file (renesas/r8a779a0-falcon-csi-dsi.dtsi) that > lacked the property uses D-PHY and have been updated. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> Reviewed-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor. > --- > * Changes since v1 > - Use symbolic names in comments for the bus-type properties. > - Improve the commit message. > - Rebased to v6.4-rc1. > --- > .../devicetree/bindings/media/i2c/maxim,max96712.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml > index 444f24838d3d..6c72e77b927c 100644 > --- a/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml > +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml > @@ -65,9 +65,14 @@ properties: > > properties: > data-lanes: true > + bus-type: > + enum: > + - 1 # MEDIA_BUS_TYPE_CSI2_CPHY > + - 4 # MEDIA_BUS_TYPE_CSI2_DPHY > > required: > - data-lanes > + - bus-type > > required: > - port@4 > @@ -82,6 +87,7 @@ additionalProperties: false > examples: > - | > #include <dt-bindings/gpio/gpio.h> > + #include <dt-bindings/media/video-interfaces.h> > > i2c@e6508000 { > #address-cells = <1>; > @@ -101,6 +107,7 @@ examples: > port@4 { > reg = <4>; > max96712_out0: endpoint { > + bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>; > clock-lanes = <0>; > data-lanes = <1 2 3 4>; > remote-endpoint = <&csi40_in>; > -- > 2.40.1 >
On 09/05/2023 21:00, Niklas Söderlund wrote: > The MAX96712 can support both a CSI-2 C-PHY and D-PHY bus. The initial > staging driver however only supported D-PHY and the bus-type property > was left optional. > > In preparation for adding C-PHY support to the staging driver make the > bus-type property mandatory as it is needed to select the correct PHY > mode. Without the bus-type property present, the driver falls-back to > D-PHY mode, so the change is functionally backward compatible with old > DTS files lacking the property. > > The only in-tree DTS file (renesas/r8a779a0-falcon-csi-dsi.dtsi) that > lacked the property uses D-PHY and have been updated. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > * Changes since v1 > - Use symbolic names in comments for the bus-type properties. > - Improve the commit message. > - Rebased to v6.4-rc1. > --- > .../devicetree/bindings/media/i2c/maxim,max96712.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Tue, May 9, 2023 at 9:03 PM Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> wrote: > The MAX96712 can support both a CSI-2 C-PHY and D-PHY bus. The initial > staging driver however only supported D-PHY and the bus-type property > was left optional. > > In preparation for adding C-PHY support to the staging driver make the > bus-type property mandatory as it is needed to select the correct PHY > mode. Without the bus-type property present, the driver falls-back to > D-PHY mode, so the change is functionally backward compatible with old > DTS files lacking the property. > > The only in-tree DTS file (renesas/r8a779a0-falcon-csi-dsi.dtsi) that > lacked the property uses D-PHY and have been updated. has > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > * Changes since v1 > - Use symbolic names in comments for the bus-type properties. > - Improve the commit message. > - Rebased to v6.4-rc1. Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Gr{oetje,eeting}s, Geert
diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml index 444f24838d3d..6c72e77b927c 100644 --- a/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max96712.yaml @@ -65,9 +65,14 @@ properties: properties: data-lanes: true + bus-type: + enum: + - 1 # MEDIA_BUS_TYPE_CSI2_CPHY + - 4 # MEDIA_BUS_TYPE_CSI2_DPHY required: - data-lanes + - bus-type required: - port@4 @@ -82,6 +87,7 @@ additionalProperties: false examples: - | #include <dt-bindings/gpio/gpio.h> + #include <dt-bindings/media/video-interfaces.h> i2c@e6508000 { #address-cells = <1>; @@ -101,6 +107,7 @@ examples: port@4 { reg = <4>; max96712_out0: endpoint { + bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>; clock-lanes = <0>; data-lanes = <1 2 3 4>; remote-endpoint = <&csi40_in>;
The MAX96712 can support both a CSI-2 C-PHY and D-PHY bus. The initial staging driver however only supported D-PHY and the bus-type property was left optional. In preparation for adding C-PHY support to the staging driver make the bus-type property mandatory as it is needed to select the correct PHY mode. Without the bus-type property present, the driver falls-back to D-PHY mode, so the change is functionally backward compatible with old DTS files lacking the property. The only in-tree DTS file (renesas/r8a779a0-falcon-csi-dsi.dtsi) that lacked the property uses D-PHY and have been updated. Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> --- * Changes since v1 - Use symbolic names in comments for the bus-type properties. - Improve the commit message. - Rebased to v6.4-rc1. --- .../devicetree/bindings/media/i2c/maxim,max96712.yaml | 7 +++++++ 1 file changed, 7 insertions(+)