Re: [PATCH 00/30] ACPI-processor: Fine-tuning for several function implementations

SF Markus Elfring
2017-08-06T10:05:24Z

> Date: Tue, 9 Sep 2016 20:15:05 +0200
>
> Several update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (30):
> Use kmalloc_array() in acpi_processor_get_performance_states()
> Improve two size determinations in acpi_processor_get_performance_states()
> Rename jump labels in acpi_processor_get_performance_states()
> Delete two unnecessary initialisations in acpi_processor_get_performance_states()
> Rename jump labels in acpi_processor_preregister_performance()
> Move a success indication in acpi_processor_preregister_performance()
> Rename jump labels in acpi_processor_get_psd()
> Delete two unnecessary initialisations in acpi_processor_get_psd()
> Improve a size determination in acpi_processor_get_psd()
> Rename jump labels in acpi_processor_get_performance_control()
> Delete two unnecessary initialisations in acpi_processor_get_performance_control()
> Rename jump labels in acpi_processor_ppc_notifier()
> Delete an unnecessary initialisation in acpi_processor_ppc_notifier()
> Delete an unnecessary initialisation in acpi_processor_get_performance_info()
> Delete an unnecessary initialisation in acpi_processor_get_platform_limit()
> Use kmalloc_array() in acpi_processor_get_throttling_states()
> Improve another size determination in acpi_processor_get_throttling_states()
> Rename jump labels in acpi_processor_get_throttling_states()
> Delete two unnecessary initialisations in acpi_processor_get_throttling_states()
> Fix jump targets in acpi_processor_get_throttling_info()
> Rename jump labels in acpi_processor_get_tsd()
> Delete two unnecessary initialisations in acpi_processor_get_tsd()
> Improve a size determination in acpi_processor_get_tsd()
> Rename jump labels in acpi_processor_get_throttling_control()
> Delete two unnecessary initialisations in acpi_processor_get_throttling_control()
> Fix jump targets in acpi_processor_reevaluate_tstate()
> Delete an unnecessary initialisation in acpi_processor_reevaluate_tstate()
> Rename a jump label in acpi_processor_get_platform_limit()
> Delete an unnecessary initialisation in acpi_processor_get_platform_limit()
> Improve jump targets in acpi_processor_update_tsd_coord()
>
> drivers/acpi/processor_perflib.c | 87 +++++++++++++--------------
> drivers/acpi/processor_throttling.c | 116 ++++++++++++++++--------------------
> 2 files changed, 93 insertions(+), 110 deletions(-)

How do you think about to take another look at change possibilities
for these source files?

Regards,
Markus