On Fri, Oct 02, 2015 at 01:45:55PM -0700, Stephen Boyd wrote: > On 09/29, Maxime Ripard wrote: > > + * (at your option) any later version. > > + * > > + * This program is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + * GNU General Public License for more details. > > + */ > > + > > +#include <linux/clk-provider.h> > > +#include <linux/clkdev.h> > > Is this include used?
Fixed. > > + if (!gate) > > + goto err_free_mux; > > + > > + of_property_read_string(node, "clock-output-names", &clk_name); > > + > > + while (i < SUN4I_MOD1_MAX_PARENTS && > > + (parents[i] = of_clk_get_parent_name(node, i)) != NULL) > > Can we use of_clk_parent_fill() here? We can, I switched to it. Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
signature.asc
Description: Digital signature
