Received-SPF: pass (google.com: domain of linux-kernel+bounces-185219-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Tue, 21 May 2024 09:35:13 -0700
From: Dan Williams <dan.j.williams@intel.com>
To: Ira Weiny <ira.weiny@intel.com>, <nifan.cxl@gmail.com>,
	<dave.jiang@intel.com>, <alison.schofield@intel.com>,
	<vishal.l.verma@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: <664ccd41d9721_e8be2942d@dwillia2-mobl3.amr.corp.intel.com.notmuch>
References: <20240520193106.994172-1-nifan.cxl@gmail.com>
 <664cbd06b5342_364af29491@iweiny-mobl.notmuch>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <664cbd06b5342_364af29491@iweiny-mobl.notmuch>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?rp25o9PkncFwjfZF5hc9DBGmUKcQG3msIRidEBfvVm0txz9YpRWA9vse+PGt?=
 =?us-ascii?Q?SLJL14sET+5uUdQylDrggd3nmUGQz5JlZSjhf7yFhyLPHyY0dxu6T7aLBKUm?=
 =?us-ascii?Q?ohVeWMM4sKQX1O1By5d4HN/tlxXyKy/H3HLHQnq4Yad6QKVsvy/fXeRBBjS5?=
 =?us-ascii?Q?hFdmzk7bATNqxs3nafuetQEiooXvusAwc1lJhMc/9C20y0k5X9GRUEOShJvK?=
 =?us-ascii?Q?JzDMLr2d7xeYSvL0u6NmfvbZHmBaK6uEqsRT96Y34KxRPKbGogT4WLccJL+W?=
 =?us-ascii?Q?j+8HIcPG4C5x7YRNG94XD18ogCu7NgAHUtoUiZBB5g9IIHUfB5oOdxW5t63N?=
 =?us-ascii?Q?L2yW1hudyOeGunXuErddaS4ngBGZGRrW8L+P75zMQpc9Q1qDvDZDL8EgzS7S?=
 =?us-ascii?Q?e860NWCXqqzTSQrmiLXcc9ranwJwISz5nkXUEUJXWTA0aXkGrNfUUjIaKd4w?=
 =?us-ascii?Q?wK8ctLl05JZtk/cMSZBx9XGTZQE/C1vJn94EuTRig3tJ438RABPftyqgrNvR?=
 =?us-ascii?Q?YCXbSA19kbu+SbNIw8TgHi8yG+GWl5tav1ue/OTlKb4bhcDXuKn5F9If03/v?=
 =?us-ascii?Q?v4A/wkxWDaJDvr/T7N+96s3rU6YTR2ih0DcbvKMrbf1GpOmRJk8Uo7vVaDkU?=
 =?us-ascii?Q?4wE/BAcHm17NpM74WmLtV7QewHNOEgwZgw+I3iCRnX9aHNmestbNbtNeeuaU?=
 =?us-ascii?Q?OgFMSfzTpb5g6qwB/Iv6QUcJIKSlakj4yMfu+9aA2W+cwqNh5qzXa+lRTtE7?=
 =?us-ascii?Q?wbPpnrZSr1T/+och9Q/1kRPotVQnF6RB9uUeE6eySqrKqsPaz7ZIJNNb7/gl?=
 =?us-ascii?Q?vK9MTQut92wpH//BvPZdiuVK2ANK09yqfJ5Wt1T/Ydklz7Df+hMlqFAZR7nP?=
 =?us-ascii?Q?+u51T/jWCJS2irmHgO0fP6/jbYbTibwcR7oCg2IxcC6fO43J6zVUW38AdFb0?=
 =?us-ascii?Q?Ku+Kl6+l1wBf8PDUcVM9a1htOmPV/Y3OaBKzAsbNCVKXLSBu0O1MDm2bR0nq?=
 =?us-ascii?Q?+DcvvFngPxz8Y62zi2ZDUFTiVytw00XG9G8hazjUj2cFbdj7s4LCoyp2Mha8?=
 =?us-ascii?Q?zOQO54cZKqUDYy7ScBZF6573XDqQBtQ2D54KZ9X3Z1551sSaNp5LrHCibwhI?=
 =?us-ascii?Q?dt4vrlQNq6yaqEcfZWUydl6uLOKCHxKcwCJHjdEvthc/PtDxvL05Pb9Ee4+p?=
 =?us-ascii?Q?y0uJtOGA4ijKM4//0neGL5rnblCpxYMXULa0Qkh0o4AzY4wLrwxl09GFbUNG?=
 =?us-ascii?Q?TBq1w8ZJ+C6pk6cIq19fZCE653MopQcRMVCmvzvquQ2//QF7eV4IG0N3lRa7?=
 =?us-ascii?Q?EJs+jOao2bB5LhdLNWGhYDZkwhiCKWNfYB+WUXgexjeewrP/YL24IZRbcYTg?=
 =?us-ascii?Q?0TO++JLPeI/+Cy9E28O4PvLEE0ZuzgWT6yQstO3X0xFE3oeYnM2m/ew12BaG?=
 =?us-ascii?Q?7mPMp40sFyNOmDMzwHezVsYH5B9OD85gGRieiZ3u7B/v82t7/FFpPQPHs5tT?=
 =?us-ascii?Q?Er8XpqWXTsIC/yvtaWlaqGp/nDlf79Xtn3dnhj3TIHPld4hwqU/3eKWXIgsv?=
 =?us-ascii?Q?h+LbRHrvj+Xzj/bUSq1oCsRVYDlVaQHhJbcwo2z5OZ5C0loMl0QFD8A1QQwv?=
 =?us-ascii?Q?hw=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e85d8643-e643-4d77-a109-08dc79b3feff
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2024 16:35:16.5532
 (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: nsJIaHGsCL71IvTY/iZtjTsuPOPUL1grIl0ggPtfhFmAkOPYk4SOHO29gAfaizsmHEZL5NTyVI/TY4Hs9effCDM48Le6jtp+bQK8vMQNJQY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB8072
X-OriginatorOrg: intel.com

Ira Weiny wrote:
> 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.

Right, and region.c has:

// SPDX-License-Identifier: GPL-2.0-only