iio: proximity: sx9324: Switch to device_property_match_property_string()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 8 Aug 2023 16:28:00 +0000 (19:28 +0300)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Thu, 16 Nov 2023 19:10:26 +0000 (19:10 +0000)
Replace open coded device_property_match_property_string().

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20230808162800.61651-7-andriy.shevchenko@linux.intel.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/proximity/sx9324.c

index 438f9c9..ac2ed2d 100644 (file)
@@ -888,7 +888,6 @@ sx9324_get_default_reg(struct device *dev, int idx,
        char prop[] = SX9324_PROXRAW_DEF;
        u32 start = 0, raw = 0, pos = 0;
        int ret, count, ph, pin;
-       const char *res;
 
        memcpy(reg_def, &sx9324_default_regs[idx], sizeof(*reg_def));
 
@@ -915,24 +914,21 @@ sx9324_get_default_reg(struct device *dev, int idx,
                reg_def->def = raw;
                break;
        case SX9324_REG_AFE_CTRL0:
-               ret = device_property_read_string(dev,
-                               "semtech,cs-idle-sleep", &res);
-               if (!ret)
-                       ret = match_string(sx9324_csidle, ARRAY_SIZE(sx9324_csidle), res);
+               ret = device_property_match_property_string(dev, "semtech,cs-idle-sleep",
+                                                           sx9324_csidle,
+                                                           ARRAY_SIZE(sx9324_csidle));
                if (ret >= 0) {
                        reg_def->def &= ~SX9324_REG_AFE_CTRL0_CSIDLE_MASK;
                        reg_def->def |= ret << SX9324_REG_AFE_CTRL0_CSIDLE_SHIFT;
                }
 
-               ret = device_property_read_string(dev,
-                               "semtech,int-comp-resistor", &res);
-               if (ret)
-                       break;
-               ret = match_string(sx9324_rints, ARRAY_SIZE(sx9324_rints), res);
-               if (ret < 0)
-                       break;
-               reg_def->def &= ~SX9324_REG_AFE_CTRL0_RINT_MASK;
-               reg_def->def |= ret << SX9324_REG_AFE_CTRL0_RINT_SHIFT;
+               ret = device_property_match_property_string(dev, "semtech,int-comp-resistor",
+                                                           sx9324_rints,
+                                                           ARRAY_SIZE(sx9324_rints));
+               if (ret >= 0) {
+                       reg_def->def &= ~SX9324_REG_AFE_CTRL0_RINT_MASK;
+                       reg_def->def |= ret << SX9324_REG_AFE_CTRL0_RINT_SHIFT;
+               }
                break;
        case SX9324_REG_AFE_CTRL4:
        case SX9324_REG_AFE_CTRL7: