Re: [PATCH] eeprom: at24: code shrink

From: Uwe Kleine-König
Date: Mon Dec 11 2017 - 03:08:56 EST


Hello,

On Sun, Dec 10, 2017 at 08:29:47PM +0100, Bartosz Golaszewski wrote:
> @@ -625,7 +610,7 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
> at24->offset_adj = at24_get_offset_adj(chip.flags, chip.byte_len);
>
> at24->client[0].client = client;
> - at24->client[0].regmap = devm_regmap_init_i2c(client, config);
> + at24->client[0].regmap = devm_regmap_init_i2c(client, &config);
> if (IS_ERR(at24->client[0].regmap))
> return PTR_ERR(at24->client[0].regmap);
>

One side effect here that the lockdep name changes from (unhelpful)
"config" to (unhelpful and ugly) "&config". Probably doesn't matter much
...

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |