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=-8.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_PASS,USER_AGENT_NEOMUTT autolearn=ham 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 C2139C4360F for ; Mon, 11 Mar 2019 14:10:09 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 8FE2C2075C for ; Mon, 11 Mar 2019 14:10:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="a/0NJlF0"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="WCSjUnup" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8FE2C2075C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.20170209; h=Sender: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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/6GkZSHLhKbAoMb+p/5yuCAdS/41obY40gd01NNBqJg=; b=a/0NJlF0srL1Horz0IYt9n+EK dwg4laxTSGVTTtiXOMUyuRj9TVqVMFFlid/e11fxj4EUw1F4qe3hMfo1RCBm+fNwgVQ/50XE+bEea usKRmrnq7ibVU9ym2h7d5xihDjJ/JHknOnz789W2K8p8JI6+sVTnpmil4B4z+gT1LASX9pVbtcLs2 zrsRoDXwl4bvLfUoeaEHhj7OmupkesAUMYVriZhjni7EIp/Qt9Om1Egiioa+WflpJp4pVwry2lZb9 He18FVgMNpd2Ixof3zKwG9doWNJbSWhzpVWKcsagE42YEhGHLlDRPlq1mm85lKLVEEdZsEKaHWUG3 7ErYADTgQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h3Lcu-00036F-1g; Mon, 11 Mar 2019 14:10:00 +0000 Received: from casper.infradead.org ([2001:8b0:10b:1236::1]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h3Lcr-000360-GB for linux-arm-kernel@bombadil.infradead.org; Mon, 11 Mar 2019 14:09:57 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=SFgTBm1hWxJuLmO7u9CFyQFnBqheCdXAS534kUlA0y0=; b=WCSjUnupBA/ucBVknOPsuyifE PisIQk9I1vebfX0rIVnwPC5hdR3CWd+wFOUKs5e6zm7Rx62d4XaYn67cdsvB4KXqa7b+cp5248kly JrJAt5CMRNn32lU49uvbm0gdpT9OO3mEUUy5lq8PIQAQAM4Ej67Rzq/qUTFiIYLQCe7UVlptRqZD8 aHmFLgk3jls7JwS7fhdBOt8njtusCbmybFQWSGkyR3WQDjDY01L/ch1WTtfsyhRaWmrG96apg5cBs cExDBQVGKxMVe0WsIU4ASDCwXznHGDaRRh6KxjO1PGpt9dmZeMkp69euyTRdvQWDimKIN18NgqjF1 BCWkHC4rA==; Received: from relay7-d.mail.gandi.net ([217.70.183.200]) by casper.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h3Lcn-0007CN-PJ for linux-arm-kernel@lists.infradead.org; Mon, 11 Mar 2019 14:09:55 +0000 X-Originating-IP: 90.88.150.179 Received: from localhost (aaubervilliers-681-1-31-179.w90-88.abo.wanadoo.fr [90.88.150.179]) (Authenticated sender: maxime.ripard@bootlin.com) by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id ADA8220016; Mon, 11 Mar 2019 14:09:23 +0000 (UTC) Date: Mon, 11 Mar 2019 15:09:23 +0100 From: Maxime Ripard To: Jagan Teki Subject: Re: [PATCH v9 2/5] drm/sun4i: sun6i_mipi_dsi: Fix TCON DRQ set bits Message-ID: <20190311140923.vvtubx7wndos2rfr@flea> References: <20190303173527.31055-1-jagan@amarulasolutions.com> <20190303173527.31055-3-jagan@amarulasolutions.com> <20190304154320.uledvwz5jhzt22o3@flea> <20190307153935.uxh2hsmahw5l7h4q@flea> MIME-Version: 1.0 In-Reply-To: User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190311_140953_868702_88AD8E28 X-CRM114-Status: GOOD ( 44.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: David Airlie , linux-sunxi , linux-kernel , dri-devel , Chen-Yu Tsai , Daniel Vetter , Michael Trimarchi , linux-amarula@amarulasolutions.com, linux-arm-kernel Content-Type: multipart/mixed; boundary="===============8321786705563646890==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============8321786705563646890== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="lbfkunzo5uuh54qm" Content-Disposition: inline --lbfkunzo5uuh54qm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Mar 07, 2019 at 09:24:02PM +0530, Jagan Teki wrote: > On Thu, Mar 7, 2019 at 9:09 PM Maxime Ripard = wrote: > > > > On Thu, Mar 07, 2019 at 05:49:07PM +0530, Jagan Teki wrote: > > > On Mon, Mar 4, 2019 at 9:13 PM Maxime Ripard wrote: > > > > > > > > On Sun, Mar 03, 2019 at 11:05:24PM +0530, Jagan Teki wrote: > > > > > TCON DRQ for non-burst DSI mode can computed based on horizontal > > > > > front porch value, but the current driver trying to include sync > > > > > timings along with front porch resulting wrong drq. > > > > > > > > > > This patch is trying to update the drq by subtracting hsync_start > > > > > with hdisplay, which is horizontal front porch. > > > > > > > > > > Current code: > > > > > ------------ > > > > > mode->hsync_end - mode->hdisplay =3D> horizontal front porch + sy= nc > > > > > > > > > > With this patch: > > > > > ---------------- > > > > > mode->hsync_start - mode->hdisplay =3D> horizontal front porch > > > > > > > > > > BSP code form BPI-M64-bsp is computing TCON DRQ set bits > > > > > for non-burts as (from linux-sunxi/ > > > > > drivers/video/sunxi/disp2/disp/de/lowlevel_sun50iw1/de_dsi.c) > > > > > > > > > > =3D> panel->lcd_ht - panel->lcd_x - panel->lcd_hbp > > > > > =3D> (timmings->hor_front_porch + panel->lcd_hbp + panel->lcd_x) > > > > ^ + sync length + > > > > > - panel->lcd_x - panel->hbp > > > > > =3D> timmings->hor_front_porch > > > > ^ + sync > > > > > =3D> mode->hsync_start - mode->hdisplay > > > > > > > > s/hsync_start/hsync_end/ > > > > > > No, it should be front porch so it is hsync_start. This change is > > > trying to update DRQ set to use front porch and above evaluation from > > > BSP, result the same front front porch > > > > > > Current driver has hsync_end - hdisplay which is not front porch > > > timing (it is adding extra sync timing). > > > > It would be if you considered that the back porch actually was the > > back porch plus the sync length. I have found no such evidence, quite > > the opposite actually, everything seems to point at the fact that > > unlike the TCON, the DSI block uses the back porch as only the back > > porch. >=20 > Sorry, I'm not clear about back porch here. >=20 > The current code has mode->hsync_end - mode->hdisplay which is Front > porch + sync time do you think it's not? It is. > DRQ set time is pure front porch value (according BSP) as I didn't > see any information about DRQ set bits in manual or anywhere. This is what I'm telling you. If you consider the back porch as only the back porch, then the result of that BSP calculation you mentionned earlier is the front porch and the sync length. You imply that the back porch should actually be treated as the back porch and the sync length in your calculation. What makes you say so? > fyi: atleast if you didn't trust me, here is existing applied patch > for about equations. > https://cgit.freedesktop.org/drm/drm-misc/commit/drivers/gpu/drm/sun4i/su= n6i_mipi_dsi.c?id=3D2cfdc24d2f8d9b14704567c065beb2a118a578fa I'm aware of that patch, but I have no idea why do you bring it up in that discussion. > So, this patch fixed to remove sync time by updating hsync_start - > hdisplay which is pure front porch. and it's clear that pane is not > working without this. This is the same discussion over and over again: which panel, which datasheet, not working how? > > > I believe this is something similar like fixed patches for VBP, HBLK > > > timings. > > > > > > > Did you encounter any panel where this was fixing something? If so, > > > > which one, and what is the matching timings and / or datasheet? > > > > > > W/O this change Bananapi s070wv20 panel has issue on striped lines on > > > the panel[1] and timings are > > > > > > static const struct drm_display_mode s070wv20_default_mode =3D { > > > .clock =3D 30000, > > > .vrefresh =3D 60, > > > > > > .hdisplay =3D 800, > > > .hsync_start =3D 800 + 40, > > > .hsync_end =3D 800 + 40 + 48, > > > .htotal =3D 800 + 40 + 48 + 40, > > > > > > .vdisplay =3D 480, > > > .vsync_start =3D 480 + 13, > > > .vsync_end =3D 480 + 13 + 3, > > > .vtotal =3D 480 + 13 + 3 + 29, > > > }; > > > > > > Which is similar like in panel-simple "bananapi,s070wv20-ct16" > > > > > > Here is the DSI panel patches and sequence: > > > [pixel clock is 30Mhz] https://patchwork.kernel.org/patch/10680331/ > > > https://github.com/yesnoandor/x300/blob/master/kernel/arch/arm/boot/d= ts/erobbing/x300/x300.dtsi#L81 > > > https://github.com/wxzed/Raspberry_5MIPI_Display/blob/master/I2C_Slav= e/USER/main.c#L15 > > > https://github.com/eliot-shao/qcom/blob/master/icn6211_cxn0102/kernel= /drivers/video/msm/mdss/mdss_i2c_interface.c#L152 > > > > What are those supposed to be? It doesn't look like timings but rather = initialization sequences > > > > > matches timings for > > > https://github.com/eliot-shao/qcom/blob/master/icn6211_cxn0102/kernel= /arch/arm/boot/dts/qcom/dsi-mipi-2-rgb_1280p_video.dtsi#L20 > > > > That's not even the same resolution.. >=20 > fyi about the sequence. How is that relevant? > > > https://github.com/zestroly/micromat/blob/master/test/raspberry/ICN62= 11.cpp#L169 > > > > And this isn't a set of timings either. > > > > > Attached is panel datasheet. > > > > Which is for an RGB panel... not a MIPI-DSI one. >=20 > Same panel timings It is a DSI ICN6211 bridge, I have attached the > patch above https://patchwork.kernel.org/patch/10680331/ for > information about the display please find previous mail attachment. The previous attachment was for a display that doesn't even have the same bus. How is that relevant? Maxime --=20 Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --lbfkunzo5uuh54qm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXIZsEwAKCRDj7w1vZxhR xU0rAP0UGfRrSN11Vi+XBZeaQtUO9PDFLMhBdOKHqiRjuRZjWQD/W9Bu12DCZKaD 2xf6tvt1J8CPlDPBdpa4hz4awiityws= =rEj7 -----END PGP SIGNATURE----- --lbfkunzo5uuh54qm-- --===============8321786705563646890== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============8321786705563646890==--