Quoting Codrin Ciubotariu (2018-06-04 01:20:29)

On 31.05.2018 18:26, Stephen Boyd wrote:

quoted

Quoting Codrin Ciubotariu (2018-05-25 05:34:23)

quoted

+       .get_parent = clk_i2s_mux_get_parent,
+       .set_parent = clk_i2s_mux_set_parent,
+       .determine_rate = __clk_mux_determine_rate,
+};
+
+static struct clk_hw * __init
+at91_clk_i2s_mux_register(struct regmap *regmap, const char *name,
+                         const char * const *parent_names,
+                         unsigned int num_parents, u32 bus_id)
+{
+       struct clk_init_data init = {};
+       struct clk_i2s_mux *i2s_ck;
+       int ret;
+
+       i2s_ck = kzalloc(sizeof(*i2s_ck), GFP_KERNEL);
+       if (!i2s_ck)
+               return ERR_PTR(-ENOMEM);
+
+       init.name = name;
+       init.ops = &clk_i2s_mux_ops;
+       init.parent_names = parent_names;
+       init.num_parents = num_parents;
+       init.flags = CLK_IGNORE_UNUSED;

Really? Why?

I am thinking that there is no need to gate this clock, since there is 
no way to gate this clock in HW.

This flag is not necessary if the clk can't be gated via hardware
control registers. Please remove the flag.