24 Jan
2014
24 Jan
'14
12:52 p.m.
On 24 January 2014 16:40, Mark Brown broonie@kernel.org wrote:
On Fri, Jan 24, 2014 at 02:09:24PM +0530, Sachin Kamat wrote:
+#if IS_ENABLED(CONFIG_I2C) +static const char *wm8580_supply_names[WM8580_NUM_SUPPLIES] = {
"AVDD","DVDD","PVDD",+};
This is obviously not sensible - the device doesn't suddenly stop needing power if it is using SPI.
This structure is being used only in wm8580_i2c_probe which is guarded by IS_ENABLED(CONFIG_I2C). Do you mean the existing implementation is wrong or we just need to move it out of the #if?
--
With warm regards,
Sachin