Commit 5674e089 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Andi Shyti
Browse files

i2c: designware: Drop return value from i2c_dw_acpi_configure() 



i2c_dw_acpi_configure() is called without checking of the returned
value, hence just drop it by converting to void.

Reviewed-by: default avatarAndi Shyti <andi.shyti@kernel.org>
Acked-by: default avatarJarkko Nikula <jarkko.nikula@linux.intel.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarAndi Shyti <andi.shyti@kernel.org>
parent f2330bfb

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

+1 −3
Original line number Diff line number Diff line
@@ -255,7 +255,7 @@ static void i2c_dw_acpi_params(struct device *device, char method[], 
	kfree(buf.pointer);

}



int i2c_dw_acpi_configure(struct device *device)

void i2c_dw_acpi_configure(struct device *device)

{

	struct dw_i2c_dev *dev = dev_get_drvdata(device);

	struct i2c_timings *t = &dev->timings;
@@ -285,8 +285,6 @@ int i2c_dw_acpi_configure(struct device *device) 
		dev->sda_hold_time = fs_ht;

		break;

	}



	return 0;

}

EXPORT_SYMBOL_GPL(i2c_dw_acpi_configure);

drivers/i2c/busses/i2c-designware-core.h

+2 −2
Original line number Diff line number Diff line
@@ -420,7 +420,7 @@ int i2c_dw_validate_speed(struct dw_i2c_dev *dev); 
void i2c_dw_adjust_bus_speed(struct dw_i2c_dev *dev);



#if IS_ENABLED(CONFIG_ACPI)

int i2c_dw_acpi_configure(struct device *device);

void i2c_dw_acpi_configure(struct device *device);

#else

static inline int i2c_dw_acpi_configure(struct device *device) { return -ENODEV; }

static inline void i2c_dw_acpi_configure(struct device *device) { }

#endif