Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp3779644imm; Mon, 6 Aug 2018 10:26:36 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeNtFl9pTRwCvLFfaJW0gjq20kJKhgoYsACUNGKzWDDAEBEiFy5Ig5ChP0j+NNwzTPAve3N X-Received: by 2002:a63:d613:: with SMTP id q19-v6mr15203969pgg.327.1533576395986; Mon, 06 Aug 2018 10:26:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533576395; cv=none; d=google.com; s=arc-20160816; b=gg03KMKp2sJ2/0LPDvKgqJ51Ibj4RFpznEIB9odKRr5eQI02f7abI4I5Ppgeu0yUo/ jzl/l0sip8LyPJ7DjIp0DwhewKV1j8bohif/tlEO2ZKu9BX65Tc49dEsfGWktNhVjIVh 2o5q3JZ4S9BfZ5+Yp185LjPckIQ0rF++FGIygxeinB+/7KzBEFe+qXDS++QwZ+b8L73d SnPWFDxmApzdgLIhpxzMZZAJPRPhE9Q9nC/5i/f20BYp4oGHcLRSyGl7t+HSnEQPLsnG 3EHmmW5QQ6FkyFhsGUIoLcS2+cxJsVpj/wJg82ffcNMC/hTfkvRzlkUIJuDugyVsv0MV jJSA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:mail-followup-to :message-id:subject:cc:to:from:date:dmarc-filter:dkim-signature :dkim-signature:arc-authentication-results; bh=dWLHvnwhVD553SsIASh3H/f1jnwwcx0zj8CBCW1vsvo=; b=vhnej+0U4HLzEfpitu9VReOWB4Cs0UqsOO2ZsyM+s2+TpqSzWp5dEXpXXos/gtroYu oWY46i649l4WzfwfSrNS9ncwks49jxid6bfAyQ78wI2JHsViEuEu5Ol/cr+4wAwdzIjG p1VbyqHoCX0L2frBY4ZJEBQdAbv8s8Y6fJQjeoG8GoUCmAGxVCdi5jsUurDjOL7PYIds /5WHfyBMnP3dFTG5n4L+Qsh2nKwNc50uAHrTiyNLDbciS2OQ6VIcukrHfpSg1cuim2dh 4KW0o9jCuGti1A4i5FTyxELRA3X/MFu4jSD2Yh89LfFrdlk/6ixuBPQuSVIgUMLMULzH 7jaA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=XOwgtVfi; dkim=pass header.i=@codeaurora.org header.s=default header.b=XOwgtVfi; 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 d16-v6si14279030pfe.267.2018.08.06.10.26.21; Mon, 06 Aug 2018 10:26:35 -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=XOwgtVfi; dkim=pass header.i=@codeaurora.org header.s=default header.b=XOwgtVfi; 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 S1732609AbeHFRON (ORCPT + 99 others); Mon, 6 Aug 2018 13:14:13 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:45542 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732377AbeHFRON (ORCPT ); Mon, 6 Aug 2018 13:14:13 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id E8B2C60541; Mon, 6 Aug 2018 15:04:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1533567881; bh=Ks+M+x+/4l8vooNjxrJS767hdHiAjuZNtDgXwYpIyjg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XOwgtVfik3qtzLcxFO6PffoBnQJMpdHU2UVA6+8XxMUEp4zQ1CyjO5s/0Td0zGufi 69Y6WTX8SiTYG7BYx0DidlsdKDc3xlYAHnJlJemC+bqqLSeTq5lNgBosYcDyuLWvDK ijU0RdJ3rlE6GtUQzvY77prM5ZHyDgvKLbIP+2IQ= 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 jcrouse-lnx.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: jcrouse@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 498526037C; Mon, 6 Aug 2018 15:04:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1533567881; bh=Ks+M+x+/4l8vooNjxrJS767hdHiAjuZNtDgXwYpIyjg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XOwgtVfik3qtzLcxFO6PffoBnQJMpdHU2UVA6+8XxMUEp4zQ1CyjO5s/0Td0zGufi 69Y6WTX8SiTYG7BYx0DidlsdKDc3xlYAHnJlJemC+bqqLSeTq5lNgBosYcDyuLWvDK ijU0RdJ3rlE6GtUQzvY77prM5ZHyDgvKLbIP+2IQ= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 498526037C 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=jcrouse@codeaurora.org Date: Mon, 6 Aug 2018 09:04:37 -0600 From: Jordan Crouse To: Amit Nischal Cc: Stephen Boyd , Michael Turquette , Andy Gross , David Brown , Rajendra Nayak , Odelu Kukatla , Taniya Das , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk-owner@vger.kernel.org Subject: Re: [PATCH 2/4] clk: qcom: Add clk_rcg2_gfx3d_ops for SDM845 Message-ID: <20180806150437.GE21283@jcrouse-lnx.qualcomm.com> Mail-Followup-To: Amit Nischal , Stephen Boyd , Michael Turquette , Andy Gross , David Brown , Rajendra Nayak , Odelu Kukatla , Taniya Das , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk-owner@vger.kernel.org References: <1528285308-25477-1-git-send-email-anischal@codeaurora.org> <1528285308-25477-3-git-send-email-anischal@codeaurora.org> <153111693472.143105.11303543263643845656@swboyd.mtv.corp.google.com> <1e6d9fc284c3c118203728867f504ec6@codeaurora.org> <153250192252.48062.9210075387954345932@swboyd.mtv.corp.google.com> <07e0321116993d27d6585bd1a186328d@codeaurora.org> <153324986956.10763.5124619734269160725@swboyd.mtv.corp.google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 06, 2018 at 02:37:18PM +0530, Amit Nischal wrote: > On 2018-08-03 04:14, Stephen Boyd wrote: > >Quoting Amit Nischal (2018-07-30 04:28:56) > >>On 2018-07-25 12:28, Stephen Boyd wrote: > >>> > >>> Ok. Sounds good! Is the rate range call really needed? It can't be > >>> determined in the PLL code with some table or avoided by making sure > >>> GPU > >>> uses OPP table with only approved frequencies? > >>> > >> > >>Currently fabia PLL code does not have any table to check this and > >>intention > >>was to avoid relying on the client to call set_rate with only approved > >>frequencies so we have added the set_rate_range() call in the GPUCC > >>driver > >>in order to set the rate range. > >> > > > >But GPU will use OPP so it doesn't seem like it really buys us anything > >here. And it really doesn't matter when the clk driver implementation > >doesn't use the min/max to clamp the values of the round_rate() > >call. Is > >that being done here? I need to double check. I would be more convinced > >if the implementation was looking at min/max to constrain the rate > >requested. > > > > So our understanding is that GPU(client) driver will always call the > set_rate with approved frequencies only and we can completely rely > on the > client. Is our understanding is correct? First: on sdm845 the software doesn't set the GPU clocks - we rely on the GMU firmware to do that on our behalf but for the GPU at least this is an academic exercise. But that said: traditionally we've expected that the clock driver correctly clamp the requested rate to the correct values. In the past we have taken advantage of this and we may in the future. I don't think it is reasonable to require the leaf driver to only pass "approved" frequencies especially since we depend on our own OPP table that may or may not be similar to the one used by the clock driver. Jordan -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project