commit 69fdf9bcc11aff38ce9961c1fdd78befe36d065a
author Jerzy Kasenberg <jerzy@apache.org> Sat Oct 11 20:52:49 2025 +0200
committer kasjer <kasjer@users.noreply.github.com> Fri Oct 24 18:47:21 2025 +0200
tree 32943a743e938ab4b3eff190fb67179762055d39
parent 352fb7cad2262946688b3051db534296215c6b70

drivers/ina226: Add bus driver support

Sensor was only supporting hal i2c.
Now it can be used with bus drivers.

Signed-off-by: Jerzy Kasenberg <jerzy@apache.org>

hw/drivers/sensors/ina226/include/ina226/ina226.h

diff --git a/hw/drivers/sensors/ina226/include/ina226/ina226.h b/hw/drivers/sensors/ina226/include/ina226/ina226.h
index 1cb8f7e..209e1f4 100644
--- a/hw/drivers/sensors/ina226/include/ina226/ina226.h
+++ b/hw/drivers/sensors/ina226/include/ina226/ina226.h
@@ -131,7 +131,11 @@
 STATS_SECT_END
 
 struct ina226_dev {
+#if MYNEWT_VAL(BUS_DRIVER_PRESENT)
+    struct bus_i2c_node i2c_node;
+#else
     struct os_dev dev;
+#endif
     struct sensor sensor;
     /* Hardware wiring config, (pin, shunt, i2c) */
     struct ina226_hw_cfg hw_cfg;
@@ -258,6 +262,12 @@
  */
 int ina226_stop_continuous_mode(struct ina226_dev *ina226);
 
+#if MYNEWT_VAL(BUS_DRIVER_PRESENT)
+int ina226_create_sensor_dev(struct ina226_dev *ina226, const char *name,
+                             const struct bus_i2c_node_cfg *i2c_cfg,
+                             const struct ina226_hw_cfg *hw_cfg);
+#endif
+
 #if MYNEWT_VAL(INA226_CLI)
 /**
  * Initialize the INA226 shell extensions.

hw/drivers/sensors/ina226/src/ina226.c

diff --git a/hw/drivers/sensors/ina226/src/ina226.c b/hw/drivers/sensors/ina226/src/ina226.c
index 6777d1c..d2d72b0 100644
--- a/hw/drivers/sensors/ina226/src/ina226.c
+++ b/hw/drivers/sensors/ina226/src/ina226.c
@@ -29,6 +29,14 @@
 #include <ina226/ina226.h>
 #include <hal/hal_gpio.h>
 
+#if MYNEWT_VAL(BUS_DRIVER_PRESENT)
+#define OS_DEV(dev_)   (&dev_->i2c_node.bnode.odev)
+#define I2C_ADDR(dev_) (dev_->i2c_node.addr)
+#else
+#define OS_DEV(dev_)   (&dev_->dev)
+#define I2C_ADDR(dev_) (dev_->hw_cfg.itf.si_addr)
+#endif
+
 /* Define stat names for querying */
 STATS_NAME_START(ina226_stat_section)
     STATS_NAME(ina226_stat_section, read_count)
@@ -133,6 +141,41 @@
 #define ina226_init_interrupt(i) 0
 #endif
 
+#if MYNEWT_VAL(BUS_DRIVER_PRESENT)
+int
+ina226_write_reg(struct ina226_dev *ina226, uint8_t reg, uint16_t reg_val)
+{
+    int rc;
+    uint8_t payload[3] = { reg, reg_val >> 8, (uint8_t)reg_val };
+
+    STATS_INC(ina226->stats, write_count);
+    rc = bus_node_simple_write(OS_DEV(ina226), payload, sizeof(payload));
+    if (rc) {
+        STATS_INC(ina226->stats, write_errors);
+        INA226_LOG_ERROR("INA226 write I2C failed\n");
+    }
+
+    return rc;
+}
+
+int
+ina226_read_reg(struct ina226_dev *ina226, uint8_t reg, uint16_t *reg_val)
+{
+    int rc;
+    uint8_t buf[2];
+
+    STATS_INC(ina226->stats, read_count);
+    rc = bus_node_simple_write_read_transact(OS_DEV(ina226), &reg, 1, buf, 2);
+    if (rc) {
+        STATS_INC(ina226->stats, read_errors);
+        INA226_LOG_ERROR("INA226 read I2C failed\n");
+    } else {
+        *reg_val = (buf[0] << 8) | buf[1];
+    }
+
+    return rc;
+}
+#else
 int
 ina226_write_reg(struct ina226_dev *ina226, uint8_t reg, uint16_t reg_val)
 {
@@ -203,6 +246,7 @@
 
     return rc;
 }
+#endif
 
 int
 ina226_reset(struct ina226_dev *ina226)
@@ -445,7 +489,7 @@
     }
     if (mfg != INA226_MANUFACTURER_ID) {
         INA226_LOG_ERROR("INA226 read ID failed, no INA226 at 0x%X, found 0x%X 0x%X\n",
-                         ina226->hw_cfg.itf.si_addr, mfg, die);
+                         I2C_ADDR(ina226), mfg, die);
         rc = SYS_ENODEV;
         goto exit;
     }
@@ -495,7 +539,7 @@
         STATS_NAME_INIT_PARMS(ina226_stat_section));
     SYSINIT_PANIC_ASSERT(rc == SYS_EOK);
     /* Register the entry with the stats registry */
-    rc = stats_register(ina226->dev.od_name, STATS_HDR(ina226->stats));
+    rc = stats_register(OS_DEV(ina226)->od_name, STATS_HDR(ina226->stats));
     SYSINIT_PANIC_ASSERT(rc == SYS_EOK);
 
     rc = ina226_init_interrupt(ina226);
@@ -579,3 +623,26 @@
     }
     return rc;
 }
+
+#if MYNEWT_VAL(BUS_DRIVER_PRESENT)
+
+static void
+init_node_cb(struct bus_node *bnode, void *arg)
+{
+    ina226_init(&bnode->odev, arg);
+}
+
+int
+ina226_create_sensor_dev(struct ina226_dev *ina226, const char *name,
+                         const struct bus_i2c_node_cfg *i2c_cfg,
+                         const struct ina226_hw_cfg *hw_cfg)
+{
+    int rc;
+    static struct sensor_itf itf;
+
+    rc = sensor_create_i2c_device(&ina226->i2c_node, name, i2c_cfg,
+                                  init_node_cb, (void *)hw_cfg, &itf);
+
+    return rc;
+}
+#endif