summaryrefslogtreecommitdiffstats
path: root/drivers/misc/eeprom
diff options
context:
space:
mode:
authorBartosz Golaszewski <brgl@bgdev.pl>2017-12-18 18:24:43 +0100
committerBartosz Golaszewski <brgl@bgdev.pl>2018-01-01 19:40:49 +0100
commiteef6939849b04bb1ba36f35f5c864eb0a66d1d83 (patch)
treed477a04dd4a343eb56b24e7d794596838a378686 /drivers/misc/eeprom
parentec3c2d518b163dd2be48ac1fee8d7f1686cd769f (diff)
downloadtalos-obmc-linux-eef6939849b04bb1ba36f35f5c864eb0a66d1d83.tar.gz
talos-obmc-linux-eef6939849b04bb1ba36f35f5c864eb0a66d1d83.zip
eeprom: at24: code shrink
A regmap_config struct is pretty big and declaring two of them statically just to tweak the reg_bits value adds unnecessary bloat. Declare the regmap config locally in at24_probe() instead. Bloat-o-meter output for ARM: add/remove: 0/2 grow/shrink: 1/0 up/down: 4/-272 (-268) Function old new delta at24_probe 1560 1564 +4 regmap_config_8 136 - -136 regmap_config_16 136 - -136 Total: Before=7012, After=6744, chg -3.82% Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
Diffstat (limited to 'drivers/misc/eeprom')
-rw-r--r--drivers/misc/eeprom/at24.c23
1 files changed, 6 insertions, 17 deletions
diff --git a/drivers/misc/eeprom/at24.c b/drivers/misc/eeprom/at24.c
index 657074ec0e64..b44a3d2b2b20 100644
--- a/drivers/misc/eeprom/at24.c
+++ b/drivers/misc/eeprom/at24.c
@@ -514,16 +514,6 @@ static unsigned int at24_get_offset_adj(u8 flags, unsigned int byte_len)
}
}
-static const struct regmap_config regmap_config_8 = {
- .reg_bits = 8,
- .val_bits = 8,
-};
-
-static const struct regmap_config regmap_config_16 = {
- .reg_bits = 16,
- .val_bits = 8,
-};
-
static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
{
struct at24_platform_data chip;
@@ -532,7 +522,7 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
struct at24_data *at24;
int err;
unsigned int i, num_addresses;
- const struct regmap_config *config;
+ struct regmap_config regmap_config = { };
u8 test_byte;
if (client->dev.platform_data) {
@@ -601,10 +591,8 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
num_addresses = DIV_ROUND_UP(chip.byte_len,
(chip.flags & AT24_FLAG_ADDR16) ? 65536 : 256);
- if (chip.flags & AT24_FLAG_ADDR16)
- config = &regmap_config_16;
- else
- config = &regmap_config_8;
+ regmap_config.val_bits = 8;
+ regmap_config.reg_bits = (chip.flags & AT24_FLAG_ADDR16) ? 16 : 8;
at24 = devm_kzalloc(&client->dev, sizeof(struct at24_data) +
num_addresses * sizeof(struct at24_client), GFP_KERNEL);
@@ -617,7 +605,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, &regmap_config);
if (IS_ERR(at24->client[0].regmap))
return PTR_ERR(at24->client[0].regmap);
@@ -647,7 +635,8 @@ static int at24_probe(struct i2c_client *client, const struct i2c_device_id *id)
goto err_clients;
}
at24->client[i].regmap = devm_regmap_init_i2c(
- at24->client[i].client, config);
+ at24->client[i].client,
+ &regmap_config);
if (IS_ERR(at24->client[i].regmap)) {
err = PTR_ERR(at24->client[i].regmap);
goto err_clients;
OpenPOWER on IntegriCloud