Re: [PATCH v4 06/15] clk: Add Ingenic jz4770 CGU driver

From: Paul Cercueil
Date: Fri Dec 29 2017 - 10:02:38 EST



Dear Mr. Ombredanne,

Le ven. 29 déc. 2017 à 13:55, Philippe Ombredanne <pombredanne@xxxxxxxx> a écrit :
Dear Mr Crapouillou-Cercueil-Sir,

On Thu, Dec 28, 2017 at 2:56 PM, Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote:
Add support for the clocks provided by the CGU in the Ingenic JZ4770
SoC.

<snip>

--- /dev/null
+++ b/drivers/clk/ingenic/jz4770-cgu.c
@@ -0,0 +1,487 @@
+/*
+ * JZ4770 SoC CGU driver
+ *
+ * Copyright 2017, Paul Cercueil <paul@xxxxxxxxxxxxxxx>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 or later
+ * as published by the Free Software Foundation.
+ */

Do you mind using a simpler one-line SPDX identifier instead of this
fine but clearly crapouillish legalese boilerplate? Unless you are
trying to turn the kernel in a legal compendium, of course ;)

This is documented in Thomas doc patches. This would apply to your
entire patch set.
Thank you for your kind consideration!

Sure, I wasn't aware about that new SPDX identifier thing. I'll do it in V5.

Regards,
-Paul