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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A5840C433EF for ; Tue, 28 Sep 2021 12:49:13 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6B800601FF for ; Tue, 28 Sep 2021 12:49:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 6B800601FF Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 69E1F6E898; Tue, 28 Sep 2021 12:49:12 +0000 (UTC) X-Greylist: delayed 12418 seconds by postgrey-1.36 at gabe; Tue, 28 Sep 2021 12:49:11 UTC Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8B6666E898 for ; Tue, 28 Sep 2021 12:49:11 +0000 (UTC) Received: from [IPv6:::1] (p578adb1c.dip0.t-ipconnect.de [87.138.219.28]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: marex@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 15F728312F; Tue, 28 Sep 2021 14:49:07 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1632833348; bh=HjmkWr+4r7r/atswVfbKvnliuyMeP34ao4ebbIjZUtM=; h=Subject:To:References:Cc:From:Date:In-Reply-To:From; b=IOUnQ3AXIukz2Q6cbalRMcafkhbVnnKHazpvZGHH8S8s3UisQ5pNlZLcMSlIkgQis LpLOZqzqcfLGef6vV0ovLFia7jJVRey1BTtYpQhuUReCDKZQ2srX2/962ii8r86Rvs dxEBhTY7Vx9GE1UVaCNkubjzXF00BJggQU0e31pgUHZ9RY18UMKlWT+Q3rLIiDkgox yz2RGAPIXUfQ8zBBAXjK3lDwIrMgQt4Avg3RpejNmAYMbwSB47TLHJQsGC97JmG7R4 sVPCt03h45R7EQZQusYaZPZby88qY6JlHbeGto+1aW9Z87Mw9wRRCyEVne4iidRRUy uXxwGxkc+TDXg== Subject: Re: [PATCH v1 2/5] drm: mxsfb: Set proper default bus format when using a bridge To: =?UTF-8?Q?Guido_G=c3=bcnther?= References: <1fd5457b925249bce77315663adabecdd29ec0f3.1632828477.git.agx@sigxcpu.org> Cc: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Stefan Agner , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Thierry Reding , Sam Ravnborg , Ondrej Jirman , Lucas Stach , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org From: Marek Vasut Message-ID: <67e6056a-6157-795d-908d-d65cc803c03d@denx.de> Date: Tue, 28 Sep 2021 14:49:06 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <1fd5457b925249bce77315663adabecdd29ec0f3.1632828477.git.agx@sigxcpu.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-Virus-Scanned: clamav-milter 0.103.2 at phobos.denx.de X-Virus-Status: Clean X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On 9/28/21 2:16 PM, Guido Günther wrote: > If a bridge doesn't do any bus format handling MEDIA_BUS_FMT_FIXED is > returned. Fallback to a reasonable default (MEDIA_BUS_FMT_RGB888_1X24) in > that case. > > This unbreaks e.g. using mxsfb with the nwl bridge and mipi panels. > > Fixes: b776b0f00f24 ("drm: mxsfb: Use bus_format from the nearest bridge if present") > > Reported-by: Martin Kepplinger > Signed-off-by: Guido Günther > --- > drivers/gpu/drm/mxsfb/mxsfb_kms.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_kms.c b/drivers/gpu/drm/mxsfb/mxsfb_kms.c > index d6abd2077114..f4be16f5c20b 100644 > --- a/drivers/gpu/drm/mxsfb/mxsfb_kms.c > +++ b/drivers/gpu/drm/mxsfb/mxsfb_kms.c > @@ -369,6 +369,11 @@ static void mxsfb_crtc_atomic_enable(struct drm_crtc *crtc, > drm_atomic_get_new_bridge_state(state, > mxsfb->bridge); > bus_format = bridge_state->input_bus_cfg.format; > + if (bus_format == MEDIA_BUS_FMT_FIXED) { > + dev_warn_once(drm->dev, > + "Bridge does not provide bus format. Please fix."); Fix what ? Oh ... the bridge driver ? Maybe such a hint should be in the warning message. And it might be useful to explain how to fix that bridge driver, or at least provide some reference to an example (like this nwl patch series). > + bus_format = MEDIA_BUS_FMT_RGB888_1X24; The warning should also mention that the driver is falling back to this mode.