Received-SPF: pass (google.com: domain of linux-kernel+bounces-185148-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 21 May 2024 10:25:58 -0500
From: Ira Weiny <ira.weiny@intel.com>
To: <nifan.cxl@gmail.com>, <dave.jiang@intel.com>,
	<alison.schofield@intel.com>, <vishal.l.verma@intel.com>,
	<ira.weiny@intel.com>, <dan.j.williams@intel.com>,
	<Jonathan.Cameron@huawei.com>
CC: <linux-cxl@vger.kernel.org>, <a.manzanares@samsung.com>,
	<dave@stgolabs.net>, <linux-kernel@vger.kernel.org>, <nifan.cxl@gmail.com>,
	Fan Ni <fan.ni@samsung.com>
Subject: Re: [PATCH] cxl/region: Add module license declaration to cxl region
 module source code
Message-ID: <664cbd06b5342_364af29491@iweiny-mobl.notmuch>
References: <20240520193106.994172-1-nifan.cxl@gmail.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240520193106.994172-1-nifan.cxl@gmail.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?nvgpOuZhdHPZxHmlufE2JNcYAYnWiR1SzrOb6gSPfJDfLR3p5QG63zEBsFtV?=
 =?us-ascii?Q?tXMHYHyeGAehQIy2LLEMzmbQZlHnGCv3/IxsGk+vqXhSiHAP1jyeZxpV8eqx?=
 =?us-ascii?Q?/HNk9bUHf+HnARVeEpbaPkUsIbEBZdo/BqE32fBwbNEL3ExQU6Op7khkX4FF?=
 =?us-ascii?Q?AK+pxAaHuTE4rJg3PG5+oZr5BXZyixpaahD/8xHsul3Vm7J4+2zFXwwC8sEp?=
 =?us-ascii?Q?Sn8RCj78ZgK+Jfn9OT+0Qdgeh0LwL+xKkVPNtZmrgCNJ08UmICNcIHpI3NMV?=
 =?us-ascii?Q?83kFLX6K88mQghIWYE4Rnod8ZlhcAJWoCwLaBue/71Zgk4MArv8/uSf9hZjI?=
 =?us-ascii?Q?+9JStRwhvNl7lFBBBwVgw1YEiih0QBpil+jz5cEMyMg8iYDzJYL3nNW9ZIx0?=
 =?us-ascii?Q?DY5slZ4cOGi6hGutuHzVyzwRc8Bifzsj9EvchkaqGrSTJ1H1tluB9qMO9JEP?=
 =?us-ascii?Q?Evr1JrbLH8AdKsI0PemQ3Yr3Ebk0IO3slTSUZh5e3ZTK7Spf1jAGNc1qJfYP?=
 =?us-ascii?Q?KeCki1GgVEsBcfkKYBeNr/26auX4NAybhpZZIilHrdfUwcY4j33+fWUH6dfT?=
 =?us-ascii?Q?SHFZ3HY5XzGXjOBY6loO4+1d3AQarTaexseLOVczyPvFjdCCWLyJwVvlSGIF?=
 =?us-ascii?Q?uoABV4t4cza5Enc8sAW+nmlEFKKanGUsTiQIJojpjX4IJcsQ9+KbR4+W+MMh?=
 =?us-ascii?Q?1scCqFqJPxwMbVH4UYEIQGRAaRO7c5MsUTSybmL2R+yOSr3ic/BkjQuZ/SCo?=
 =?us-ascii?Q?l8ngmKz9SdPr3EvHMw1XoMqZ6elTHkKGIj+McyXakrjNfsdWU7u2/2yxdcKT?=
 =?us-ascii?Q?rBFMIbdiUzujKU7dbE/Kc96psnML5RFGPLezdcsNuMXweG0cHNmZ+dsmz01H?=
 =?us-ascii?Q?ua/jdXKEbdths4ZH9kM9S4TSXzUI8n82MPwNZM5EhuluujMv6uLRpw0gWExv?=
 =?us-ascii?Q?2dg3N61ncgA9wb/Xg3LdZF2mlxRB+oUYs5x2q2QFX8lBWxt+SlomcLEDnn/N?=
 =?us-ascii?Q?ZVYJMrh1+5ochyuD7jU5skH6TOjCMZ85+9XGjjoWCLJ+CZdbtWo6pL2WY+dy?=
 =?us-ascii?Q?X4+kyovt7xLjhcegO0VsH7SZN2jDTa6sF3Nu4qgYWcTaqO4k4TTjZbY6p61P?=
 =?us-ascii?Q?Jq7/rnZbEkFizzeAaWfDy6d6L8JHQMuchUspPj6v6efjzIo92AESym6eVEjF?=
 =?us-ascii?Q?1ryV7IWuxG8YOFtGEueZ5oaBZVkNeAvr4RdUwDVqJbL++jMrerT1fH+wMjGN?=
 =?us-ascii?Q?Gchm9/iiJaXJoucgsmsl0+g5Oa6CECfPGtlswmN8cTtWMJamSGN6BBww+tks?=
 =?us-ascii?Q?X09/0LEmGPPrIarme4rICmDeDJlqwX7gobRJWM21OTeW7awVy4xOuQJ4+oFT?=
 =?us-ascii?Q?bCBmqNppcb9hYzJWB8t3rY5FlU5uIasxPCj2NC9ZXtEAK9jT9hbIs9bvIlGO?=
 =?us-ascii?Q?ZaosJwyyrZHExHHM9kHwv5EjWA9ZQXfIcsCdhjmyg9W8TQMs2qSWKHt2yFJH?=
 =?us-ascii?Q?cHElP7eZTn2iuqigo41j5I4a7Ac3vBYTsWRWYstPc/Wiytmpkjlkkv8o2meH?=
 =?us-ascii?Q?HIQzhP0SZ4SEwRacCFfhg/00Gt9vzGa9ngyH/tf3?=
X-MS-Exchange-CrossTenant-Network-Message-Id: b777b81a-4415-410b-6e15-08dc79aa5369
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2024 15:26:03.2975
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: BDCmm99mOB7RpLIxDWptATrlsMfQ/gpzksKuw7T0O/VUs1xwOV8LCbsT3UvlxFhVnzglAVNcpiH4xRZKtG7LKg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR11MB7749
X-OriginatorOrg: intel.com

nifan.cxl@ wrote:
> From: Fan Ni <fan.ni@samsung.com>
> 
> This change explicitly specifies the licensing terms for the CXL region
> module as GPL v2. This commit does not introduce any functional changes
> to the behavior or functionality of the CXL region module.
> It solely adds the MODULE_LICENSE declaration to the module source file.

Why is this needed?  The region code is added to cxl_core when built and
that module already reports the correct module license.

10:24:55 > modinfo cxl_core | grep lic
license:        GPL v2

Ira

> 
> Signed-off-by: Fan Ni <fan.ni@samsung.com>
> ---
>  drivers/cxl/core/region.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c
> index 5c186e0a39b9..e299ddcbd1a2 100644
> --- a/drivers/cxl/core/region.c
> +++ b/drivers/cxl/core/region.c
> @@ -3204,3 +3204,4 @@ void cxl_region_exit(void)
>  MODULE_IMPORT_NS(CXL);
>  MODULE_IMPORT_NS(DEVMEM);
>  MODULE_ALIAS_CXL(CXL_DEVICE_REGION);
> +MODULE_LICENSE("GPL v2");
> -- 
> 2.43.0
>