Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit 58e04f3

Browse files
andy-shevrafaeljw
authored andcommitted
ACPI: PMIC: Use sizeof() instead of hard coded value
It's better to use sizeof() of a given buffer than spreading a hard coded value. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com> Reviewed-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 83a7eef commit 58e04f3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

drivers/acpi/pmic/intel_pmic_chtdc_ti.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ static int chtdc_ti_pmic_get_raw_temp(struct regmap *regmap, int reg)
8787
{
8888
u8 buf[2];
8989

90-
if (regmap_bulk_read(regmap, reg, buf, 2))
90+
if (regmap_bulk_read(regmap, reg, buf, sizeof(buf)))
9191
return -EIO;
9292

9393
/* stored in big-endian */

drivers/acpi/pmic/intel_pmic_xpower.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@ static int intel_xpower_pmic_get_raw_temp(struct regmap *regmap, int reg)
255255
if (ret)
256256
return ret;
257257

258-
ret = regmap_bulk_read(regmap, AXP288_GP_ADC_H, buf, 2);
258+
ret = regmap_bulk_read(regmap, AXP288_GP_ADC_H, buf, sizeof(buf));
259259
if (ret == 0)
260260
ret = (buf[0] << 4) + ((buf[1] >> 4) & 0x0f);
261261

0 commit comments

Comments
 (0)