On Tue, Feb 11, 2014 at 11:37:09AM +0000, Mark Rutland wrote: > - pl022->clk = devm_clk_get(&adev->dev, NULL); > + /* > + * For compatibility with old DTBs and platform data, fall back to the > + * first clock if there's not an explicitly named "sspclk" entry. > + */ > + pl022->clk = devm_clk_get(&adev->dev, "sspclk"); > + if (IS_ERR(pl022->clk)) > + pl022->clk = devm_clk_get(&adev->dev, NULL); > + I'll just have a bit of a grumble here and point out that this sort of stuff always worries me with the convention of using nameless clocks - it causes hassle adding further clocks. In any case you didn't CC me on the cover letter or any of the non-SPI patches so I'm not sure what the dependencies are here (if there are any), does the series need to go in as one?