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 X-Spam-Level: X-Spam-Status: No, score=-9.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 42F7BC433E1 for ; Mon, 17 Aug 2020 00:31:24 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 10DDF2078A for ; Mon, 17 Aug 2020 00:31:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="eyk298My"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="H+S5q7Tc" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 10DDF2078A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject: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=dFYGWi2WrGFvym9bQxd2jc2HSgFgIsEwrIaVwQE6YCY=; b=eyk298My0i2wRF7BdCPpJil/4 XmV69AFg+D6+aLnZpQu+e2d8YOZX1BEH8vV2pPI91WsxvvS7b7M5O2ftP+54sML1TnSc844vKMrdK 06htM7syrbYNxnrLyYeH8VNptlzQP1lZBUltoRIoao39kRy1tQNUtqMfsbJ+DOAz6xwH+gkJe7rs5 7X4JYo0XdwUXwQUhfvBb0vUtwrdk/nu2z/LhLZCK7pEwye+VGHIi3zBeM8ulmooV3meu0485jWIT1 Lxa840GXnPFF/MD40bPZcEuERH54x08W7Th/rNWBGvkN6j2+RzZ36IC0X/SH+yRUNGw8CHdjZ7HYy CuS3CRZeQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k7T2N-0001z1-2v; Mon, 17 Aug 2020 00:30:07 +0000 Received: from perceval.ideasonboard.com ([2001:4b98:dc2:55:216:3eff:fef7:d647]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k7T2K-0001yG-FU for linux-arm-kernel@lists.infradead.org; Mon, 17 Aug 2020 00:30:05 +0000 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 19974F9; Mon, 17 Aug 2020 02:30:03 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1597624203; bh=8BQyhx7lyZCv4L88FZ8xKeHNkAxqK4InxbmPXHnl1Dc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=H+S5q7TcBovYm1nmIx4gfwlcA0VNdQv555uCHqICq4PvEAZs7Gd3yM4AScnZuLbJV GHrNX7eIMGYs3Z0NaeEVex8fGz6ytJtdXAeQs3kOlvamtkgZjknkquVEA+IkyVsynp 8ECA896D4pKV4Xxe66KLqJjhb/HbddEBFFwxkbow= Date: Mon, 17 Aug 2020 03:29:47 +0300 From: Laurent Pinchart To: Sam Ravnborg Subject: Re: [PATCH 8/8] drm: mxsfb: Add support for the bus-width DT property Message-ID: <20200817002947.GF7729@pendragon.ideasonboard.com> References: <20200813012910.13576-1-laurent.pinchart@ideasonboard.com> <20200813012910.13576-9-laurent.pinchart@ideasonboard.com> <20200816074630.GI1201814@ravnborg.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200816074630.GI1201814@ravnborg.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200816_203004_638998_6894A215 X-CRM114-Status: GOOD ( 28.06 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Marek Vasut , devicetree@vger.kernel.org, Guido =?utf-8?Q?G=C3=BCnther?= , dri-devel@lists.freedesktop.org, Rob Herring , Pengutronix Kernel Team , linux-arm-kernel@lists.infradead.org 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 Sam, On Sun, Aug 16, 2020 at 09:46:30AM +0200, Sam Ravnborg wrote: > On Thu, Aug 13, 2020 at 04:29:10AM +0300, Laurent Pinchart wrote: > > A new bus-width DT property has been introduced in the bindings to allow > > overriding the bus width. Support it. > > > > Signed-off-by: Laurent Pinchart > > We already reads the bus-width in following files in drm: > atmel-hlcdc/atmel_hlcdc_output.c bridge/ti-tfp410.c > > So this calls for a common helper to do this like: > > int drm_of_bus_fmt(const struct device_node *ep) > { > } > > This helper could then read bus-width, data-shift (if relevant) > and return the relevant bus format. > > I can see that bridge/ti-tfp410.c assumes 12 equals > MEDIA_BUS_FMT_RGB888_2X12_LE where as mxsfb assumes 12 is > MEDIA_BUS_FMT_RGB444_1X12. > I do not know why neither how to handle this difference. > Maybe this is something to do with DVI as this is what tfp410 > seems to support. I think the mapping from bus width to format is device-specific, I'm not sure a common helper is possible. And now that I think about this, I wonder if data-shift = <2> wouldn't be a better option than bus-width = <24>, as a 18-bpp panel will always be connected with 18 lines, not 24. One issue, however, is that for the 565 case, I'd need a different data-shift value for R/B and for G, which isn't possible. I don't think that turning data-shift into a multi-integers property is worth it though. Given that DT properties are interpreted in the context of a particular binding using bus-width could be fine here, but I'm open to better alternatives if someone has a good proposal. > > --- > > drivers/gpu/drm/mxsfb/mxsfb_drv.c | 26 ++++++++++++++++++++++++++ > > drivers/gpu/drm/mxsfb/mxsfb_drv.h | 2 ++ > > drivers/gpu/drm/mxsfb/mxsfb_kms.c | 8 ++++++-- > > 3 files changed, 34 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c b/drivers/gpu/drm/mxsfb/mxsfb_drv.c > > index 8c549c3931af..fab3aae8cf73 100644 > > --- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c > > +++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c > > @@ -95,10 +95,36 @@ static int mxsfb_attach_bridge(struct mxsfb_drm_private *mxsfb) > > { > > struct drm_device *drm = mxsfb->drm; > > struct drm_connector_list_iter iter; > > + struct device_node *ep; > > struct drm_panel *panel; > > struct drm_bridge *bridge; > > + u32 bus_width = 0; > > int ret; > > > > + ep = of_graph_get_endpoint_by_regs(drm->dev->of_node, 0, 0); > > + if (!ep) > > + return -ENODEV; > > + > > + of_property_read_u32(ep, "bus-width", &bus_width); > > + of_node_put(ep); > > + > > + switch (bus_width) { > > + case 16: > > + mxsfb->bus_format = MEDIA_BUS_FMT_RGB565_1X16; > > + break; > > + case 18: > > + mxsfb->bus_format = MEDIA_BUS_FMT_RGB666_1X18; > > + break; > > + case 24: > > + mxsfb->bus_format = MEDIA_BUS_FMT_RGB888_1X24; > > + break; > > + case 0: > > + break; > > + default: > > + DRM_DEV_ERROR(drm->dev, "Invalid bus-width %u", bus_width); > > + return -ENODEV; > > + } > > + > > ret = drm_of_find_panel_or_bridge(drm->dev->of_node, 0, 0, &panel, > > &bridge); > > if (ret) > > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.h b/drivers/gpu/drm/mxsfb/mxsfb_drv.h > > index 399d23e91ed1..c4f7a8a0c891 100644 > > --- a/drivers/gpu/drm/mxsfb/mxsfb_drv.h > > +++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.h > > @@ -32,6 +32,8 @@ struct mxsfb_drm_private { > > struct clk *clk_axi; > > struct clk *clk_disp_axi; > > > > + u32 bus_format; > > + > > struct drm_device *drm; > > struct { > > struct drm_plane primary; > > diff --git a/drivers/gpu/drm/mxsfb/mxsfb_kms.c b/drivers/gpu/drm/mxsfb/mxsfb_kms.c > > index b721b8b262ce..6d512f346918 100644 > > --- a/drivers/gpu/drm/mxsfb/mxsfb_kms.c > > +++ b/drivers/gpu/drm/mxsfb/mxsfb_kms.c > > @@ -50,11 +50,15 @@ static void mxsfb_set_formats(struct mxsfb_drm_private *mxsfb) > > { > > struct drm_device *drm = mxsfb->drm; > > const u32 format = mxsfb->crtc.primary->state->fb->format->format; > > - u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24; > > + u32 bus_format; > > u32 ctrl, ctrl1; > > > > - if (mxsfb->connector->display_info.num_bus_formats) > > + if (mxsfb->bus_format) > > + bus_format = mxsfb->bus_format; > > + else if (mxsfb->connector->display_info.num_bus_formats) > > bus_format = mxsfb->connector->display_info.bus_formats[0]; > > + else > > + bus_format = MEDIA_BUS_FMT_RGB888_1X24; > > > > DRM_DEV_DEBUG_DRIVER(drm->dev, "Using bus_format: 0x%08X\n", > > bus_format); -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel