Hi Mani, Am 01.11.2017 um 03:54 schrieb Manivannan Sadhasivam: > This series adds clock driver for Actions Semi OWL series > S900 SoC with relevant clock bindings and device tree data. > > This series also addresses the review comments from previous > submission happened last year. > > https://patchwork.kernel.org/patch/9254471/ Interesting - I had searched for any S500/S900 patches before starting to work on this but came up empty on LAKML. It seems you missed at least one review comment from 2016 about the example indentation, so please double-check for anything Stephen already pointed out. > Driver has been validated on Bubblegum-96 board. > > Thanks, > Mani > > Manivannan Sadhasivam (3): > arm64: dts: actions: add s900 clock controller nodes > clk: owl: add clock driver for Actions S900 SoC > Documentation: add Actions S900 clock bindings Please reverse the patch order. Have you checked S500 code whether the S900 vs. common split is correct? Many thanks for your work on this, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- 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