Received-SPF: pass (google.com: domain of linux-kernel+bounces-24151-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <3d74f069-b760-45ab-8d62-5d7d143e27e5@intel.com>
Date: Thu, 11 Jan 2024 16:54:45 -0700
User-Agent: Betterbird (Linux)
Subject: Re: [PATCH -next] cxl: Remove duplicated include in cdat.c
To: Yang Li <yang.lee@linux.alibaba.com>, <jonathan.cameron@huawei.com>,
	<dave@stgolabs.net>
CC: <linux-cxl@vger.kernel.org>, <linux-kernel@vger.kernel.org>
References: <20240109004728.54703-1-yang.lee@linux.alibaba.com>
Content-Language: en-US
From: Dave Jiang <dave.jiang@intel.com>
In-Reply-To: <20240109004728.54703-1-yang.lee@linux.alibaba.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Q3E5MzUyQjJRQi9aZnpmeDZHclBVcFdnRy9yREVMUXlOYkhrWExYdk96cXZR?=
 =?utf-8?B?TjhyMkc1L1EzUmN5TVFOcVV4KzZmcjJKMytGWVcwMTRBMkVLM0w1bW5qTWd5?=
 =?utf-8?B?NkQvMXdFYXIzd2lXbDlzN3JaaXFETVlXUVc0ZFFUL25abnlsTWNqTkVicFBJ?=
 =?utf-8?B?L3h6UUFJNExVOW90UkpNSmRwOUVldm1MN0Qwemsvd0ZVTUthOE9sa1EvSUd0?=
 =?utf-8?B?SENzaktucWpKbzZxWFcxT25URkJ0QVN5Nm9VSTJFVXVBTXNTRTd4NThKMWtE?=
 =?utf-8?B?MVMzN1djY1IzMS9xTm82cWptSHU1YlIySzZxK0ZTa3BHWVkrTWxwSHlWRUFP?=
 =?utf-8?B?UGFjM1FXK2l1Y0N6TlVFZzlyaWxpWWFpU3N1UXArWFJOVWNjTk9VUmJJQTZv?=
 =?utf-8?B?dGZBdkFCZFphSUU2b2xmVS9XQW9qTE5DMitoWHVZcDhEc1hHWmVVcFROWGE0?=
 =?utf-8?B?ZkIyY1RKYjNPc2FEUCtuMlkzczY5UkhsbGpVenNqblZDOGhCL3h1enBMNFpB?=
 =?utf-8?B?eXlCeXFnZUdLT3NOK2t2RldtMjBwVW0rUzJKQWpQbGg4RFBZQ2ZiZGNtVFNh?=
 =?utf-8?B?YjFZMUdQcTE0dnNuK2NLV0pSUTFNdVplMXMzVWFMOGxSSjdvOUI2THM4Vmdn?=
 =?utf-8?B?ZWx1cHl0NUhYeGU4elRYcStheHhkaVRGWmtialdJMzJxSVhTT1FsZEpiemdF?=
 =?utf-8?B?dldWdm5jY2REQkdFS2hWSEcvSUI0elJ6V3Rvd3hkYWErekptUHU1OTJUeFYr?=
 =?utf-8?B?LzJVemdsWHVQd0tzWmJBb2txSTJvR2lMUk5EMXE5aUlLUEwvOHhIcnMrajVi?=
 =?utf-8?B?UXZiK2piaW5LVGovRzN4bTRiQmR3WEhlVlQ0UE5qcHdZZHFqYjNLc3lnTWRG?=
 =?utf-8?B?U1VlbDRlck1DRmZNblRJckRlUUh4Ykl5bjZvZTB2bTluZUk5dnh1emhyNUZH?=
 =?utf-8?B?c1MwOFMzRFVJUDJ2czJ6eE5Sc3REUjVlWWpwTkQxRDQ2elRkWW9rQ1NteTBo?=
 =?utf-8?B?T3RObnluVU42OGJvVWlGWjk2c3hqemhONWFXMnYrWlcwdHkzRW9pZzVnMnlk?=
 =?utf-8?B?eTMralVJYTBEd0pNMC9BaDh3akdhTEZjL1BxdWNqRWxOU0M4bGRGajNnYnR6?=
 =?utf-8?B?WkZLbWk2TWZVVGg5bkgzMWhTUUdXa3BHVFpJQ2xjN3I0Rzc2cEZNaFZSdGFk?=
 =?utf-8?B?ZitoU3NvejdPbENaei9wcEl6aW4xeW95TlVDUWtwYnhhK0txa21IRWxLS2JU?=
 =?utf-8?B?SkVpZFNGZ0M3blJOcE9DL3JpRWVTbitIdFphYWNybUEvbWtYdm15ZVpmTFR2?=
 =?utf-8?B?RG1QQzdjT2Y4azU1SEdVdHlWa1NRQUhvV0JYZTBYL2pxNWlXd2tmVnJpdWE5?=
 =?utf-8?B?czVKcFFYdXlnUWIxcU1rc0ZpZTNDZUxvUWV1bkhnVFc2eVllV0VGMUIxVVkz?=
 =?utf-8?B?UklJR0gwSFh5T0QyVVNCcVpQc0NLb3lETWQzKzNwUExuT2J4b0h2a0FwTHE1?=
 =?utf-8?B?bGNJeko4OURJMUMrYUJETEpCZnRpRmdtMlRsbXIyMW1UdUZpOXEvYWFaNXN6?=
 =?utf-8?B?NGNQTDVnUWpjSnBBVzdMdGRvZ0VrR3BLTjc5ckRwUjVZTTNWUTVYcE5NZjhK?=
 =?utf-8?B?cGFCbG9mbmV3RDN3VnpCNitIcHJjbXh6V2V1WVplSEd4WDNFL0kxYWNUY0M5?=
 =?utf-8?B?U1daV1FsWVBVOHlTemlYVVIwY1J3ay83VDJwdDBoaGQyaHlIWFdrZU1XTzEr?=
 =?utf-8?B?NzQzT2dkTnRlSmYyOFdlMGsvQW1OM0Q3ME83aktVME9UdGdEbjd5T3FxY1Iz?=
 =?utf-8?B?OVJxTEQ1NTNIMTBvcGxxNlNwdDYyWndRT3lYdzFMd09ITWZtZmloTkxmblNV?=
 =?utf-8?B?T041VE5kY2c1SXlMQzh5VzFCRkRQVUNobkpxamRBd201UjI5amlVT3plMHdx?=
 =?utf-8?B?QUZLZmNwNUdPcEhzNkhJVVpVcGkyWW1TKzBtVHo1enRoMzVVLzhQS0wxZDVK?=
 =?utf-8?B?Q3hKbTNsNFpqQ2JINUZWazh1MENVN0lJZXBjUFZDUE5vc3BKQ3NQcmllb0t2?=
 =?utf-8?B?cmxxd29qalg2cmJTQ1ZaQ1Vqc0dMQmVRNlgwUGdGUHIvbzdCSWRJdjFvS1Zm?=
 =?utf-8?Q?AK/4MjtLxDMWqSXJK5VNIOUIV?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 63272b0f-2a65-4fcf-8a69-08dc1300b233
X-MS-Exchange-CrossTenant-AuthSource: PH7PR11MB5984.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2024 23:54:49.2603
 (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: wF6otex6N0WLcwceZHbhFjZ/Z2SEob07MVAJQdT/R6MfkBA0olm0p4stUuSaW+l+4YaX/zg7nqzqmAzAghPklQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR11MB8285
X-OriginatorOrg: intel.com


On 1/8/24 17:47, Yang Li wrote:
> The header files core.h is included twice in cdat.c,
> so one inclusion can be removed.
> 
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>

Reviewed-by: Dave Jiang <dave.jiang@intel.com>
> ---
>  drivers/cxl/core/cdat.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/cxl/core/cdat.c b/drivers/cxl/core/cdat.c
> index f5e649407b8b..6fe11546889f 100644
> --- a/drivers/cxl/core/cdat.c
> +++ b/drivers/cxl/core/cdat.c
> @@ -9,7 +9,6 @@
>  #include "cxlmem.h"
>  #include "core.h"
>  #include "cxl.h"
> -#include "core.h"
>  
>  struct dsmas_entry {
>  	struct range dpa_range;