lkml.org 
[lkml]   [2018]   [Feb]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 05/19] clk: meson: add regmap clocks
From
Date
On Thu, 2018-02-08 at 15:33 +0800, Yixun Lan wrote:
> > +/**
> > + * struct clk_regmap_mux_data - regmap backed multiplexer clock specific data
> > + *
> > + * @hw: handle between common and hardware-specific interfaces
> > + * @offset: offset of theregister controlling multiplexer
> > + * @table: array of parent indexed register values
> > + * @shift: shift to multiplexer bit field
> > + * @width: width of mutliplexer bit field
>
> ~~~~~~ this is wrong, please update to keep it sync with the
> struct definition

Indeed, it is.
This is just a copy/paste of the mux documentation provided in
include/linux/clk-provider.h ... which is also inaccurate.

Thanks for pointing it out

>
> > + * @flags: hardware-specific flags
> > + *
> > + * Flags:
> > + * Same as clk_divider except CLK_MUX_HIWORD_MASK which is ignored
> > + */
> > +struct clk_regmap_mux_data {
> > + unsigned int offset;
> > + u32 *table;
> > + u32 mask;
>
> ~~~~~ here
> > + u8 shift;
> > + u8 flags;
> > +};
> > +


\
 
 \ /
  Last update: 2018-02-08 09:07    [W:0.066 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site