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 <krzk@xxxxxxxxxx> --- drivers/mfd/twl6040.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mfd/twl6040.c b/drivers/mfd/twl6040.c index b9c6d94b4002..702f9e216d42 100644 --- a/drivers/mfd/twl6040.c +++ b/drivers/mfd/twl6040.c @@ -794,8 +794,8 @@ static int twl6040_probe(struct i2c_client *client, regcache_cache_only(twl6040->regmap, true); regcache_mark_dirty(twl6040->regmap); - ret = mfd_add_devices(&client->dev, -1, twl6040->cells, children, - NULL, 0, NULL); + ret = mfd_add_devices(&client->dev, PLATFORM_DEVID_NONE, twl6040->cells, + children, NULL, 0, NULL); if (ret) goto readyirq_err; -- 2.25.1