<div dir="ltr">Hi Rob<br><br>Rob Herring <<a href="mailto:robh%2Bdt@kernel.org">robh+dt@kernel.org</a>> 于2020年12月1日周二 上午4:29写道：<br>><br>> On Mon, Nov 30, 2020 at 7:28 AM Kevin Tang <<a href="mailto:kevin3.tang@gmail.com">kevin3.tang@gmail.com</a>> wrote:<br>> ><br>> > From: Kevin Tang <<a href="mailto:kevin.tang@unisoc.com">kevin.tang@unisoc.com</a>><br>><br>> Once again, DT patches must Cc the DT list if you want them reviewed.<div>Ok, i will add DT list to my Cc list.</div><div>><br>> ><br>> > The Unisoc DRM master device is a virtual device needed to list all<br>> > DPU devices or other display interface nodes that comprise the<br>> > graphics subsystem<br>> ><br>> > Cc: Orson Zhai <<a href="mailto:orsonzhai@gmail.com">orsonzhai@gmail.com</a>><br>> > Cc: Chunyan Zhang <<a href="mailto:zhang.lyra@gmail.com">zhang.lyra@gmail.com</a>><br>> > Signed-off-by: Kevin Tang <<a href="mailto:kevin.tang@unisoc.com">kevin.tang@unisoc.com</a>><br>> > ---<br>> >  .../display/sprd/sprd,display-subsystem.yaml       | 39 ++++++++++++++++++++++<br>> >  1 file changed, 39 insertions(+)<br>> >  create mode 100644 Documentation/devicetree/bindings/display/sprd/sprd,display-subsystem.yaml<br>> ><br>> > diff --git a/Documentation/devicetree/bindings/display/sprd/sprd,display-subsystem.yaml b/Documentation/devicetree/bindings/display/sprd/sprd,display-subsystem.yaml<br>> > new file mode 100644<br>> > index 0000000..9487a39<br>> > --- /dev/null<br>> > +++ b/Documentation/devicetree/bindings/display/sprd/sprd,display-subsystem.yaml<br>> > @@ -0,0 +1,39 @@<br>> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)<br>> > +%YAML 1.2<br>> > +---<br>> > +$id: <a href="http://devicetree.org/schemas/display/sprd/sprd,display-subsystem.yaml#">http://devicetree.org/schemas/display/sprd/sprd,display-subsystem.yaml#</a><br>> > +$schema: <a href="http://devicetree.org/meta-schemas/core.yaml#">http://devicetree.org/meta-schemas/core.yaml#</a><br>> > +<br>> > +title: Unisoc DRM master device<br>> > +<br>> > +maintainers:<br>> > +  - Kevin Tang <<a href="mailto:kevin.tang@unisoc.com">kevin.tang@unisoc.com</a>><br>> > +<br>> > +description: |<br>> > +  The Unisoc DRM master device is a virtual device needed to list all<br>> > +  DPU devices or other display interface nodes that comprise the<br>> > +  graphics subsystem.<br>> > +<br>> > +properties:<br>> > +  compatible:<br>> > +    const: sprd,display-subsystem<br>><br>> As I said before, we try to avoid these virtual nodes. Make the DRM<br>> driver bind to the DPU node.<br>><br>> The case where this might be needed is if you have h/w that's a<br>> mixture of shared and discrete blocks. I don't see anything here<br>> matching that.<br><br>This is how I understand this sentence: "h/w that's a mixture of shared and discrete blocks"<br>We have multiple display controllers and corresponding physical interfaces, typical dual-screen display scenario<div>E.g:</div><div>1. dpu0 and dpu1 both binding to DSI for dual mipi-dsi display;</div><div>2. dpu0 binding to DSI for primary display, and dpu1 binding to DP for external display;<br></div><div><br>It looks like this:<br>dpu0------>MIPI-DSI------>DPHY/CPHY Combo------>Panel0<br>                        ^<br>    ----------------|<br>    |<br>dpu1------>DisplayPort------>PHY------>Panel1<br><br>For DTS like this:<br>display-subsystem {<br>compatible = "sprd,display-subsystem";<br>ports = <&dpu0_out>, <&dpu1_out>;<br>};<br><br>But this is our first time submitting code to the community, following the suggestions of other reviewers, we only submitted a basic version. Other virtual nodes and corresponding drivers will be submitted later.<div><div>><br></div><div>> Rob</div></div></div></div></div>