lkml.org 
[lkml]   [2013]   [Mar]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] clk: add table lookup to mux
On Wed, Mar 20, 2013 at 04:51:58PM +0100, Mike Turquette wrote:
> Quoting Peter De Schrijver (2013-03-20 02:49:57)
> > On Wed, Mar 20, 2013 at 12:51:10AM +0100, Mike Turquette wrote:
> > > Quoting Peter De Schrijver (2013-03-12 11:42:23)
> > > > diff --git a/include/linux/clk-private.h b/include/linux/clk-private.h
> > > > index 9c7f580..53d39c2 100644
> > > > --- a/include/linux/clk-private.h
> > > > +++ b/include/linux/clk-private.h
> > > > @@ -144,12 +144,13 @@ struct clk {
> > > >
> > > > #define DEFINE_CLK_MUX(_name, _parent_names, _parents, _flags, \
> > > > _reg, _shift, _width, \
> > > > - _mux_flags, _lock) \
> > > > + _mux_flags, _table, _lock) \
> > > > static struct clk _name; \
> > > > static struct clk_mux _name##_hw = { \
> > > > .hw = { \
> > > > .clk = &_name, \
> > > > }, \
> > > > + .table = _table, \
> > > > .reg = _reg, \
> > > > .shift = _shift, \
> > > > .width = _width, \
> > >
> > > This breaks OMAP horribly since OMAP already uses this macro. There are
> > > two options:
> > >
> > > 1) stop using statically initialized data and no longer use
> > > clk-private.h macros. I was under the impression that the tegra clock
> > > data no longer required this?
> >
> > Ok. We don't do that anymore. Maybe I just modified it out of habbit...
> >
>
> Cool. Will you send a V2 that drops the macro changes?
>

Yes. Coming soon :)

Cheers,

Peter.


\
 
 \ /
  Last update: 2013-03-21 11:21    [W:0.044 / U:0.824 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site