diff for duplicates of <1449171008.15393.158.camel@intel.com> diff --git a/a/1.txt b/N1/1.txt index bb6ecf7..60d3536 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,5 +1,5 @@ On Fri, 2015-11-27 at 11:56 +0200, Andy Shevchenko wrote: -> > > > > +> > > > > > On Fri, 2015-11-27 at 00:15 +0100, Rafael J. Wysocki wrote: > > On Thursday, November 26, 2015 06:45:17 PM Andy Shevchenko wrote: > > > On Thu, 2015-11-26 at 18:30 +0200, Jarkko Nikula wrote: @@ -8,7 +8,7 @@ On Fri, 2015-11-27 at 11:56 +0200, Andy Shevchenko wrote: Jarkko, I will split this one to the revert (with Fixes tag) and new patch to target DMA issue. - +   > > > > Primary problem here > > > > is that there is no explicit power on at all during LPSS device @@ -33,10 +33,10 @@ patch to target DMA issue. > > > > > > > > > > > driver_probe_device -> > > > platform_drv_prove -> > > > dev_pm_domain_attach -> > > > acpi_dev_pm_attach -> > > > returns instantly because of dev->pm_domain is set +> > > >    platform_drv_prove +> > > >      dev_pm_domain_attach +> > > >        acpi_dev_pm_attach +> > > >          returns instantly because of dev->pm_domain is set > > > > This looks like a candidate for the new PM domain callbacks, > > ->activate and @@ -63,9 +63,9 @@ custom PM domain and call them explicitly in acpi_lpss.c. The code will be the same as we are using now to bring device from runtime suspend resume. This means whenever we call probe for e.g. I2C we end up in a sequence similar to: - pm_runtime_resume(I2C); - ->probe(I2C); - pm_runtime_suspend(I2C); + pm_runtime_resume(I2C); + ->probe(I2C); + pm_runtime_suspend(I2C); I will try to mock up this and check if it will work, though have no idea what to do if I2C during probe calls pm_runtime_forbid(). @@ -85,3 +85,4 @@ This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. +ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥ diff --git a/a/content_digest b/N1/content_digest index 1281364..58346da 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -19,7 +19,7 @@ "\00:1\0" "b\0" "On Fri, 2015-11-27 at 11:56 +0200, Andy Shevchenko wrote:\n" - "> > > > >\302\240\n" + "> > > > >\303\202\302\240\n" "> On Fri, 2015-11-27 at 00:15 +0100, Rafael J. Wysocki wrote:\n" "> > On Thursday, November 26, 2015 06:45:17 PM Andy Shevchenko wrote:\n" "> > > On Thu, 2015-11-26 at 18:30 +0200, Jarkko Nikula wrote:\n" @@ -28,7 +28,7 @@ "\n" "Jarkko, I will split this one to the revert (with Fixes tag) and new\n" "patch to target DMA issue.\n" - "\302\240 \302\240\302\240\n" + "\303\202\302\240 \303\202\302\240\303\202\302\240\n" "\n" "> > > > Primary problem here \n" "> > > > is that there is no explicit power on at all during LPSS device\n" @@ -53,10 +53,10 @@ "> > > \n" "> > > > \n" "> > > > driver_probe_device\n" - "> > > > \302\240\302\240\302\240platform_drv_prove\n" - "> > > > \302\240\302\240\302\240\302\240\302\240dev_pm_domain_attach\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240acpi_dev_pm_attach\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240returns instantly because of dev->pm_domain is set\n" + "> > > > \303\202\302\240\303\202\302\240\303\202\302\240platform_drv_prove\n" + "> > > > \303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240dev_pm_domain_attach\n" + "> > > > \303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240acpi_dev_pm_attach\n" + "> > > > \303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240\303\202\302\240returns instantly because of dev->pm_domain is set\n" "> > \n" "> > This looks like a candidate for the new PM domain callbacks,\n" "> > ->activate and\n" @@ -83,9 +83,9 @@ "The code will be the same as we are using now to bring device from\n" "runtime suspend resume. This means whenever we call probe for e.g. I2C\n" "we end up in a sequence similar to:\n" - "\302\240pm_runtime_resume(I2C);\n" - "\302\240->probe(I2C);\n" - "\302\240pm_runtime_suspend(I2C);\n" + "\303\202\302\240pm_runtime_resume(I2C);\n" + "\303\202\302\240->probe(I2C);\n" + "\303\202\302\240pm_runtime_suspend(I2C);\n" "\n" "I will try to mock up this and check if it will work, though have no\n" "idea what to do if I2C during probe calls pm_runtime_forbid().\n" @@ -104,6 +104,7 @@ "This e-mail and any attachments may contain confidential material for\n" "the sole use of the intended recipient(s). Any review or distribution\n" "by others is strictly prohibited. If you are not the intended\n" - recipient, please contact the sender and delete all copies. + "recipient, please contact the sender and delete all copies.\n" + "\303\277\303\264\303\250\302\272{.n\303\207+\302\211\302\267\302\237\302\256\302\211\302\255\302\206+%\302\212\303\213\303\277\302\261\303\251\303\235\302\266\027\302\245\302\212w\303\277\302\272{.n\303\207+\302\211\302\267\302\245\302\212{\302\261\303\276G\302\253\302\235\303\251\303\277\302\212{ay\302\272\035\303\212\302\207\303\232\302\231\303\253,j\a\302\255\302\242f\302\243\302\242\302\267h\302\232\302\217\303\257\302\201\303\252\303\277\302\221\303\252\303\247z_\303\250\302\256\003(\302\255\303\251\302\232\302\216\302\212\303\235\302\242j\"\302\235\303\272\032\302\266\033m\302\247\303\277\303\277\302\276\a\302\253\303\276G\302\253\302\235\303\251\303\277\302\242\302\270?\302\231\302\250\303\250\302\255\303\232&\302\243\303\270\302\247~\302\217\303\241\302\266iO\302\225\303\246\302\254z\302\267\302\232v\303\230^\024\004\032\302\266\033m\302\247\303\277\303\277\303\203\f\303\277\302\266\303\254\303\277\302\242\302\270?\302\226I\302\245" -00332483d24e8d773fde01ab606061858f9c583b78ec6ace4c529d93fa712a24 +ef8ccebac4112cd9f679641ce0616719eaf1a122f4f8d84e440b930bb8266dbd
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.