One more thing: On Tue, Sep 08, 2015 at 05:49:54PM +0800, Bayi Cheng wrote: > +- clocks: spi nor source clock ^^ you only list one clock here > +- clock-names: "spi_clk", "axi_clk", "mux_clk", "sf_clk" But you have 4 names here. ... > + clocks = <&pericfg CLK_PERI_SPI>, > + <&topckgen CLK_TOP_AXI_SEL>, > + <&topckgen CLK_TOP_UNIVPLL2_D8>, > + <&topckgen CLK_TOP_SPINFI_IFR_SEL>; And you provide 4 clocks. > + clock-names = "spi_clk", "axi_clk", "mux_clk", "sf_clk"; Please list all 4 under the "clocks" property, not just under the "clock-names" property. Brian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html