Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 5 Feb 2021 19:38:06 +0800
From:   Xin Ji <xji@analogixsemi.com>
To:     Robert Foss <robert.foss@linaro.org>
Cc:     Rob Herring <robh+dt@kernel.org>, David Airlie <airlied@linux.ie>,
        Nicolas Boichat <drinkcat@google.com>,
        Hsin-Yi Wang <hsinyi@chromium.org>,
        Daniel Vetter <daniel@ffwll.ch>,
        Sam Ravnborg <sam@ravnborg.org>,
        Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
        Maxime Ripard <mripard@kernel.org>,
        Mark Brown <broonie@kernel.org>,
        Ricardo =?iso-8859-1?Q?Ca=F1uelo?= 
        <ricardo.canuelo@collabora.com>, dri-devel@lists.freedesktop.org,
        devicetree@vger.kernel.org, Bernie Liang <bliang@analogixsemi.com>,
        Sheng Pan <span@analogixsemi.com>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 1/3] dt-bindings:drm/bridge:anx7625:add vendor define
 flags
Message-ID: <20210205113805.GB18836@zhaomy-pc>
References: <cover.1611802321.git.xji@analogixsemi.com>
 <246b8bd6e51ed5c8cb3618f4259adf8aba319511.1611802321.git.xji@analogixsemi.com>
 <CAG3jFyu0qj_ipqz8eKWtmkjKDBBAF_X20f1i-d=83vksgh6nxg@mail.gmail.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAG3jFyu0qj_ipqz8eKWtmkjKDBBAF_X20f1i-d=83vksgh6nxg@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from zhaomy-pc (61.148.116.10) by HKAPR03CA0017.apcprd03.prod.outlook.com (2603:1096:203:c8::22) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA) id 15.20.3846.11 via Frontend Transport; Fri, 5 Feb 2021 11:38:17 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 4eb6efa4-e560-42f4-2e5a-08d8c9ca880b
