Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751802AbdLNKzO (ORCPT ); Thu, 14 Dec 2017 05:55:14 -0500 Received: from osg.samsung.com ([64.30.133.232]:61762 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751173AbdLNKzM (ORCPT ); Thu, 14 Dec 2017 05:55:12 -0500 Date: Thu, 14 Dec 2017 08:55:03 -0200 From: Mauro Carvalho Chehab To: "Takiguchi, Yasunari" Cc: "akpm@linux-foundation.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-media@vger.kernel.org" , "tbird20d@gmail.com" , "frowand.list@gmail.com" , "Yamamoto, Masayuki" , "Nozawa, Hideki (STWN)" , "Yonezawa, Kota" , "Matsumoto, Toshihiko" , "Watanabe, Satoshi (SSS)" , "Bird, Timothy" Subject: Re: [PATCH v4 00/12] [dt-bindings] [media] Add document file and driver for Sony CXD2880 DVB-T2/T tuner + demodulator Message-ID: <20171214085503.289f06f8@vento.lan> In-Reply-To: <02699364973B424C83A42A84B04FDA85431742@JPYOKXMS113.jp.sony.com> References: <20171013054635.20946-1-Yasunari.Takiguchi@sony.com> <20171213173633.57edca85@vento.lan> <02699364973B424C83A42A84B04FDA85431742@JPYOKXMS113.jp.sony.com> Organization: Samsung X-Mailer: Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 933 Lines: 27 Em Thu, 14 Dec 2017 09:59:32 +0000 "Takiguchi, Yasunari" escreveu: > Dear Mauro > > Thanks for your review. > > We will refer to your comments and consider how to respond for them. > I want to confirm one thing about SPDX license text > > We will add SPDX license text to our files, > Is it necessary to add SPDX not only .c .h Makefile but also Kconfig? > When I checked current files in driver/media, there is no Kconfig file which has SPDX. SPDX is a new requirement that started late on Kernel 4.14 development cycle (and whose initial changes were merged directly at Linus tree). Not all existing files have it yet, as identifying the right license on existing files is a complex task, but if you do a: $ git grep SPDX $(find . -name Makefile) $(find . -name Kconfig) You'll see that lot of such files have it already. So, yes, please add it to both Makefile and Kconfig. Thanks, Mauro