From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,USER_AGENT_SANE_2 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 15E63C63798 for ; Thu, 22 Jul 2021 11:44:19 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id EDA776100C for ; Thu, 22 Jul 2021 11:44:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231773AbhGVLDm (ORCPT ); Thu, 22 Jul 2021 07:03:42 -0400 Received: from mailgw01.mediatek.com ([60.244.123.138]:55996 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S231566AbhGVLDk (ORCPT ); Thu, 22 Jul 2021 07:03:40 -0400 X-UUID: 6b079f1006b049b8abefc36d8306d277-20210722 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=sVejKnwGWH8m17twQO8kI259z77f4UObKDcLs86o0BA=; b=hZmDse7Bfa/vOwW9FbIlbTMiw6GHLOiMWFTf9wHC1rt4p1tj3N7CpwrLqTiKegRhpNJND76PSiFiDBlIi5cBVsvQX+fslD6ShUQmdtfOsR5/P3dMVu0sEqon+bJONI/ZxSqQ+mk7DTMNbJNSsrNgjDGABj2qUSfQr/3r8StdkPo=; X-UUID: 6b079f1006b049b8abefc36d8306d277-20210722 Received: from mtkmbs10n1.mediatek.inc [(172.21.101.34)] by mailgw01.mediatek.com (envelope-from ) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-GCM-SHA384 256/256) with ESMTP id 570380403; Thu, 22 Jul 2021 19:44:11 +0800 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs07n2.mediatek.inc (172.21.101.141) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 22 Jul 2021 19:44:10 +0800 Received: from [10.17.3.153] (10.17.3.153) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 22 Jul 2021 19:44:09 +0800 Message-ID: <1626954248.32022.5.camel@mhfsdcap03> Subject: Re: [PATCH v2, 00/14] Using component framework to support multi hardware decode From: mtk12024 To: Dafna Hirschfeld CC: Alexandre Courbot , Hans Verkuil , Tzung-Bi Shih , "Tiffany Lin" , Andrew-CT Chen , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Tomasz Figa , Hsin-Yi Wang , Fritz Koenig , Irui Wang , , , , , , , , Collabora Kernel ML , Enric Balletbo i Serra Date: Thu, 22 Jul 2021 19:44:08 +0800 In-Reply-To: References: <20210717081233.7809-1-yunfei.dong@mediatek.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-MTK: N Content-Transfer-Encoding: base64 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org T24gTW9uLCAyMDIxLTA3LTE5IGF0IDA5OjAwICswMjAwLCBEYWZuYSBIaXJzY2hmZWxkIHdyb3Rl Og0KPiANCj4gT24gMTcuMDcuMjEgMTA6MTIsIFl1bmZlaSBEb25nIHdyb3RlOg0KPiA+IFRoaXMg c2VyaWVzIGFkZHMgc3VwcG9ydCBmb3IgbXVsdGkgaGFyZHdhcmUgZGVjb2RlIGludG8gbXRrLXZj b2RlYywgYnkgZmlyc3QNCj4gPiBhZGRpbmcgY29tcG9uZW50IGZyYW1ld29yayB0byBtYW5hZ2Ug ZWFjaCBoYXJkd2FyZSBpbmZvcm1hdGlvbjogaW50ZXJydXB0LA0KPiA+IGNsb2NrLCByZWdpc3Rl ciBiYXNlcyBhbmQgcG93ZXIuIFNlY29uZGx5IGFkZCBjb3JlIHRocmVhZCB0byBkZWFsIHdpdGgg Y29yZQ0KPiA+IGhhcmR3YXJlIG1lc3NhZ2UsIGF0IHRoZSBzYW1lIHRpbWUsIGFkZCBtc2cgcXVl dWUgZm9yIGRpZmZlcmVudCBoYXJkd2FyZQ0KPiA+IHNoYXJlIG1lc3NhZ2VzLiBMYXN0bHksIHRo ZSBhcmNoaXRlY3R1cmUgb2YgZGlmZmVyZW50IHNwZWNzIGFyZSBub3QgdGhlIHNhbWUsDQo+ID4g dXNpbmcgc3BlY3MgdHlwZSB0byBzZXBhcmF0ZSB0aGVtLg0KPiA+IA0KPiA+IFRoaXMgc2VyaWVz IGhhcyBiZWVuIHRlc3RlZCB3aXRoIGJvdGggTVQ4MTgzIGFuZCBNVDgxNzMuIERlY29kaW5nIHdh cyB3b3JraW5nDQo+ID4gZm9yIGJvdGggY2hpcHMuDQo+IA0KPiBIaSwgSSBhbSBjdXJyZW50bHkg d29ya2luZyBvbiBzdGFiaWxpemluZyB0aGUgY29kZWMgZHJpdmVycyBvbiBtdDgxNzMgY2hyb21l b3MNCj4gICBkZXZpY2UgKGVsbSkuIEl0IHdvdWxkIGJlIHZlcnkgaGVscGZ1bCBpZiB5b3UgcHJv dmlkZSBpbmZvcm1hdGlvbiBvbiBob3cgeW91DQo+IHRlc3RlZCB0aGUgY29kZS4NCj4gRGlkIHlv dSB0ZXN0IGl0IG9uIGNocm9tZW9zIHVzZXJzcGFjZT8NCj4gRGlkIHlvdSB1c2UgdGhlIHRlc3Qt dGFzdCBmcmFtZXdvcms/DQo+IFdoYXQgdGVzdHMgZGlkIHlvdSBydW4/DQo+IA0KPiBUaGFua3Ms DQo+IERhZm5hDQo+IA0KSGkgRGFmbmEsDQoNCkFkZGluZyBjb21wb25lbnQgZnJhbWV3b3JrIGlz IGp1c3QgdXNlZCBmb3IgbXQ4MTkyIGFuZCBtdDgxOTUuDQpNVDgxNzMgYW5kIE1UODE4MyBzdGls bCB1c2UgdGhlIHRoZSBvbGRlciBhcmNoaXRlY3R1cmUuIEluIGFub3RoZXIgd2F5LA0KbXQ4MTky IGFuZCBtdDgxOTUgaXMgc3RhdGVsZXNzIGRlY29kZSwgbXQ4MTczIGlzIHN0YXRlZnVsIGRlY29k ZSwgaXQgaXMNCmRpZmZlcmVudC4gSSBqdXN0IHBvcnRpbmcgdGhlc2UgcGF0Y2hlcyB0byBtdDgx NzMgYnJhbmNoLCBhbmQganVzdCBuZWVkDQp0byBkbyBzZXZlcmFsIHNpbXBsZSB0ZXN0Lg0KDQpU aGFua3MsDQpZdW5mZWkgRG9uZw0KPiA+IA0KPiA+IFBhdGNoZXMgMSwyIHJld3JpdGUgZ2V0IHJl Z2lzdGVyIGJhc2VzIGFuZCBwb3dlciBvbi9vZmYgaW50ZXJmYWNlLg0KPiA+IA0KPiA+IFBhdGNo IDMtNSBhZGQgY29tcG9uZW50IGZyYW1ld29yayB0byBzdXBwb3J0IG11bHRpIGhhcmR3YXJlLg0K PiA+IA0KPiA+IFBhdGNoZXMgNi0xNCBhZGQgaW50ZXJmYWNlcyB0byBzdXBwb3J0IGNvcmUgaGFy ZHdhcmUuDQo+ID4gLS0tLQ0KPiA+IFRoaXMgcGF0Y2ggZGVwZW5kZW50cyBvbiAibWVkaWE6IG10 ay12Y29kZWM6IHN1cHBvcnQgZm9yIE1UODE4MyBkZWNvZGVyIlsxXS4NCj4gPiANCj4gPiBNdWx0 aSBoYXJkd2FyZSBkZWNvZGUgaXMgYmFzZWQgb24gc3RhdGVsZXNzIGRlY29kZXIsIE1UODE4MyBp cyB0aGUgZmlyc3QNCj4gPiB0aW1lIHRvIGFkZCBzdGF0ZWxlc3MgZGVjb2Rlci4gT3RoZXJ3aXNl IGl0IHdpbGwgY2F1c2UgY29uZmxpY3QuDQo+ID4gUGxlYXNlIGFsc28gYWNjZXB0IHRoaXMgcGF0 Y2ggdG9nZXRoZXIgd2l0aCBbMV0uDQo+ID4gDQo+ID4gWzFdaHR0cHM6Ly9sb3JlLmtlcm5lbC5v cmcvcGF0Y2h3b3JrL3Byb2plY3QvbGttbC9saXN0Lz9zZXJpZXM9NTA3MDg0DQo+ID4gLS0tLQ0K PiA+IA0KPiA+IENoYW5nZXMgY29tcGFyZWQgd2l0aCB2MToNCj4gPiAtIEZpeCBtYW55IGNvbW1l bnRzIGZvciBwYXRjaCAzLzE0DQo+ID4gLSBSZW1vdmUgdW5uZWNlc3NhcnkgY29kZSBmb3IgcGF0 Y2ggNC8xNA0KPiA+IC0gVXNpbmcgZW51bSBtdGtfdmRlY19od19jb3VudCBpbnN0ZWFkIG9mIG1h Z2ljIG51bWJlcnMgZm9yIHBhdGNoIDYvMTQNCj4gPiAtIFJlY29uc3RydWN0ZWQgZ2V0L3B1dCBs YXQgYnVmZmVyIGZvciBsYXQgYW5kIGNvcmUgaGFyZHdhcmUgZm9yIHBhdGNoIDcvMTQNCj4gPiAt IFVzaW5nIHlhbWwgZm9ybWF0IHRvIGluc3RlYWQgb2YgdHh0IGZpbGUgZm9yIHBhdGNoIDEyLzE0 DQo+ID4gDQo+ID4gWXVuZmVpIERvbmcgKDE0KToNCj4gPiAgICBtZWRpYTogbXRrLXZjb2RlYzog R2V0IG51bWJlcnMgb2YgcmVnaXN0ZXIgYmFzZXMgZnJvbSBEVA0KPiA+ICAgIG1lZGlhOiBtdGst dmNvZGVjOiBSZWZhY3RvciB2Y29kZWMgcG0gaW50ZXJmYWNlDQo+ID4gICAgbWVkaWE6IG10ay12 Y29kZWM6IFVzZSBjb21wb25lbnQgZnJhbWV3b3JrIHRvIG1hbmFnZSBlYWNoIGhhcmR3YXJlDQo+ ID4gICAgICBpbmZvcm1hdGlvbg0KPiA+ICAgIGR0LWJpbmRpbmdzOiBtZWRpYTogbXRrLXZjb2Rl YzogU2VwYXJhdGUgdmlkZW8gZW5jb2RlciBhbmQgZGVjb2Rlcg0KPiA+ICAgICAgZHQtYmluZGlu Z3MNCj4gPiAgICBtZWRpYTogbXRrLXZjb2RlYzogVXNlIHB1cmUgc2luZ2xlIGNvcmUgZm9yIE1U ODE4Mw0KPiA+ICAgIG1lZGlhOiBtdGstdmNvZGVjOiBBZGQgaXJxIGludGVyZmFjZSBmb3IgY29y ZSBoYXJkd2FyZQ0KPiA+ICAgIG1lZGlhOiBtdGstdmNvZGVjOiBBZGQgbXNnIHF1ZXVlIGZlYXR1 cmUgZm9yIGxhdCBhbmQgY29yZSBhcmNoaXRlY3R1cmUNCj4gPiAgICBtZWRpYTogbXRrLXZjb2Rl YzogR2VuZXJhbGl6ZSBwb3dlciBhbmQgY2xvY2sgb24vb2ZmIGludGVyZmFjZXMNCj4gPiAgICBt ZWRpYTogbXRrLXZjb2RlYzogQWRkIG5ldyBpbnRlcmZhY2UgdG8gbG9jayBkaWZmZXJlbnQgaGFy ZHdhcmUNCj4gPiAgICBtZWRpYTogbXRrLXZjb2RlYzogQWRkIGNvcmUgdGhyZWFkDQo+ID4gICAg bWVkaWE6IG10ay12Y29kZWM6IFN1cHBvcnQgMzRiaXRzIGRtYSBhZGRyZXNzIGZvciB2ZGVjDQo+ ID4gICAgZHQtYmluZGluZ3M6IG1lZGlhOiBtdGstdmNvZGVjOiBBZGRzIGRlY29kZXIgZHQtYmlu ZGluZ3MgZm9yIG10ODE5Mg0KPiA+ICAgIG1lZGlhOiBtdGstdmNvZGVjOiBBZGQgY29yZSBkZWMg YW5kIGRlYyBlbmQgaXBpIG1zZw0KPiA+ICAgIG1lZGlhOiBtdGstdmNvZGVjOiBVc2UgY29kZWMg dHlwZSB0byBzZXBhcmF0ZSBkaWZmZXJlbnQgaGFyZHdhcmUNCj4gPiANCj4gPiAgIC4uLi9tZWRp YS9tZWRpYXRlay12Y29kZWMtY29tcC1kZWNvZGVyLnlhbWwgICB8IDE1MCArKysrKysrKysNCj4g PiAgIC4uLnZjb2RlYy50eHQgPT4gbWVkaWF0ZWstdmNvZGVjLWRlY29kZXIudHh0fSB8ICA2MiAr LS0tDQo+ID4gICAuLi4vbWVkaWEvbWVkaWF0ZWstdmNvZGVjLWVuY29kZXIudHh0ICAgICAgICAg fCAgNzMgKysrKysNCj4gPiAgIGRyaXZlcnMvbWVkaWEvcGxhdGZvcm0vbXRrLXZjb2RlYy9NYWtl ZmlsZSAgICB8ICAgMiArDQo+ID4gICAuLi4vcGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVj X2RlYy5jICAgICAgfCAgIDQgKy0NCj4gPiAgIC4uLi9wbGF0Zm9ybS9tdGstdmNvZGVjL210a192 Y29kZWNfZGVjLmggICAgICB8ICAgMSArDQo+ID4gICAuLi4vcGxhdGZvcm0vbXRrLXZjb2RlYy9t dGtfdmNvZGVjX2RlY19kcnYuYyAgfCAyOTcgKysrKysrKysrKysrKysrLS0tDQo+ID4gICAuLi4v cGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVjX2RlY19ody5jICAgfCAxOTQgKysrKysrKysr KysrDQo+ID4gICAuLi4vcGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVjX2RlY19ody5oICAg fCAgNTAgKysrDQo+ID4gICAuLi4vcGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVjX2RlY19w bS5jICAgfCAgOTggKysrKy0tDQo+ID4gICAuLi4vcGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNv ZGVjX2RlY19wbS5oICAgfCAgMTMgKy0NCj4gPiAgIC4uLi9tdGstdmNvZGVjL210a192Y29kZWNf ZGVjX3N0YXRlZnVsLmMgICAgICB8ICAgMSArDQo+ID4gICAuLi4vbXRrLXZjb2RlYy9tdGtfdmNv ZGVjX2RlY19zdGF0ZWxlc3MuYyAgICAgfCAgIDEgKw0KPiA+ICAgLi4uL3BsYXRmb3JtL210ay12 Y29kZWMvbXRrX3Zjb2RlY19kcnYuaCAgICAgIHwgIDcyICsrKystDQo+ID4gICAuLi4vcGxhdGZv cm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVjX2VuY19wbS5jICAgfCAgIDEgLQ0KPiA+ICAgLi4uL3Bs YXRmb3JtL210ay12Y29kZWMvbXRrX3Zjb2RlY19pbnRyLmMgICAgIHwgIDMyICstDQo+ID4gICAu Li4vcGxhdGZvcm0vbXRrLXZjb2RlYy9tdGtfdmNvZGVjX2ludHIuaCAgICAgfCAgIDQgKy0NCj4g PiAgIC4uLi9wbGF0Zm9ybS9tdGstdmNvZGVjL210a192Y29kZWNfdXRpbC5jICAgICB8ICA4NyAr KysrLQ0KPiA+ICAgLi4uL3BsYXRmb3JtL210ay12Y29kZWMvbXRrX3Zjb2RlY191dGlsLmggICAg IHwgICA4ICstDQo+ID4gICAuLi4vbWVkaWEvcGxhdGZvcm0vbXRrLXZjb2RlYy92ZGVjX2Rydl9p Zi5jICAgfCAgMjEgKy0NCj4gPiAgIC4uLi9tZWRpYS9wbGF0Zm9ybS9tdGstdmNvZGVjL3ZkZWNf aXBpX21zZy5oICB8ICAxNiArLQ0KPiA+ICAgLi4uL3BsYXRmb3JtL210ay12Y29kZWMvdmRlY19t c2dfcXVldWUuYyAgICAgIHwgMjg2ICsrKysrKysrKysrKysrKysrDQo+ID4gICAuLi4vcGxhdGZv cm0vbXRrLXZjb2RlYy92ZGVjX21zZ19xdWV1ZS5oICAgICAgfCAxNDMgKysrKysrKysrDQo+ID4g ICAuLi4vbWVkaWEvcGxhdGZvcm0vbXRrLXZjb2RlYy92ZGVjX3ZwdV9pZi5jICAgfCAgNDYgKyst DQo+ID4gICAuLi4vbWVkaWEvcGxhdGZvcm0vbXRrLXZjb2RlYy92ZGVjX3ZwdV9pZi5oICAgfCAg MjIgKysNCj4gPiAgIDI1IGZpbGVzIGNoYW5nZWQsIDE1MTYgaW5zZXJ0aW9ucygrKSwgMTY4IGRl bGV0aW9ucygtKQ0KPiA+ICAgY3JlYXRlIG1vZGUgMTAwNjQ0IERvY3VtZW50YXRpb24vZGV2aWNl dHJlZS9iaW5kaW5ncy9tZWRpYS9tZWRpYXRlay12Y29kZWMtY29tcC1kZWNvZGVyLnlhbWwNCj4g PiAgIHJlbmFtZSBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvbWVkaWEve21lZGlh dGVrLXZjb2RlYy50eHQgPT4gbWVkaWF0ZWstdmNvZGVjLWRlY29kZXIudHh0fSAoNTYlKQ0KPiA+ ICAgY3JlYXRlIG1vZGUgMTAwNjQ0IERvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9t ZWRpYS9tZWRpYXRlay12Y29kZWMtZW5jb2Rlci50eHQNCj4gPiAgIGNyZWF0ZSBtb2RlIDEwMDY0 NCBkcml2ZXJzL21lZGlhL3BsYXRmb3JtL210ay12Y29kZWMvbXRrX3Zjb2RlY19kZWNfaHcuYw0K PiA+ICAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRyaXZlcnMvbWVkaWEvcGxhdGZvcm0vbXRrLXZjb2Rl Yy9tdGtfdmNvZGVjX2RlY19ody5oDQo+ID4gICBjcmVhdGUgbW9kZSAxMDA2NDQgZHJpdmVycy9t ZWRpYS9wbGF0Zm9ybS9tdGstdmNvZGVjL3ZkZWNfbXNnX3F1ZXVlLmMNCj4gPiAgIGNyZWF0ZSBt b2RlIDEwMDY0NCBkcml2ZXJzL21lZGlhL3BsYXRmb3JtL210ay12Y29kZWMvdmRlY19tc2dfcXVl dWUuaA0KPiA+IA0KDQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED, USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9E545C63793 for ; Thu, 22 Jul 2021 12:50:22 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 690D161378 for ; Thu, 22 Jul 2021 12:50:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 690D161378 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:CC:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Rgfz5gu8h/WcT5TSwokQ7U9t8f8BUsfF4ENOicWt69Q=; b=yfU5C0V3fBccxE WZ7jiKtuyt3BKbmskKH363jvOTzQAb4lVEwJRczi/Wh/kmN+wzMKDeuTLjj6yila6oSfD6fth9Pcr OIs0YDh5w96RUj1kEfEERC3bu/w5puAwohEXs+WJfRWscxjMu++e2Y8RSohdJVT7FjS4gBVwUnCPR W9cwFgm2uz6+lY+AJFtXZ9GXGBhlS0S+1glOP2VX93StqH0dzXaUDsselFI4bCccqLOh/OxvLzdZD tGwGBUvcf/6D5BSkS/+VMjyXr4Py/hZ3pMjXRWcaXBGYE7ZZHErbltGwnnpSctZvIGEvTuLmxOXPM ajm6YaY/ed/iL64Mx4JQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6Y9P-001XfI-CJ; Thu, 22 Jul 2021 12:50:07 +0000 Received: from mailgw02.mediatek.com ([216.200.240.185]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6X7l-001OVi-8B; Thu, 22 Jul 2021 11:44:28 +0000 X-UUID: 3d1e6a55904c44bdb07317669a74bdcb-20210722 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=sVejKnwGWH8m17twQO8kI259z77f4UObKDcLs86o0BA=; b=EHRTMlUnReWQ0/AZzpR5ZH0pJZ1bqzhqLQfF3LKLSDMrVwwpItYrSXc/YAoei3uu2w1ly83E93EXNpAe15aDe9248ZDNGXQWUW+N5/kHpgYWhXUQmgLF5+XaDwBNHmI8ojfA1vrQ8DWTUqySAkr2Nqg633JwP6yIRYpqR73UJzI=; X-UUID: 3d1e6a55904c44bdb07317669a74bdcb-20210722 Received: from mtkcas67.mediatek.inc [(172.29.193.45)] by mailgw02.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 553344500; Thu, 22 Jul 2021 04:44:13 -0700 Received: from MTKMBS07N2.mediatek.inc (172.21.101.141) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 22 Jul 2021 04:44:11 -0700 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs07n2.mediatek.inc (172.21.101.141) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 22 Jul 2021 19:44:10 +0800 Received: from [10.17.3.153] (10.17.3.153) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 22 Jul 2021 19:44:09 +0800 Message-ID: <1626954248.32022.5.camel@mhfsdcap03> Subject: Re: [PATCH v2, 00/14] Using component framework to support multi hardware decode From: mtk12024 To: Dafna Hirschfeld CC: Alexandre Courbot , Hans Verkuil , Tzung-Bi Shih , "Tiffany Lin" , Andrew-CT Chen , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Tomasz Figa , Hsin-Yi Wang , "Fritz Koenig" , Irui Wang , , , , , , , , Collabora Kernel ML , Enric Balletbo i Serra Date: Thu, 22 Jul 2021 19:44:08 +0800 In-Reply-To: References: <20210717081233.7809-1-yunfei.dong@mediatek.com> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210722_044425_144768_C99EB604 X-CRM114-Status: GOOD ( 35.00 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Mon, 2021-07-19 at 09:00 +0200, Dafna Hirschfeld wrote: > > On 17.07.21 10:12, Yunfei Dong wrote: > > This series adds support for multi hardware decode into mtk-vcodec, by first > > adding component framework to manage each hardware information: interrupt, > > clock, register bases and power. Secondly add core thread to deal with core > > hardware message, at the same time, add msg queue for different hardware > > share messages. Lastly, the architecture of different specs are not the same, > > using specs type to separate them. > > > > This series has been tested with both MT8183 and MT8173. Decoding was working > > for both chips. > > Hi, I am currently working on stabilizing the codec drivers on mt8173 chromeos > device (elm). It would be very helpful if you provide information on how you > tested the code. > Did you test it on chromeos userspace? > Did you use the test-tast framework? > What tests did you run? > > Thanks, > Dafna > Hi Dafna, Adding component framework is just used for mt8192 and mt8195. MT8173 and MT8183 still use the the older architecture. In another way, mt8192 and mt8195 is stateless decode, mt8173 is stateful decode, it is different. I just porting these patches to mt8173 branch, and just need to do several simple test. Thanks, Yunfei Dong > > > > Patches 1,2 rewrite get register bases and power on/off interface. > > > > Patch 3-5 add component framework to support multi hardware. > > > > Patches 6-14 add interfaces to support core hardware. > > ---- > > This patch dependents on "media: mtk-vcodec: support for MT8183 decoder"[1]. > > > > Multi hardware decode is based on stateless decoder, MT8183 is the first > > time to add stateless decoder. Otherwise it will cause conflict. > > Please also accept this patch together with [1]. > > > > [1]https://lore.kernel.org/patchwork/project/lkml/list/?series=507084 > > ---- > > > > Changes compared with v1: > > - Fix many comments for patch 3/14 > > - Remove unnecessary code for patch 4/14 > > - Using enum mtk_vdec_hw_count instead of magic numbers for patch 6/14 > > - Reconstructed get/put lat buffer for lat and core hardware for patch 7/14 > > - Using yaml format to instead of txt file for patch 12/14 > > > > Yunfei Dong (14): > > media: mtk-vcodec: Get numbers of register bases from DT > > media: mtk-vcodec: Refactor vcodec pm interface > > media: mtk-vcodec: Use component framework to manage each hardware > > information > > dt-bindings: media: mtk-vcodec: Separate video encoder and decoder > > dt-bindings > > media: mtk-vcodec: Use pure single core for MT8183 > > media: mtk-vcodec: Add irq interface for core hardware > > media: mtk-vcodec: Add msg queue feature for lat and core architecture > > media: mtk-vcodec: Generalize power and clock on/off interfaces > > media: mtk-vcodec: Add new interface to lock different hardware > > media: mtk-vcodec: Add core thread > > media: mtk-vcodec: Support 34bits dma address for vdec > > dt-bindings: media: mtk-vcodec: Adds decoder dt-bindings for mt8192 > > media: mtk-vcodec: Add core dec and dec end ipi msg > > media: mtk-vcodec: Use codec type to separate different hardware > > > > .../media/mediatek-vcodec-comp-decoder.yaml | 150 +++++++++ > > ...vcodec.txt => mediatek-vcodec-decoder.txt} | 62 +--- > > .../media/mediatek-vcodec-encoder.txt | 73 +++++ > > drivers/media/platform/mtk-vcodec/Makefile | 2 + > > .../platform/mtk-vcodec/mtk_vcodec_dec.c | 4 +- > > .../platform/mtk-vcodec/mtk_vcodec_dec.h | 1 + > > .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 297 +++++++++++++++--- > > .../platform/mtk-vcodec/mtk_vcodec_dec_hw.c | 194 ++++++++++++ > > .../platform/mtk-vcodec/mtk_vcodec_dec_hw.h | 50 +++ > > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.c | 98 ++++-- > > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.h | 13 +- > > .../mtk-vcodec/mtk_vcodec_dec_stateful.c | 1 + > > .../mtk-vcodec/mtk_vcodec_dec_stateless.c | 1 + > > .../platform/mtk-vcodec/mtk_vcodec_drv.h | 72 ++++- > > .../platform/mtk-vcodec/mtk_vcodec_enc_pm.c | 1 - > > .../platform/mtk-vcodec/mtk_vcodec_intr.c | 32 +- > > .../platform/mtk-vcodec/mtk_vcodec_intr.h | 4 +- > > .../platform/mtk-vcodec/mtk_vcodec_util.c | 87 ++++- > > .../platform/mtk-vcodec/mtk_vcodec_util.h | 8 +- > > .../media/platform/mtk-vcodec/vdec_drv_if.c | 21 +- > > .../media/platform/mtk-vcodec/vdec_ipi_msg.h | 16 +- > > .../platform/mtk-vcodec/vdec_msg_queue.c | 286 +++++++++++++++++ > > .../platform/mtk-vcodec/vdec_msg_queue.h | 143 +++++++++ > > .../media/platform/mtk-vcodec/vdec_vpu_if.c | 46 ++- > > .../media/platform/mtk-vcodec/vdec_vpu_if.h | 22 ++ > > 25 files changed, 1516 insertions(+), 168 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-vcodec-comp-decoder.yaml > > rename Documentation/devicetree/bindings/media/{mediatek-vcodec.txt => mediatek-vcodec-decoder.txt} (56%) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-vcodec-encoder.txt > > create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_hw.c > > create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_hw.h > > create mode 100644 drivers/media/platform/mtk-vcodec/vdec_msg_queue.c > > create mode 100644 drivers/media/platform/mtk-vcodec/vdec_msg_queue.h > > _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED, USER_AGENT_SANE_2 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7BB1DC63793 for ; Thu, 22 Jul 2021 12:50:51 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 43F7561378 for ; Thu, 22 Jul 2021 12:50:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 43F7561378 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:CC:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ozw5cnbpaaMYDAHrbgr4s0eYJcms44DXlKuwbkugEUM=; b=pouUskHerNxt2e y33D+HdWASAML1P+kr5MmzEuwaM8GgXT6ZHtljmEo6CJKA1d6N9FNh2UAgE7yoU3+AbFk3B3m8yPk 9uUx2ozxGupuO0UDwGXpFrxQFWbXozCeVZT4vpAq5GHg4AvqT1QEBN4MO/RrNSZcBEWG9fBzqphov ToWxrpJuCoALiahyRt+JogYwA2MIAeXiO7r99XH1woIIL6boOyNBJ5tZZ/STjbrKDrSpJhjki4DkR fsOsC0YbbhA2mJwMifkuI7JnJhcbIMA2HOlhBoS27h+9mPOOX0omeSZnZWETgZxvtYKWsAU3lKKjX ptRL34o6+wl03KBUAEzQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6Y6Y-001X53-JM; Thu, 22 Jul 2021 12:47:13 +0000 Received: from mailgw02.mediatek.com ([216.200.240.185]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6X7l-001OVi-8B; Thu, 22 Jul 2021 11:44:28 +0000 X-UUID: 3d1e6a55904c44bdb07317669a74bdcb-20210722 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=sVejKnwGWH8m17twQO8kI259z77f4UObKDcLs86o0BA=; b=EHRTMlUnReWQ0/AZzpR5ZH0pJZ1bqzhqLQfF3LKLSDMrVwwpItYrSXc/YAoei3uu2w1ly83E93EXNpAe15aDe9248ZDNGXQWUW+N5/kHpgYWhXUQmgLF5+XaDwBNHmI8ojfA1vrQ8DWTUqySAkr2Nqg633JwP6yIRYpqR73UJzI=; X-UUID: 3d1e6a55904c44bdb07317669a74bdcb-20210722 Received: from mtkcas67.mediatek.inc [(172.29.193.45)] by mailgw02.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 553344500; Thu, 22 Jul 2021 04:44:13 -0700 Received: from MTKMBS07N2.mediatek.inc (172.21.101.141) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 22 Jul 2021 04:44:11 -0700 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs07n2.mediatek.inc (172.21.101.141) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 22 Jul 2021 19:44:10 +0800 Received: from [10.17.3.153] (10.17.3.153) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 22 Jul 2021 19:44:09 +0800 Message-ID: <1626954248.32022.5.camel@mhfsdcap03> Subject: Re: [PATCH v2, 00/14] Using component framework to support multi hardware decode From: mtk12024 To: Dafna Hirschfeld CC: Alexandre Courbot , Hans Verkuil , Tzung-Bi Shih , "Tiffany Lin" , Andrew-CT Chen , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Tomasz Figa , Hsin-Yi Wang , "Fritz Koenig" , Irui Wang , , , , , , , , Collabora Kernel ML , Enric Balletbo i Serra Date: Thu, 22 Jul 2021 19:44:08 +0800 In-Reply-To: References: <20210717081233.7809-1-yunfei.dong@mediatek.com> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210722_044425_144768_C99EB604 X-CRM114-Status: GOOD ( 35.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 2021-07-19 at 09:00 +0200, Dafna Hirschfeld wrote: > > On 17.07.21 10:12, Yunfei Dong wrote: > > This series adds support for multi hardware decode into mtk-vcodec, by first > > adding component framework to manage each hardware information: interrupt, > > clock, register bases and power. Secondly add core thread to deal with core > > hardware message, at the same time, add msg queue for different hardware > > share messages. Lastly, the architecture of different specs are not the same, > > using specs type to separate them. > > > > This series has been tested with both MT8183 and MT8173. Decoding was working > > for both chips. > > Hi, I am currently working on stabilizing the codec drivers on mt8173 chromeos > device (elm). It would be very helpful if you provide information on how you > tested the code. > Did you test it on chromeos userspace? > Did you use the test-tast framework? > What tests did you run? > > Thanks, > Dafna > Hi Dafna, Adding component framework is just used for mt8192 and mt8195. MT8173 and MT8183 still use the the older architecture. In another way, mt8192 and mt8195 is stateless decode, mt8173 is stateful decode, it is different. I just porting these patches to mt8173 branch, and just need to do several simple test. Thanks, Yunfei Dong > > > > Patches 1,2 rewrite get register bases and power on/off interface. > > > > Patch 3-5 add component framework to support multi hardware. > > > > Patches 6-14 add interfaces to support core hardware. > > ---- > > This patch dependents on "media: mtk-vcodec: support for MT8183 decoder"[1]. > > > > Multi hardware decode is based on stateless decoder, MT8183 is the first > > time to add stateless decoder. Otherwise it will cause conflict. > > Please also accept this patch together with [1]. > > > > [1]https://lore.kernel.org/patchwork/project/lkml/list/?series=507084 > > ---- > > > > Changes compared with v1: > > - Fix many comments for patch 3/14 > > - Remove unnecessary code for patch 4/14 > > - Using enum mtk_vdec_hw_count instead of magic numbers for patch 6/14 > > - Reconstructed get/put lat buffer for lat and core hardware for patch 7/14 > > - Using yaml format to instead of txt file for patch 12/14 > > > > Yunfei Dong (14): > > media: mtk-vcodec: Get numbers of register bases from DT > > media: mtk-vcodec: Refactor vcodec pm interface > > media: mtk-vcodec: Use component framework to manage each hardware > > information > > dt-bindings: media: mtk-vcodec: Separate video encoder and decoder > > dt-bindings > > media: mtk-vcodec: Use pure single core for MT8183 > > media: mtk-vcodec: Add irq interface for core hardware > > media: mtk-vcodec: Add msg queue feature for lat and core architecture > > media: mtk-vcodec: Generalize power and clock on/off interfaces > > media: mtk-vcodec: Add new interface to lock different hardware > > media: mtk-vcodec: Add core thread > > media: mtk-vcodec: Support 34bits dma address for vdec > > dt-bindings: media: mtk-vcodec: Adds decoder dt-bindings for mt8192 > > media: mtk-vcodec: Add core dec and dec end ipi msg > > media: mtk-vcodec: Use codec type to separate different hardware > > > > .../media/mediatek-vcodec-comp-decoder.yaml | 150 +++++++++ > > ...vcodec.txt => mediatek-vcodec-decoder.txt} | 62 +--- > > .../media/mediatek-vcodec-encoder.txt | 73 +++++ > > drivers/media/platform/mtk-vcodec/Makefile | 2 + > > .../platform/mtk-vcodec/mtk_vcodec_dec.c | 4 +- > > .../platform/mtk-vcodec/mtk_vcodec_dec.h | 1 + > > .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 297 +++++++++++++++--- > > .../platform/mtk-vcodec/mtk_vcodec_dec_hw.c | 194 ++++++++++++ > > .../platform/mtk-vcodec/mtk_vcodec_dec_hw.h | 50 +++ > > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.c | 98 ++++-- > > .../platform/mtk-vcodec/mtk_vcodec_dec_pm.h | 13 +- > > .../mtk-vcodec/mtk_vcodec_dec_stateful.c | 1 + > > .../mtk-vcodec/mtk_vcodec_dec_stateless.c | 1 + > > .../platform/mtk-vcodec/mtk_vcodec_drv.h | 72 ++++- > > .../platform/mtk-vcodec/mtk_vcodec_enc_pm.c | 1 - > > .../platform/mtk-vcodec/mtk_vcodec_intr.c | 32 +- > > .../platform/mtk-vcodec/mtk_vcodec_intr.h | 4 +- > > .../platform/mtk-vcodec/mtk_vcodec_util.c | 87 ++++- > > .../platform/mtk-vcodec/mtk_vcodec_util.h | 8 +- > > .../media/platform/mtk-vcodec/vdec_drv_if.c | 21 +- > > .../media/platform/mtk-vcodec/vdec_ipi_msg.h | 16 +- > > .../platform/mtk-vcodec/vdec_msg_queue.c | 286 +++++++++++++++++ > > .../platform/mtk-vcodec/vdec_msg_queue.h | 143 +++++++++ > > .../media/platform/mtk-vcodec/vdec_vpu_if.c | 46 ++- > > .../media/platform/mtk-vcodec/vdec_vpu_if.h | 22 ++ > > 25 files changed, 1516 insertions(+), 168 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-vcodec-comp-decoder.yaml > > rename Documentation/devicetree/bindings/media/{mediatek-vcodec.txt => mediatek-vcodec-decoder.txt} (56%) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-vcodec-encoder.txt > > create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_hw.c > > create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_hw.h > > create mode 100644 drivers/media/platform/mtk-vcodec/vdec_msg_queue.c > > create mode 100644 drivers/media/platform/mtk-vcodec/vdec_msg_queue.h > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel