* [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info()
[not found] <20220617025152.1908638-1-luriwen@hotmail.com>
@ 2022-06-17 2:51 ` Riwen Lu
2022-06-17 9:42 ` Punit Agrawal
0 siblings, 1 reply; 4+ messages in thread
From: Riwen Lu @ 2022-06-17 2:51 UTC (permalink / raw)
To: rafael, lenb, rui.zhang, robert.moore
Cc: linux-acpi, linux-kernel, devel, punit.agrawal, Riwen Lu
From: Riwen Lu <luriwen@kylinos.cn>
Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
left behind this, remove it.
Signed-off-by: Riwen Lu <luriwen@kylinos.cn>
---
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
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info()
2022-06-17 2:51 ` [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info() Riwen Lu
@ 2022-06-17 9:42 ` Punit Agrawal
0 siblings, 0 replies; 4+ messages in thread
From: Punit Agrawal @ 2022-06-17 9:42 UTC (permalink / raw)
To: Riwen Lu
Cc: rafael, lenb, rui.zhang, robert.moore, linux-acpi, linux-kernel,
devel, punit.agrawal, Riwen Lu
Riwen Lu <luriwen@hotmail.com> writes:
> From: Riwen Lu <luriwen@kylinos.cn>
>
> Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> left behind this, remove it.
>
> Signed-off-by: Riwen Lu <luriwen@kylinos.cn>
>
> ---
> 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@bytedance.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Devel] Re: [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info()
2022-06-17 9:42 ` Punit Agrawal
@ 2022-06-29 16:56 ` Rafael J. Wysocki
-1 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2022-06-29 16:56 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 1868 bytes --]
On Fri, Jun 17, 2022 at 11:42 AM Punit Agrawal
<punit.agrawal(a)bytedance.com> wrote:
>
> Riwen Lu <luriwen(a)hotmail.com> writes:
>
> > From: Riwen Lu <luriwen(a)kylinos.cn>
> >
> > Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> > left behind this, remove it.
> >
> > Signed-off-by: Riwen Lu <luriwen(a)kylinos.cn>
> >
> > ---
> > 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(a)bytedance.com>
Applied as 5.20 material with some edits in the subject and changelog.
Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info()
@ 2022-06-29 16:56 ` Rafael J. Wysocki
0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2022-06-29 16:56 UTC (permalink / raw)
To: Punit Agrawal
Cc: Riwen Lu, Rafael J. Wysocki, Len Brown, Zhang, Rui, Robert Moore,
ACPI Devel Maling List, Linux Kernel Mailing List,
open list:ACPI COMPONENT ARCHITECTURE (ACPICA), Riwen Lu
On Fri, Jun 17, 2022 at 11:42 AM Punit Agrawal
<punit.agrawal@bytedance.com> wrote:
>
> Riwen Lu <luriwen@hotmail.com> writes:
>
> > From: Riwen Lu <luriwen@kylinos.cn>
> >
> > Commit 22e7551eb6fd ("ACPI / processor: Remove acpi_processor_get_limit_info()"),
> > left behind this, remove it.
> >
> > Signed-off-by: Riwen Lu <luriwen@kylinos.cn>
> >
> > ---
> > 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@bytedance.com>
Applied as 5.20 material with some edits in the subject and changelog.
Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-06-29 16:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20220617025152.1908638-1-luriwen@hotmail.com>
2022-06-17 2:51 ` [PATCH v2] ACPI/processor: Remove unused function acpi_processor_get_limit_info() Riwen Lu
2022-06-17 9:42 ` Punit Agrawal
2022-06-29 16:56 [Devel] " Rafael J. Wysocki
2022-06-29 16:56 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.