{"id":818002,"url":"http://patchwork.ozlabs.org/api/covers/818002/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-i2c/cover/65494652.vEfz4tCBDb@aspire.rjw.lan/","project":{"id":35,"url":"http://patchwork.ozlabs.org/api/projects/35/?format=json","name":"Linux I2C development","link_name":"linux-i2c","list_id":"linux-i2c.vger.kernel.org","list_email":"linux-i2c@vger.kernel.org","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<65494652.vEfz4tCBDb@aspire.rjw.lan>","list_archive_url":null,"date":"2017-09-24T23:13:29","name":"[v4,0/3] i2c: designware: Runtime PM aware system sleep handling","submitter":{"id":26536,"url":"http://patchwork.ozlabs.org/api/people/26536/?format=json","name":"Rafael J. Wysocki","email":"rjw@rjwysocki.net"},"mbox":"http://patchwork.ozlabs.org/project/linux-i2c/cover/65494652.vEfz4tCBDb@aspire.rjw.lan/mbox/","series":[{"id":4853,"url":"http://patchwork.ozlabs.org/api/series/4853/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-i2c/list/?series=4853","date":"2017-09-24T23:20:35","name":"i2c: designware: Runtime PM aware system sleep handling","version":4,"mbox":"http://patchwork.ozlabs.org/series/4853/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/818002/comments/","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=linux-i2c-owner@vger.kernel.org;\n\treceiver=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3y0kHS2Z3xz9t3t\n\tfor ;\n\tMon, 25 Sep 2017 09:40:52 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S932386AbdIXXkt (ORCPT );\n\tSun, 24 Sep 2017 19:40:49 -0400","from cloudserver094114.home.net.pl ([79.96.170.134]:43108 \"EHLO\n\tcloudserver094114.home.net.pl\" rhost-flags-OK-OK-OK-OK)\n\tby vger.kernel.org with ESMTP id S1753552AbdIXXkn (ORCPT\n\t); Sun, 24 Sep 2017 19:40:43 -0400","from 79.184.252.54.ipv4.supernova.orange.pl (79.184.252.54) (HELO\n\taspire.rjw.lan)\n\tby serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer\n\t0.82) id 90c134c2f38884a3; Mon, 25 Sep 2017 01:40:41 +0200"],"From":"\"Rafael J. Wysocki\" ","To":"linux-pm@vger.kernel.org","Cc":"linux-i2c@vger.kernel.org, Wolfram Sang ,\n\tlinux-acpi@vger.kernel.org, Kevin Hilman ,\n\tJarkko Nikula ,\n\tAndy Shevchenko ,\n\tMika Westerberg ,\n\tJisheng Zhang ,\n\tJohn Stultz , Guodong Xu ,\n\tSumit Semwal ,\n\tHaojian Zhuang ,\n\tJohannes Stezenbach , Ulf Hansson ,\n\tLee Jones , Rajat Jain ","Subject":"[PATCH v4 0/3] i2c: designware: Runtime PM aware system sleep\n\thandling","Date":"Mon, 25 Sep 2017 01:13:29 +0200","Message-ID":"<65494652.vEfz4tCBDb@aspire.rjw.lan>","In-Reply-To":"<3958866.l2qnKDbinI@aspire.rjw.lan>","References":"<3023226.l5IfJK6GIc@aspire.rjw.lan>\n\t<3958866.l2qnKDbinI@aspire.rjw.lan>","MIME-Version":"1.0","Content-Transfer-Encoding":"7Bit","Content-Type":"text/plain; charset=\"us-ascii\"","Sender":"linux-i2c-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"linux-i2c@vger.kernel.org"},"content":"Hi,\n\nThe original purpose of this set was as stated below:\n\nOn Wednesday, September 6, 2017 1:41:37 AM CEST Rafael J. Wysocki wrote:\n> On Monday, September 4, 2017 1:05:57 AM CEST Rafael J. Wysocki wrote:\n> > Hi,\n> \n> An update related to the dependency on intel-lpss.\n> \n> > The point here is to avoid runtime resuming i2c designware devices during\n> > system suspend in the driver's ->suspend callback in the case when the\n> > ACPI PM domain is not present. That will allow us to deal with the\n> > ACPI PM domain case going forward, among other things.\n> > \n> > The first patch cleans up the runtime PM handling in the i2c-designware-platdrv\n> > _probe() routine so as to make it enable runtime PM in all cases, but prevent\n> > the device from being runtime suspended via pm_runtime_forbid() if\n> > pm_disabled is set.\n> \n> The second one pushed the intel-lpss system suspend/resume callbacks to the\n> late/early stages of suspend/resume, respectively, so that the i2c-designware-platdrv\n> ones can be pushed too.\n> \n> > The second one get rids of some ugly code and makes the PM callbacks of the\n> > driver handle runtime-suspended devices during system suspend/resume.\n> \n> This is the third one and it does less than it did before now.\n> \n\nIn the meantime, though, Rajat reported that there actually were problems with\ni2c-designware being suspended too early (as in some cases dependent device\ndrivers attempted to use i2c during the late suspend phase of system suspend),\nso here it goes again as a fix.\n\nPatches [1-2/3] have not changed since v3 (other than by acquiring tags) and\npatch [3/3] has only been rebased on top of 4.14-rc1 (and has acquired some\ntags too).\n\nThanks,\nRafael"}