[PATCH v5 0/2] Add TPS68470 power supply support for ov13b10 sensor
From: Arun T
Date: Fri Mar 27 2026 - 09:51:04 EST
This patch series adds power supply management support for the OV13B10 camera sensor when used with TPS68470 PMIC, commonly used on Intel-based camera platforms.
The OV13B10 sensor requires three power supplies (DOVDD, AVDD, DVDD) which are provided by the TPS68470 PMIC.
This series implements proper regulator framework integration to manage these power supplies during sensor power on/off sequences.
Changes in V5:
- keep supply names to only those consumed for sensor driver like dvdd, avdd, dovdd regulators
- Removed always_on flag for other power sources except only VSIO power rail
- Removed unused gpio names (s_idle/s_enable) in gpio lookup table
- changed gpio lookup table name as intel_nvl_ovti13b1_gpios
Suggested by Dan Scally:
- Limit regulator supplies to only those consumed by sensor driver: Keep only dvdd, avdd, dovdd; remove unused vdd, vsio, vddd, vdda, vdddo mapping
- Only VSIO should be forced always-on; drop it from CORE, ANA, VCM, and VIO rails to reduce power draw.
- Rename table to intel_nvl_ovti13b1_gpios, keep only reset GPIO (remove unsupported s_idle and s_enable)
- Remove gpio lookup table name like intel_nvl_ovti13b1_gpios
Suggested by Sakari Alius:
- Asked to remove always_on flag for all power sources except vsio
Changes in v4:
- Discrete INT3472 supplying regulator to i2c-OVTI13B1:00 and tps68470 supplying regulator to i2c-OVTI13B1:01.
- Change Analog regulator name as avdd for tps68470 pmic
- Fixed regulator_consumer_supply maps to i2c-OVTI13B1:01 device.
- Removed vdd regulator from Power supply names list
Suggested by Hans:
- No "avdd" found, only "vdda" and "ana" - check if "ana" should be changed to "avdd".
- Drop unused regulator maps. Only keep the 3 maps used by sensor-driver.
- For "i2c-OVTI13B1:01" consumer, only map "dovdd", "avdd", and "dvdd" supplies.
Changes in v3:
- Fixed check patch errors as updated by sakari
- Enabled VCM regulator for ov13b10 sensor to enable autofocus to work
Suggested by Sakari:
- Add include header file in alphabetical order
- Remove extra space
- Asked to check vcm instance
Changes in v2:
- Added power supply names array for the three required regulators
- Implement regulator handling with devm_regulator_bulk_get()
- Add regulator_bulk_enable() and regulator_bulk_disable() for sensor power on/off sequence
Suggested by Hans:
- Add multiple regulators you should use the bulk regulator API
- Use use "reset" instead of "s_resetn" for parse gpio name.
- Add diffent regulator power supplies in single array
Arun T (2):
platform/x86: int3472: Add TPS68470 board data for Intel nvl
media: i2c: ov13b10: support tps68470 regulator and gpio
drivers/media/i2c/ov13b10.c | 47 ++++---
.../x86/intel/int3472/tps68470_board_data.c | 127 ++++++++++++++++++
2 files changed, 153 insertions(+), 21 deletions(-)
--
2.43.0