From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Zhang Subject: Re: [PATCH 2/5] clk: tegra: Add reset only clock node flag and COP Date: Thu, 8 Aug 2013 13:50:22 +0800 Message-ID: <5203319E.6050002@nvidia.com> References: <1375874709-10438-1-git-send-email-markz@nvidia.com> <1375874709-10438-2-git-send-email-markz@nvidia.com> <52027C9B.2030009@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <52027C9B.2030009-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: "mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , Peter De Schrijver , Prashant Gaikwad , Andrew Chew , Paul Walmsley , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: linux-tegra@vger.kernel.org Okay, I don't know these background infos. If so, there is no reason to upstream this kind of patches. On 08/08/2013 12:58 AM, Stephen Warren wrote: > On 08/07/2013 05:25 AM, Mark Zhang wrote: >> From: Peter De Schrijver >> >> COP is a reset only clock. So this patch adds NO_CLK support >> then adds the COP clock. > > Do we actually need this clock upstream yet? > > IIRC, Prashant was working on implementing the common reset API, so I'd > prefer not to add any reset-only "clocks" to the clock driver, but > rather to simply make sure that the new reset driver exposes them. > >> diff --git a/drivers/clk/tegra/clk-tegra114.c b/drivers/clk/tegra/clk-tegra114.c > >> enum tegra114_clk { >> - rtc = 4, timer = 5, uarta = 6, sdmmc2 = 9, i2s1 = 11, i2c1 = 12, >> + cop = 1, rtc = 4, timer = 5, uarta = 6, sdmmc2 = 9, i2s1 = 11, i2c1 = 12, > > To make this change, you would also need to edit > include/dt-bindings/clock/tegra114-car.h. BTW, why doesn't the Tegra > clock driver include that file rather than defining its own enum? > From mboxrd@z Thu Jan 1 00:00:00 1970 From: markz@nvidia.com (Mark Zhang) Date: Thu, 8 Aug 2013 13:50:22 +0800 Subject: [PATCH 2/5] clk: tegra: Add reset only clock node flag and COP In-Reply-To: <52027C9B.2030009@wwwdotorg.org> References: <1375874709-10438-1-git-send-email-markz@nvidia.com> <1375874709-10438-2-git-send-email-markz@nvidia.com> <52027C9B.2030009@wwwdotorg.org> Message-ID: <5203319E.6050002@nvidia.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Okay, I don't know these background infos. If so, there is no reason to upstream this kind of patches. On 08/08/2013 12:58 AM, Stephen Warren wrote: > On 08/07/2013 05:25 AM, Mark Zhang wrote: >> From: Peter De Schrijver >> >> COP is a reset only clock. So this patch adds NO_CLK support >> then adds the COP clock. > > Do we actually need this clock upstream yet? > > IIRC, Prashant was working on implementing the common reset API, so I'd > prefer not to add any reset-only "clocks" to the clock driver, but > rather to simply make sure that the new reset driver exposes them. > >> diff --git a/drivers/clk/tegra/clk-tegra114.c b/drivers/clk/tegra/clk-tegra114.c > >> enum tegra114_clk { >> - rtc = 4, timer = 5, uarta = 6, sdmmc2 = 9, i2s1 = 11, i2c1 = 12, >> + cop = 1, rtc = 4, timer = 5, uarta = 6, sdmmc2 = 9, i2s1 = 11, i2c1 = 12, > > To make this change, you would also need to edit > include/dt-bindings/clock/tegra114-car.h. BTW, why doesn't the Tegra > clock driver include that file rather than defining its own enum? >