drivers: Standardize device identification to device_id().

Author: nedseb

lib/apds9960/apds9960/device.py

@@ -79,6 +79,9 @@ def __init__(self, i2c, address=APDS9960_I2C_ADDR, valid_id=APDS9960_DEV_ID):
         self._write_reg(APDS9960_REG_GCONF3, APDS9960_DEFAULT_GCONF3)
         self.set_gesture_int_enable(APDS9960_DEFAULT_GIEN)
 
+    def device_id(self):
+        return self._read_reg(APDS9960_REG_ID)
+
     def get_mode(self):
         return self._read_reg(APDS9960_REG_ENABLE)
 

lib/hts221/hts221/device.py

@@ -57,7 +57,7 @@ def _read_reg16(self, reg):
         return (hi << 8) + lo
 
     # Device identification
-    def who_am_i(self):
+    def device_id(self):
         return self._read_reg(HTS221_WHO_AM_I)
 
     # get STATUS register

lib/ism330dl/ism330dl/device.py

@@ -77,11 +77,11 @@ def _read_vector(self, reg):
     # Device check
     # --------------------------------------------------
 
-    def who_am_i(self):
+    def device_id(self):
         return self._read_u8(REG_WHO_AM_I)
 
     def check_device(self):
-        if self.who_am_i() != ISM330DL_WHO_AM_I_VALUE:
+        if self.device_id() != ISM330DL_WHO_AM_I_VALUE:
             raise ISM330DLNotFound("ISM330DL not detected")
 
     # --------------------------------------------------

lib/lis2mdl/examples/magnet_test.py

@@ -234,7 +234,7 @@ def test_reads(dev):
     print("\n=== TEST READS ===")
 
     # WHO_AM_I
-    who = dev.read_who_am_i()
+    who = dev.device_id()
     print(f"WHO_AM_I=0x{who:02X}  expected 0x40 =>", "OK" if who == 0x40 else "FAIL")
     ok &= who == 0x40
 
@@ -652,7 +652,7 @@ def test_reboot(dev):
     ok &= reboot_cleared
 
     # WHO_AM_I still correct
-    who = dev.read_who_am_i()
+    who = dev.device_id()
     print(f"WHO_AM_I=0x{who:02X} expected 0x40 =>", "OK" if who == 0x40 else "FAIL")
     ok &= who == 0x40
 

lib/lis2mdl/lis2mdl/device.py

@@ -290,7 +290,7 @@ def calibrate_temperature(self, ref_low, measured_low, ref_high, measured_high):
 
     # --- IDENTITY & HARDWARE OFFSETS ---
 
-    def read_who_am_i(self) -> int:
+    def device_id(self) -> int:
         """Reads WHO_AM_I (should be 0x40)."""
         return self._read_reg(LIS2MDL_WHO_AM_I)
 

lib/vl53l1x/vl53l1x/device.py

@@ -103,7 +103,7 @@ def __init__(self, i2c, address=0x29):
         self.address = address
         self.reset()
         machine.lightsleep(1)
-        if self.read_model_id() != 0xEACC:
+        if self.device_id() != 0xEACC:
             raise RuntimeError("Failed to find expected ID register values. Check wiring!")
         # write default configuration
         self.i2c.writeto_mem(self.address, 0x2D, VL53L1X_DEFAULT_CONFIGURATION, addrsize=16)
@@ -126,7 +126,7 @@ def _read_reg16(self, reg):
         data = self.i2c.readfrom_mem(self.address, reg, 2, addrsize=16)
         return (data[0] << 8) + data[1]
 
-    def read_model_id(self):
+    def device_id(self):
         return self._read_reg16(0x010F)
 
     def reset(self):

tests/scenarios/hts221.yaml

@@ -51,7 +51,7 @@ tests:
 
   - name: "Read WHO_AM_I via method"
     action: call
-    method: who_am_i
+    method: device_id
     expect: 0xBC
     mode: [mock, hardware]
 

tests/scenarios/ism330dl.yaml

@@ -36,7 +36,7 @@ tests:
 
   - name: "Read WHO_AM_I via method"
     action: call
-    method: who_am_i
+    method: device_id
     expect: 0x6A
     mode: [mock, hardware]
 

tests/scenarios/lis2mdl.yaml

@@ -42,7 +42,7 @@ tests:
 
   - name: "Read WHO_AM_I via method"
     action: call
-    method: read_who_am_i
+    method: device_id
     expect: 0x40
     mode: [mock, hardware]
 
@@ -165,15 +165,15 @@ tests:
     action: hardware_script
     script: |
       dev.soft_reset()
-      result = dev.read_who_am_i()
+      result = dev.device_id()
     expect: 0x40
     mode: [hardware]
 
   - name: "Reboot then WHO_AM_I"
     action: hardware_script
     script: |
       dev.reboot()
-      result = dev.read_who_am_i()
+      result = dev.device_id()
     expect: 0x40
     mode: [hardware]
 

tests/scenarios/vl53l1x.yaml

@@ -37,7 +37,7 @@ mock_registers:
 tests:
   - name: "Verify model ID register"
     action: call
-    method: read_model_id
+    method: device_id
     expect: 0xEACC
     mode: [mock, hardware]