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(-) >