On 5/8/2018 9:08 PM, Deucher, Alexander wrote: >> -----Original Message----- >> From: Agrawal, Akshu >> Sent: Tuesday, May 8, 2018 12:04 AM >> To: Deucher, Alexander <Alexander.Deucher@xxxxxxx> >> Cc: djkurtz@xxxxxxxxxxxx; mturquette@xxxxxxxxxxxx; sboyd@xxxxxxxxxx; >> Koenig, Christian <Christian.Koenig@xxxxxxx>; airlied@xxxxxxxxxx; Liu, >> Shaoyun <Shaoyun.Liu@xxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; linux- >> clk@xxxxxxxxxxxxxxx; rjw@xxxxxxxxxxxxx; lenb@xxxxxxxxxx; linux- >> acpi@xxxxxxxxxxxxxxx >> Subject: Re: [PATCH v4 1/2] clk: x86: Add ST oscout platform clock >> >> >> >> On 5/8/2018 3:14 AM, Deucher, Alexander wrote: >>>> -----Original Message----- >>>> From: Agrawal, Akshu >>>> Sent: Monday, May 7, 2018 6:14 AM >>>> Cc: djkurtz@xxxxxxxxxxxx; Agrawal, Akshu <Akshu.Agrawal@xxxxxxx>; >>>> Deucher, Alexander <Alexander.Deucher@xxxxxxx>; >>>> mturquette@xxxxxxxxxxxx; sboyd@xxxxxxxxxx; Koenig, Christian >>>> <Christian.Koenig@xxxxxxx>; airlied@xxxxxxxxxx; Liu, Shaoyun >>>> <Shaoyun.Liu@xxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; linux- >>>> clk@xxxxxxxxxxxxxxx; rjw@xxxxxxxxxxxxx; lenb@xxxxxxxxxx; linux- >>>> acpi@xxxxxxxxxxxxxxx >>>> Subject: [PATCH v4 1/2] clk: x86: Add ST oscout platform clock >>>> >>>> Stoney SoC provides oscout clock. This clock can support 25Mhz and >>>> 48Mhz of frequency. >>>> The clock is available for general system use. >>>> >>>> Signed-off-by: Akshu Agrawal <akshu.agrawal@xxxxxxx> >>>> --- >>>> v2: config change, added SPDX tag and used clk_hw_register_. >>>> v3: Fix kbuild warning for checking of NULL pointer >>>> v4: unregister clk_hw in driver remove, add .suppress_bind_attrs >>>> drivers/clk/x86/Makefile | 3 +- >>>> drivers/clk/x86/clk-st.c | 85 >>>> ++++++++++++++++++++++++++++++++++++ >>>> include/linux/platform_data/clk-st.h | 17 ++++++++ >>>> 3 files changed, 104 insertions(+), 1 deletion(-) create mode >>>> 100644 drivers/clk/x86/clk-st.c create mode 100644 >>>> include/linux/platform_data/clk-st.h >>>> >>>> diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile >>>> index 1367afb..00303bc 100644 >>>> --- a/drivers/clk/x86/Makefile >>>> +++ b/drivers/clk/x86/Makefile >>>> @@ -1,3 +1,4 @@ >>>> +obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o >>>> +obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE) += clk-st.o >>>> clk-x86-lpss-objs := clk-lpt.o >>>> obj-$(CONFIG_X86_INTEL_LPSS) += clk-x86-lpss.o >>>> -obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o >>>> diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c new >>>> file mode >>>> 100644 index 0000000..8a7795c >>>> --- /dev/null >>>> +++ b/drivers/clk/x86/clk-st.c >>>> @@ -0,0 +1,85 @@ >>>> +// SPDX-License-Identifier: GPL-2.0 >>> >>> Should this be MIT? The original license was MIT. >>> >>> Alex >>> >> >> We are adding SPDX tag, while license remains same GPL-2.0 >> >> What I have read is this is "to provide license identifiers inside the source >> code that could be easily parsed by machines and would allow checking for >> license compliance of an open source project easier." > > My point as just that the original license on the file that you first sent out was MIT so the SPDX tag should be MIT rather than GPL. E.g., > SPDX-License-Identifier: MIT > Oh right it should be MIT, will change it. Thanks, Akshu -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
