[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <799c897d-facf-e6d1-6c13-ccbc09630389@amd.com>
Date: Tue, 16 Jul 2024 09:50:47 +0100
From: Alejandro Lucero Palau <alucerop@....com>
To: Andrew Lunn <andrew@...n.ch>, alejandro.lucero-palau@....com
Cc: linux-cxl@...r.kernel.org, netdev@...r.kernel.org,
dan.j.williams@...el.com, martin.habets@...inx.com, edward.cree@....com,
davem@...emloft.net, kuba@...nel.org, pabeni@...hat.com,
edumazet@...gle.com, richard.hughes@....com
Subject: Re: [PATCH v2 01/15] cxl: add type2 device basic support
On 7/15/24 19:48, Andrew Lunn wrote:
>> +++ b/include/linux/cxl_accel_mem.h
>> @@ -0,0 +1,22 @@
>> +/* SPDX-License-Identifier: GPL-2.0 */
>> +/* Copyright(c) 2024 Advanced Micro Devices, Inc. */
>> +
>> +#include <linux/cdev.h>
> That is generally a red flag that something not good is about to be
> found. But it does not appear to be used in this patch....
>
> Andrew
>
I have no explanation about how it ended up there. I suspect it comes
from V1 --> V2 transition. cxlmem.h includes it and V1 was moving that
file to include/linux.
Anyway, I'll get rid of it.
Thanks
Powered by blists - more mailing lists