From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jisheng Zhang Subject: Re: [PATCH 3/3] clk: berlin: bg2q: remove non-exist "smemc" gate clock Date: Wed, 7 Jan 2015 22:22:09 +0800 Message-ID: <20150107222209.64ed24a0@xhacker> References: <1420016272-6789-1-git-send-email-jszhang@marvell.com> <1420016272-6789-4-git-send-email-jszhang@marvell.com> <54AD3EAE.6090009@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <54AD3EAE.6090009-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sebastian Hesselbarth Cc: "mturquette-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , "sboyd-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org" , "alexandre.belloni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org" , "antoine.tenart-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org" , "robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" , "pawel.moll-5wv7dgnIgG8@public.gmane.org" , "mark.rutland-5wv7dgnIgG8@public.gmane.org" , "ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org" , "galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org" , "linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org Dear Sebastian, On Wed, 7 Jan 2015 06:11:58 -0800 Sebastian Hesselbarth wrote: > On 31.12.2014 09:57, Jisheng Zhang wrote: > > The "smemc" clock is removed on BG2Q SoCs. In fact, bit19 of clkenable > > register is for nfc. Current code use bit19 for non-exist "smemc" > > incorrectly, this prevents eMMC from working due to the sdhci's > > "core" clk is still gated. > > > > Signed-off-by: Jisheng Zhang > > Cc: stable-u79uwXL29TY76Z2rM5mHXA@public.gmane.org # 3.16+ > > --- > > drivers/clk/berlin/bg2q.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/clk/berlin/bg2q.c b/drivers/clk/berlin/bg2q.c > > index 21784e4..440ef81 100644 > > --- a/drivers/clk/berlin/bg2q.c > > +++ b/drivers/clk/berlin/bg2q.c > > @@ -285,7 +285,6 @@ static const struct berlin2_gate_data bg2q_gates[] > > __initconst = { { "pbridge", "perif", 15, > > CLK_IGNORE_UNUSED }, { "sdio", "perif", 16, > > CLK_IGNORE_UNUSED }, { "nfc", "perif", 18 }, The nfc here is really confusing, we call it as nfccore internally. Is it better to rename it as nfccore? > > - { "smemc", "perif", 19 }, > > Jisheng, > > if bit 19 is for nfc, how does that work out with bit 18 which is > still assigned to nfc? Can you re-evaluate clkenable registers for bit 19 is for nfcEcc, the "io" clock; bit 18 is for nfcCore, the "core" clock. > BG2Q and fix it up accordingly? I'd suggest to still disable as many I'll recheck the clk driver for BG2Q. Thanks very much, Jisheng -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html