Re: [PATCH 5/5] power: supply: bd99954: Drop bad register fields

From: Andy Shevchenko

Date: Mon Aug 10 2026 - 14:05:30 EST


On Mon, Aug 10, 2026 at 01:10:03PM +0300, Matti Vaittinen wrote:

> The bd99954 driver creates regmap fields for all the registers. All the
> fields for VBUS_UCD_FCTRL_SET (extended command 0x33) are accidentally
> created for VCC_UCD_FCTRL_SET (extended command 0x2b), causing all
> reads/writes for the fields to access wrong register. Luckily the fields
> for VBUS_UCD_FCTRL_SET are unused.
>
> Taking a look at the data-sheet:
> https://fscdn.rohm.com/en/products/databook/datasheet/ic/power/battery_management/bd99954xxx-e.pdf

Make it a Link or Datasheet tag.

> all the fields in the VBUS_UCD_FCTRL_SET are marked as:
> "In normal operation, please don’t set these registers" with not much of
> additional documentation. As the fields are wrong and because there seems
> to be no users of the fields - it is better to just drop them. This will
> also decrease the memory footprint.
>
> Drop incorrect VBUS_UCD_FCTRL_SET fields.

--
With Best Regards,
Andy Shevchenko