Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp2035754ybl; Sat, 7 Dec 2019 07:05:01 -0800 (PST) X-Google-Smtp-Source: APXvYqw+9wlheT1mTa6YjLnAfuMq/7suffWNaDkSXO2TAhA9JnuWPWQ0TYhhhcmOlr0nMR5Fj9qP X-Received: by 2002:a9d:f61:: with SMTP id 88mr11889990ott.234.1575731101458; Sat, 07 Dec 2019 07:05:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1575731101; cv=none; d=google.com; s=arc-20160816; b=i6QmlASTSLx8yzE0sdI9Ynbx1gEI5tXwDom38XUZ5gscDFXGBRo9Ry43Vgjx6u3J8Z Kjg2efX4pkZkTkQ2KtxvvJsXLZXJVEdDXKLQSD0RLSyWK7aShOGSxoo3wvYp685tfqDR B87FpmEOaVoxYXv9rzego1aB03eUlMssf4PT0Y9K2NjSydX3fR1Op/T9GOUCxGDxOoi8 lSpreOPdHiyf4OOQIc+/Uwjx3sTJo5h3vSksUfBUTniq3l82hnPW2f2zQzoZZBcXovmk mc4HEYYm7JhFFxDVVThJG4R0A+T2VRJMWlWLklZ1N+ySN3O5VA3ZLhwXIkIpVYy07ImZ R8Ng== 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:references:cc:to:from:subject:dkim-signature; bh=n8a+Wo0Uh6h7aFSS48D/6SstLsiUjh8g7OoKXGq8H00=; b=sMKgbOzR/GvvH1d3ZAuDeP5TdBzIOEHj2xrqkTwx0tH0Qwat0phOBQAiQOTqzpa7cA sWFyevBBScU9J5DnAWbbPWrLL0lf6L7QV5oLZMh3RavMilpWhCZQVjOt92hCJj1M9Lpy jw3vCqA0UUjALPwUumKOXiAjOkq+C3vykqWEUGxcCQems58mHiacrD9XwHkGw5HOFeVU VW+4UtzDW3JTVriCVF5v4t10bEZCtc5sk9/MM47kD8+k4nIor5TNiSvqdDtVtY6PlDqn yJVEHj/YG6yETYoBu/YbFQxxk9peT0/+gJRL7LWczY/H8XRNFERUr7jO2j/pZvmMBAz7 rX6Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=F6+K7JPq; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u20si8164061otg.193.2019.12.07.07.04.49; Sat, 07 Dec 2019 07:05:01 -0800 (PST) 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=@gmail.com header.s=20161025 header.b=F6+K7JPq; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726476AbfLGPET (ORCPT + 99 others); Sat, 7 Dec 2019 10:04:19 -0500 Received: from mail-lj1-f193.google.com ([209.85.208.193]:39919 "EHLO mail-lj1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726400AbfLGPET (ORCPT ); Sat, 7 Dec 2019 10:04:19 -0500 Received: by mail-lj1-f193.google.com with SMTP id e10so10798446ljj.6; Sat, 07 Dec 2019 07:04:17 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:from:to:cc:references:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=n8a+Wo0Uh6h7aFSS48D/6SstLsiUjh8g7OoKXGq8H00=; b=F6+K7JPqGvnQPCj28mwmw64tWflHmRmyDRI8MKlZXjoZzQqgZFfel3SZ4iaYF1AlRg se5iYaUtMXNNsTEwCwgnyVAZHCG6BoMefsmABDu8mCKuq44oE48Doj9GUhZ3RZNXZZsn Os2c9bNKrc4rUcmbifRJYrZags2606oCFJDSI8PgmOVo3lDwHUmPwvkw2emjqpxUmiQi asRj4d90tqVR976m0ftDXNrJPP/8dmeIe6JHHVdHru6kwcFXBSm0/NQtsIWpWnzlDNYI gUln+JQDYnkTtJjvWPat3QEQan9Ma7wsD7UKvsevW1SMhLQWs3/5cOXDVnEa+8azxRuS k5RQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=n8a+Wo0Uh6h7aFSS48D/6SstLsiUjh8g7OoKXGq8H00=; b=XMChmrqd+gfL/ckhGqKj5bR7mlXWQnoJq+hLWXoSyMN1jwkVDIfSr7yOurCrNjWYW7 LxbU2vEUEjJ4cgVfT0KKwulTY02DuhOcruh03gwZYm56dAYZYYcaF0IuUCmVW3AGb+3K bJl/XvsmRHYAHrAfx858naD4RhjjgGt5IJiRC6iuSqiv3D7zxEnzght86G6w9B8uujF3 kNpm1Ri9XDGdaT45Bv6eNNQTv/A6673Z+s7ZzILwh127R36tVu08YiTaPJiWAdmtDb23 YloLgrDjfG5nSI+BnNrQAYkcgJ2lTu16yVcrhIbi7ojGpS8mHvA22r7ZeBCPm8LSVNLD nVIg== X-Gm-Message-State: APjAAAUGzP9/QpbKXefUydOldTYUOGMyxO9T1EXjg36EQzdv4+uE+GrV asYGoxJ3JIDGjSDS3Yd5jjE= X-Received: by 2002:a2e:90c6:: with SMTP id o6mr11710121ljg.93.1575731056973; Sat, 07 Dec 2019 07:04:16 -0800 (PST) Received: from [192.168.2.145] (79-139-233-37.dynamic.spd-mgts.ru. [79.139.233.37]) by smtp.googlemail.com with ESMTPSA id z13sm8188887ljh.21.2019.12.07.07.04.15 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 07 Dec 2019 07:04:16 -0800 (PST) Subject: Re: [PATCH v3 06/15] clk: tegra: Remove tegra_pmc_clk_init along with clk ids From: Dmitry Osipenko To: Sowjanya Komatineni , thierry.reding@gmail.com, jonathanh@nvidia.com, mperttunen@nvidia.com, gregkh@linuxfoundation.org, sboyd@kernel.org, tglx@linutronix.de, robh+dt@kernel.org, mark.rutland@arm.com Cc: allison@lohutok.net, pdeschrijver@nvidia.com, pgaikwad@nvidia.com, mturquette@baylibre.com, horms+renesas@verge.net.au, Jisheng.Zhang@synaptics.com, krzk@kernel.org, arnd@arndb.de, spujar@nvidia.com, josephl@nvidia.com, vidyas@nvidia.com, daniel.lezcano@linaro.org, mmaddireddy@nvidia.com, markz@nvidia.com, devicetree@vger.kernel.org, linux-clk@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, alexios.zavras@intel.com, alsa-devel@alsa-project.org References: <1575600535-26877-1-git-send-email-skomatineni@nvidia.com> <1575600535-26877-7-git-send-email-skomatineni@nvidia.com> <3880aa15-c47a-5ab2-dd39-e8a47f6a3d6a@gmail.com> Message-ID: <5938df22-2474-3950-fc33-3e19cbf3da9c@gmail.com> Date: Sat, 7 Dec 2019 18:04:14 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 07.12.2019 17:43, Dmitry Osipenko пишет: > 07.12.2019 17:33, Dmitry Osipenko пишет: >> 06.12.2019 05:48, Sowjanya Komatineni пишет: >>> Current Tegra clock driver registers PMC clocks clk_out_1, clk_out_2, >>> clk_out_3 and blink output in tegra_pmc_init() which does direct Tegra >>> PMC access during clk_ops and these PMC register read and write access >>> will not happen when PMC is in secure mode. >>> >>> Any direct PMC register access from non-secure world will not go >>> through and all the PMC clocks and blink control are done in Tegra PMC >>> driver with PMC as clock provider. >>> >>> This patch removes tegra_pmc_clk_init along with corresponding clk ids >>> from Tegra clock driver. >>> >>> Signed-off-by: Sowjanya Komatineni >>> --- >> >> [snip] >> >>> @@ -1230,9 +1222,6 @@ static struct tegra_clk_init_table init_table[] __initdata = { >> >>> { TEGRA30_CLK_PLL_A, TEGRA30_CLK_CLK_MAX, 564480000, 1 }, >>> { TEGRA30_CLK_PLL_A_OUT0, TEGRA30_CLK_CLK_MAX, 11289600, 1 }, >>> { TEGRA30_CLK_EXTERN1, TEGRA30_CLK_PLL_A_OUT0, 0, 1 }, >> >> Perhaps these clocks do not need to be always-enabled? >> >> [snip] >> > > Also, EXTERN1 parent configuration should be moved to the audio > driver/device-tree as well. Ah, I missed that it's done in the patch #10. > Maybe it even makes sense to move the whole configuration, including > PLLA. I don't see why clk driver need to do something for the audio driver.