Re: [PATCH 1/1] cxl/acpi: Remove cxl_acpi_set_cache_size()

From: Jonathan Cameron

Date: Thu Jan 15 2026 - 12:31:35 EST


On Fri, 9 Jan 2026 08:53:06 -0700
Dave Jiang <dave.jiang@xxxxxxxxx> wrote:

> On 1/9/26 8:40 AM, Li Ming wrote:
> > cxl_acpi_set_cache_size() returns an error only when the size of the
> > cache range is not matched with the CXL address range. Almost all
> > implementation of setting cache size is in cxl_acpi_set_cache_size(),
> > cxl_setup_extended_linear_size() does nothing except printing a warning
> > in above error case, but cxl_acpi_set_cache_size() also prints a warning
> > at the same time. So can consolidates these two functions into one, keep
> > the function name as cxl_setup_extended_linear_size().
> >
> > Signed-off-by: Li Ming <ming.li@xxxxxxxxxxxx>
>
> Thanks for the cleanup. Should've happened when I moved it from core to acpi.
>
> Reviewed-by: Dave Jiang <dave.jiang@xxxxxxxxx>
>
Reviewed-by: Jonathan Cameron <jonathan.cameron@xxxxxxxxxx>