Use PLATFORM_DEVID_NONE define instead of "-1" value because:
 - it brings some meaning,
 - it might point attention why auto device ID was not used.

Signed-off-by: Krzysztof Kozlowski <k...@kernel.org>
---
 drivers/mfd/rc5t583.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/mfd/rc5t583.c b/drivers/mfd/rc5t583.c
index d0dc48f99096..a4915f411ee8 100644
--- a/drivers/mfd/rc5t583.c
+++ b/drivers/mfd/rc5t583.c
@@ -270,8 +270,9 @@ static int rc5t583_i2c_probe(struct i2c_client *i2c,
                        dev_warn(&i2c->dev, "IRQ init failed: %d\n", ret);
        }
 
-       ret = devm_mfd_add_devices(rc5t583->dev, -1, rc5t583_subdevs,
-                                  ARRAY_SIZE(rc5t583_subdevs), NULL, 0, NULL);
+       ret = devm_mfd_add_devices(rc5t583->dev, PLATFORM_DEVID_NONE,
+                                  rc5t583_subdevs, ARRAY_SIZE(rc5t583_subdevs),
+                                  NULL, 0, NULL);
        if (ret) {
                dev_err(&i2c->dev, "add mfd devices failed: %d\n", ret);
                return ret;
-- 
2.25.1

Reply via email to