lkml.org 
[lkml]   [2020]   [Apr]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: R: [PATCH v2] ARM: qcom: Disable i2c device on gsbi4 for ipq806x
From
Date
Quoting ansuelsmth@gmail.com (2020-04-02 18:39:04)
>
>
> > -----Messaggio originale-----
> > Da: Stephen Boyd <sboyd@kernel.org>
> > Inviato: venerdì 3 aprile 2020 03:34
> > A: Andy Gross <agross@kernel.org>; Ansuel Smith
> > <ansuelsmth@gmail.com>
> > Cc: Ansuel Smith <ansuelsmth@gmail.com>; Mathieu Olivari
> > <mathieu@codeaurora.org>; Bjorn Andersson
> > <bjorn.andersson@linaro.org>; Rob Herring <robh+dt@kernel.org>; Mark
> > Rutland <mark.rutland@arm.com>; Michael Turquette
> > <mturquette@baylibre.com>; linux-arm-msm@vger.kernel.org;
> > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-
> > clk@vger.kernel.org
> > Oggetto: Re: [PATCH v2] ARM: qcom: Disable i2c device on gsbi4 for
> > ipq806x
> >
> > Quoting Ansuel Smith (2020-03-30 13:56:46)
> > > diff --git a/drivers/clk/qcom/gcc-ipq806x.c b/drivers/clk/qcom/gcc-
> > ipq806x.c
> > > index b0eee0903807..f7d7a2bc84c1 100644
> > > --- a/drivers/clk/qcom/gcc-ipq806x.c
> > > +++ b/drivers/clk/qcom/gcc-ipq806x.c
> > > @@ -991,6 +991,7 @@ static struct clk_branch gsbi4_h_clk = {
> > > .hw.init = &(struct clk_init_data){
> > > .name = "gsbi4_h_clk",
> > > .ops = &clk_branch_ops,
> > > + .flags = CLK_IGNORE_UNUSED,
> >
> > Is this necessary? Shouldn't we skip clks that are protected during the
> > unused phase?
> >
>
> gsbi4_h_clk is not protected. gsbi4_h_clk needs to not be disabled if unused
> (as it's used by rpm) but can't be protected since it's used by uart gsbi4.
> (With some test protecting also this clk cause the malfunction of uart gsb4)
>

Who owns gsbi4 on this platform? Is it RPM? If so, it should be
protected and we shouldn't touch this clk from the kernel.

\
 
 \ /
  Last update: 2020-04-05 05:11    [W:0.068 / U:0.188 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site