X-MS-TrafficTypeDiagnostic: BYAPR04MB3927:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <BYAPR04MB3927DE5E849F9F987F948609C7B29@BYAPR04MB3927.namprd04.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:6790;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: D9enVoPgIs0VSY8vm8z6cXrRhzZDMFQqkOuS6hs83AyIDRYg/Bl41Pivj5gy2sC0rt1RR6yzXZur8vwY0DcGKxNl8bILtgAUMcY9imSI4xlgf6uR5lnffKtfzcbhVVykTLPdisGQlVqTj0Zy4VSyIwHtGewkLMMoqMVE0vTO+uL0modHIXqrxUJ3J0rH5pdU4TrguvzYLcz6wD+Ypnc9w56TtFtn+OCL1hpTKXJqAuhmlhf198YVU6IAzHRnjzzLqNkjDO8klYFnYwwzBGkeII1a63HzBxGLIdv2XzwXatGYDwepu0+SvL01HtMDm9nO8UJ5wpn3YH2d/bGxQBOfnmNkqyRLOP0b9N0ZFefJetOlm+NA8QiCLXsyy2Sur+ZaYYlCCAGa2jZTUkFmeA1xrNhFsOvXkdHivv+qjSSlZ7462SRnO4d1546PgQGeJxg3MBGnQEWIwKY5BpbEeTojskKQWVpJZXI5opVFd6+Wt/SN1WMWJAYkvDrm+LpyXTA7VXizN3h7PKFriaQnGvEYlNrVhH/lwOM4V1nYB4UaP8bVZgnsKl0NO7SPxi+Br6nA
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BY5PR04MB6739.namprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(376002)(39840400004)(346002)(366004)(396003)(136003)(66476007)(52116002)(4326008)(6666004)(33656002)(54906003)(83380400001)(316002)(956004)(66556008)(186003)(66946007)(2906002)(5660300002)(26005)(9686003)(1076003)(8676002)(16526019)(33716001)(86362001)(478600001)(8936002)(6916009)(6496006)(7416002)(55016002)(16060500001);DIR:OUT;SFP:1102;
X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?CB2jGHgEfKyhB206Gp8/BjQ+yomA02D1IYXcD/gMJMN4y3v2daWhqu7EC5LR?=
 =?us-ascii?Q?9yKtofCy5uZFj+SRr9UG2y8ixDdJzLLEq/XI0wd82x2N6FBiw3IvOblJ+wld?=
 =?us-ascii?Q?IhJpwmNFPzmeCTxHqJ3wzhTWLdHH6zPoIM2GqYofg7n4bV4F2koSrfhO2PzT?=
 =?us-ascii?Q?7+Kl2o1IzwWnngf82e0o/ffVNEswx8Az8THN5toRI2acaCE1A4gsumeP5p70?=
 =?us-ascii?Q?fqq7NDi6HkzM26IRI6SMlZhQX3Pu0PytvQr6mBZA1CIDvdvc6srqRoFUTaJc?=
 =?us-ascii?Q?h2T0nunXCRgPJKeGw1PyD28+77cr4O2eOaf20kg9+cvHPP8FW1hvWWOHyZAf?=
 =?us-ascii?Q?R1wh7gqsthwnlvDNTPof8PvnvegzvxzufXpQYYNoE7giHMztHjfqlh+8Ybon?=
 =?us-ascii?Q?fOT0w6u6HW7TQbhD6Lu0ElSMJEgklXxo/EuGZeWNfKR4lhbmq21joN3r1Tow?=
 =?us-ascii?Q?aT3o9PXz2HYdxOkNWRAMaY1y822p/bhaPdPi19fkFskAvACCfGSLP6+IkSLu?=
 =?us-ascii?Q?5VObiwq88J4qCpVD9vLHBn8dOhm7ydVah1ktqR7c/PNLgCqVUAPbv5EeFyPY?=
 =?us-ascii?Q?VQx6aoPRJV7y2RlqHaA39kyAWP95YvA5VfB08u2Xu6UUSNOiGHMshdqkChly?=
 =?us-ascii?Q?9cwItG56HvliCALxjwwyAMxvvPqBUhIZ/sFwF4SZeWSsEJbXUfXIyoOLz6bd?=
 =?us-ascii?Q?1FbdKCCHgZfASgpS6cA/abGxRSe+XgRH3bYDLMHfJiZwHq/wDxRtjzjuD25A?=
 =?us-ascii?Q?wkOgPlNoATSF9nlAxyVWhDrbmccxAy9RbiedayS8xqcN+M+5/WmkIbEnVO5+?=
 =?us-ascii?Q?ifN0botdu93LKZatQzhuZ3Dd84AfAXxP4x7rs96ne17LMLxgC2LYm9HMjr4B?=
 =?us-ascii?Q?WSnBOd5gpiwBd1PWtdThTsTFKyEGy8wdKun1DE95hf8zsUeNbCWVP/vOPBkP?=
 =?us-ascii?Q?9kV8Ws+QrvF/PX6+SjWdWIRvNilMMHxW4jIPPvhmEYg1q6bHF/pg1pqLmMfE?=
 =?us-ascii?Q?xofWw2Dq2UWJ7GnJgOG+Fsajl+ECR+vxkyXu+1sJ7PvVLrSMEESa2PWGM5at?=
 =?us-ascii?Q?T039CekS/2dWhTIMd7XFcWOMyId1FiGku0G/1u781Y3JYkJ1moQesOnxPatB?=
 =?us-ascii?Q?dKWczOwNwsaMyaney8ZRZW/+aRM+F/REtZhzm0xEXKIFoF/XmI8CavW1W9mi?=
 =?us-ascii?Q?sZiGF528Z0+nn3jx5v5SZHzGkFRONdCtcCwvv0jhn4tLUSaiI5mMKpZkZ3qN?=
 =?us-ascii?Q?Aq9thGdqJay05aF8VZPRzCMLvrW7xjVrZMx/VggQO5us1XN2Vp8MRHQ4rl00?=
 =?us-ascii?Q?JLcmyxwCEADVmseRd+xcOr2h?=
