Hi, On 14 June 2018 at 07:13, Manivannan Sadhasivam <manivannan.sadhasi...@linaro.org> wrote: > Hi Simon, > > On Thu, Jun 14, 2018 at 06:58:40AM -0600, Simon Glass wrote: >> Hi, >> >> On 12 June 2018 at 22:15, Manivannan Sadhasivam >> <manivannan.sadhasi...@linaro.org> wrote: >> > This commit adds Actions Semi OWL family base clock and S900 SoC specific >> > clock support. For S900 peripheral clock support, only UART clock has been >> > added for now. >> > >> > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasi...@linaro.org> >> > --- >> > arch/arm/include/asm/arch-owl/clk_s900.h | 57 +++++++++ >> > arch/arm/include/asm/arch-owl/regs_s900.h | 64 ++++++++++ >> > drivers/clk/Kconfig | 1 + >> > drivers/clk/Makefile | 1 + >> > drivers/clk/owl/Kconfig | 12 ++ >> > drivers/clk/owl/Makefile | 3 + >> > drivers/clk/owl/clk_s900.c | 138 ++++++++++++++++++++++ >> > 7 files changed, 276 insertions(+) >> > create mode 100644 arch/arm/include/asm/arch-owl/clk_s900.h >> > create mode 100644 arch/arm/include/asm/arch-owl/regs_s900.h >> > create mode 100644 drivers/clk/owl/Kconfig >> > create mode 100644 drivers/clk/owl/Makefile >> > create mode 100644 drivers/clk/owl/clk_s900.c >> >> This says v2 but I don't see a change log. Have you tried using >> patman? It does this for you easily. >> > > Sorry, I have added the changelog in coverletter of the patchseries! Is it a > bad practice in U-Boot?
I'm not sure about bad practice, but without it, it's hard to know what you've changed in this patch. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot