From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 1/2] drm/i915/audio: extend get_saved_enc() to support more scenarios Date: Tue, 29 Nov 2016 18:35:51 +0200 Message-ID: <87wpfms0pk.fsf@intel.com> References: <1479193483-136968-1-git-send-email-libin.yang@intel.com> <87vavbty7g.fsf@intel.com> <96A12704CE18D347B625EE2D4A099D194FA6760A@SHSMSX103.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id 82EF06E7F1 for ; Tue, 29 Nov 2016 16:35:55 +0000 (UTC) In-Reply-To: <96A12704CE18D347B625EE2D4A099D194FA6760A@SHSMSX103.ccr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: "Yang, Libin" , "intel-gfx@lists.freedesktop.org" , "ville.syrjala@linux.intel.com" , "Vetter, Daniel" , "Pandiyan, Dhinakaran" , "Kp, Jeeja" , "tiwai@suse.de" Cc: Libin Yang List-Id: intel-gfx@lists.freedesktop.org T24gVHVlLCAyOSBOb3YgMjAxNiwgIllhbmcsIExpYmluIiA8bGliaW4ueWFuZ0BpbnRlbC5jb20+ IHdyb3RlOgo+Pi0tLS0tT3JpZ2luYWwgTWVzc2FnZS0tLS0tCj4+RnJvbTogSmFuaSBOaWt1bGEg W21haWx0bzpqYW5pLm5pa3VsYUBsaW51eC5pbnRlbC5jb21dCj4+U2VudDogRnJpZGF5LCBOb3Zl bWJlciAyNSwgMjAxNiAxMDo0NiBQTQo+PlRvOiBZYW5nLCBMaWJpbiA8bGliaW4ueWFuZ0BpbnRl bC5jb20+OyBpbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnOwo+PnZpbGxlLnN5cmphbGFA bGludXguaW50ZWwuY29tOyBWZXR0ZXIsIERhbmllbCA8ZGFuaWVsLnZldHRlckBpbnRlbC5jb20+ Owo+PlBhbmRpeWFuLCBEaGluYWthcmFuIDxkaGluYWthcmFuLnBhbmRpeWFuQGludGVsLmNvbT47 IEtwLCBKZWVqYQo+PjxqZWVqYS5rcEBpbnRlbC5jb20+OyB0aXdhaUBzdXNlLmRlCj4+Q2M6IFlh bmcsIExpYmluIDxsaWJpbi55YW5nQGludGVsLmNvbT47IExpYmluIFlhbmcgPGxpYmluLnlhbmdA bGludXguaW50ZWwuY29tPgo+PlN1YmplY3Q6IFJlOiBbUEFUQ0ggMS8yXSBkcm0vaTkxNS9hdWRp bzogZXh0ZW5kIGdldF9zYXZlZF9lbmMoKSB0byBzdXBwb3J0Cj4+bW9yZSBzY2VuYXJpb3MKPj4K Pj5PbiBUdWUsIDE1IE5vdiAyMDE2LCBsaWJpbi55YW5nQGludGVsLmNvbSB3cm90ZToKPj4+IEZy b206IExpYmluIFlhbmcgPGxpYmluLnlhbmdAbGludXguaW50ZWwuY29tPgo+Pj4KPj4+IFdoZW4g Ym9vdHVwLCBhdWRpbyBkcml2ZXIgbWF5IG5vdCBrbm93IGl0IGlzIE1TVCBvciBub3QuIFRoZSBh dWRpbwo+Pj4gZHJpdmVyIHdpbGwgcG9sbCBhbGwgdGhlIHBvcnQgJiBwaXBlIGNvbWJpbmF0aW9u cyBpbiBlaXRoZXIgTVNUIG9yCj4+PiBOb24tTVNUIG1vZGUuIGdldF9zYXZlZF9lbmMoKSBzaG91 bGQgaGFuZGxlIHRoaXMgc2l0dWF0aW9uLgo+Pj4KPj4+IFNpZ25lZC1vZmYtYnk6IExpYmluIFlh bmcgPGxpYmluLnlhbmdAbGludXguaW50ZWwuY29tPgo+Pj4gLS0tCj4+PiAgZHJpdmVycy9ncHUv ZHJtL2k5MTUvaW50ZWxfYXVkaW8uYyB8IDMyCj4+PiArKysrKysrKysrKysrKysrKysrKysrKysr KysrLS0tLQo+Pj4gIDEgZmlsZSBjaGFuZ2VkLCAyOCBpbnNlcnRpb25zKCspLCA0IGRlbGV0aW9u cygtKQo+Pj4KPj4+IGRpZmYgLS1naXQgYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9hdWRp by5jCj4+PiBiL2RyaXZlcnMvZ3B1L2RybS9pOTE1L2ludGVsX2F1ZGlvLmMKPj4+IGluZGV4IDQ5 ZjEwNTMuLmM4YTEzNDUgMTAwNjQ0Cj4+PiAtLS0gYS9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRl bF9hdWRpby5jCj4+PiArKysgYi9kcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9hdWRpby5jCj4+ PiBAQCAtNzM3LDI1ICs3MzcsNDkgQEAgc3RhdGljIGludAo+Pmk5MTVfYXVkaW9fY29tcG9uZW50 X2dldF9jZGNsa19mcmVxKHN0cnVjdCBkZXZpY2UgKmtkZXYpCj4+PiAgCXJldHVybiBkZXZfcHJp di0+Y2RjbGtfZnJlcTsKPj4+ICB9Cj4+Pgo+Pj4gKy8qCj4+PiArICogZ2V0IHRoZSBpbnRlbF9l bmNvZGVyIGFjY29yZGluZyB0byB0aGUgcGFyYW1ldGVyIHBvcnQgYW5kIHBpcGUKPj4+ICsgKiBp bnRlbF9lbmNvZGVyIGlzIHNhdmVkIGJ5IHRoZSBpbmRleCBvZiBwaXBlCj4+PiArICogTVNUICYg KHBpcGUgPj0gMCk6IHJldHVybiB0aGUgYXZfZW5jX21hcFtwaXBlXSwKPj4+ICsgKiAgIHdoZW4g cG9ydCBpcyBtYXRjaGVkCj4+PiArICogTVNUICYgKHBpcGUgPCAwKTogdGhpcyBpcyBpbnZhbGlk Cj4+PiArICogTm9uLU1TVCAmIChwaXBlID49IDApOiBvbmx5IHBpcGUgPSAwICh0aGUgZmlyc3Qg ZGV2aWNlIGVudHJ5KQo+Pj4gKyAqICAgd2lsbCBnZXQgdGhlIHJpZ2h0IGludGVsX2VuY29kZXIg d2l0aCBwb3J0IG1hdGNoZWQKPj4+ICsgKiBOb24tTVNUICYgKHBpcGUgPCAwKTogZ2V0IHRoZSBy aWdodCBpbnRlbF9lbmNvZGVyIHdpdGggcG9ydAo+Pj4gK21hdGNoZWQKPj4KPj5UaGlzIGVzc2Vu dGlhbGx5IGRlc2NyaWJlcyB0aGUgY29kZSBpbiBFbmdsaXNoLiBJIGNhbiByZWFkIHRoZSBjb2Rl LiBJIG5lZWQgdG8KPj5rbm93IG1vcmUgYWJvdXQgdGhlIGhpZ2hlciBsZXZlbCBvZiB3aGF0IHRo aXMgZG9lcyBhbmQgKndoeSouCj4KPiBJbiBpbml0aWFsaXphdGlvbiwgYXVkaW8gZHJpdmVyIHdp bGwgY2FsbCBmdW5jdGlvbnMgZ2V0X2VsZCgpIGFuZCBldGMuIEJ1dAo+IGF0IHRoYXQgdGltZSwg YXVkaW8gZHJpdmVyIG1heSBub3Qga25vdyB3aGV0aGVyIGl0IGlzIERQIE1TVCBvciBub3QuCj4g SW4gdGhlIG9yaWdpbmFsIGZ1bmN0aW9uIGdldF9zYXZlZF9lbmMoKSwgaWYgaXQgaXMgRFAgTVNU LCBpdCB3aWxsIHNldCB0aGUgcGlwZQo+IHRvIHRoZSBjb3JyZWN0IHZhbHVlLCBvdGhlcndpc2Us IGl0IHdpbGwgYmUgc2V0IC0xLgo+Cj4gQWx0aG91Z2ggYXVkaW8gZHJpdmVyIGNhbiBnZXQgdGhl IGtub3dsZWRnZSB3aGV0aGVyIGl0IGlzIGluIERQIE1TVCAKPiBtb2RlIG9yIG5vdCBieSByZWFk aW5nIHRoZSBjb2RlYyByZWdpc3Rlci4gQnV0IGl0IHdpbGwgZHJvcCBwZXJmb3JtYW5jZQo+IGVh Y2ggdGltZSBpdCBjYWxsIHRoZSBnZXRfZWxkIGFuZCBvdGhlciBmdW5jdGlvbnMuIEFzIGdmeCBk cml2ZXIgY2FuIGVhc2lseQo+IGtub3cgd2hldGhlciBpdCBpcyBpbiBEUCBNU1QgbW9kZSBvciBu b3QuIEkgd291bGQgbGlrZSB0byBleHRlbmQgdGhlCj4gZ2V0X3NhdmVkX2VuYygpIGZ1bmN0aW9u IHRvIGhhbmRsZSB0aGUgc2l0dWF0aW9uIHRoYXQgYXVkaW8gZHJpdmVyIHN0aWxsCj4gc2VuZCB0 aGUgZGV2aWNlIGlkIGluZm8gZXZlbiBpdCBpcyBpbiBEUCBtb2RlLiBJbiB0aGlzIHNpdHVhdGlv biwgCj4gZ2V0X3NhdmVkX2VuYygpIHdpbGwgcmV0dXJuIHRoZSBjb3JyZWN0IGludGVsX2VuY29k ZXIgaW5zdGVhZCBvZiBwYW5pYy4KClBsZWFzZSBkZXNjcmliZSB0aGF0IGluIHRoZSBjb21taXQg bWVzc2FnZSBhbmQvb3IgY29tbWVudHMgaW4gdGhlIHBhdGNoLgoKPj4KPj5JIGFsc28gbG9vayBh dCB0aGUgY2FsbCBzaXRlcyBvZiBnZXRfc2F2ZWRfZW5jIGFuZCB3b25kZXIgd2h5IHRoZXkgaGF2 ZQo+PmRpZmZlcmVudCBjaGVja3MgZm9yIHRoZSByZXR1cm4gdmFsdWVzLiBBbHNvIHBhdGNoIDIv MiBvbmx5IG1vZGlmaWVzIG9uZSBvZgo+PnRoZW0sIG5vdCBib3RoLiBXaHk/Cj4KPiBUaGUgMiBj aGVjayBzaG91bGQgYmUgc2ltaWxhcj8gSW4gc3luY19hdWRpb19yYXRlKCksIGl0IGNhbGxzIGdl dF9zYXZlZF9lbmMoKQo+IGFuZCBjaGVja2VkIGJhc2UuY3J0YywgSSB0aGluayBpdCBpcyBiZWNh dXNlIGl0IHdpbGwgdXNlIGJhc2UuY3J0YyBsYXRlci4KPgo+IEFuZCBpdCB3aWxsIGNoZWNrIHRo ZSBpbnRlbF9lbmNvZGVyLT50eXBlIGJlY2F1c2UgdGhpcyBmdW5jdGlvbiBvbmx5IG5lZWQKPiBo YW5kbGUgSERNSS9EUC9EUCBNU1Qgc2l0dWF0aW9uIHRvIHNldHVwICB0aGUgYXVkaW8gY29uZmln LgoKSSB0aGluayBhZGRyZXNzaW5nIG15IHJldmlldyBjb21tZW50IG9uIHBhdGNoIDIgd2lsbCBo ZWxwIHRoaXMuCgpCUiwKSmFuaS4KCj4KPiBSZWdhcmRzLAo+IExpYmluCj4KPj4KPj5TbyBtYXli ZSBJJ20ganVzdCB0b28gY2x1ZWxlc3MgYWJvdXQgRFAgKE1TVCkgYXVkaW8gaW4gZ2VuZXJhbCwg YW5kIEkKPj53b3VsZG4ndCBldmVuIGdldCBvZmZlbmRlZCBpZiB5b3UgdG9sZCBtZSB0aGF0LiBC dXQgSSBkb24ndCB0aGluayB0aGUgY29tbWl0Cj4+bWVzc2FnZSBhbmQgdGhlIGNvbW1lbnRzIGlu IHRoaXMgcGF0Y2ggcHJvcGVybHkgaGVscCBtZSBlaXRoZXIuCj4+Cj4+QlIsCj4+SmFuaS4KPj4K Pj4KPj4+ICsgKi8KPj4+ICBzdGF0aWMgc3RydWN0IGludGVsX2VuY29kZXIgKmdldF9zYXZlZF9l bmMoc3RydWN0IGRybV9pOTE1X3ByaXZhdGUKPj4qZGV2X3ByaXYsCj4+PiAgCQkJCQkgICAgICAg aW50IHBvcnQsIGludCBwaXBlKQo+Pj4gIHsKPj4+ICsJc3RydWN0IGludGVsX2VuY29kZXIgKmVu Y29kZXI7Cj4+Pgo+Pj4gIAlpZiAoV0FSTl9PTihwaXBlID49IEk5MTVfTUFYX1BJUEVTKSkKPj4+ ICAJCXJldHVybiBOVUxMOwo+Pj4KPj4+ICAJLyogTVNUICovCj4+PiAtCWlmIChwaXBlID49IDAp Cj4+PiAtCQlyZXR1cm4gZGV2X3ByaXYtPmF2X2VuY19tYXBbcGlwZV07Cj4+PiArCWlmIChwaXBl ID49IDApIHsKPj4+ICsJCWVuY29kZXIgPSBkZXZfcHJpdi0+YXZfZW5jX21hcFtwaXBlXTsKPj4+ ICsJCS8qCj4+PiArCQkgKiB3aGVuIGJvb3R1cCwgYXVkaW8gZHJpdmVyIG1heSBub3Qga25vdyBp dCBpcwo+Pj4gKwkJICogTVNUIG9yIG5vdC4gU28gaXQgd2lsbCBwb2xsIGFsbCB0aGUgcG9ydCAm IHBpcGUKPj4+ICsJCSAqIGNvbWJpbmF0aW9ucwo+Pj4gKwkJICovCj4+PiArCQlpZiAoZW5jb2Rl ciAhPSBOVUxMICYmIGVuY29kZXItPnBvcnQgPT0gcG9ydCAmJgo+Pj4gKwkJICAgIGVuY29kZXIt PnR5cGUgPT0gSU5URUxfT1VUUFVUX0RQX01TVCkKPj4+ICsJCQlyZXR1cm4gZW5jb2RlcjsKPj4+ ICsJfQo+Pj4KPj4+ICAJLyogTm9uLU1TVCAqLwo+Pj4gLQlmb3JfZWFjaF9waXBlKGRldl9wcml2 LCBwaXBlKSB7Cj4+PiAtCQlzdHJ1Y3QgaW50ZWxfZW5jb2RlciAqZW5jb2RlcjsKPj4+ICsJaWYg KHBpcGUgPiAwKQo+Pj4gKwkJcmV0dXJuIE5VTEw7Cj4+Pgo+Pj4gKwlmb3JfZWFjaF9waXBlKGRl dl9wcml2LCBwaXBlKSB7Cj4+PiAgCQllbmNvZGVyID0gZGV2X3ByaXYtPmF2X2VuY19tYXBbcGlw ZV07Cj4+PiAgCQlpZiAoZW5jb2RlciA9PSBOVUxMKQo+Pj4gIAkJCWNvbnRpbnVlOwo+Pj4KPj4+ ICsJCWlmIChlbmNvZGVyLT50eXBlID09IElOVEVMX09VVFBVVF9EUF9NU1QpCj4+PiArCQkJY29u dGludWU7Cj4+PiArCj4+PiAgCQlpZiAocG9ydCA9PSBlbmNvZGVyLT5wb3J0KQo+Pj4gIAkJCXJl dHVybiBlbmNvZGVyOwo+Pj4gIAl9Cj4+Cj4+LS0KPj5KYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBT b3VyY2UgVGVjaG5vbG9neSBDZW50ZXIKCi0tIApKYW5pIE5pa3VsYSwgSW50ZWwgT3BlbiBTb3Vy Y2UgVGVjaG5vbG9neSBDZW50ZXIKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRl c2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v aW50ZWwtZ2Z4Cg==