Messages in this thread | | | From | "dbasehore ." <> | Date | Fri, 27 Jul 2018 09:17:12 -0700 | Subject | Re: [PATCH] i2c: enable async suspend/resume on i2c devices |
| |
<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 27, 2018 at 5:07 AM Hans de Goede <<a href="mailto:hdegoede@redhat.com">hdegoede@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br> <br> On 07/27/2018 12:44 PM, Andy Shevchenko wrote:<br> > On Fri, Jul 27, 2018 at 1:55 AM, Derek Basehore <<a href="mailto:dbasehore@chromium.org" target="_blank">dbasehore@chromium.org</a>> wrote:<br> >> This enables the async suspend property for i2c devices. This reduces<br> >> the suspend/resume time considerably on platforms with multiple i2c<br> >> devices (such as a trackpad or touchscreen).<br> > <br> > How did you test this?<br></blockquote><div><br></div><div>It was tested on Apollolake Chromebook devices.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> > <br> > Especially on Chromebooks based on Intel Cherrytrail / Braswell<br> > platforms, they have a painful PMIC vs. OS design solution.<br> > <br> > +Cc: Hans, who did a lot in this area.<br> <br> We disable suspend of the i2c controller for the i2c-bus<br> to which the PMIc is connected on these platforms, so I do not<br> expect this to cause any new issues.<br> <br> But this is something to keep an eye on,<br> <br> Regards,<br> <br> Hans<br> <br> </blockquote></div></div>
| |