X-OriginatorOrg: analogixsemi.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4eb6efa4-e560-42f4-2e5a-08d8c9ca880b
X-MS-Exchange-CrossTenant-AuthSource: BY5PR04MB6739.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Feb 2021 11:38:17.8325
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: b099b0b4-f26c-4cf5-9a0f-d5be9acab205
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: grpvmzO/S9hANqhEq4H9Ut3BxcyBftU3Z/zkWZqqhj1dfKRIB19WhoM4lb7no/UoqebGIj/Z1Vfu3wZ5pwlm+Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR04MB3927
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Feb 04, 2021 at 01:38:36PM +0100, Robert Foss wrote:
> Hey Xin,
> 
> On Thu, 28 Jan 2021 at 04:10, Xin Ji <xji@analogixsemi.com> wrote:
> >
> > Add 'bus-type' and 'data-lanes' define for port0, add HDCP support
> > flag and DP tx lane0 and lane1 swing register array define.
> >
> > Signed-off-by: Xin Ji <xji@analogixsemi.com>
> > ---
> >  .../bindings/display/bridge/analogix,anx7625.yaml  | 54 +++++++++++++++++++++-
> >  1 file changed, 53 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > index c789784..048deec 100644
> > --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > @@ -34,6 +34,24 @@ properties:
> >      description: used for reset chip control, RESET_N pin B7.
> >      maxItems: 1
> >
> > +  analogix,lane0-swing:
> > +    $ref: /schemas/types.yaml#/definitions/uint32-array
> > +    maxItems: 20
> > +    description:
> > +      an array of swing register setting for DP tx lane0 PHY, please don't
> > +      add this property, or contact vendor.
> > +
> > +  analogix,lane1-swing:
> > +    $ref: /schemas/types.yaml#/definitions/uint32-array
> > +    maxItems: 20
> > +    description:
> > +      an array of swing register setting for DP tx lane1 PHY, please don't
> > +      add this property, or contact vendor.
> > +
> > +  analogix,hdcp-support:
> > +    type: boolean
> > +    description: indicate the DP tx HDCP support or not.
> > +
> >    ports:
> >      $ref: /schemas/graph.yaml#/properties/ports
> >
> > @@ -41,13 +59,45 @@ properties:
> >        port@0:
> >          $ref: /schemas/graph.yaml#/properties/port
> >          description:
> > -          Video port for MIPI DSI input.
> > +          Video port for MIPI input.
> 
> Maybe MIPI DSI/DPI input is more correct, since there are a lot of
> other MIPI standards.
Hi Rob, OK, I'll change it in the next series.
Thanks,
Xin
> 
> > +
> > +        properties:
> > +          endpoint:
> > +            type: object
> > +            additionalProperties: false
> > +
> > +            # Properties described in
> > +            # Documentation/devicetree/bindings/media/video-interfaces.txt
> > +            properties:
> > +              remote-endpoint: true
> > +              bus-type: true
> > +              data-lanes: true
> > +
> > +            required:
> > +              - remote-endpoint
> > +
> > +        required:
> > +          - endpoint
> > +
> >
> >        port@1:
> >          $ref: /schemas/graph.yaml#/properties/port
> >          description:
> >            Video port for panel or connector.
> >
> > +        properties:
> > +          endpoint:
> > +            type: object
> > +            additionalProperties: false
> > +
> > +            # Properties described in
> > +            # Documentation/devicetree/bindings/media/video-interfaces.txt
> > +            properties:
> > +              remote-endpoint: true
> > +
> > +            required:
> > +              - remote-endpoint
> > +
> >      required:
> >        - port@0
> >        - port@1
> > @@ -81,6 +131,8 @@ examples:
> >                      reg = <0>;
> >                      anx7625_in: endpoint {
> >                          remote-endpoint = <&mipi_dsi>;
> > +                        bus-type = <5>;
> > +                        data-lanes = <0 1 2 3>;
> >                      };
> >                  };
> >
> > --
> > 2.7.4
> >