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 0B25DC43334 for ; Mon, 18 Jul 2022 18:01:56 +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=+Vp/KfALWvQXIgLNzeU6BqIYE2Sfjc6En5Msc1dSF84=; b=nZUHOfijBhCUov 8pMsYRmgobM73C0ZlQ8MzdHNkK7pnyVNRpg+pPkIjlKi3Okyg9rwVOKMURmVzh542BWaUw+SqybyZ jSIT+jmnZVF3C8IoV+aeSzP6if8fw2otSBIB1wJWTmtAeYHuXzcYNi8VXaeOgGaiZuk6l64pQg9+9 Dj3aGbiueLa9Q/TDeVvpWKWsaSfjH1xB4XQKrnBZ5+smkv28O2NMQa56ZJm+/EUdRgWGjyHz3Tnlt 4rD+F0emHWKd8JtEjt89XGLm6G1MiKnGIhpYRt7kKgWPRvMDw9aY9BC2og2tMp9cmR7K3yb27Pmxz BRRn4hZeIdppfWWaD3hA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oDV2s-00HPYG-NF; Mon, 18 Jul 2022 18:00:38 +0000 Received: from mailrelay1-1.pub.mailoutpod1-cph3.one.com ([46.30.210.182]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oDV2k-00HPNn-1L for linux-arm-kernel@lists.infradead.org; Mon, 18 Jul 2022 18:00:33 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ravnborg.org; s=rsa1; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=1k7ygd7TjVU/0OeLc8vN7pGFtYh4D8DdF58kU5beJu0=; b=VxuZdz71LN8l4OHMA30tuoiK0Nr/nISJyIbR4puSlqXqtLNJ3R9E8Ki5Y/fB//0ToLWnFzY6Ksx2v q15Tp+N3bjxKD5yb8cNSHhlL3Fn5bRvCp2wA20B/b8kotzJ8eyHD3W0jwgxlwEAS7y8yp9aYUCPRij D6w4iTRCC6nKQbdz1oAl9YkjdJBz4iHoVhli1MdsAc3QdBqKAH5OIBdeojkex0vvXW5oFokXNW+08W aIVUQfOp1MMmHhJ2Eo8oeIrL8vAjG4XGjO59HSG5vDKD2i9lhICvJrNx36s5unvTPFFMNLi1SgQk// otsKVYz1SQ0DcHOoKxmADC+NiOX02UA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=ravnborg.org; s=ed1; h=in-reply-to:content-type:mime-version:references:message-id:subject:cc:to: from:date:from; bh=1k7ygd7TjVU/0OeLc8vN7pGFtYh4D8DdF58kU5beJu0=; b=jL/3JsawqxR14ggldJXwk5X9UoFKRKl4/PCzdAI+J8QjN2tXmDPVlJnoASEn5F/EpOlFEotQOERIw yI/yZMXCA== X-HalOne-Cookie: fb423e04bf4d0af1a45399369f4e4409c2acada4 X-HalOne-ID: 7dbadd68-06c3-11ed-a6c8-d0431ea8a283 Received: from mailproxy1.cst.dirpod4-cph3.one.com (2-105-2-98-cable.dk.customer.tdc.net [2.105.2.98]) by mailrelay1.pub.mailoutpod1-cph3.one.com (Halon) with ESMTPSA id 7dbadd68-06c3-11ed-a6c8-d0431ea8a283; Mon, 18 Jul 2022 18:00:22 +0000 (UTC) Date: Mon, 18 Jul 2022 20:00:20 +0200 From: Sam Ravnborg To: Dave Stevenson Cc: Dafna Hirschfeld , Neil Armstrong , David Airlie , Tomi Valkeinen , DRI Development , Laurent Pinchart , Andrzej Hajda , Guenter Roeck , chrome-platform@lists.linux.dev, Jernej Skrabec , Chun-Kuang Hu , Jitao Shi , Arnd Bergmann , Jonas Karlman , linux-mediatek@lists.infradead.org, Matthias Brugger , Linux ARM , Philip Chen , Robert Foss , linux-renesas-soc@vger.kernel.org, Kieran Bingham , Thomas Zimmermann , Enric Balletbo i Serra , Cai Huoqing Subject: Re: [PATCH v1 12/12] drm/todo: Add bridge related todo items Message-ID: References: <20220717174454.46616-1-sam@ravnborg.org> <20220717175801.78668-1-sam@ravnborg.org> <20220717175801.78668-5-sam@ravnborg.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220718_110030_486260_2072DFB8 X-CRM114-Status: GOOD ( 29.54 ) 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 Dave, On Mon, Jul 18, 2022 at 11:27:37AM +0100, Dave Stevenson wrote: > Hi Sam > > On Sun, 17 Jul 2022 at 18:58, Sam Ravnborg wrote: > > > > Add todo in the hope someone will help updating the bridge drivers. > > > > v2: > > - Updated descriptions in todo.rst > > > > Signed-off-by: Sam Ravnborg > > Acked-by: Maxime Ripard > > Cc: Laurent Pinchart > > Cc: Maarten Lankhorst > > Cc: Maxime Ripard > > Cc: Thomas Zimmermann > > Cc: David Airlie > > Cc: Daniel Vetter > > --- > > Documentation/gpu/todo.rst | 20 ++++++++++++++++++++ > > 1 file changed, 20 insertions(+) > > > > diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst > > index 10bfb50908d1..fbcc232e0bc1 100644 > > --- a/Documentation/gpu/todo.rst > > +++ b/Documentation/gpu/todo.rst > > @@ -480,6 +480,26 @@ Contact: Thomas Zimmermann > > > > Level: Starter > > > > +Drop use of deprecated operations in bridge drivers > > +-------------------------------------------------- > > + > > +&struct drm_bridge_funcs contains a number of deprecated operations > > +which can be replaced by the atomic variants. > > + > > +The following is more or less 1:1 replacements with the arguments > > +and names adjusted: > > +* pre_enable => atomic_pre_enable > > +* enable => atomic_enable > > +* disable => atomic_disable > > +* post_disable => atomic_post_disable > > + > > +* mode_set is no longer required and the implementation shall be merged > > + with atomic_enable. > > The dw-mipi-dsi and msm DSI host controller bridge drivers are > currently relying on mode_set as a convenient hook to power up the DSI > host prior to pre_enable of the bridge chain. Removing it is therefore > going to break those. > > There is a proposed mechanism to allow for the removal of this hack > [1], but it's still waiting on R-B tags and/or discussion from bridge > maintainers (gentle nudge as you are one of those maintainers). I have over time gained some knowledge of how bridges works and have applied a few patches - but the maintainer role belongs to others. I just try to help a bit. I will review the referenced series - could you then in return review this series? > > And do you mean merge with atomic_enable, or merge with > atomic_pre_enable? atomic_pre_enable would be more applicable for > almost all the bridges I'm aware of as they want to be configured > before video starts. Thanks, yes you are right. I will update the text to refer to pre_enable as this is often the right choice. Looking at how many bridges who implements mode_set, or are not yet atomic, this will take a while before we can drop it. Sam _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel