On Thu, Nov 9, 2017 at 2:18 AM, Chunyan Zhang <zhang.lyra@xxxxxxxxx> wrote: > Hi Rob and Stephen, > > On 7 November 2017 at 05:43, Rob Herring <robh@xxxxxxxxxx> wrote: >> On Thu, Nov 02, 2017 at 02:56:24PM +0800, Chunyan Zhang wrote: >>> This patch added the list of clocks for Spreadtrum's SC9860 SoC, >>> together with clock initialization code. >>> >>> Signed-off-by: Chunyan Zhang <chunyan.zhang@xxxxxxxxxxxxxx> >>> --- >>> drivers/clk/sprd/Kconfig | 10 + >>> drivers/clk/sprd/Makefile | 3 + >>> drivers/clk/sprd/sc9860-clk.c | 1987 +++++++++++++++++++++++++++ >>> include/dt-bindings/clock/sprd,sc9860-clk.h | 408 ++++++ >> >> This belongs in the binding doc patch. > > I noticed that this kind of files were submitted in driver patch > before when adding a new driver, how about I just put it into a single > patch in the next version? It's been done all ways, but the correct way is as I told you. Binding doc and header in one, driver in another patch. Rob -- 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
