Date:   Thu, 13 Jan 2022 11:32:26 +0800
From:   Xin Ji <xji@analogixsemi.com>
To:     Hsin-Yi Wang <hsinyi@chromium.org>,
        Robert Foss <robert.foss@linaro.org>,
        Rob Herring <robh+dt@kernel.org>
Cc:     David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, Andrzej Hajda <a.hajda@samsung.com>,
        Neil Armstrong <narmstrong@baylibre.com>,
        Jonas Karlman <jonas@kwiboo.se>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Sam Ravnborg <sam@ravnborg.org>,
        Maxime Ripard <maxime@cerno.tech>
Subject: Re: [PATCH 3/3] dt-bindings: drm/bridge: anx7625: Add aux-bus node
Message-ID: <20220113033226.GB2478030@anxtwsw-Precision-3640-Tower>
References: <20220111112701.1064458-1-hsinyi@chromium.org>
 <20220111112701.1064458-3-hsinyi@chromium.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220111112701.1064458-3-hsinyi@chromium.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Q1FzugVauG1mbfhlcBLL5tqeXqSympDfgqfb4RvC+hfjstGwoU0tVybwwLCF?=
 =?us-ascii?Q?+72Z2sg1bhSfaKVjTVO/vUx2ZDluLIE21pUl9mMhd5Qoj6DQPNEyMyJO9T8Q?=
 =?us-ascii?Q?L/ettVKYSNRTxRwCX4W3hh3rLYqYKvHh6tmgw/BTHW/M1igBiWwrzERP9/7G?=
 =?us-ascii?Q?/5OVa8FJsogpVa8AlyiM1OOgdBSozTIMe0+gJbNbet3u+YVT3Xz/ffY4X9TW?=
 =?us-ascii?Q?qRXERoBdcTnetusHpzTZugmI+wdL3K6hMjMbMdl6ypaHnzx7SBZhGh0bbXaM?=
 =?us-ascii?Q?ibb3CHQQvvTk05J2wnsl6Q8Uq/QpBGaGWuykOERyuY9sv1WB22IsTQrvgDys?=
 =?us-ascii?Q?iseYGupHMqD53cZ5gaVVcouRHvrfU1WcHYtV7rm4ckMbQKUH0UTPNYKA9vTb?=
 =?us-ascii?Q?o75h0VEuGh5m5NsX2LJQulTVWODNM0JWJUxQ2GM+nC+NL3oFIsNYxK+HglCL?=
 =?us-ascii?Q?APTnPcij6fOXLu01FwpIyKlAKmg8BufQzWxpAynRo4HFhsWNsI5nn2rj1OBR?=
 =?us-ascii?Q?E/FwKl/OZjMYY6u9uWY6JYVTsozA/jZKkZU1DnIXeqHaHgH/BIbtQb3yTQr6?=
 =?us-ascii?Q?Eruj91XaC65MT97i8scrdhN869kXpCgkgkkHfUTrhhuDyT7buAEJW6wW2Q/j?=
 =?us-ascii?Q?wBL0u8422wG1oL3qGMomG/AXkocD2fylpdhXWSSzBPdrXHJEODhGdGQjWou0?=
 =?us-ascii?Q?OpsE3ZK9uuu8D6nvyclFXj3XSaaaBEYq7iNqgcW/XEeXC63tgfFVqtYgwPo5?=
 =?us-ascii?Q?oGdpiM0PltTAe4ftm+Bpu1jD3QGkV4uzH7NtucTCxSM16eu5u/PIKe8swIao?=
 =?us-ascii?Q?5U2BzBEPoI8KExJ8ybqHpiat84JQxhPVqNbYyUYKBnMD9bWRSrFZk4BEfNZv?=
 =?us-ascii?Q?wB8P9r8gUtNOwGm2Er/CZ3GT1fhaY/hr3QSXcp+vqXwB/SWqws91KkrbU9nJ?=
 =?us-ascii?Q?p3lumt/dj2/kg6mJmGWVniVaIziwVlsuUNkzXKk+GCaKiepLBPMQJz6GxCMc?=
 =?us-ascii?Q?L3JEwzCCyZ9k8tTU9EM3CkN/X18VMIb+Mlh9eQgzmEssNVTlYhfkqeyU+50B?=
 =?us-ascii?Q?bo/XKJt1yraO3oV3Mx2oOdGeBvfKO3ngvqPtY95bBPcexqTmpFtyuX03KU3W?=
 =?us-ascii?Q?jLEtroLbbGphOEB6FlY+CphAcUWRGNu+rqEWFy9uQWcGABWdPuJRRGQdxEY1?=
 =?us-ascii?Q?vgKeePMkN5kdcMCvxfxpGOpsDciQCCKD1j5zOXpMjUXEsNdowJusJsuZELSq?=
 =?us-ascii?Q?MX4+QiYqkgMYbpVzSWtwgpQc51xa0B4eLNw8nYsG6Fw5eoOdbPvf6s88yer7?=
 =?us-ascii?Q?QocC/C9vM6vUbdmq5wHwQFNphxuMWiDWC9sqR0PKrG50lnWWxn0uueWnbT4A?=
 =?us-ascii?Q?oRVFaBuoAAX6OCzkuGj9ePaqWKJzWhVOBe7mZTphjT2tbSavFzkutkuYq/rA?=
 =?us-ascii?Q?l13HoKjH7vw1cR+09VJNuHIfmH0run1x54s7B8FFiuOVl/OXAxlJmBF+TLCK?=
 =?us-ascii?Q?CjXgTGHpXvIm4MWibAvRWrXlce+2jMUHJAHJh0kD6c/jrYJ56O0FrErthGtG?=
 =?us-ascii?Q?x0LT+dXevDo4X8xLrMdTgZ10lTLwYRx0yymyIrcranHcA9Qt46FbcpXdiDzM?=
 =?us-ascii?Q?Mn9/VUktJquKJit5UybPqxs=3D?=
