Re: [PATCH 5/5] power: supply: bd99954: Drop bad register fields
Andy Shevchenko <[email protected]>
| Newsgroups | org.kernel.vger.linux-pm,org.kernel.vger.linux-kernel |
|---|---|
| Organization | Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs, Bertel Jungin Aukio 5, 02600 Espoo |
| Message-ID | <[email protected]> |
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