Hi Philippe,
On Tue, Feb 20, 2018 at 08:55:57AM +0100, Philippe Ombredanne wrote:

Manivannan,

On Sat, Feb 17, 2018 at 9:44 PM, Manivannan Sadhasivam
[off-list ref] wrote:

quoted

Add gpio driver for Actions Semi OWL family S900 SoC. Set of registers
controlling the gpio shares the same register range with pinctrl block.

GPIO registers are organized as 6 banks and each bank controls the
maximum of 32 gpios.

Signed-off-by: Manivannan Sadhasivam <redacted>

<snip>

quoted

--- /dev/null
+++ b/drivers/gpio/gpio-owl.c

@@ -0,0 +1,258 @@
+// SPDX-License-Identifier: GPL-2.0+

<snip>

quoted

+MODULE_LICENSE("GPL v2");

Please sync this with your SPDX id above. nodule.h has the doc on the
meaning of MODULE_LICENSE macros.
Here you have stated a combo of GPL-2.0 and GPL-2.0 or later

Sorry. It should be MODULE_LICENSE("GPL"). Will get it fixed in next revision.

Thanks,
Mani

-- 
Cordially
Philippe Ombredanne