X-OriginatorOrg: analogixsemi.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9d1e94e6-beee-4f7d-5591-08d9d645560d
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB6748.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Jan 2022 03:32:33.5374
 (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: H9facUCFIvEVdarVKRXn576KV9QywxGTIxLrQPW2vfy5bHxmH80tNGaqo1KU07RoxJ+Pocm46kFjCxGPbNL96w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR04MB5706
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Hsin-Yi, thanks for the patch! It's OK for me!
Reviewed-by: Xin Ji <xji@analogixsemi.com>

Thanks,
Xin

On Tue, Jan 11, 2022 at 07:27:01PM +0800, Hsin-Yi Wang wrote:
> List panel under aux-bus node if it's connected to anx7625's aux bus.
> 
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> ---
>  .../display/bridge/analogix,anx7625.yaml        | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> index 1d3e88daca041a..0d38d6fe39830f 100644
> --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> @@ -83,6 +83,9 @@ properties:
>      type: boolean
>      description: let the driver enable audio HDMI codec function or not.
>  
> +  aux-bus:
> +    $ref: /schemas/display/dp-aux-bus.yaml#
> +
>    ports:
>      $ref: /schemas/graph.yaml#/properties/ports
>  
> @@ -167,5 +170,19 @@ examples:
>                      };
>                  };
>              };
> +
> +            aux-bus {
> +                panel {
> +                    compatible = "innolux,n125hce-gn1";
> +                    power-supply = <&pp3300_disp_x>;
> +                    backlight = <&backlight_lcd0>;
> +
> +                    port {
> +                        panel_in: endpoint {
> +                            remote-endpoint = <&anx7625_out>;
> +                        };
> +                    };
> +                };
> +            };
>          };
>      };
> -- 
> 2.34.1.575.g55b058a8bb-goog