diff mbox series

About enabling async suspend/resume on i2c devices

Message ID e766f6cc821cfc57246f4b18eac03a29d46641ac.camel@collabora.com
State Superseded
Headers show
Series About enabling async suspend/resume on i2c devices | expand

Commit Message

Ricardo CaƱuelo March 26, 2020, 3:36 p.m. UTC
Hi all,

I came across this patch: https://marc.info/?l=linux-i2c&m=153270824822381&w=2
and I'm evaluating if it's a good candidate for upstreaming:

----------------------------------------------------------------------
 	if (info->properties) {
----------------------------------------------------------------------

Was this ever tested on something other than a Chromebook? Reading the thread I
got the impression that the patch was intended for Chromebooks only. Wouldn't
adding that line to i2c_new_device affect all i2c clients in every platform?

Thanks!

Best,
Ricardo
diff mbox series

Patch

diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c
index 1ba40bb2b966..3382bb7e1dcc 100644
--- a/drivers/i2c/i2c-core-base.c
+++ b/drivers/i2c/i2c-core-base.c
@@ -749,6 +749,7 @@   i2c_new_device(struct i2c_adapter *adap, struct
i2c_board_info const *info)
 	client->dev.of_node = info->of_node;
 	client->dev.fwnode = info->fwnode;
 
+	device_enable_async_suspend(&client->dev);
 	i2c_dev_set_name(adap, client, info);