xen/riscv: Updated the license header

[XEN] xen/riscv: Updated the license header

Posted by Ayan Kumar Halder 1 year ago

Updated the license header in a separate comment of its own.

Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@amd.com>
---

This was highlighted in the following review -
https://www.mail-archive.com/xen-devel@lists.xenproject.org/msg141930.html

 xen/arch/riscv/include/asm/csr.h            | 3 +--
 xen/arch/riscv/include/asm/riscv_encoding.h | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/xen/arch/riscv/include/asm/csr.h b/xen/arch/riscv/include/asm/csr.h
index 8215562343..be57dcce1c 100644
--- a/xen/arch/riscv/include/asm/csr.h
+++ b/xen/arch/riscv/include/asm/csr.h
@@ -1,6 +1,5 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
 /*
- * SPDX-License-Identifier: GPL-2.0-only
- *
  * Copyright (C) 2015 Regents of the University of California
  */
 
diff --git a/xen/arch/riscv/include/asm/riscv_encoding.h b/xen/arch/riscv/include/asm/riscv_encoding.h
index 43dd4f6981..58abe5eccc 100644
--- a/xen/arch/riscv/include/asm/riscv_encoding.h
+++ b/xen/arch/riscv/include/asm/riscv_encoding.h
@@ -1,6 +1,5 @@
+/* SPDX-License-Identifier: BSD-2-Clause */
 /*
- * SPDX-License-Identifier: BSD-2-Clause
- *
  * Copyright (c) 2019 Western Digital Corporation or its affiliates.
  *
  * Authors:
-- 
2.17.1

Re: [XEN] xen/riscv: Updated the license header

Posted by Oleksii 1 year ago

Hi Ayan,

On Fri, 2023-04-28 at 19:09 +0100, Ayan Kumar Halder wrote:
> Updated the license header in a separate comment of its own.
> 
> Signed-off-by: Ayan Kumar Halder <ayan.kumar.halder@amd.com>
> ---
> 
> This was highlighted in the following review -
> https://www.mail-archive.com/xen-devel@lists.xenproject.org/msg141930.html
> 
>  xen/arch/riscv/include/asm/csr.h            | 3 +--
>  xen/arch/riscv/include/asm/riscv_encoding.h | 3 +--
>  2 files changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/xen/arch/riscv/include/asm/csr.h
> b/xen/arch/riscv/include/asm/csr.h
> index 8215562343..be57dcce1c 100644
> --- a/xen/arch/riscv/include/asm/csr.h
> +++ b/xen/arch/riscv/include/asm/csr.h
> @@ -1,6 +1,5 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
>  /*
> - * SPDX-License-Identifier: GPL-2.0-only
> - *
>   * Copyright (C) 2015 Regents of the University of California
>   */
>  
> diff --git a/xen/arch/riscv/include/asm/riscv_encoding.h
> b/xen/arch/riscv/include/asm/riscv_encoding.h
> index 43dd4f6981..58abe5eccc 100644
> --- a/xen/arch/riscv/include/asm/riscv_encoding.h
> +++ b/xen/arch/riscv/include/asm/riscv_encoding.h
> @@ -1,6 +1,5 @@
> +/* SPDX-License-Identifier: BSD-2-Clause */
>  /*
> - * SPDX-License-Identifier: BSD-2-Clause
> - *
>   * Copyright (c) 2019 Western Digital Corporation or its affiliates.
>   *
>   * Authors:

Looks fine to me.

I'll do my best to stick to what was mentioned here:
https://www.mail-archive.com/xen-devel@lists.xenproject.org/msg141930.html

Reviewed-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>

~ Oleksii