Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp5366307ybi; Tue, 4 Jun 2019 05:42:57 -0700 (PDT) X-Google-Smtp-Source: APXvYqzfNLgOf2goAP0HteeZDhlto+74RKzatq54euLOMbyZZ4vnMlAbAyZRP6TqCtQW+kpicoUn X-Received: by 2002:a17:90a:8d0d:: with SMTP id c13mr18336403pjo.137.1559652177158; Tue, 04 Jun 2019 05:42:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1559652177; cv=none; d=google.com; s=arc-20160816; b=t3IBnfPfgYPqUs6vJUu6Shz5kp2OcDPqkT4Ylx0UkTIy3YZphBgUsoY+zzvt+zwcjt W33afc67zwZJ7EY6Vgvbvk20nTmGg1eGU816gWYv2C+hni6+PxMUoCsVT72VYks1n5to L0aBDZjhCM0MfaWN6QCAEets/Zl0Bt0aAdGvnlapVwY7MZgHl5C1Mzey8sJ91n0+Bnux k1CZ5Wq4OWRF6rI6HBGmH+jqcUvYa2h9g6l/k24lndH0UyDWvYUykDVgUKnOFI6Psxl9 cyb4ZICnTp5OWaHpiEQEoL6fHbxlnQyTvjCs1FIby774f3Cw5X/7527s96FgCg6zQ4DI WMAg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:dkim-signature:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=eYlU/EiszkB9G/gXKiyqydlxvhzbz9Z6m/D0NdsuBOk=; b=ZrDJXzqwb41y/ds1LamDJa9jJ6FRYHzupSXRU+tU95BKQwU3aMgHxZNf1b6KlHK+Ys ksJruu7bJ+6zMgLr9Smw50/l04jS5tpef1c+q2HvR+fOz8ZyYtmb0sN2OWnE0spQ1RFX EGgOpcqRc5vVyIeDUMxZPNnZofnDhTvHYbQfl629ozqmtxOQW2L2vFCoATD18P9XGOv4 BcsjOapx2wzQEUIo6BtZFKbVIK47D9iI6yYOVAhR/vb6HUsl5JiCld7/YbFj8s/b/bsr 29EFoP5cEzwQMEiIhwPVZmUrIp9TYSKjRAR1RNPHZDHBVqXe0IdcJDefmgc4+2PpWrPi AFdg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b="esUE5NN/"; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p11si11830212plk.67.2019.06.04.05.42.39; Tue, 04 Jun 2019 05:42:57 -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=@nvidia.com header.s=n1 header.b="esUE5NN/"; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727785AbfFDMlY (ORCPT + 99 others); Tue, 4 Jun 2019 08:41:24 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:15574 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727394AbfFDMlY (ORCPT ); Tue, 4 Jun 2019 08:41:24 -0400 Received: from hqpgpgate101.nvidia.com (Not Verified[216.228.121.13]) by hqemgate15.nvidia.com (using TLS: TLSv1.2, DES-CBC3-SHA) id ; Tue, 04 Jun 2019 05:41:10 -0700 Received: from hqmail.nvidia.com ([172.20.161.6]) by hqpgpgate101.nvidia.com (PGP Universal service); Tue, 04 Jun 2019 05:41:23 -0700 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Tue, 04 Jun 2019 05:41:23 -0700 Received: from tbergstrom-lnx.Nvidia.com (10.124.1.5) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 4 Jun 2019 12:41:22 +0000 Received: by tbergstrom-lnx.Nvidia.com (Postfix, from userid 1000) id 5CB9740609; Tue, 4 Jun 2019 15:41:20 +0300 (EEST) Date: Tue, 4 Jun 2019 15:41:20 +0300 From: Peter De Schrijver To: Sowjanya Komatineni CC: , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH V2 06/12] clk: tegra: add suspend resume support for DFLL clock Message-ID: <20190604124120.GD29894@pdeschrijver-desktop.Nvidia.com> References: <1559084936-4610-1-git-send-email-skomatineni@nvidia.com> <1559084936-4610-7-git-send-email-skomatineni@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <1559084936-4610-7-git-send-email-skomatineni@nvidia.com> X-NVConfidentiality: public User-Agent: Mutt/1.9.4 (2018-02-28) X-Originating-IP: [10.124.1.5] X-ClientProxiedBy: HQMAIL105.nvidia.com (172.20.187.12) To HQMAIL107.nvidia.com (172.20.187.13) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1559652070; bh=eYlU/EiszkB9G/gXKiyqydlxvhzbz9Z6m/D0NdsuBOk=; h=X-PGP-Universal:Date:From:To:CC:Subject:Message-ID:References: MIME-Version:Content-Type:Content-Disposition:In-Reply-To: X-NVConfidentiality:User-Agent:X-Originating-IP:X-ClientProxiedBy; b=esUE5NN/po/BVVuGgXZj6/xY8iQFNYfEOb8bOaIfgqDg7+xR5PSCikiNVn22FYDuE p1+7DJQUheTr5EkYYqL2NyeXrCyncRGRZhXe1jGDl63d7hFLjGWFxVtpmsQzFDk4PE yWr2NmC3oSdqPZOkJqOTaD1+vv02ASYmqOOnLs6cgfBJBoEm+BabGSTnrv2BrXmACP XrJ++f/Kp7mLLxVDO/gz1QOFscuSUJo/H2zgiPpCO/vNF3h3VDWSUVmODpmgwShoV9 i0G/wo0avsjzadfiIXVRNB6admgk7KcyKPj53CWF7oZxukhfPjsrIGWaIsWgb22gZo hEU26H/y00CRg== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 28, 2019 at 04:08:50PM -0700, Sowjanya Komatineni wrote: > This patch adds support for suspend and resume for DFLL clock. > > Signed-off-by: Sowjanya Komatineni > --- > drivers/clk/tegra/clk-dfll.c | 82 ++++++++++++++++++++++++++++++++++++++++++++ > drivers/clk/tegra/clk-dfll.h | 2 ++ > 2 files changed, 84 insertions(+) > > diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c > index 1fc71baae13b..d92a5a05fbbc 100644 > --- a/drivers/clk/tegra/clk-dfll.c > +++ b/drivers/clk/tegra/clk-dfll.c > @@ -286,6 +286,7 @@ struct tegra_dfll { > unsigned long dvco_rate_min; > > enum dfll_ctrl_mode mode; > + enum dfll_ctrl_mode resume_mode; > enum dfll_tune_range tune_range; > struct dentry *debugfs_dir; > struct clk_hw dfll_clk_hw; > @@ -1873,6 +1874,87 @@ static int dfll_fetch_common_params(struct tegra_dfll *td) > } > > /* > + * tegra_dfll_suspend > + * @pdev: DFLL instance > + * > + * dfll controls clock/voltage to other devices, including CPU. Therefore, > + * dfll driver pm suspend callback does not stop cl-dvfs operations. It is > + * only used to enforce cold voltage limit, since SoC may cool down during > + * suspend without waking up. The correct temperature zone after suspend will > + * be updated via dfll cooling device interface during resume of temperature > + * sensor. Temperature dependent cl-dvfs is not yet implemented in upstream, so leave out the part about cold voltage limits and temperature zones. Peter.