On 5/1/2015 11:12 PM, Jim Lin wrote: > >> +static void clk_plle_tegra210_is_enabled(struct struct clk_hw *hw) { > Returned type is "int" instead of "void". > Also one "struct" only for "clk_hw *hw"? > >> + struct tegra_clk_pll *pll = to_clk_pll(hw); >> + u32 val; >> + >> + val = pll_readl_base(pll); >> + >> + return val & PLLE_BASE_ENABLE ? 1 : 0; } >> + > --nvpublic > thanks! will fix. -rhyland -- nvpublic -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html