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

From: Stephen Boyd
Date: Mon May 09 2016 - 18:13:46 EST


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@xxxxxxxxxxxx>
> > > ---
> >
> > 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