Re: [PATCH v4 0/3] ACPI: PMIC: a small refactoring

From: Hans de Goede
Date: Fri Jun 07 2024 - 05:06:43 EST


Hi,

On 6/6/24 10:54 PM, Andy Shevchenko wrote:
> Use sizeof(), dev_err()/dev_warn(), and regmap bulk read
> where it makes sense.
>
> In v4:
> - dropped controversial changes (Hans)

Thanks, the whole series looks good to me:

Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>

for the series.

Regards,

Hans




>
> Andy Shevchenko (3):
> ACPI: PMIC: Use sizeof() instead of hard coded value
> ACPI: PMIC: Convert pr_*() to dev_*() printing macros
> ACPI: PMIC: Replace open coded be16_to_cpu()
>
> drivers/acpi/pmic/intel_pmic_chtdc_ti.c | 13 ++++++++-----
> drivers/acpi/pmic/intel_pmic_chtwc.c | 5 +++--
> drivers/acpi/pmic/intel_pmic_xpower.c | 7 ++++---
> 3 files changed, 15 insertions(+), 10 deletions(-)
>