<div dir="ltr">Hi,<br><div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 8, 2018 at 3:21 PM Gao Xiang <<a href="mailto:gaoxiang25@huawei.com">gaoxiang25@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Loic,<br>
<br>
On 2018/10/8 18:41, Loic Tourlonias wrote:<br>
> Add SPDX identifier to simplify header and remove whole license text<br>
> <br>
> Signed-off-by: Loic Tourlonias <<a href="mailto:loic.tourlonias.lkml@gmail.com" target="_blank">loic.tourlonias.lkml@gmail.com</a>><br>
> ---<br>
>  drivers/staging/erofs/lz4defs.h | 23 ++---------------------<br>
>  1 file changed, 2 insertions(+), 21 deletions(-)<br>
> <br>
> diff --git a/drivers/staging/erofs/lz4defs.h b/drivers/staging/erofs/lz4defs.h<br>
> index 00a0b58a0871..bdb1022981d8 100644<br>
> --- a/drivers/staging/erofs/lz4defs.h<br>
> +++ b/drivers/staging/erofs/lz4defs.h<br>
> @@ -1,32 +1,13 @@<br>
>  #ifndef __LZ4DEFS_H__<br>
>  #define __LZ4DEFS_H__<br>
>  <br>
> +// SPDX-License-Identifier: BSD-2-Clause<br>
<br>
For *.h files, it seems SPDX Identifier should be /* SPDX-License-Identifier: */<br>
rather than // SPDX-License-Identifier:<br>
<br>
And I think it should be placed at the first line of the line, but I am not sure of that.<br>
<br></blockquote><div>I would make the change and resubmit my patch in V2</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,<br>
Gao Xiang<br>
<br></blockquote><div>Thanks,</div><div>Loic<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>  /*<br>
>   * lz4defs.h -- common and architecture specific defines for the kernel usage<br>
>  <br>
>   * LZ4 - Fast LZ compression algorithm<br>
>   * Copyright (C) 2011-2016, Yann Collet.<br>
> - * BSD 2-Clause License (<a href="http://www.opensource.org/licenses/bsd-license.php" rel="noreferrer" target="_blank">http://www.opensource.org/licenses/bsd-license.php</a>)<br>
> - * Redistribution and use in source and binary forms, with or without<br>
> - * modification, are permitted provided that the following conditions are<br>
> - * met:<br>
> - *   * Redistributions of source code must retain the above copyright<br>
> - *     notice, this list of conditions and the following disclaimer.<br>
> - *   * Redistributions in binary form must reproduce the above<br>
> - * copyright notice, this list of conditions and the following disclaimer<br>
> - * in the documentation and/or other materials provided with the<br>
> - * distribution.<br>
> - * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS<br>
> - * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT<br>
> - * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR<br>
> - * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT<br>
> - * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,<br>
> - * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT<br>
> - * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,<br>
> - * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY<br>
> - * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT<br>
> - * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE<br>
> - * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.<br>
> + *<br>
>   * You can contact the author at :<br>
>   *   - LZ4 homepage : <a href="http://www.lz4.org" rel="noreferrer" target="_blank">http://www.lz4.org</a><br>
>   *   - LZ4 source repository : <a href="https://github.com/lz4/lz4" rel="noreferrer" target="_blank">https://github.com/lz4/lz4</a><br>
> <br>
</blockquote></div></div></div>