[PATCH v2 2/2] i2c: mux: ltc4306: Drop enum ltc_type and split chips[]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Drop enum ltc_type and split the array chips[] as individual
variables, and make lines shorter by referring to e.g. &ltc_4305_chip
instead of &chips[ltc_4305].

Suggested-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
---
v2:
 * New patch
---
 drivers/i2c/muxes/i2c-mux-ltc4306.c | 28 +++++++++++-----------------
 1 file changed, 11 insertions(+), 17 deletions(-)

diff --git a/drivers/i2c/muxes/i2c-mux-ltc4306.c b/drivers/i2c/muxes/i2c-mux-ltc4306.c
index c7dfd5eba413..c4f090e8d6db 100644
--- a/drivers/i2c/muxes/i2c-mux-ltc4306.c
+++ b/drivers/i2c/muxes/i2c-mux-ltc4306.c
@@ -34,11 +34,6 @@
 #define LTC_GPIO_ALL_INPUT	0xC0
 #define LTC_SWITCH_MASK		0xF0
 
-enum ltc_type {
-	ltc_4305,
-	ltc_4306,
-};
-
 struct chip_desc {
 	u8 nchans;
 	u8 num_gpios;
@@ -50,14 +45,13 @@ struct ltc4306 {
 	const struct chip_desc *chip;
 };
 
-static const struct chip_desc chips[] = {
-	[ltc_4305] = {
-		.nchans = LTC4305_MAX_NCHANS,
-	},
-	[ltc_4306] = {
-		.nchans = LTC4306_MAX_NCHANS,
-		.num_gpios = 2,
-	},
+static const struct chip_desc ltc_4305_chip = {
+	.nchans = LTC4305_MAX_NCHANS
+};
+
+static const struct chip_desc ltc_4306_chip = {
+	.nchans = LTC4306_MAX_NCHANS,
+	.num_gpios = 2
 };
 
 static bool ltc4306_is_volatile_reg(struct device *dev, unsigned int reg)
@@ -192,15 +186,15 @@ static int ltc4306_deselect_mux(struct i2c_mux_core *muxc, u32 chan)
 }
 
 static const struct i2c_device_id ltc4306_id[] = {
-	{ "ltc4305", .driver_data = (kernel_ulong_t)&chips[ltc_4305] },
-	{ "ltc4306", .driver_data = (kernel_ulong_t)&chips[ltc_4306] },
+	{ "ltc4305", .driver_data = (kernel_ulong_t)&ltc_4305_chip },
+	{ "ltc4306", .driver_data = (kernel_ulong_t)&ltc_4306_chip },
 	{ }
 };
 MODULE_DEVICE_TABLE(i2c, ltc4306_id);
 
 static const struct of_device_id ltc4306_of_match[] = {
-	{ .compatible = "lltc,ltc4305", .data = &chips[ltc_4305] },
-	{ .compatible = "lltc,ltc4306", .data = &chips[ltc_4306] },
+	{ .compatible = "lltc,ltc4305", .data = &ltc_4305_chip },
+	{ .compatible = "lltc,ltc4306", .data = &ltc_4306_chip },
 	{ }
 };
 MODULE_DEVICE_TABLE(of, ltc4306_of_match);
-- 
2.25.1




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux