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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0CA39C433FE for ; Wed, 3 Nov 2021 08:03:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E7FBB61156 for ; Wed, 3 Nov 2021 08:03:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231396AbhKCIGK (ORCPT ); Wed, 3 Nov 2021 04:06:10 -0400 Received: from mailgw02.mediatek.com ([210.61.82.184]:58656 "EHLO mailgw02.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S229504AbhKCIGJ (ORCPT ); Wed, 3 Nov 2021 04:06:09 -0400 X-UUID: 5c85692030744872b371dba8f43547ed-20211103 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=VYJwlQewBN6GJbnEUto0c9JoA556ekjEp/y7UAh+224=; b=bj0vc9j62aleMBBXSbenQ5oxkjw85uvy2HNkv4Qs90H3YZ8NiZNK1maFF47B317hf+z2zRHJqBCgnc0ZGNXEhh6i8JXEZHJkeN1R8Zx9qK1i4mvMwrYX9JYnMq79YvIMpSbVFDW35cf098SuzdIP73PdwFhs6Kp3EmZN7jrm2Gs=; X-UUID: 5c85692030744872b371dba8f43547ed-20211103 Received: from mtkmbs10n2.mediatek.inc [(172.21.101.183)] by mailgw02.mediatek.com (envelope-from ) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-GCM-SHA384 256/256) with ESMTP id 1714250672; Wed, 03 Nov 2021 16:03:30 +0800 Received: from mtkmbs10n1.mediatek.inc (172.21.101.34) by mtkmbs10n2.mediatek.inc (172.21.101.183) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.3; Wed, 3 Nov 2021 16:03:29 +0800 Received: from mhfsdcap04 (10.17.3.154) by mtkmbs10n1.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.2.792.15 via Frontend Transport; Wed, 3 Nov 2021 16:03:28 +0800 Message-ID: Subject: Re: [PATCH v4] media: mtk-vpu: Ensure alignment of 8 for DTCM buffer From: houlong wei To: , Dafna Hirschfeld , Alexandre Courbot , Hans Verkuil CC: Linux Media Mailing List , "moderated list:ARM/Mediatek SoC support" , LKML , "kernel@collabora.com" , Enric Balletbo i Serra , Dafna Hirschfeld , "Tiffany Lin =?UTF-8?Q?=28=E6=9E=97=E6=85=A7=E7=8F=8A=29?=" , Andrew-CT Chen =?UTF-8?Q?=28=E9=99=B3=E6=99=BA=E8=BF=AA=29?= , Minghsiu Tsai =?UTF-8?Q?=28=E8=94=A1=E6=98=8E=E4=BF=AE=29?= , Mauro Carvalho Chehab , Matthias Brugger , Date: Wed, 3 Nov 2021 16:03:28 +0800 In-Reply-To: <9475ac5b-79fe-da0e-ed1c-a91275cad46e@collabora.com> References: <20210920170408.1561-1-dafna.hirschfeld@collabora.com> <9475ac5b-79fe-da0e-ed1c-a91275cad46e@collabora.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.28.5-0ubuntu0.18.04.2 MIME-Version: 1.0 X-MTK: N Content-Transfer-Encoding: base64 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org QWRkIG10ay12cHUgZHJpdmVyIGV4cGVydCBpcnVpLndhbmcgaW4gdGhlIGxvb3AuDQoNCk9uIE1v biwgMjAyMS0xMC0xOCBhdCAxNTowNyArMDgwMCwgRGFmbmEgSGlyc2NoZmVsZCB3cm90ZToNCj4g DQo+IE9uIDE4LjEwLjIxIDAzOjE2LCBBbGV4YW5kcmUgQ291cmJvdCB3cm90ZToNCj4gPiBIaSBI YW5zIQ0KPiA+IA0KPiA+IE9uIE1vbiwgT2N0IDQsIDIwMjEgYXQgNjozNyBQTSBIYW5zIFZlcmt1 aWwgPGh2ZXJrdWlsQHhzNGFsbC5ubD4NCj4gPiB3cm90ZToNCj4gPiA+IA0KPiA+ID4gT24gMjAv MDkvMjAyMSAxOTowNCwgRGFmbmEgSGlyc2NoZmVsZCB3cm90ZToNCj4gPiA+ID4gRnJvbTogQWxl eGFuZHJlIENvdXJib3QgPGFjb3VyYm90QGNocm9taXVtLm9yZz4NCj4gPiA+ID4gDQo+ID4gPiA+ IFdoZW4gcnVubmluZyBtZW1jcHlfdG9pbzoNCj4gPiA+ID4gbWVtY3B5X3RvaW8oc2VuZF9vYmot PnNoYXJlX2J1ZiwgYnVmLCBsZW4pOw0KPiA+ID4gPiBpdCB3YXMgZm91bmQgdGhhdCBlcnJvcnMg YXBwZWFyIGlmIGxlbiBpcyBub3QgYSBtdWx0aXBsZSBvZiA4Og0KPiA+ID4gPiANCj4gPiA+ID4g WzU4LjM1MDg0MV0gbXRrLW1kcCAxNDAwMTAwMC5yZG1hOiBwcm9jZXNzaW5nIGZhaWxlZDogLTIy DQo+ID4gPiANCj4gPiA+IFdoeSBkbyBlcnJvcnMgYXBwZWFyPyBJcyB0aGF0IGR1ZSB0byBhIEhX IGJ1Zz8gU29tZSBvdGhlciByZWFzb24/DQo+ID4gDQo+ID4gTVRLIGZvbGtzIHdvdWxkIGJlIHRo ZSBiZXN0IHBsYWNlZCB0byBhbnN3ZXIgdGhpcywgYnV0IHNpbmNlIHRoZQ0KPiA+IGZhaWx1cmUg aXMgcmVwb3J0ZWQgYnkgdGhlIGZpcm13YXJlIEknZCBzdXNwZWN0IGVpdGhlciBhIGZpcm13YXJl DQo+ID4gb3INCj4gPiBoYXJkd2FyZSBsaW1pdGF0aW9uLg0KPiA+IA0KPiA+ID4gDQo+ID4gPiA+ IA0KPiA+ID4gPiBUaGlzIHBhdGNoIGVuc3VyZXMgdGhlIGNvcHkgb2YgYSBtdWx0aXBsZSBvZiA4 IHNpemUgYnkgY2FsbGluZw0KPiA+ID4gPiByb3VuZF91cChsZW4sIDgpIHdoZW4gY29weWluZw0K PiA+ID4gPiANCj4gPiA+ID4gRml4ZXM6IGU2NTk5YWRmYWQzMCAoIm1lZGlhOiBtdGstdnB1OiBh dm9pZCB1bmFsaWduZWQgYWNjZXNzIHRvDQo+ID4gPiA+IERUQ00gYnVmZmVyLiIpDQo+ID4gPiA+ IFNpZ25lZC1vZmYtYnk6IEFsZXhhbmRyZSBDb3VyYm90IDxhY291cmJvdEBjaHJvbWl1bS5vcmc+ DQo+ID4gPiA+IFNpZ25lZC1vZmYtYnk6IEVucmljIEJhbGxldGJvIGkgU2VycmEgPA0KPiA+ID4g PiBlbnJpYy5iYWxsZXRib0Bjb2xsYWJvcmEuY29tPg0KPiA+ID4gPiBTaWduZWQtb2ZmLWJ5OiBE YWZuYSBIaXJzY2hmZWxkIDxkYWZuYS5oaXJzY2hmZWxkQGNvbGxhYm9yYS5jb20NCj4gPiA+ID4g Pg0KPiA+ID4gPiBSZXZpZXdlZC1ieTogSG91bG9uZyBXZWkgPGhvdWxvbmcud2VpQG1lZGlhdGVr LmNvbT4NCj4gPiA+ID4gLS0tDQo+ID4gPiA+IGNoYW5nZXMgc2luY2UgdjM6DQo+ID4gPiA+IDEu IG11bHRpbGUgLT4gbXVsdGlwbGUNCj4gPiA+ID4gMi4gYWRkIGlubGluZSBkb2MNCj4gPiA+ID4g DQo+ID4gPiA+IGNoYW5nZXMgc2luY2UgdjI6DQo+ID4gPiA+IDEuIGRvIHRoZSBleHRyYSBjb3B5 IG9ubHkgaWYgbGVuIGlzIG5vdCBtdWx0aXBsZSBvZiA4DQo+ID4gPiA+IA0KPiA+ID4gPiBjaGFu Z2VzIHNpbmNlIHYxOg0KPiA+ID4gPiAxLiBjaGFuZ2Ugc2lnbi1vZmYtYnkgdGFncw0KPiA+ID4g PiAyLiBjaGFuZ2UgdmFsdWVzIHRvIG1lbXNldA0KPiA+ID4gPiANCj4gPiA+ID4gICBkcml2ZXJz L21lZGlhL3BsYXRmb3JtL210ay12cHUvbXRrX3ZwdS5jIHwgMTUgKysrKysrKysrKysrKystDQo+ ID4gPiA+ICAgMSBmaWxlIGNoYW5nZWQsIDE0IGluc2VydGlvbnMoKyksIDEgZGVsZXRpb24oLSkN Cj4gPiA+ID4gDQo+ID4gPiA+IGRpZmYgLS1naXQgYS9kcml2ZXJzL21lZGlhL3BsYXRmb3JtL210 ay12cHUvbXRrX3ZwdS5jDQo+ID4gPiA+IGIvZHJpdmVycy9tZWRpYS9wbGF0Zm9ybS9tdGstdnB1 L210a192cHUuYw0KPiA+ID4gPiBpbmRleCBlYzI5MGRkZTU5Y2YuLjFkZjAzMTcxNmM4ZiAxMDA2 NDQNCj4gPiA+ID4gLS0tIGEvZHJpdmVycy9tZWRpYS9wbGF0Zm9ybS9tdGstdnB1L210a192cHUu Yw0KPiA+ID4gPiArKysgYi9kcml2ZXJzL21lZGlhL3BsYXRmb3JtL210ay12cHUvbXRrX3ZwdS5j DQo+ID4gPiA+IEBAIC0zNDksNyArMzQ5LDIwIEBAIGludCB2cHVfaXBpX3NlbmQoc3RydWN0IHBs YXRmb3JtX2RldmljZQ0KPiA+ID4gPiAqcGRldiwNCj4gPiA+ID4gICAgICAgICAgICAgICAgfQ0K PiA+ID4gPiAgICAgICAgfSB3aGlsZSAodnB1X2NmZ19yZWFkbCh2cHUsIEhPU1RfVE9fVlBVKSk7 DQo+ID4gPiA+IA0KPiA+ID4gPiAtICAgICBtZW1jcHlfdG9pbyhzZW5kX29iai0+c2hhcmVfYnVm LCBidWYsIGxlbik7DQo+ID4gPiA+ICsgICAgIC8qDQo+ID4gPiA+ICsgICAgICAqIHdoZW4gY29w eWluZyBkYXRhIHRvIHRoZSB2cHUgaGFyZHdhcmUsIHRoZSBtZW1jcHlfdG9pbw0KPiA+ID4gPiBv cGVyYXRpb24gbXVzdCBjb3B5DQo+ID4gPiA+ICsgICAgICAqIGEgbXVsdGlwbGUgb2YgOC4gT3Ro ZXJ3aXNlIHRoZSBwcm9jZXNzaW5nIGZhaWxzDQo+ID4gPiANCj4gPiA+IFNhbWUgaGVyZTogaXQg bmVlZHMgdG8gZXhwbGFpbiB3aHkgdGhlIHByb2Nlc3NpbmcgZmFpbHMuDQo+IA0KPiBJcyB3cml0 aW5nICdkdWUgdG8gaGFyZHdhcmUgb3IgZmlybXdhcmUgbGltaXRhdGlvbicgZW5vdWdoPw0KPiBJ ZiBub3QsIHRoZW4gd2Ugc2hvdWxkIHdhaXQgZm9yIG1lZGlhdGVrIHBlb3BsZSdzIHJlc3BvbnNl IHRvIGV4cGxhaW4NCj4gaWYgdGhleSBrbm93IG1vcmUNCj4gDQo+ID4gPiANCj4gPiA+ID4gKyAg ICAgICovDQo+ID4gPiA+ICsgICAgIGlmIChsZW4gJSA4ICE9IDApIHsNCj4gPiA+ID4gKyAgICAg ICAgICAgICB1bnNpZ25lZCBjaGFyIGRhdGFbU0hBUkVfQlVGX1NJWkVdOw0KPiA+ID4gDQo+ID4g PiBXb3VsZG4ndCBpdCBiZSBtb3JlIHJvYnVzdCBpZiB5b3Ugc2F5Og0KPiA+ID4gDQo+ID4gPiAg ICAgICAgICAgICAgICAgIHVuc2lnbmVkIGNoYXIgZGF0YVtzaXplb2Yoc2VuZF9vYmotPnNoYXJl X2J1ZildOw0KPiA+IA0KPiA+IERlZmluaXRlbHkgeWVzLg0KPiANCj4gSSdsbCBzZW5kIHY1IGZp eGluZyB0aGlzDQo+IA0KPiA+IA0KPiA+ID4gDQo+ID4gPiBJIGFsc28gdGhpbmsgdGhhdCB0aGUg U0hBUkVfQlVGX1NJWkUgZGVmaW5lIG5lZWRzIGEgY29tbWVudA0KPiA+ID4gc3RhdGluZyB0aGF0 IGl0IG11c3QgYmUgYQ0KPiA+ID4gbXVsdGlwbGUgb2YgOCwgb3RoZXJ3aXNlIHVuZXhwZWN0ZWQg dGhpbmdzIGNhbiBoYXBwZW4uDQo+ID4gPiANCj4gPiA+IFlvdSBhbHNvIG5vdGljZWQgdGhhdCB0 aGUgY3VycmVudCBTSEFSRV9CVUZfU0laRSBkZWZpbmUgaXMgdG9vDQo+ID4gPiBsb3csIGJ1dCBJ IHNhdw0KPiA+ID4gbm8gcGF0Y2ggY29ycmVjdGluZyB0aGlzLiBTaG91bGRuJ3QgdGhhdCBiZSBm aXhlZCBhcyB3ZWxsPw0KPiA+IA0KPiA+IEFGQUlDVCB0aGUgZmlybXdhcmUgZXhwZWN0cyB0aGlz IGV4YWN0IHNpemUgb24gaXRzIGVuZCwgc28gSSBkb24ndA0KPiA+IGJlbGlldmUgaXQgY2FuIGJl IGNoYW5nZWQgdGhhdCBlYXNpbHkuIEJ1dCBtYXliZSBzb21lb25lIGZyb20gTVRLDQo+ID4gY2Fu DQo+ID4gcHJvdmUgbWUgd3JvbmcuDQo+ID4gDQo+IA0KPiBJIGxvb2tlZCBmdXJ0aGVyIGFuZCBu b3RlZCB0aGF0IHRoZSBzdHJ1Y3RzIHRoYXQgYXJlIGxhcmdlciB0aGFuDQo+ICdTSEFSRV9CVUZf U0laRScNCj4gKHZlbmNfYXBfaXBpX21zZ19lbmNfZXh0IHZlbmNfYXBfaXBpX21zZ19zZXRfcGFy YW1fZXh0KQ0KPiBhcmUgdXNlZCBieSBkcml2ZXJzIHRoYXQgZG9uJ3QgdXNlIHRoaXMgdnB1IGFw aSwgc28gYWN0dWFsbHkNCj4gU0hBUkVfQlVGX1NJWkUgaXMNCj4gbm90IHRvbyBsb3cgYW5kIGFz IENvcnVyYm90IHdvcnRlIHByb2JhYmx5IG5vdCBjaGFuZ2VhYmxlLg0KPiANCj4gDQo+IFRoYW5r cywNCj4gRGFmbmENCj4gDQo+ID4gQ2hlZXJzLA0KPiA+IEFsZXguDQo+ID4gDQo= 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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 517DBC433F5 for ; Wed, 3 Nov 2021 08:11: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 07F48610E5 for ; Wed, 3 Nov 2021 08:11:50 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 07F48610E5 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=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=q2jblrCsdF3ZLYWBf8Jemw2jqBFer6ESJ9q4NLymN/k=; b=nN37m8dvrUKUvH PkpV78Tb8jDR22b0A9dA1lFUBEA21S2iwu09U0dnJzoyQKJJWrNUQxcPqY/6AuvolaXiUV9zMHzUq dCsSLKU4Hv45ymS1yrschkCEA5LzAYI9cG2wdXD2/VLOpIrlnIOnkgNn4jz+xsCcvHiTUoUdxC7fV 8f1Fm2KMFn4r634FIyBcNnBg7p1FAoYj9VMhYRv7fg2Tx9ydPWYgb6yag/bi5RC+89vZ+jUmpZ+hj v+RShp80xu8K6+yMfRHt2/+KSIah9s68bZlGKxJ+MuO6Vkk6Z7LT4Um+Ff+UIJdatvk51uoYsf3dy dfBLh6QHIsvc/LhFlaxQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1miBMu-004Tjr-T7; Wed, 03 Nov 2021 08:11:36 +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 1miBMs-004Tht-9y for linux-mediatek@lists.infradead.org; Wed, 03 Nov 2021 08:11:35 +0000 X-UUID: 37bc2ad6d988485a99356acaf783b288-20211103 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=VYJwlQewBN6GJbnEUto0c9JoA556ekjEp/y7UAh+224=; b=bj0vc9j62aleMBBXSbenQ5oxkjw85uvy2HNkv4Qs90H3YZ8NiZNK1maFF47B317hf+z2zRHJqBCgnc0ZGNXEhh6i8JXEZHJkeN1R8Zx9qK1i4mvMwrYX9JYnMq79YvIMpSbVFDW35cf098SuzdIP73PdwFhs6Kp3EmZN7jrm2Gs=; X-UUID: 37bc2ad6d988485a99356acaf783b288-20211103 Received: from mtkcas66.mediatek.inc [(172.29.193.44)] by mailgw02.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1596613237; Wed, 03 Nov 2021 01:11:27 -0700 Received: from mtkmbs10n2.mediatek.inc (172.21.101.183) by MTKMBS62N1.mediatek.inc (172.29.193.41) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Wed, 3 Nov 2021 01:03:30 -0700 Received: from mtkmbs10n1.mediatek.inc (172.21.101.34) by mtkmbs10n2.mediatek.inc (172.21.101.183) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.792.3; Wed, 3 Nov 2021 16:03:29 +0800 Received: from mhfsdcap04 (10.17.3.154) by mtkmbs10n1.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.2.792.15 via Frontend Transport; Wed, 3 Nov 2021 16:03:28 +0800 Message-ID: Subject: Re: [PATCH v4] media: mtk-vpu: Ensure alignment of 8 for DTCM buffer From: houlong wei To: , Dafna Hirschfeld , Alexandre Courbot , Hans Verkuil CC: Linux Media Mailing List , "moderated list:ARM/Mediatek SoC support" , LKML , "kernel@collabora.com" , Enric Balletbo i Serra , Dafna Hirschfeld , "Tiffany Lin =?UTF-8?Q?=28=E6=9E=97=E6=85=A7=E7=8F=8A=29?=" , Andrew-CT Chen =?UTF-8?Q?=28=E9=99=B3=E6=99=BA=E8=BF=AA=29?= , Minghsiu Tsai =?UTF-8?Q?=28=E8=94=A1=E6=98=8E=E4=BF=AE=29?= , Mauro Carvalho Chehab , Matthias Brugger , Date: Wed, 3 Nov 2021 16:03:28 +0800 In-Reply-To: <9475ac5b-79fe-da0e-ed1c-a91275cad46e@collabora.com> References: <20210920170408.1561-1-dafna.hirschfeld@collabora.com> <9475ac5b-79fe-da0e-ed1c-a91275cad46e@collabora.com> X-Mailer: Evolution 3.28.5-0ubuntu0.18.04.2 MIME-Version: 1.0 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211103_011134_362805_2D7ECC87 X-CRM114-Status: GOOD ( 41.65 ) 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 Add mtk-vpu driver expert irui.wang in the loop. On Mon, 2021-10-18 at 15:07 +0800, Dafna Hirschfeld wrote: > > On 18.10.21 03:16, Alexandre Courbot wrote: > > Hi Hans! > > > > On Mon, Oct 4, 2021 at 6:37 PM Hans Verkuil > > wrote: > > > > > > On 20/09/2021 19:04, Dafna Hirschfeld wrote: > > > > From: Alexandre Courbot > > > > > > > > When running memcpy_toio: > > > > memcpy_toio(send_obj->share_buf, buf, len); > > > > it was found that errors appear if len is not a multiple of 8: > > > > > > > > [58.350841] mtk-mdp 14001000.rdma: processing failed: -22 > > > > > > Why do errors appear? Is that due to a HW bug? Some other reason? > > > > MTK folks would be the best placed to answer this, but since the > > failure is reported by the firmware I'd suspect either a firmware > > or > > hardware limitation. > > > > > > > > > > > > > This patch ensures the copy of a multiple of 8 size by calling > > > > round_up(len, 8) when copying > > > > > > > > Fixes: e6599adfad30 ("media: mtk-vpu: avoid unaligned access to > > > > DTCM buffer.") > > > > Signed-off-by: Alexandre Courbot > > > > Signed-off-by: Enric Balletbo i Serra < > > > > enric.balletbo@collabora.com> > > > > Signed-off-by: Dafna Hirschfeld > > > > > > > > Reviewed-by: Houlong Wei > > > > --- > > > > changes since v3: > > > > 1. multile -> multiple > > > > 2. add inline doc > > > > > > > > changes since v2: > > > > 1. do the extra copy only if len is not multiple of 8 > > > > > > > > changes since v1: > > > > 1. change sign-off-by tags > > > > 2. change values to memset > > > > > > > > drivers/media/platform/mtk-vpu/mtk_vpu.c | 15 ++++++++++++++- > > > > 1 file changed, 14 insertions(+), 1 deletion(-) > > > > > > > > diff --git a/drivers/media/platform/mtk-vpu/mtk_vpu.c > > > > b/drivers/media/platform/mtk-vpu/mtk_vpu.c > > > > index ec290dde59cf..1df031716c8f 100644 > > > > --- a/drivers/media/platform/mtk-vpu/mtk_vpu.c > > > > +++ b/drivers/media/platform/mtk-vpu/mtk_vpu.c > > > > @@ -349,7 +349,20 @@ int vpu_ipi_send(struct platform_device > > > > *pdev, > > > > } > > > > } while (vpu_cfg_readl(vpu, HOST_TO_VPU)); > > > > > > > > - memcpy_toio(send_obj->share_buf, buf, len); > > > > + /* > > > > + * when copying data to the vpu hardware, the memcpy_toio > > > > operation must copy > > > > + * a multiple of 8. Otherwise the processing fails > > > > > > Same here: it needs to explain why the processing fails. > > Is writing 'due to hardware or firmware limitation' enough? > If not, then we should wait for mediatek people's response to explain > if they know more > > > > > > > > + */ > > > > + if (len % 8 != 0) { > > > > + unsigned char data[SHARE_BUF_SIZE]; > > > > > > Wouldn't it be more robust if you say: > > > > > > unsigned char data[sizeof(send_obj->share_buf)]; > > > > Definitely yes. > > I'll send v5 fixing this > > > > > > > > > I also think that the SHARE_BUF_SIZE define needs a comment > > > stating that it must be a > > > multiple of 8, otherwise unexpected things can happen. > > > > > > You also noticed that the current SHARE_BUF_SIZE define is too > > > low, but I saw > > > no patch correcting this. Shouldn't that be fixed as well? > > > > AFAICT the firmware expects this exact size on its end, so I don't > > believe it can be changed that easily. But maybe someone from MTK > > can > > prove me wrong. > > > > I looked further and noted that the structs that are larger than > 'SHARE_BUF_SIZE' > (venc_ap_ipi_msg_enc_ext venc_ap_ipi_msg_set_param_ext) > are used by drivers that don't use this vpu api, so actually > SHARE_BUF_SIZE is > not too low and as Corurbot worte probably not changeable. > > > Thanks, > Dafna > > > Cheers, > > Alex. > > _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek