* [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI
@ 2023-11-26 12:53 Fabio Estevam
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-11-26 12:53 UTC (permalink / raw)
To: sakari.ailus
Cc: rfoss, robh+dt, krzysztof.kozlowski+dt, conor+dt, mchehab,
laurent.pinchart, devicetree, linux-media, Fabio Estevam
From: Fabio Estevam <festevam@denx.de>
Commit 18860529a599 ("media: dt-bindings: media: video-interfaces: Add
new bus-type") introduced a new bus-type for DPI video bus.
Introduce MEDIA_BUS_TYPE_DPI into video-interfaces.h to complete
the list of video interfaces.
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
include/dt-bindings/media/video-interfaces.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/dt-bindings/media/video-interfaces.h b/include/dt-bindings/media/video-interfaces.h
index 68ac4e05e37f..0dace694a18f 100644
--- a/include/dt-bindings/media/video-interfaces.h
+++ b/include/dt-bindings/media/video-interfaces.h
@@ -12,5 +12,6 @@
#define MEDIA_BUS_TYPE_CSI2_DPHY 4
#define MEDIA_BUS_TYPE_PARALLEL 5
#define MEDIA_BUS_TYPE_BT656 6
+#define MEDIA_BUS_TYPE_DPI 7
#endif /* __DT_BINDINGS_MEDIA_VIDEO_INTERFACES_H__ */
--
2.34.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h
2023-11-26 12:53 [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Fabio Estevam
@ 2023-11-26 12:53 ` Fabio Estevam
2023-11-27 22:40 ` Rob Herring
2023-11-28 0:00 ` Laurent Pinchart
2023-11-27 22:39 ` [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Rob Herring
2023-11-27 23:58 ` Laurent Pinchart
2 siblings, 2 replies; 8+ messages in thread
From: Fabio Estevam @ 2023-11-26 12:53 UTC (permalink / raw)
To: sakari.ailus
Cc: rfoss, robh+dt, krzysztof.kozlowski+dt, conor+dt, mchehab,
laurent.pinchart, devicetree, linux-media, Fabio Estevam
From: Fabio Estevam <festevam@denx.de>
To improve readability, replace hardcoding numerical constants with the
corresponding definitions from video-interfaces.h.
Based on commit c4cfd47eb4ac ("media: dt-bindings: Use new video interface
bus type macros in examples").
Signed-off-by: Fabio Estevam <festevam@denx.de>
---
.../devicetree/bindings/display/bridge/analogix,anx7625.yaml | 3 ++-
.../devicetree/bindings/media/i2c/toshiba,tc358746.yaml | 3 ++-
Documentation/devicetree/bindings/media/microchip,csi2dc.yaml | 4 +++-
.../devicetree/bindings/media/starfive,jh7110-camss.yaml | 4 +++-
.../devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml | 3 ++-
5 files changed, 12 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
index a1ed1004651b..e18cc0e8a520 100644
--- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
@@ -132,6 +132,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/media/video-interfaces.h>
i2c {
#address-cells = <1>;
@@ -157,7 +158,7 @@ examples:
reg = <0>;
anx7625_in: endpoint {
remote-endpoint = <&mipi_dsi>;
- bus-type = <7>;
+ bus-type = <MEDIA_BUS_TYPE_DPI>;
data-lanes = <0 1 2 3>;
};
};
diff --git a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
index 1c476b635b69..9e1874946017 100644
--- a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
@@ -129,6 +129,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
+ #include <dt-bindings/media/video-interfaces.h>
i2c {
#address-cells = <1>;
@@ -161,7 +162,7 @@ examples:
remote-endpoint = <&sensor_out>;
hsync-active = <0>;
vsync-active = <0>;
- bus-type = <5>;
+ bus-type = <MEDIA_BUS_TYPE_PARALLEL>;
};
};
diff --git a/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml b/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
index b3a345fc6464..2e694479d82d 100644
--- a/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
+++ b/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
@@ -142,6 +142,8 @@ examples:
# Example for connecting to a parallel sensor controller block (video pipe)
# and the input is received from Synopsys IDI interface
- |
+ #include <dt-bindings/media/video-interfaces.h>
+
csi2dc@e1404000 {
compatible = "microchip,sama7g5-csi2dc";
reg = <0xe1404000 0x500>;
@@ -154,7 +156,7 @@ examples:
port@0 {
reg = <0>; /* must be 0, first child port */
csi2dc_in: endpoint { /* input from IDI interface */
- bus-type = <4>; /* MIPI CSI2 D-PHY */
+ bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>;
remote-endpoint = <&csi2host_out>;
};
};
diff --git a/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
index c66586d90fa2..4964e59c3ae3 100644
--- a/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
+++ b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
@@ -120,6 +120,8 @@ additionalProperties: false
examples:
- |
+ #include <dt-bindings/media/video-interfaces.h>
+
isp@19840000 {
compatible = "starfive,jh7110-camss";
reg = <0x19840000 0x10000>,
@@ -161,7 +163,7 @@ examples:
reg = <0>;
vin_from_sc2235: endpoint {
remote-endpoint = <&sc2235_to_vin>;
- bus-type = <5>;
+ bus-type = <MEDIA_BUS_TYPE_PARALLEL>;
bus-width = <8>;
data-shift = <2>;
hsync-active = <1>;
diff --git a/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml b/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
index f762fdc05e4d..5fdb9ae758e3 100644
--- a/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
+++ b/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
@@ -58,6 +58,7 @@ additionalProperties: false
examples:
- |
+ #include <dt-bindings/media/video-interfaces.h>
#include <dt-bindings/soc/ti,sci_pm_domain.h>
ti_csi2rx0: ticsi2rx@4500000 {
@@ -90,7 +91,7 @@ examples:
csi2rx0_in_sensor: endpoint {
remote-endpoint = <&csi2_cam0>;
- bus-type = <4>; /* CSI2 DPHY. */
+ bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>;
clock-lanes = <0>;
data-lanes = <1 2>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI
2023-11-26 12:53 [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Fabio Estevam
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
@ 2023-11-27 22:39 ` Rob Herring
2023-11-27 23:58 ` Laurent Pinchart
2 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2023-11-27 22:39 UTC (permalink / raw)
To: Fabio Estevam
Cc: conor+dt, mchehab, Fabio Estevam, devicetree, linux-media,
krzysztof.kozlowski+dt, sakari.ailus, laurent.pinchart, robh+dt,
rfoss
On Sun, 26 Nov 2023 09:53:19 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> Commit 18860529a599 ("media: dt-bindings: media: video-interfaces: Add
> new bus-type") introduced a new bus-type for DPI video bus.
>
> Introduce MEDIA_BUS_TYPE_DPI into video-interfaces.h to complete
> the list of video interfaces.
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> include/dt-bindings/media/video-interfaces.h | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
@ 2023-11-27 22:40 ` Rob Herring
2023-11-28 0:00 ` Laurent Pinchart
1 sibling, 0 replies; 8+ messages in thread
From: Rob Herring @ 2023-11-27 22:40 UTC (permalink / raw)
To: Fabio Estevam
Cc: conor+dt, sakari.ailus, Fabio Estevam, devicetree,
laurent.pinchart, krzysztof.kozlowski+dt, robh+dt, rfoss, mchehab,
linux-media
On Sun, 26 Nov 2023 09:53:20 -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> To improve readability, replace hardcoding numerical constants with the
> corresponding definitions from video-interfaces.h.
>
> Based on commit c4cfd47eb4ac ("media: dt-bindings: Use new video interface
> bus type macros in examples").
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> .../devicetree/bindings/display/bridge/analogix,anx7625.yaml | 3 ++-
> .../devicetree/bindings/media/i2c/toshiba,tc358746.yaml | 3 ++-
> Documentation/devicetree/bindings/media/microchip,csi2dc.yaml | 4 +++-
> .../devicetree/bindings/media/starfive,jh7110-camss.yaml | 4 +++-
> .../devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml | 3 ++-
> 5 files changed, 12 insertions(+), 5 deletions(-)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI
2023-11-26 12:53 [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Fabio Estevam
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
2023-11-27 22:39 ` [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Rob Herring
@ 2023-11-27 23:58 ` Laurent Pinchart
2 siblings, 0 replies; 8+ messages in thread
From: Laurent Pinchart @ 2023-11-27 23:58 UTC (permalink / raw)
To: Fabio Estevam
Cc: sakari.ailus, rfoss, robh+dt, krzysztof.kozlowski+dt, conor+dt,
mchehab, devicetree, linux-media, Fabio Estevam
Hi Fabio,
Thank you for the patch.
On Sun, Nov 26, 2023 at 09:53:19AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> Commit 18860529a599 ("media: dt-bindings: media: video-interfaces: Add
> new bus-type") introduced a new bus-type for DPI video bus.
>
> Introduce MEDIA_BUS_TYPE_DPI into video-interfaces.h to complete
> the list of video interfaces.
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> include/dt-bindings/media/video-interfaces.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/include/dt-bindings/media/video-interfaces.h b/include/dt-bindings/media/video-interfaces.h
> index 68ac4e05e37f..0dace694a18f 100644
> --- a/include/dt-bindings/media/video-interfaces.h
> +++ b/include/dt-bindings/media/video-interfaces.h
> @@ -12,5 +12,6 @@
> #define MEDIA_BUS_TYPE_CSI2_DPHY 4
> #define MEDIA_BUS_TYPE_PARALLEL 5
> #define MEDIA_BUS_TYPE_BT656 6
> +#define MEDIA_BUS_TYPE_DPI 7
>
> #endif /* __DT_BINDINGS_MEDIA_VIDEO_INTERFACES_H__ */
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
2023-11-27 22:40 ` Rob Herring
@ 2023-11-28 0:00 ` Laurent Pinchart
2023-11-28 0:31 ` Fabio Estevam
1 sibling, 1 reply; 8+ messages in thread
From: Laurent Pinchart @ 2023-11-28 0:00 UTC (permalink / raw)
To: Fabio Estevam
Cc: sakari.ailus, rfoss, robh+dt, krzysztof.kozlowski+dt, conor+dt,
mchehab, devicetree, linux-media, Fabio Estevam
Hi Fabio,
Thank you for the patch.
On Sun, Nov 26, 2023 at 09:53:20AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
>
> To improve readability, replace hardcoding numerical constants with the
> corresponding definitions from video-interfaces.h.
>
> Based on commit c4cfd47eb4ac ("media: dt-bindings: Use new video interface
> bus type macros in examples").
>
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---
> .../devicetree/bindings/display/bridge/analogix,anx7625.yaml | 3 ++-
> .../devicetree/bindings/media/i2c/toshiba,tc358746.yaml | 3 ++-
> Documentation/devicetree/bindings/media/microchip,csi2dc.yaml | 4 +++-
> .../devicetree/bindings/media/starfive,jh7110-camss.yaml | 4 +++-
> .../devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml | 3 ++-
> 5 files changed, 12 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> index a1ed1004651b..e18cc0e8a520 100644
> --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> @@ -132,6 +132,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/media/video-interfaces.h>
>
> i2c {
> #address-cells = <1>;
> @@ -157,7 +158,7 @@ examples:
> reg = <0>;
> anx7625_in: endpoint {
> remote-endpoint = <&mipi_dsi>;
> - bus-type = <7>;
7 seems to have been a mistake, as this is a DSI endpoint, not a DPI
endpoint.
> + bus-type = <MEDIA_BUS_TYPE_DPI>;
> data-lanes = <0 1 2 3>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> index 1c476b635b69..9e1874946017 100644
> --- a/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/toshiba,tc358746.yaml
> @@ -129,6 +129,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> + #include <dt-bindings/media/video-interfaces.h>
>
> i2c {
> #address-cells = <1>;
> @@ -161,7 +162,7 @@ examples:
> remote-endpoint = <&sensor_out>;
> hsync-active = <0>;
> vsync-active = <0>;
> - bus-type = <5>;
> + bus-type = <MEDIA_BUS_TYPE_PARALLEL>;
> };
> };
>
> diff --git a/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml b/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
> index b3a345fc6464..2e694479d82d 100644
> --- a/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
> +++ b/Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
> @@ -142,6 +142,8 @@ examples:
> # Example for connecting to a parallel sensor controller block (video pipe)
> # and the input is received from Synopsys IDI interface
> - |
> + #include <dt-bindings/media/video-interfaces.h>
> +
> csi2dc@e1404000 {
> compatible = "microchip,sama7g5-csi2dc";
> reg = <0xe1404000 0x500>;
> @@ -154,7 +156,7 @@ examples:
> port@0 {
> reg = <0>; /* must be 0, first child port */
> csi2dc_in: endpoint { /* input from IDI interface */
> - bus-type = <4>; /* MIPI CSI2 D-PHY */
> + bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>;
> remote-endpoint = <&csi2host_out>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
> index c66586d90fa2..4964e59c3ae3 100644
> --- a/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
> +++ b/Documentation/devicetree/bindings/media/starfive,jh7110-camss.yaml
> @@ -120,6 +120,8 @@ additionalProperties: false
>
> examples:
> - |
> + #include <dt-bindings/media/video-interfaces.h>
> +
> isp@19840000 {
> compatible = "starfive,jh7110-camss";
> reg = <0x19840000 0x10000>,
> @@ -161,7 +163,7 @@ examples:
> reg = <0>;
> vin_from_sc2235: endpoint {
> remote-endpoint = <&sc2235_to_vin>;
> - bus-type = <5>;
> + bus-type = <MEDIA_BUS_TYPE_PARALLEL>;
> bus-width = <8>;
> data-shift = <2>;
> hsync-active = <1>;
> diff --git a/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml b/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
> index f762fdc05e4d..5fdb9ae758e3 100644
> --- a/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
> +++ b/Documentation/devicetree/bindings/media/ti,j721e-csi2rx-shim.yaml
> @@ -58,6 +58,7 @@ additionalProperties: false
>
> examples:
> - |
> + #include <dt-bindings/media/video-interfaces.h>
> #include <dt-bindings/soc/ti,sci_pm_domain.h>
>
> ti_csi2rx0: ticsi2rx@4500000 {
> @@ -90,7 +91,7 @@ examples:
>
> csi2rx0_in_sensor: endpoint {
> remote-endpoint = <&csi2_cam0>;
> - bus-type = <4>; /* CSI2 DPHY. */
> + bus-type = <MEDIA_BUS_TYPE_CSI2_DPHY>;
> clock-lanes = <0>;
> data-lanes = <1 2>;
> };
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h
2023-11-28 0:00 ` Laurent Pinchart
@ 2023-11-28 0:31 ` Fabio Estevam
2023-12-01 12:16 ` Xin Ji
0 siblings, 1 reply; 8+ messages in thread
From: Fabio Estevam @ 2023-11-28 0:31 UTC (permalink / raw)
To: Laurent Pinchart, Xin Ji
Cc: sakari.ailus, rfoss, robh+dt, krzysztof.kozlowski+dt, conor+dt,
mchehab, devicetree, linux-media, Fabio Estevam
Hi Laurent,
On Mon, Nov 27, 2023 at 9:00 PM Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> > @@ -157,7 +158,7 @@ examples:
> > reg = <0>;
> > anx7625_in: endpoint {
> > remote-endpoint = <&mipi_dsi>;
> > - bus-type = <7>;
>
> 7 seems to have been a mistake, as this is a DSI endpoint, not a DPI
> endpoint.
bus-type = <7> was added in commit 0a61ef9cc30d
("dt-bindings:drm/bridge:anx7625: add port@0 property").
Adding Xin Ji on CC for clarification.
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h
2023-11-28 0:31 ` Fabio Estevam
@ 2023-12-01 12:16 ` Xin Ji
0 siblings, 0 replies; 8+ messages in thread
From: Xin Ji @ 2023-12-01 12:16 UTC (permalink / raw)
To: Fabio Estevam, Laurent Pinchart
Cc: sakari.ailus@linux.intel.com, rfoss@kernel.org,
robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
conor+dt@kernel.org, mchehab@kernel.org,
devicetree@vger.kernel.org, linux-media@vger.kernel.org,
Fabio Estevam
Hi Fabio Estevam, yes, 7 is for DPI, please remove it.
Reviewed-by: Xin Ji <xji@analogixsemi.com>
> -----Original Message-----
> From: Fabio Estevam <festevam@gmail.com>
> Sent: Tuesday, November 28, 2023 8:31 AM
> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>; Xin Ji
> <xji@analogixsemi.com>
> Cc: sakari.ailus@linux.intel.com; rfoss@kernel.org; robh+dt@kernel.org;
> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org; mchehab@kernel.org;
> devicetree@vger.kernel.org; linux-media@vger.kernel.org; Fabio Estevam
> <festevam@denx.de>
> Subject: Re: [PATCH 2/2] media: dt-bindings: Use bus-types definitions from
> video-interfaces.h
>
> CAUTION: This email originated from outside of the organization. Please do not
> click links or open attachments unless you recognize the sender, and know the
> content is safe.
>
>
> Hi Laurent,
>
> On Mon, Nov 27, 2023 at 9:00 PM Laurent Pinchart
> <laurent.pinchart@ideasonboard.com> wrote:
>
> > > @@ -157,7 +158,7 @@ examples:
> > > reg = <0>;
> > > anx7625_in: endpoint {
> > > remote-endpoint = <&mipi_dsi>;
> > > - bus-type = <7>;
> >
> > 7 seems to have been a mistake, as this is a DSI endpoint, not a DPI
> > endpoint.
>
> bus-type = <7> was added in commit 0a61ef9cc30d
> ("dt-bindings:drm/bridge:anx7625: add port@0 property").
>
> Adding Xin Ji on CC for clarification.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-12-01 12:17 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-26 12:53 [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Fabio Estevam
2023-11-26 12:53 ` [PATCH 2/2] media: dt-bindings: Use bus-types definitions from video-interfaces.h Fabio Estevam
2023-11-27 22:40 ` Rob Herring
2023-11-28 0:00 ` Laurent Pinchart
2023-11-28 0:31 ` Fabio Estevam
2023-12-01 12:16 ` Xin Ji
2023-11-27 22:39 ` [PATCH 1/2] media: dt-bindings: media: Introduce MEDIA_BUS_TYPE_DPI Rob Herring
2023-11-27 23:58 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).