Re: [PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

Hi Paul, Thank you for the patch! Yet something to improve: [auto build test ERROR on tip/irq/core] [also build test ERROR on v4.15-rc6 next-20171222] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

Hi Paul, Thank you for the patch! Yet something to improve: [auto build test ERROR on tip/irq/core] [also build test ERROR on v4.15-rc6 next-20171222] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

[...] + tcu->clocks.clk_num = nb_clks; + tcu->clocks.clks = kcalloc(nb_clocks, sizeof(struct clk *), GFP_KERNEL); + if (!tcu->clocks.clks) { + pr_err("%s: cannot allocate memory\n", __func__); + goto err_free_tcu; + } Facepalm. A quick edit

Re: [PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

[...] + tcu->clocks.clk_num = nb_clks; + tcu->clocks.clks = kcalloc(nb_clocks, sizeof(struct clk *), GFP_KERNEL); + if (!tcu->clocks.clks) { + pr_err("%s: cannot allocate memory\n", __func__); + goto err_free_tcu; + } Facepalm. A quick edit

[PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

The TCU (Timer Counter Unit) of the Ingenic JZ47xx SoCs features 8 channels, each one having its own clock, that can be started and stopped, reparented, and reclocked. This driver only modifies the bits of the registers of the TCU that are related to clocks control. It provides one clock per TCU

[PATCH 4/6] clk: ingenic: Add JZ47xx TCU clocks driver

The TCU (Timer Counter Unit) of the Ingenic JZ47xx SoCs features 8 channels, each one having its own clock, that can be started and stopped, reparented, and reclocked. This driver only modifies the bits of the registers of the TCU that are related to clocks control. It provides one clock per TCU