2017-11-30 08:23:16

by Philippe Ombredanne

[permalink] [raw]
Subject: Re: [patch v12 2/4] drivers: jtag: Add Aspeed SoC 24xx and 25xx families JTAG master driver

On Wed, Nov 29, 2017 at 11:51 PM, Kun Yi <[email protected]> wrote:
> On Tue, Nov 14, 2017 at 8:11 AM, Oleksandr Shamray
> <[email protected]> wrote:
[]
>> diff --git a/drivers/jtag/jtag-aspeed.c b/drivers/jtag/jtag-aspeed.c
>> new file mode 100644
>> index 0000000..a6e2417
>> --- /dev/null
>> +++ b/drivers/jtag/jtag-aspeed.c
>> @@ -0,0 +1,782 @@
>> +/*
>> + * drivers/jtag/aspeed-jtag.c
>> + *
>> + * Copyright (c) 2017 Mellanox Technologies. All rights reserved.
>> + * Copyright (c) 2017 Oleksandr Shamray <[email protected]>
>> + *
>> + * Released under the GPLv2 only.
>> + * SPDX-License-Identifier: GPL-2.0
>> + */

I think the SPDX id should be on the first line as requested by Linus
and documented by Thomas (tglx) and Greg (greg-kh). And it should use
// comments, e.g:

// SPDX-License-Identifier: GPL-2.0

See the threads discussing all these.


--
Cordially
Philippe Ombredanne

From 1585442684311096653@xxx Wed Nov 29 22:52:48 +0000 2017
X-GM-THRID: 1584061622077044088
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread