Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1283285 > unrolled thread

Re: [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain during ->probe()

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain  during ->probe() "Shevchenko, Andriy" <andriy.shevchenko@intel.com> - 2015-12-03 20:40 +0100
    Re: [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain  during ->probe() Jarkko Nikula <jarkko.nikula@linux.intel.com> - 2015-12-04 14:10 +0100

#1283285 — Re: [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain during ->probe()

T24gRnJpLCAyMDE1LTExLTI3IGF0IDExOjU2ICswMjAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6
DQo+ID4gPiA+ID7CoA0KPiBPbiBGcmksIDIwMTUtMTEtMjcgYXQgMDA6MTUgKzAxMDAsIFJhZmFl
bCBKLiBXeXNvY2tpIHdyb3RlOg0KPiA+IE9uIFRodXJzZGF5LCBOb3ZlbWJlciAyNiwgMjAxNSAw
Njo0NToxNyBQTSBBbmR5IFNoZXZjaGVua28gd3JvdGU6DQo+ID4gPiBPbiBUaHUsIDIwMTUtMTEt
MjYgYXQgMTg6MzAgKzAyMDAsIEphcmtrbyBOaWt1bGEgd3JvdGU6DQo+ID4gPiA+IE9uIDExLzI2
LzIwMTUgMDU6MTkgUE0sIEFuZHkgU2hldmNoZW5rbyB3cm90ZToNCj4gPiA+ID4gVGhpcyB3b24n
dCBmaXggbGlrZSByZXZlcnQgb2Ygb3JpZ2luYWwgY29tbWl0IGRvZXMuDQoNCkphcmtrbywgSSB3
aWxsIHNwbGl0IHRoaXMgb25lIHRvIHRoZSByZXZlcnQgKHdpdGggRml4ZXMgdGFnKSBhbmQgbmV3
DQpwYXRjaCB0byB0YXJnZXQgRE1BIGlzc3VlLg0KwqAgwqDCoA0KDQo+ID4gPiA+IFByaW1hcnkg
cHJvYmxlbSBoZXJlIA0KPiA+ID4gPiBpcyB0aGF0IHRoZXJlIGlzIG5vIGV4cGxpY2l0IHBvd2Vy
IG9uIGF0IGFsbCBkdXJpbmcgTFBTUyBkZXZpY2UNCj4gPiA+ID4gcHJvYmUNCj4gPiA+ID4gYmVj
YXVzZSBkZXYtPnBtX2RvbWFpbiBpcyBzZXQgYmVmb3JlIHByb2JpbmcuDQo+ID4gPiANCj4gPiA+
IEFuZCB3ZSBjYW4ndCBkbyB0aGlzIGFzIGluIHZlcnkgb3JpZ2luYWwgY29kZSBvZiBhY3BpX2xw
c3MuYw0KPiA+ID4gc2luY2UNCj4gPiA+IERNQQ0KPiA+ID4gaGFzIHRvIGJlIHN1cmUgaXQncyBw
b3dlcmVkIG9uIHdoaWxlIHByb2JpbmcuIFdlIGNvdWxkIGd1YXJhbnRlZQ0KPiA+ID4gdGhpcw0K
PiA+ID4gb25seSBpbiBjYXNlIHdoZW4gUE0gZG9tYWluIGlzIGFzc2lnbmVkIGFscmVhZHkgYW5k
IHdlIGRvIG91cg0KPiA+ID4gcXVpcmsNCj4gPiA+IGZvcg0KPiA+ID4gaXQuDQo+ID4gPiANCj4g
PiA+IEZyb20gbXkgcG9pbnQgb2YgdmlldyB3ZSBoYXZlIHRvIGZpeCBoYW5nIGZpcnN0IHNpbmNl
IGl0J3MgbW9zdA0KPiA+ID4gcGFpbmZ1bA0KPiA+ID4gY2FzZSBmb3IgdXNlcnMgYW5kIHRoZWly
IGV4cGVyaWVuY2UuIFRob3VnaCBJJ20gb3BlbiB0byBhbnkNCj4gPiA+IGJldHRlcg0KPiA+ID4g
c29sdXRpb24gaWYgeW91IGhhdmUgYW55IGluIG1pbmQuDQo+ID4gPiANCj4gPiA+ID4gDQo+ID4g
PiA+IGRyaXZlcl9wcm9iZV9kZXZpY2UNCj4gPiA+ID4gwqDCoMKgcGxhdGZvcm1fZHJ2X3Byb3Zl
DQo+ID4gPiA+IMKgwqDCoMKgwqBkZXZfcG1fZG9tYWluX2F0dGFjaA0KPiA+ID4gPiDCoMKgwqDC
oMKgwqDCoGFjcGlfZGV2X3BtX2F0dGFjaA0KPiA+ID4gPiDCoMKgwqDCoMKgwqDCoMKgwqByZXR1
cm5zIGluc3RhbnRseSBiZWNhdXNlIG9mIGRldi0+cG1fZG9tYWluIGlzIHNldA0KPiA+IA0KPiA+
IFRoaXMgbG9va3MgbGlrZSBhIGNhbmRpZGF0ZSBmb3IgdGhlIG5ldyBQTSBkb21haW4gY2FsbGJh
Y2tzLA0KPiA+IC0+YWN0aXZhdGUgYW5kDQo+ID4gLT5kaXNtaXNzLg0KPiA+IA0KPiA+IC0+YWN0
aXZhdGUoKSBpcyBjYWxsZWQgYmVmb3JlIHRoZSBwcm9iZSwgc28gaXQgbWF5IHBvd2VyIHVwIHRo
aW5ncy4NCj4gPiANCj4gPiAtPmRpc21pc3MoKSBpbiB0dXJuIGlzIGNhbGxlZCBpbiB0aGUgZmFp
bGVkIHByb2JlIGNhc2UsIHNvIGl0IGNhbg0KPiA+IGRvDQo+ID4gdGhlDQo+ID4gY2xlYW51cC4N
Cj4gPiANCj4gPiBIYXZlIHlvdSBjb25zaWRlcmVkIHVzaW5nIHRoZXNlPw0KPiANCj4gVGhhbmtz
IGZvciB0aGUgaGludC4gV2Ugd2lsbCBjaGVjayB0aGlzLg0KDQpJIGJyaWVmbHkgY2hlY2tlZCB0
aGlzIGZvciBETUEgaXNzdWUuIEl0IHdpbGwgbm90IGhlbHAgYW55aG93LCBzbyB3ZQ0KKmhhdmUg
dG8qIG1vdmUgYSBwb3dlciBkb21haW4gYXNzaWdubWVudCB0byB0aGUgQklORCBzdGFnZS4NCg0K
Rm9yIEkyQyBhbmQgcmVzdCBMUFNTIGRldmljZXMgdGhpcyBtaWdodCBoZWxwICh0aG91Z2ggZGlk
bid0IGxvb2sNCmRlZXBseSkuIE15IHVuZGVyc3RhbmRpbmcgdGhhdCB3ZSBhc3NpZ24gdGhvc2Ug
Y2FsbGJhY2tzIGluIHRoZSBMUFNTDQpjdXN0b20gUE0gZG9tYWluIGFuZCBjYWxsIHRoZW0gZXhw
bGljaXRseSBpbiBhY3BpX2xwc3MuYy4NCg0KVGhlIGNvZGUgd2lsbCBiZSB0aGUgc2FtZSBhcyB3
ZSBhcmUgdXNpbmcgbm93IHRvIGJyaW5nIGRldmljZSBmcm9tDQpydW50aW1lIHN1c3BlbmQgcmVz
dW1lLiBUaGlzIG1lYW5zIHdoZW5ldmVyIHdlIGNhbGwgcHJvYmUgZm9yIGUuZy4gSTJDDQp3ZSBl
bmQgdXAgaW4gYSBzZXF1ZW5jZSBzaW1pbGFyIHRvOg0KwqBwbV9ydW50aW1lX3Jlc3VtZShJMkMp
Ow0KwqAtPnByb2JlKEkyQyk7DQrCoHBtX3J1bnRpbWVfc3VzcGVuZChJMkMpOw0KDQpJIHdpbGwg
dHJ5IHRvIG1vY2sgdXAgdGhpcyBhbmQgY2hlY2sgaWYgaXQgd2lsbCB3b3JrLCB0aG91Z2ggaGF2
ZSBubw0KaWRlYSB3aGF0IHRvIGRvIGlmIEkyQyBkdXJpbmcgcHJvYmUgY2FsbHMgcG1fcnVudGlt
ZV9mb3JiaWQoKS4NCg0KSmFya2tvLCB3aGF0IGRvIHlvdSB0aGluaz8NCg0KLS0gDQpBbmR5IFNo
ZXZjaGVua28gPGFuZHJpeS5zaGV2Y2hlbmtvQGludGVsLmNvbT4NCkludGVsIEZpbmxhbmQgT3kN
Ci0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLQpJbnRlbCBGaW5sYW5kIE95ClJlZ2lzdGVyZWQgQWRkcmVzczogUEwgMjgx
LCAwMDE4MSBIZWxzaW5raSAKQnVzaW5lc3MgSWRlbnRpdHkgQ29kZTogMDM1NzYwNiAtIDQgCkRv
bWljaWxlZCBpbiBIZWxzaW5raSAKClRoaXMgZS1tYWlsIGFuZCBhbnkgYXR0YWNobWVudHMgbWF5
IGNvbnRhaW4gY29uZmlkZW50aWFsIG1hdGVyaWFsIGZvcgp0aGUgc29sZSB1c2Ugb2YgdGhlIGlu
dGVuZGVkIHJlY2lwaWVudChzKS4gQW55IHJldmlldyBvciBkaXN0cmlidXRpb24KYnkgb3RoZXJz
IGlzIHN0cmljdGx5IHByb2hpYml0ZWQuIElmIHlvdSBhcmUgbm90IHRoZSBpbnRlbmRlZApyZWNp
cGllbnQsIHBsZWFzZSBjb250YWN0IHRoZSBzZW5kZXIgYW5kIGRlbGV0ZSBhbGwgY29waWVzLgo=

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1283802

On 12/03/2015 09:29 PM, Shevchenko, Andriy wrote:
> I briefly checked this for DMA issue. It will not help anyhow, so we
> *have to* move a power domain assignment to the BIND stage.
>
> For I2C and rest LPSS devices this might help (though didn't look
> deeply). My understanding that we assign those callbacks in the LPSS
> 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);
>
> 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().
>
> Jarkko, what do you think?
>
I suppose device core will handle it. If the runtime PM is forbidden or 
not initialized at all the device shouldn't idle.

-- 
Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web