Skip to content

Commit 982959f

Browse files
andy-shevAndi Shyti
authored andcommitted
i2c: designware: Drop return value from dw_i2c_of_configure()
dw_i2c_of_configure() is called without checking of the returned value, hence just drop it by converting to void. Reviewed-by: Andi Shyti <andi.shyti@kernel.org> Reviewed-by: Mario Limonciello <mario.limonciello@amd.com> Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
1 parent 5674e08 commit 982959f

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

drivers/i2c/busses/i2c-designware-platdrv.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ static int mscc_twi_set_sda_hold_time(struct dw_i2c_dev *dev)
110110
return 0;
111111
}
112112

113-
static int dw_i2c_of_configure(struct platform_device *pdev)
113+
static void dw_i2c_of_configure(struct platform_device *pdev)
114114
{
115115
struct dw_i2c_dev *dev = platform_get_drvdata(pdev);
116116

@@ -123,18 +123,15 @@ static int dw_i2c_of_configure(struct platform_device *pdev)
123123
default:
124124
break;
125125
}
126-
127-
return 0;
128126
}
129127
#else
130128
static int bt1_i2c_request_regs(struct dw_i2c_dev *dev)
131129
{
132130
return -ENODEV;
133131
}
134132

135-
static inline int dw_i2c_of_configure(struct platform_device *pdev)
133+
static inline void dw_i2c_of_configure(struct platform_device *pdev)
136134
{
137-
return -ENODEV;
138135
}
139136
#endif
140137

0 commit comments

Comments
 (0)