Re: [PATCH 2/2] regulator: pf0900: Modify volatile register range definition
From: Francesco Dolcini
Date: Thu Jun 18 2026 - 02:54:40 EST
Hello Joy,
thanks for the patch
On Thu, Jun 18, 2026 at 10:03:06AM +0800, joy.zou@xxxxxxxxxxx wrote:
> From: Joy Zou <joy.zou@xxxxxxx>
>
> The pf0900_range was incorrectly defined as a single continuous range
> from PF0900_REG_DEV_ID to PF0900_REG_SYS_DIAG, which includes many
> non-volatile registers. This could lead to unnecessary I2C/SPI reads
> and potential performance issues.
>
> Ensures only volatile registers are read from the device
> on each access, while other registers can be cached by regmap.
>
> Fixes: 162e23657e53 ("regulator: pf0900: Add PMIC PF0900 support")
>
> Signed-off-by: Joy Zou <joy.zou@xxxxxxx>
No empty lines in-between tags. Remove the newline after Fixes:
Francesco