On 11-09-18, 10:17, Geert Uytterhoeven wrote: > Hi Vinod, > > On Tue, Sep 11, 2018 at 9:48 AM Vinod <vkoul@xxxxxxxxxx> wrote: > > On 07-09-18, 01:58, Kuninori Morimoto wrote: > > > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > > > > > This patch updates license to use SPDX-License-Identifier > > > instead of verbose license text. > > > > Thanks but the style is not consistent in files :( > > > > Can we use one only? > > Please read > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst#n69 > > 2. Style: > > The SPDX license identifier is added in form of a comment. The comment > style depends on the file type:: > > C source: // SPDX-License-Identifier: <SPDX License Expression> > C header: /* SPDX-License-Identifier: <SPDX License Expression> */ Right and I missed patch is doing for both c & h files, my bad Applying this now. Btw I have sent the series for dma_slave_config direction removal, sh/ drivers turned to be bit more, can you guys take care of it please? -- ~Vinod