From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 15990C25B7A for ; Tue, 21 May 2024 12:30:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=FWC3hSYOg2p5er9BIiG2k8l3H806LBJfpKwbnwtUrYk=; b=CJ4tdrLX0zcP0U aoesS/lY4WpZj1Y+6qXoATrLI620Ow/W54iw6Hf3GaheG8fhWgmovLzuPnTN+qYQODWDNLQVAnswf yAtEUoqsKEW/FHkgpPi9kE1gHhI8wC9BWfz9X77hJRbRBncpUTR/XjQrrT8LCf6Pc6E4PgXG/aNKL t9iO8K+GKkZsnKQ/YPTe3ZUk5Eov/a4uQe7IaLTHlawDxjqXpgpQFPXbJUcpj3ljNAbHWOksOoYgy ogXoXNcBS9/5npmYhWx8x34RWo1ZLVY8hvqW73lsvHMcYs8YeaYj9QrZn+EmZEBD/sAbnKS0mw2b6 3DkI1w/+tF+eQJ2Ld8Qw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s9OdC-0000000HRpR-0CVb; Tue, 21 May 2024 12:30:14 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s9Od7-0000000HRla-49he for linux-arm-kernel@lists.infradead.org; Tue, 21 May 2024 12:30:11 +0000 Received: from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi [81.175.209.231]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id C58CA5A4; Tue, 21 May 2024 14:29:52 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1716294593; bh=inFia2SI4gyrSS+NM4R+gTmGK3sXAfInB0c0Z9vRUF4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tcJsehu86vNyA8ftNK8aNoX/SW6csZjhvaEfv5aXD57/Wx52uzcC7PEFQyVNVtUCU H5NOfbm8vhWrbWwABpwvX7obbu/iDTWIkuXqfoig38h5WexNC4FFqXrjexS32GyNU9 oFmO+4pj/zJ0xAGHB7JdQje3WWMERGt6tJInqYOs= Date: Tue, 21 May 2024 15:29:55 +0300 From: Laurent Pinchart To: Marek Vasut Cc: devicetree@vger.kernel.org, Andrzej Hajda , Conor Dooley , Daniel Vetter , David Airlie , Fabio Estevam , Jernej Skrabec , Jonas Karlman , Krzysztof Kozlowski , Liu Ying , Maarten Lankhorst , Mark Yao , Maxime Ripard , Neil Armstrong , Pengutronix Kernel Team , Philipp Zabel , Rob Herring , Robert Foss , Sascha Hauer , Shawn Guo , Thomas Zimmermann , dri-devel@lists.freedesktop.org, imx@lists.linux.dev, kernel@dh-electronics.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] dt-bindings: display: synopsys,dw-hdmi: Mark ddc-i2c-bus as deprecated Message-ID: <20240521122955.GH16345@pendragon.ideasonboard.com> References: <20240521104057.10485-1-marex@denx.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240521104057.10485-1-marex@denx.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240521_053010_206526_1F1C27C4 X-CRM114-Status: GOOD ( 17.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Marek, Thank you for the patch. On Tue, May 21, 2024 at 12:40:47PM +0200, Marek Vasut wrote: > The ddc-i2c-bus property should be placed in connector node, > mark the HDMI TX side property as deprecated. > > Signed-off-by: Marek Vasut > --- > Cc: Andrzej Hajda > Cc: Conor Dooley > Cc: Daniel Vetter > Cc: David Airlie > Cc: Fabio Estevam > Cc: Jernej Skrabec > Cc: Jonas Karlman > Cc: Krzysztof Kozlowski > Cc: Laurent Pinchart > Cc: Liu Ying > Cc: Maarten Lankhorst > Cc: Mark Yao > Cc: Maxime Ripard > Cc: Neil Armstrong > Cc: Pengutronix Kernel Team > Cc: Philipp Zabel > Cc: Rob Herring > Cc: Robert Foss > Cc: Sascha Hauer > Cc: Shawn Guo > Cc: Thomas Zimmermann > Cc: devicetree@vger.kernel.org > Cc: dri-devel@lists.freedesktop.org > Cc: imx@lists.linux.dev > Cc: kernel@dh-electronics.com > Cc: linux-arm-kernel@lists.infradead.org > --- > .../devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml b/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml > index 828709a8ded26..d09a0bee54247 100644 > --- a/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml > @@ -47,6 +47,7 @@ properties: > > ddc-i2c-bus: > $ref: /schemas/types.yaml#/definitions/phandle > + deprecated: true > description: > The HDMI DDC bus can be connected to either a system I2C master or the > functionally-reduced I2C master contained in the DWC HDMI. When connected How about adding an additional sentence here to explain what should be used instead ? to a system I2C master this property contains a phandle to that I2C master controller. + + This property is deprecated, the system I2C master controller should + be referenced through the ddc-i2c-bus property of the HDMI connector + node. Reviewed-by: Laurent Pinchart -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel