Received: by 10.213.65.68 with SMTP id h4csp267915imn; Fri, 16 Mar 2018 02:27:33 -0700 (PDT) X-Google-Smtp-Source: AG47ELtxhyuEi7MelaZS6uMMfT8QchMF6qzcO3I9oUw8oOXbNay6lezk940N/pKQ+2OY9QQckbDb X-Received: by 2002:a17:902:7142:: with SMTP id u2-v6mr1328910plm.257.1521192453732; Fri, 16 Mar 2018 02:27:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521192453; cv=none; d=google.com; s=arc-20160816; b=SSLOXdDZ4u0l+mXK5n+wKRxN/Vv0BhGmKjE77jzSNivHXktAvlnhLXt7CIDig+lXmf IMXCTOhg6bHtulo1DLTB4NVL3XgQfQEjcMgBZ1awYVYwCHJPYTIxiLczZUqqgr2p3Pb/ S0c7ktZ5KL6Kv6WIXmQdmDoQMZUNHgPbsnwvQk8tzxWimoCwKOEd+TO/izmGi8RlppU5 ELk86W3teifgsfyl1D+V5MXixMFcXiW6aQ8qTBoU+qRFPCCQ9HfHFucbSpT6rmibp2ER 3PZA32nYWrL0AUjqovkgesBZKuSDFVkjIARuXfkOrQmiA3wgD44Reu0ok1qDvlJmYh6X K3OA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dmarc-filter :dkim-signature:dkim-signature:arc-authentication-results; bh=52W6Ckmz5hkNrm0A6OLZMtiR9ZlXQ/ZYI0KvwjVsR+M=; b=X5a7bcvJdv7iAzw8VRcuYtDStCuNRCVaRXlE86OfVwaeRdwQvcPudssoSyTTMb8+GJ dJmzFdG5tdfv+iql3iCCywGz6SCORy5F9biuyyS4qSRxj8lmRpPPXbK0PqB1ym5gIAVo FyyfWzstFmH812fPbC5d6oISZXdSsjw0lY/HXQX/xSvMCRQADh/fuiDeJVmEICMnC8Rn pF4EtimdiRnASGyMvlZgXyJsgwz4VackzlA+eZ/omsPMzVHaZScLLuN+PIxHeDrrj18S 3o4fF/W9gzeh0mrmSsTBQzBC28UBVFJRTxr0WoC5CuCKw2YFdIiGqEOrAG1ahHNeDSyl i6mg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=fz16DA8E; dkim=pass header.i=@codeaurora.org header.s=default header.b=JSns9j/f; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h8si5150422pfi.117.2018.03.16.02.27.18; Fri, 16 Mar 2018 02:27:33 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=fz16DA8E; dkim=pass header.i=@codeaurora.org header.s=default header.b=JSns9j/f; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751571AbeCPJ0Y (ORCPT + 99 others); Fri, 16 Mar 2018 05:26:24 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:35710 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750847AbeCPJ0W (ORCPT ); Fri, 16 Mar 2018 05:26:22 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id E786C60C66; Fri, 16 Mar 2018 09:26:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1521192381; bh=aFrQO9lgG2FCcgBVml0+m8UBLpCXr4A6s8sf3Lq/SHc=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=fz16DA8E79hT/0fzp9dqMY80ORYnLLjU0MsVYXW9m6lNzafhQLYFswwJG9EkJrMbI upEVWYFSuDmp+fQmnlnanPCFJk93SsFn0xwv4WlP5QnYOUeYBrrD4lHM254PsfncRA 3itatemrd6dUDPrY/0+STxADQ1nXrMW6DxbdoIVc= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED,T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0 Received: from [192.168.1.84] (unknown [182.71.117.114]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: architt@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 41CC160390; Fri, 16 Mar 2018 09:26:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1521192380; bh=aFrQO9lgG2FCcgBVml0+m8UBLpCXr4A6s8sf3Lq/SHc=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=JSns9j/fdOk8NRCQZx2G8l2f/wevNlKm6CEXIxSWK2/5J/I/XUG55a/lk6n817ZAE JlZP+++JLfWo3qReOHp3AwOtrDPZhrg1UrCNcK9U5CzcSfthhSoSSgSXtfyG0xoQir JLvX0D+SKMpIucC3BVMuCwOg7GZ5DUr6FVVgT7Rc= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 41CC160390 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=architt@codeaurora.org Subject: Re: [PATCH 1/2] dt-bindings: analogix-dp: Add backlight-pwm-passthru To: Alexandru M Stan , David Airlie , Rob Herring , Mark Rutland , Andrzej Hajda , Laurent Pinchart , Sean Paul Cc: devicetree@vger.kernel.org, Enric Balletbo i Serra , Heiko Stuebner , briannorris@chromium.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, dianders@chromium.org, linux-rockchip@lists.infradead.org, marcheu@chromium.org, hoegsberg@chromium.org, Thierry Escande , Jeffy Chen References: <20180316025659.217423-1-amstan@chromium.org> <20180316025659.217423-2-amstan@chromium.org> From: Archit Taneja Message-ID: <124034c6-a0af-0142-0d04-7d8bbc19bb82@codeaurora.org> Date: Fri, 16 Mar 2018 14:56:09 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <20180316025659.217423-2-amstan@chromium.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 16 March 2018 08:26 AM, Alexandru M Stan wrote: > Documentation for the optional backlight-pwm-passthru property. > Tells the EDP panel to folow the input pwm frequency instead s/folow/follow It would be nice if we could add the details you mentioned in patch #0 in either this or the next patch. > of generating its own. This is one of those bindings which is more a knob than a HW property, but I can't think of any easy way to figure this out in SW. So, I guess it's okay to have. One thing I was wondering about was whether this prop should belong to the eDP controller or the eDP panel. I don't have any strong opinion about it, though. Reviewed-by: Archit Taneja Thanks, Archit > > Signed-off-by: Alexandru M Stan > --- > > Documentation/devicetree/bindings/display/bridge/analogix_dp.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt b/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt > index 0c7473dd0e51..3c15242f6ce3 100644 > --- a/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt > +++ b/Documentation/devicetree/bindings/display/bridge/analogix_dp.txt > @@ -23,6 +23,10 @@ Required properties for dp-controller: > from general PHY binding: Should be "dp". > > Optional properties for dp-controller: > + -backlight-pwm-passthru: > + Directly pass the PWM frequency applied to the BL_PWM_DIM > + pin to the backlight current source. Done via > + EDP_BACKLIGHT_MODE_SET_REGISTER on DPCD. > -force-hpd: > Indicate driver need force hpd when hpd detect failed, this > is used for some eDP screen which don't have hpd signal. >