Re: [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Riwen Lu <luriwen@xxxxxxxxxxx> writes:

> From: Riwen Lu <luriwen@xxxxxxxxxx>
>
> Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> left behind this, remove it.
>
> Signed-off-by: Riwen Lu <luriwen@xxxxxxxxxx>
>
> ---
> v1 -> v2:
>  - Make this patch base on ("ACPI: Split out processor thermal register
>    from ACPI PSS").

For such changes, it is better to send all the related patches as a
series so it's easy to see the dependencies . In a series the easy /
obvious fixes should be earlier so it's easier for them to be merged
while the more significant changes are still being discussed.

Hopefully in this case Rafael too agrees with the dependency patch -
otherwise, it's just extra churn on the lists.

But don't resend just yet - give some time for others to add their
feedback.

> ---
>  include/acpi/processor.h | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
> index ba1e3ed98d3d..9fa49686957a 100644
> --- a/include/acpi/processor.h
> +++ b/include/acpi/processor.h
> @@ -441,7 +441,6 @@ static inline int acpi_processor_hotplug(struct acpi_processor *pr)
>  #endif /* CONFIG_ACPI_PROCESSOR_IDLE */
>  
>  /* in processor_thermal.c */
> -int acpi_processor_get_limit_info(struct acpi_processor *pr);
>  int acpi_processor_thermal_init(struct acpi_processor *pr,
>  				struct acpi_device *device);
>  void acpi_processor_thermal_exit(struct acpi_processor *pr,

Fwiw,

Reviewed-by: Punit Agrawal <punit.agrawal@xxxxxxxxxxxxx>



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux