From patchwork Tue May 4 05:28:40 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shawn Guo X-Patchwork-Id: 430796 Delivered-To: patch@linaro.org Received: by 2002:a02:c901:0:0:0:0:0 with SMTP id t1csp3474848jao; Mon, 3 May 2021 22:29:05 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxLpr/vQIoaDBifTe6e7KdRFjuwYgR31PbYQyUAQwo4btA2lfzAjPoaz2ZhU0fUOuTs+Eeq X-Received: by 2002:a17:90a:bf0c:: with SMTP id c12mr10249281pjs.206.1620106145498; Mon, 03 May 2021 22:29:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620106145; cv=none; d=google.com; s=arc-20160816; b=Wx3Ggq70ESomqQuuWjNGzDfAQml+AYzA9/n9/pODbhFr4Vvyjv5hhtIP+7onHPlq7/ bDAs2s5UsNdo1hf+SqCHNMr151iI0maSbX9+P9WX6IK6nVMP0MuFQMLcEmdLdkejwCDX T2ivVeVoDQiLqQQa2NiywSC/CAO24VHpKmFWk6pRp1PH+oZ8CNKBQgma8IdLl50/DXbj EYXi89A3N+/l7q1SM/qQ0vaxyiyE+mIVi2uC3cYPDCR/UA9t23lx6HENqq4XKmzwgxmD tt+QYcNw0A1ytrErgY6wTRlUQQgL1ylNhVJdmTIBVrnb3Qx1Wm5ngYljVUt2S12d5479 3vKQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:references:in-reply-to:message-id:date:subject :cc:to:from:dkim-signature; bh=ts19LcXzJnm2OK0dKlHI64T4AANtdk+C1elhddnhZ4k=; b=STfLdBehgfhR6H0+ao4fQbfc5QzRdGxkidz/Wg6ebcVAeKD8Du8ce52LW+Z2UzfFWT FN2Z/QxiORhfOAvm4y/hCmySE0HDsJ6vGoB9f9RA3kn6CSpzwEffOwGfEiYj+B/9ygWn Svn2sY7khvcMR5AhpXjogFzmw9oeJwjxWCutimyQfgkFU0GL2yNfrSbVn0W+ypRLk3Hw aoxPSHS0ced4xY1IF3AFKqZbDrckEl60jWUvegj6+Eye2TWaz/fZ5aNCOgByHNzhPL+9 2CK2tPBI7JDMJLaKcwyJ+e3LwNr6EPDXJoWgDWKwBzDgpcCymcKGHYQ8khHJ5FWEEwQk hwfQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=TSRMvon4; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x7si2109914pgp.564.2021.05.03.22.29.05; Mon, 03 May 2021 22:29:05 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=TSRMvon4; spf=pass (google.com: domain of linux-arm-msm-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229773AbhEDF3v (ORCPT + 17 others); Tue, 4 May 2021 01:29:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50246 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229746AbhEDF3v (ORCPT ); Tue, 4 May 2021 01:29:51 -0400 Received: from mail-pf1-x431.google.com (mail-pf1-x431.google.com [IPv6:2607:f8b0:4864:20::431]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2E181C061574 for ; Mon, 3 May 2021 22:28:57 -0700 (PDT) Received: by mail-pf1-x431.google.com with SMTP id e15so6276549pfv.10 for ; Mon, 03 May 2021 22:28:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ts19LcXzJnm2OK0dKlHI64T4AANtdk+C1elhddnhZ4k=; b=TSRMvon4FdD8NHLI1+656d3UAALnt4POSawOiyDfsw8RDog22qyaig24zVhKgU6Wbu I29fbfqm6Q9sJ1PWouIm+m+Dy2hgxOaXdzysEXWj3o8tTXn9My7oyVirKEtAbsAH3rpR XOehcTuE6yAgqffZgx0XoRDN/nSjCFOpWDo+d2TPdlgRPZrPa7Q7ZlFCbs6bhrX2UFHn A2n1KEYY3Vl43/dTFaAVSl1EOKW00l7uxcJawHnOHEktNmS6uG+wF/Hgr1ZB84TGsMjs hh/i3GzH84QTgcEldcynV0F8RTvFjdeoI5y83pjOJedJ8ek5bhamoq+mdN3wZGu9Of0q +adQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ts19LcXzJnm2OK0dKlHI64T4AANtdk+C1elhddnhZ4k=; b=RpSANh654hM0kF3+4TueDBZC3eoTuoGqDmCIbhjvNbKLnHuOYmdtyqltZQ3eWuCfyX yoiYnbWMFoEk8//bPt63Cry3ZNEsxHARN3G2ceQXoLoTBExsVPh4IPD90TcAFXXse/Fr VKmVRwnRMq4NzvbWDfeY6AB5wOi4cZ6QVPpIP0ViKeJTg63igsNy9i/hgCgrG4uZfOV3 ZItMoX0dnxJ6zsS9RjkHe/NpNe0wdEtQRVKC2FxGp2ju50ejutIuKMvRReL/AaPes9Ym 529urb8SFA4CrzrOXNvtOFx9Sb3CmplJ0718qfTj59I3t7mMRz6xUAAgbHRCi5sioUaq YeOw== X-Gm-Message-State: AOAM5307Qdt8HZTHR/t+f7YuWh7z3nnd926KH/7OKBwOp1ILEXp0q5yn 2dtVEpOrwAGkpb1HAWfsEED0Og== X-Received: by 2002:a17:90a:1782:: with SMTP id q2mr3457058pja.73.1620106136731; Mon, 03 May 2021 22:28:56 -0700 (PDT) Received: from localhost.localdomain (80.251.214.228.16clouds.com. [80.251.214.228]) by smtp.gmail.com with ESMTPSA id 145sm5933229pfv.196.2021.05.03.22.28.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 03 May 2021 22:28:56 -0700 (PDT) From: Shawn Guo To: Stephen Boyd Cc: Bjorn Andersson , Rob Herring , Sivaprakash Murugesan , Benjamin Li , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, Shawn Guo Subject: [PATCH 1/5] clk: qcom: apcs-msm8916: Flag a53mux instead of a53pll as critical Date: Tue, 4 May 2021 13:28:40 +0800 Message-Id: <20210504052844.21096-2-shawn.guo@linaro.org> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210504052844.21096-1-shawn.guo@linaro.org> References: <20210504052844.21096-1-shawn.guo@linaro.org> Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org The clock source for MSM8916 cpu cores is like below. |\ a53pll --------| \ a53mux +------+ | |------------| cpus | gpll0_vote --------| / +------+ |/ So clock a53mux rather than a53pll is actually the clock source of cpu cores. It makes more sense to flag a53mux rather than a53pll as critical, since a53pll could be irrelevant if a53mux switches its parent clock to be gpll0_vote. Signed-off-by: Shawn Guo --- drivers/clk/qcom/a53-pll.c | 1 - drivers/clk/qcom/apcs-msm8916.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) -- 2.17.1 diff --git a/drivers/clk/qcom/a53-pll.c b/drivers/clk/qcom/a53-pll.c index 45cfc57bff92..8614b0b0e82c 100644 --- a/drivers/clk/qcom/a53-pll.c +++ b/drivers/clk/qcom/a53-pll.c @@ -70,7 +70,6 @@ static int qcom_a53pll_probe(struct platform_device *pdev) init.parent_names = (const char *[]){ "xo" }; init.num_parents = 1; init.ops = &clk_pll_sr2_ops; - init.flags = CLK_IS_CRITICAL; pll->clkr.hw.init = &init; ret = devm_clk_register_regmap(dev, &pll->clkr); diff --git a/drivers/clk/qcom/apcs-msm8916.c b/drivers/clk/qcom/apcs-msm8916.c index cf69a97d0439..d7ac6d6b15b6 100644 --- a/drivers/clk/qcom/apcs-msm8916.c +++ b/drivers/clk/qcom/apcs-msm8916.c @@ -65,7 +65,7 @@ static int qcom_apcs_msm8916_clk_probe(struct platform_device *pdev) init.parent_data = pdata; init.num_parents = ARRAY_SIZE(pdata); init.ops = &clk_regmap_mux_div_ops; - init.flags = CLK_SET_RATE_PARENT; + init.flags = CLK_IS_CRITICAL | CLK_SET_RATE_PARENT; a53cc->clkr.hw.init = &init; a53cc->clkr.regmap = regmap;