Re: [PATCH v3 1/2] dt-bindings:bridge: Add Lontiun LT7911EXC eDP to MIPI DSI bridge

杨孙运
2026-04-30T08:55:35Z

杨孙运 <yangsunyun1993@gmail.com> 于2026年4月30日周四 16:43写道：
>
> Krzysztof Kozlowski <krzk@kernel.org> 于2026年4月30日周四 15:56写道：
> >
> > On Wed, Apr 29, 2026 at 12:05:40PM +0800, syyang@lontium.com wrote:
> > > From: Sunyun Yang <syyang@lontium.com>
> > >
> > > The LT7911EXC is an I2C-controlled bridge that Receiver eDP1.4
> > > and output signal/dual port mipi.
> > >
> > > Signed-off-by: Sunyun Yang <syyang@lontium.com>
> >
> > Missing space after ':' in subject.
> >

> it will be fixed in next version.

sorry, Where is the missing space? I didn't find it.

is:
Subject: [PATCH v3 1/2] dt-bindings:bridge: Add Lontiun LT7911EXC eDP
to MIPI DSI
?

> > ...
> >
> > > + ports:
> > > + $ref: /schemas/graph.yaml#/properties/ports
> > > +
> > > + properties:
> > > + port@0:
> > > + $ref: /schemas/graph.yaml#/properties/port
> > > + description: Video port for MIPI DSI output.
> >
> > Wrong description, first is input. Look at your DTS.
> >
> yes, you are right, I didn't notice this place.
>
> it will be fixed in next version.
> thanks Krzysztof Kozlowski
>
> > > +
> > > + port@1:
> > > + $ref: /schemas/graph.yaml#/properties/port
> > > + description: Video port for eDP input.
> > > +
> > > + required:
> > > + - port@0
> > > + - port@1
> > > +
> > > +required:
> > > + - compatible
> > > + - reg
> > > + - reset-gpios
> > > + - vdd-supply
> > > + - vcc-supply
> > > + - ports
> > > +
> > > +additionalProperties: false
> > > +
> > > +examples:
> > > + - |
> > > + #include <dt-bindings/gpio/gpio.h>
> > > + i2c {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + mipi-bridge@41 {
> > > + compatible = "lontium,lt7911exc";
> > > + reg = <0x41>;
> > > + reset-gpios = <&gpy8 8 GPIO_ACTIVE_LOW>;
> > > + vdd-supply = <&lt7911exc_1v2>;
> > > + vcc-supply = <&lt7911exc_3v3>;
> > > +
> > > + ports {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > +
> > > + port@0 {
> > > + reg = <0>;
> > > + bridge_in: endpoint {
> > > + remote-endpoint = <&edp_out>;
> > > + };
> > > + };
> >
> > Best regards,
> > Krzysztof
> >