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=-3.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED 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 EB15BC31E57 for ; Mon, 17 Jun 2019 10:08:34 +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 C2EA22084D for ; Mon, 17 Jun 2019 10:08:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="XgtaCuX8" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C2EA22084D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=collabora.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-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description :Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ZsdAVfkL3wzpDiA+T++orBk+R9f4/McE2Z2KSM+CAZw=; b=XgtaCuX8702Uic +n/vWwA4Xs1FE1qaoSgw+DB7ebZkmk7c6e0PJToO5BhnWDsUGxgYPAOl4WO9QJFR6tRTgVvWerz6r sw3Fdy7ctBLIz4BpNUhGQFrrKzq5EDKTjMKfuAf8vvnkIA+MuMWiOWOLz4bOHgSjHljyGb5Te49/e PnV1+/uGJw1mNb12xinQRdH6/hO4jCpjiEL4tR43y1EtyCS41zGVjgNETn1NjPHrML/TeWI6qdXUN fuBOKArMEqpL1+drBz8rq0XwuP//Pg9P6cfIKgr33mUr9Dky4iv8V3ohzwW95GeaTgDPLLMds6TwV fE0vOBJH+lUV6kvFiFJw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hcoZ0-0006oQ-GD; Mon, 17 Jun 2019 10:08:34 +0000 Received: from bhuna.collabora.co.uk ([46.235.227.227]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hcoYx-0006na-RB; Mon, 17 Jun 2019 10:08:33 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: eballetbo) with ESMTPSA id 5D84F263992 Subject: Re: [PATCH] Revert "ARM: dts: rockchip: set PWM delay backlight settings for Minnie" To: Pavel Machek , Matthias Kaehlcke References: <20190614224533.169881-1-mka@chromium.org> <20190616154143.GA28583@atrey.karlin.mff.cuni.cz> From: Enric Balletbo i Serra Message-ID: Date: Mon, 17 Jun 2019 12:08:25 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.1 MIME-Version: 1.0 In-Reply-To: <20190616154143.GA28583@atrey.karlin.mff.cuni.cz> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190617_030832_010444_04C25EBC X-CRM114-Status: GOOD ( 14.77 ) 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: Mark Rutland , devicetree@vger.kernel.org, Heiko Stuebner , linux-kernel@vger.kernel.org, Douglas Anderson , linux-rockchip@lists.infradead.org, Rob Herring , 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+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On 16/6/19 17:41, Pavel Machek wrote: > Hi! > >> This reverts commit 288ceb85b505c19abe1895df068dda5ed20cf482. >> >> According to the commit message the AUO B101EAN01 panel on minnie >> requires a PWM delay of 200 ms, however this is not what the >> datasheet says. The datasheet mentions a *max* delay of 200 ms >> for T2 ("delay from LCDVDD to black video generation") and T3 >> ("delay from LCDVDD to HPD high"), which aren't related to the >> PWM. The backlight power sequence does not specify min/max >> constraints for T15 (time from PWM on to BL enable) or T16 >> (time from BL disable to PWM off). >> Hmm, clearly we are not looking at the same datasheet, because in the one I have I don't see any reference to T15/T16 or LCDVDD. And, I assume I am probably wrong because you might have better access to the specific panel specs for minnie. I looked at my archive and the datasheet I have is similar to this [1]. In page 21, Section 6.5 Power ON/OFF Sequence, there are two delays T3 and T4, it is *min* time between the pwm signal and the bl_en and it is 200 ms. That's the delay the patch was adding. [1] http://www.yslcd.com.tw/docs/product/B101EAN01.1.pdf >> Signed-off-by: Matthias Kaehlcke >> --- >> Enric, if you think I misinterpreted the datasheet please holler! > > Was this tested? Was previous patch tested? > IIRC, It was tested measuring the backlight power on timing (although I am not sure if I tested this on minnie or another board with better access to the pins) > Does patch being reverted actually break anything? If so, cc stable? > > Pavel > > Probably will not break anything, I don't remember the reverted patch as a fix of any specific issue. IIRC it was more a fear to be out of specs but I'll not be surprised if the datasheet lies and this delay is not needed at all. Matthias, are you reverting this to solve any problem? Could you share your datasheet? Thanks, ~Enric _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel