On Mon, Feb 04, 2013 at 08:08:55AM +0100, Prashant Gaikwad wrote: > On Friday 01 February 2013 03:48 PM, Peter De Schrijver wrote: ... > > + /* xusb_hs_src */ > > + val = readl(clk_base + CLK_SOURCE_XUSB_SS_SRC); > > + val |= BIT(25); /* always select PLLU_60M */ > > + writel(val, clk_base + CLK_SOURCE_XUSB_SS_SRC); > > + > > + clk = clk_register_fixed_factor(NULL, "xusb_hs_src", "pll_u_60M", 0, > > + 1, 1); > > + clks[xusb_hs_src] = clk; > > + > > With device tree we can directly use pll_u_60M, no need to register > clock with fixed factor 1. > Same comment for dis1-fixed, dsi2-fixed and mipi-cal-fast clocks. > Does it make sense to have separate DT IDs at all then? Cheers, Peter. -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html