Re: [PATCH 3/5] powercap/drivers/dtpm: Simplify the dtpm table

Lukasz Luba
2021-03-09T15:03:22Z

On 3/1/21 9:21 PM, Daniel Lezcano wrote:
> The dtpm table is an array of pointers, that forces the user of the
> table to define initdata along with the declaration of the table
> entry. It is more efficient to create an array of dtpm structure, so
> the declaration of the table entry can be done by initializing the
> different fields.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---
> drivers/powercap/dtpm.c | 4 ++--
> drivers/powercap/dtpm_cpu.c | 4 +++-
> include/linux/dtpm.h | 22 +++++++++-------------
> 3 files changed, 14 insertions(+), 16 deletions(-)
>

LGTM

Reviewed-by: Lukasz Luba <lukasz.luba@arm.com>