Received-SPF: pass (google.com: domain of linux-kernel+bounces-24188-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Thu, 11 Jan 2024 16:45:04 -0800
From: Dan Williams <dan.j.williams@intel.com>
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>, Yang Li
	<yang.lee@linux.alibaba.com>
Subject: RE: [PATCH -next] cxl: Remove duplicated include in cdat.c
Message-ID: <65a08b90ce17d_1eb429469@dwillia2-xfh.jf.intel.com.notmuch>
References: <20240109004728.54703-1-yang.lee@linux.alibaba.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240109004728.54703-1-yang.lee@linux.alibaba.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?dI70M/AFdhb5RZm6pvoScwxcSJxZ++O36Jx5y0y96Yjp/k4cLVzNwjkn1gf+?=
 =?us-ascii?Q?sI408AUdG3quaDrzCoZ34dTQJnq6qYbd516emh0RZS0dgGkMuzTGfotYKSiq?=
 =?us-ascii?Q?KyOzhSrR6mF6o6HVK7M4+jxLI/V4jxCriV7W7MzGGgWhjrbVXDHn8R/7KRiY?=
 =?us-ascii?Q?W9IT4JLPITYAqXvipqOfitg0HdtbQcXXervxAJZNRbxNgD+UeDMdeyZD7/mM?=
 =?us-ascii?Q?douBeb6BagOaceICmO11fDs5dKUGrBdFPAIZRK06ThdnAf3FGrQFIjHPJ0XP?=
 =?us-ascii?Q?H5mqAnNfkEctXe8GnYSy+eAa0tilUTvTbGvlhrPf4pDr/qEgwUU97+/ShVLP?=
 =?us-ascii?Q?V0DzYqpYT4g9X+VdmgqTQbBpou0mELHu8lWGvB5LMdFl0358d2rVM7/tEoXW?=
 =?us-ascii?Q?spxyADlnXa8+bxKhr0qHkm3A9aiJSKlRoNK3jm/ZvEBtFhUFUmFu70AN12BX?=
 =?us-ascii?Q?wou0saZs2PAWFmUPbgN/LR23MYwr2R0aTpi7CH+I9CmFS6Eg+jRp1UjhssN9?=
 =?us-ascii?Q?jan3bnya047NZrOkT1uzjp/9qsN+zViyHMCOXgp4S5L70H8Q3ep4r17lT+sT?=
 =?us-ascii?Q?wTCupUhfNV4sce8DGUX879eqw8W9V8eSTNUKCjcqj4oRrAzU+XyKWNEmF6Re?=
 =?us-ascii?Q?kyl+evNFlU55JoO6L3vNKYkqqHbRDg1tlM3YIk/s9e0QMmC4aJnyzKb539KW?=
 =?us-ascii?Q?saKBTrX+DIL4524T2Y9+TDaXRrFiV1Ma5/1zvObRyvWI1xdSi/vr1Fm/7v3+?=
 =?us-ascii?Q?lpoLq5tonjxXP0Tr5GdfZi+RlEZx60r4v+ym0KgQtJQVpipnf0dk9ZcsJsUt?=
 =?us-ascii?Q?MowrTel3W8vMVjR8k4iUH+ZY0jV/y4Y3g7o5UpTRFUB85sszBrvM9hCkKOZq?=
 =?us-ascii?Q?j47dl3urWw8pd3xOw+BXnuDbZNazexS/1yRbgFKsbF83E3V+VEs/FBM2hat0?=
 =?us-ascii?Q?x5sz+Db7XvHw7+hT3phJeKQJ2g53CFcJKbwXI2mohZ9OBfCv/haFeOIhdZDh?=
 =?us-ascii?Q?yyV6vWqSTQksFn/bxKhlNNuwW3pwjZQwTKR7ru/OX9++HRVehv4rRHnYQBdl?=
 =?us-ascii?Q?Y7AFEWa274NSYD9PSIXxr3H/E4m68S1pHGrM7COETi8nlcNnMiPHXzTEyAQu?=
 =?us-ascii?Q?nProo6vSL14rskPluldUp3PrNAOiApvTwYGLGlVvW9nRQRFCMCv0UKK/j09M?=
 =?us-ascii?Q?rnqao6wvkdJqLlDHXvlYpkm6ClwWKbOMCAsnozB7Wh26SJFCq1NbGLW5vMzR?=
 =?us-ascii?Q?NAdh4qb1IMXJhZ8aA7otmKBnfgM/WA1KKd9lrAzewWWI+nw1tdPFOku4GFHV?=
 =?us-ascii?Q?OZuwCC6Tilgg6n5BSlAjsnHLRIphvSTG5yMlXLdiA7VINF6nXB7sw76zJSXE?=
 =?us-ascii?Q?Tr4wJjzrYVNSNvllYNmbSSPcH6jiuhqj32shst/iyDHxzeKElYCug327mmoT?=
 =?us-ascii?Q?S9g54gCQQxX7zn3wQ4DivPm54Mh2CCHpzzwxF4v76sh6/iPXpKIgIn00/rCJ?=
 =?us-ascii?Q?v6gjoy0T/P6uf+RZ2uk+BVmUXvSUOb2RjgWZ5qQPEHxvfQ7evPD9/QNtd7AM?=
 =?us-ascii?Q?SU4sBFOI0/Z1vctIedFOq2t01ODI1yyEr9o6DsMeiqy8hh/6rdCIKCfc0QTd?=
 =?us-ascii?Q?Lg=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4209277d-6f9e-42ac-39b7-08dc1307b8fe
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jan 2024 00:45:07.1283
 (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: 1MkIFAwouJifkJ0kW2J1irTJb95D+iah4L+TIU+Vu0g8vHnu5YtB18Wl+Z4Bce/TAvDuHCZ4xwaOA6OPJt01fZs2PCTFXsgmWzhooKivIYk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR11MB5603
X-OriginatorOrg: intel.com

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>
> ---
>  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"

I believe that was due to a bad merge by me. That's fixed up now with a
better merge commit.

Thanks for spotting this though!