From: Riwen Lu <[email protected]>
Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
left behind this, remove it.
Signed-off-by: Riwen Lu <[email protected]>
---
v1 -> v2:
- Make this patch base on ("ACPI: Split out processor thermal register
from ACPI PSS").
---
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,
--
2.25.1
Riwen Lu <[email protected]> writes:
> From: Riwen Lu <[email protected]>
>
> Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> left behind this, remove it.
>
> Signed-off-by: Riwen Lu <[email protected]>
>
> ---
> 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 <[email protected]>
On Fri, Jun 17, 2022 at 11:42 AM Punit Agrawal
<[email protected]> wrote:
>
> Riwen Lu <[email protected]> writes:
>
> > From: Riwen Lu <[email protected]>
> >
> > Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> > left behind this, remove it.
> >
> > Signed-off-by: Riwen Lu <[email protected]>
> >
> > ---
> > 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 <[email protected]>
Applied as 5.20 material with some edits in the subject and changelog.
Thanks!