Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1396174 > unrolled thread

Re: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701 subsystem clocks

Started byStephen Boyd <sboyd@codeaurora.org>
First post2016-05-07 01:10 +0200
Last post2016-05-10 00:20 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701  subsystem clocks Stephen Boyd <sboyd@codeaurora.org> - 2016-05-07 01:10 +0200
    Re: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701  subsystem clocks James Liao <jamesjj.liao@mediatek.com> - 2016-05-09 07:40 +0200
      Re: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701  subsystem clocks Stephen Boyd <sboyd@codeaurora.org> - 2016-05-10 00:20 +0200

#1396174 — Re: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701 subsystem clocks

FromStephen Boyd <sboyd@codeaurora.org>
Date2016-05-07 01:10 +0200
SubjectRe: [PATCH v7 8/9] clk: mediatek: Add config options for MT2701 subsystem clocks
Message-ID<rvX2G-64J-3@gated-at.bofh.it>
On 04/14, James Liao wrote:
> MT2701 subsystem clocks are optional and should be enabled only if
> their subsystem drivers are ready to control these clocks.
> 
> Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> ---

Why is this patch split off from the patch that introduces the
file?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [next] | [standalone]


#1396594

FromJames Liao <jamesjj.liao@mediatek.com>
Date2016-05-09 07:40 +0200
Message-ID<rwM5b-6dE-1@gated-at.bofh.it>
In reply to#1396174
HI Stephen,

On Fri, 2016-05-06 at 16:02 -0700, Stephen Boyd wrote:
> On 04/14, James Liao wrote:
> > MT2701 subsystem clocks are optional and should be enabled only if
> > their subsystem drivers are ready to control these clocks.
> > 
> > Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> > ---
> 
> Why is this patch split off from the patch that introduces the
> file?

I was looking for comments about how to make subsystem clocks optional.
So I used a separated patch to do it. Is it an acceptable way to use
config options to enable subsystem clock support?


Best regards,

James

[toc] | [prev] | [next] | [standalone]


#1397467

FromStephen Boyd <sboyd@codeaurora.org>
Date2016-05-10 00:20 +0200
Message-ID<rx1GX-6f0-33@gated-at.bofh.it>
In reply to#1396594
On 05/09, James Liao wrote:
> HI Stephen,
> 
> On Fri, 2016-05-06 at 16:02 -0700, Stephen Boyd wrote:
> > On 04/14, James Liao wrote:
> > > MT2701 subsystem clocks are optional and should be enabled only if
> > > their subsystem drivers are ready to control these clocks.
> > > 
> > > Signed-off-by: James Liao <jamesjj.liao@mediatek.com>
> > > ---
> > 
> > Why is this patch split off from the patch that introduces the
> > file?
> 
> I was looking for comments about how to make subsystem clocks optional.
> So I used a separated patch to do it. Is it an acceptable way to use
> config options to enable subsystem clock support?
> 

Yes config options are fine.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web