[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9fe4744e-390a-4cf8-827b-a6299b50afce@intel.com>
Date: Wed, 23 Jul 2025 15:15:45 -0700
From: Dave Jiang <dave.jiang@...el.com>
To: dan.j.williams@...el.com, Terry Bowman <terry.bowman@....com>,
dave@...olabs.net, jonathan.cameron@...wei.com, alison.schofield@...el.com,
bhelgaas@...gle.com, shiju.jose@...wei.com, ming.li@...omail.com,
Smita.KoralahalliChannabasappa@....com, rrichter@....com,
dan.carpenter@...aro.org, PradeepVineshReddy.Kodamati@....com,
lukas@...ner.de, Benjamin.Cheatham@....com,
sathyanarayanan.kuppuswamy@...ux.intel.com, linux-cxl@...r.kernel.org
Cc: linux-kernel@...r.kernel.org, linux-pci@...r.kernel.org
Subject: Re: [PATCH v10 01/17] cxl/pci: Remove unnecessary CXL Endpoint
handling helper functions
On 7/23/25 2:58 PM, dan.j.williams@...el.com wrote:
> Terry Bowman wrote:
>> The CXL driver's cxl_handle_endpoint_cor_ras()/cxl_handle_endpoint_ras()
>> are unnecessary helper functions used only for Endpoints. Remove these
>> functions as they are not common for all CXL devices and do not provide
>> value for EP handling.
>>
>> Rename __cxl_handle_ras to cxl_handle_ras() and __cxl_handle_cor_ras()
>> to cxl_handle_cor_ras().
>>
>> Signed-off-by: Terry Bowman <terry.bowman@....com>
>> Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@...ux.intel.com>
>> Reviewed-by: Jonathan Cameron <jonathan.cameron@...wei.com>
>
> Looks good to me:
>
> Reviewed-by: Dan Williams <dan.j.williams@...el.com>
>
> Perhaps this and any other pure cleanups can go into a topic branch for
> 6.17 so that it does not need to be sent again if this set gets respun.
> Dave?
Sure. I can pick them up once you are done reviewing this series. Probably should cut things off by end of this week though.
Powered by blists - more mailing lists