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 3961CC433EF for ; Sat, 19 Feb 2022 09:33:52 +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:MIME-Version:In-Reply-To:References: Message-ID:Date:Subject:CC:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=91KHbE1gQucuxtPPehdHULuKH3mmysKJuepvzRIrx0w=; b=usRAC0XXo0E2me Ivs/wmgkH8vZ6vB2g+AOrNRFXeFtaD3eecmErdjLzH6GaOC+ZNJlMAcij6H01OA71gNMLSGuRZ+oD vNeV7o0HXuI4sM43dsvI0fw6sx/+F7JCT/UWhWFk25l5D7y825ccRHLrSkosI4LRAkzsj2QAJbqrQ SEFmOOl4R2csnlNnAZdl0w/DCAMAuGO3leIUA3p8l4lgK08iXroC7ddILC6leg7RH5ZnexgkfWCZL p7NsJearDfkcIvg5B0KrxNjH0gfSI2sbrDZjYOalb9ISpNKyraeQB9HMTX89oYPmhPRvsafKLy6P/ J6R8cYCOqUWOVBX3gvxw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLM6H-00GYx4-Ga; Sat, 19 Feb 2022 09:32:21 +0000 Received: from mx2.securetransport.de ([188.68.39.254]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLM6B-00GYvi-Sk for linux-arm-kernel@lists.infradead.org; Sat, 19 Feb 2022 09:32:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dh-electronics.com; s=dhelectronicscom; t=1645262931; bh=DvnMBoMWrC5tP4JUDp+kdkLkR3C1ZbkLyCFQs1NdNCk=; h=From:To:CC:Subject:Date:References:In-Reply-To:From; b=Wv2IOrZkyiFSsKKPGMtfvR8NPaBt4CECR+RYwjtyhKcngGGl2UbUpoL+JV/SR2Cxn 4UDhPCAk2jOm7OUHufB7BLchwddpDRJeEF/JfHn1AWBFQsnw6RBiVb6rnnyALmPcpG UU7PLigW7gVvsS5Y1Kvsnpe85kHIgZoIqogx+M5/IzeDFnNL1zfqtk+KlfyhzYLRMT 1T7r7de7HrdEdX4f0sQJYpqp4M3etsH1AyI4YQYhJLhnSPHfmjDyVm0azMa0sRO9Jx jID60SP7eMydYjMeF3WzvzAywU05S87uvLenE8Q91DzmfaTqPkzGZxhBVjTk8yQJsK a4tpx2MANYFnA== X-secureTransport-forwarded: yes From: Christoph Niedermaier Complaints-To: abuse@cubewerk.de To: Max Krummenacher , "dri-devel@lists.freedesktop.org" CC: "Marek MV. Vasut" , David Airlie , "Sascha Hauer" , Boris Brezillon , NXP Linux Team , "Shawn Guo" , Pengutronix Kernel Team , "linux-arm-kernel@lists.infradead.org" Subject: RE: [PATCH V2] drm/imx: parallel-display: Remove bus flags check in imx_pd_bridge_atomic_check() Thread-Topic: [PATCH V2] drm/imx: parallel-display: Remove bus flags check in imx_pd_bridge_atomic_check() Thread-Index: AQHYF2Auyje+6f9Uv0ak20m5iGounayK8fGAgA/C3CA= Date: Sat, 19 Feb 2022 09:28:44 +0000 Message-ID: References: <20220201113643.4638-1-cniedermaier@dh-electronics.com> <36f517300a15b2460a02512ef3c5814ad0ed2290.camel@gmail.com> In-Reply-To: <36f517300a15b2460a02512ef3c5814ad0ed2290.camel@gmail.com> Accept-Language: de-DE, en-US Content-Language: de-DE X-MS-Has-Attach: X-MS-TNEF-Correlator: MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220219_013216_245969_2EAAE9D5 X-CRM114-Status: GOOD ( 18.52 ) 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 From: Max Krummenacher [mailto:max.oss.09@gmail.com] Sent: Wednesday, February 9, 2022 10:38 AM >> If display timings were read from the devicetree using >> of_get_display_timing() and pixelclk-active is defined >> there, the flag DISPLAY_FLAGS_SYNC_POSEDGE/NEGEDGE is >> automatically generated. Through the function >> drm_bus_flags_from_videomode() e.g. called in the >> panel-simple driver this flag got into the bus flags, >> but then in imx_pd_bridge_atomic_check() the bus flag >> check failed and will not initialize the display. The >> original commit fe141cedc433 does not explain why this >> check was introduced. So remove the bus flags check, >> because it stops the initialization of the display with >> valid bus flags. >> >> Fixes: fe141cedc433 ("drm/imx: pd: Use bus format/flags provided by the bridge when available") >> Signed-off-by: Christoph Niedermaier >> Cc: Marek Vasut >> Cc: Boris Brezillon >> Cc: Philipp Zabel >> Cc: David Airlie >> Cc: Daniel Vetter >> Cc: Shawn Guo >> Cc: Sascha Hauer >> Cc: Pengutronix Kernel Team >> Cc: Fabio Estevam >> Cc: NXP Linux Team >> Cc: linux-arm-kernel@lists.infradead.org >> To: dri-devel@lists.freedesktop.org >> --- >> V2: - Add Boris to the Cc list >> --- >> drivers/gpu/drm/imx/parallel-display.c | 8 -------- >> 1 file changed, 8 deletions(-) >> >> diff --git a/drivers/gpu/drm/imx/parallel-display.c b/drivers/gpu/drm/imx/parallel-display.c >> index a8aba0141ce7..06cb1a59b9bc 100644 >> --- a/drivers/gpu/drm/imx/parallel-display.c >> +++ b/drivers/gpu/drm/imx/parallel-display.c >> @@ -217,14 +217,6 @@ static int imx_pd_bridge_atomic_check(struct drm_bridge *bridge, >> if (!imx_pd_format_supported(bus_fmt)) >> return -EINVAL; >> >> - if (bus_flags & >> - ~(DRM_BUS_FLAG_DE_LOW | DRM_BUS_FLAG_DE_HIGH | >> - DRM_BUS_FLAG_PIXDATA_DRIVE_POSEDGE | >> - DRM_BUS_FLAG_PIXDATA_DRIVE_NEGEDGE)) { >> - dev_warn(imxpd->dev, "invalid bus_flags (%x)\n", bus_flags); >> - return -EINVAL; >> - } >> - >> bridge_state->output_bus_cfg.flags = bus_flags; >> bridge_state->input_bus_cfg.flags = bus_flags; >> imx_crtc_state->bus_flags = bus_flags; > > Tested on a Colibri iMX6DL with a panel-dpi based panel. > > Tested-by: Max Krummenacher I still ask myself why this bus flag check is in the code. Is there a reason not to remove that? Regards Christoph _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel