From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751402AbcGMBvy (ORCPT ); Tue, 12 Jul 2016 21:51:54 -0400 Received: from mailgw01.mediatek.com ([210.61.82.183]:64390 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751016AbcGMBvq (ORCPT ); Tue, 12 Jul 2016 21:51:46 -0400 Message-ID: <1468374654.2462.17.camel@mtksdaap41> Subject: Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver From: tiffany lin To: CC: Wu-Cheng Li =?UTF-8?Q?=28=E6=9D=8E=E5=8B=99=E8=AA=A0=29?= , Hans Verkuil , Hans Verkuil , Daniel Thompson , "Rob Herring" , Mauro Carvalho Chehab , Matthias Brugger , Daniel Kurtz , Pawel Osciak , Eddie Huang , Yingjoe Chen , , , , , , Lin PoChun Date: Wed, 13 Jul 2016 09:50:54 +0800 In-Reply-To: <1468350511.8843.16.camel@gmail.com> References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-2-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-3-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-4-git-send-email-tiffany.lin@mediatek.com> <5a793171-24a7-4e9e-8bfd-f668c789f8e0@xs4all.nl> <1468205771.3725.8.camel@mtksdaap41> <1468350511.8843.16.camel@gmail.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 8bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Nicolas, On Tue, 2016-07-12 at 15:08 -0400, Nicolas Dufresne wrote: > Le mardi 12 juillet 2016 à 16:16 +0800, Wu-Cheng Li (李務誠) a écrit : > > Decoder hardware produces MT21 (compressed). Image processor can > > convert it to a format that can be input of display driver. Tiffany. > > When do you plan to upstream image processor (mtk-mdp)? > > > > > > It can be as input format for encoder, MDP and display drivers in > > our > > > platform. > > I remember display driver can only accept uncompressed MT21. Right? > > Basically V4L2_PIX_FMT_MT21 is compressed and is like an opaque > > format. It's not usable until it's decompressed and converted by > > image > > processor. > > Previously it was described as MediaTek block mode, and now as a > MediaTek compressed format. It makes me think you have no idea what > this pixel format really is. Is that right ? > That's not right. Its a compressed format as I document in "[PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver." In MT8173 platform, when using this format, we need Image Processor to cover it to standard format as wucheng mentioned. To prevent this ambiguous, I will change it to V4L2_PIX_FMT_M21C, it means its compressed data. Is it ok? best regards, Tiffany > The main reason why I keep asking, is that we often find similarities > between what vendor like to call their proprietary formats. Doing the > proper research helps not creating a mess like in Android where you > have a lot of formats that all point to the same format. I believe > there was the same concern when Samsung wanted to introduce their Z- > flip-Z NV12 tile format. In the end they simply provided sufficient > documentation so we could document it and implement software converters > for test and validation purpose. > > regards, > Nicolas From mboxrd@z Thu Jan 1 00:00:00 1970 From: tiffany lin Subject: Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver Date: Wed, 13 Jul 2016 09:50:54 +0800 Message-ID: <1468374654.2462.17.camel@mtksdaap41> References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-2-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-3-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-4-git-send-email-tiffany.lin@mediatek.com> <5a793171-24a7-4e9e-8bfd-f668c789f8e0@xs4all.nl> <1468205771.3725.8.camel@mtksdaap41> <1468350511.8843.16.camel@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1468350511.8843.16.camel-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+glpam-linux-mediatek=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: nicolas-dDhyB4GVkw9AFePFGvp55w@public.gmane.org Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Daniel Thompson , Mauro Carvalho Chehab , linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Hans Verkuil , Lin PoChun , Rob Herring , Hans Verkuil , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Yingjoe Chen , Matthias Brugger , Wu-Cheng Li =?UTF-8?Q?=28=E6=9D=8E=E5=8B=99=E8=AA=A0=29?= , Eddie Huang , Pawel Osciak , linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org SGkgTmljb2xhcywKCk9uIFR1ZSwgMjAxNi0wNy0xMiBhdCAxNTowOCAtMDQwMCwgTmljb2xhcyBE dWZyZXNuZSB3cm90ZToKPiBMZSBtYXJkaSAxMiBqdWlsbGV0IDIwMTYgw6AgMTY6MTYgKzA4MDAs IFd1LUNoZW5nIExpICjmnY7li5noqqApIGEgw6ljcml0IDoKPiA+IERlY29kZXIgaGFyZHdhcmUg cHJvZHVjZXMgTVQyMSAoY29tcHJlc3NlZCkuIEltYWdlIHByb2Nlc3NvciBjYW4KPiA+IGNvbnZl cnQgaXQgdG8gYSBmb3JtYXQgdGhhdCBjYW4gYmUgaW5wdXQgb2YgZGlzcGxheSBkcml2ZXIuIFRp ZmZhbnkuCj4gPiBXaGVuIGRvIHlvdSBwbGFuIHRvIHVwc3RyZWFtIGltYWdlIHByb2Nlc3NvciAo bXRrLW1kcCk/Cj4gPiA+Cj4gPiA+IEl0IGNhbiBiZSBhcyBpbnB1dCBmb3JtYXQgZm9yIGVuY29k ZXIsIE1EUCBhbmQgZGlzcGxheSBkcml2ZXJzIGluCj4gPiBvdXIKPiA+ID4gcGxhdGZvcm0uCj4g PiBJIHJlbWVtYmVyIGRpc3BsYXkgZHJpdmVyIGNhbiBvbmx5IGFjY2VwdCB1bmNvbXByZXNzZWQg TVQyMS4gUmlnaHQ/Cj4gPiBCYXNpY2FsbHkgVjRMMl9QSVhfRk1UX01UMjEgaXMgY29tcHJlc3Nl ZCBhbmQgaXMgbGlrZSBhbiBvcGFxdWUKPiA+IGZvcm1hdC4gSXQncyBub3QgdXNhYmxlIHVudGls IGl0J3MgZGVjb21wcmVzc2VkIGFuZCBjb252ZXJ0ZWQgYnkKPiA+IGltYWdlCj4gPiBwcm9jZXNz b3IuCj4gCj4gUHJldmlvdXNseSBpdCB3YXMgZGVzY3JpYmVkIGFzIE1lZGlhVGVrIGJsb2NrIG1v ZGUsIGFuZCBub3cgYXMgYQo+IE1lZGlhVGVrIGNvbXByZXNzZWQgZm9ybWF0LiBJdCBtYWtlcyBt ZSB0aGluayB5b3UgaGF2ZSBubyBpZGVhIHdoYXQKPiB0aGlzIHBpeGVsIGZvcm1hdCByZWFsbHkg aXMuIElzIHRoYXQgcmlnaHQgPwo+IApUaGF0J3Mgbm90IHJpZ2h0LgpJdHMgYSBjb21wcmVzc2Vk IGZvcm1hdCBhcyBJIGRvY3VtZW50IGluICJbUEFUQ0ggdjMgMy85XSBEb2NCb29rL3Y0bDoKQWRk IGNvbXByZXNzZWQgdmlkZW8gZm9ybWF0cyB1c2VkIG9uIE1UODE3MyBjb2RlYyBkcml2ZXIuIgpJ biBNVDgxNzMgcGxhdGZvcm0sIHdoZW4gdXNpbmcgdGhpcyBmb3JtYXQsIHdlIG5lZWQgSW1hZ2Ug UHJvY2Vzc29yIHRvCmNvdmVyIGl0IHRvIHN0YW5kYXJkIGZvcm1hdCBhcyB3dWNoZW5nIG1lbnRp b25lZC4KVG8gcHJldmVudCB0aGlzIGFtYmlndW91cywgSSB3aWxsIGNoYW5nZSBpdCB0byBWNEwy X1BJWF9GTVRfTTIxQywgaXQKbWVhbnMgaXRzIGNvbXByZXNzZWQgZGF0YS4gSXMgaXQgb2s/Cgpi ZXN0IHJlZ2FyZHMsClRpZmZhbnkKCj4gVGhlIG1haW4gcmVhc29uIHdoeSBJIGtlZXAgYXNraW5n LCBpcyB0aGF0IHdlIG9mdGVuIGZpbmQgc2ltaWxhcml0aWVzCj4gYmV0d2VlbiB3aGF0IHZlbmRv ciBsaWtlIHRvIGNhbGwgdGhlaXIgcHJvcHJpZXRhcnkgZm9ybWF0cy4gRG9pbmcgdGhlCj4gcHJv cGVyIHJlc2VhcmNoIGhlbHBzIG5vdCBjcmVhdGluZyBhIG1lc3MgbGlrZSBpbiBBbmRyb2lkIHdo ZXJlIHlvdQo+IGhhdmUgYSBsb3Qgb2YgZm9ybWF0cyB0aGF0IGFsbCBwb2ludCB0byB0aGUgc2Ft ZSBmb3JtYXQuIEkgYmVsaWV2ZQo+IHRoZXJlIHdhcyB0aGUgc2FtZSBjb25jZXJuIHdoZW4gU2Ft c3VuZyB3YW50ZWQgdG8gaW50cm9kdWNlIHRoZWlyIFotCj4gZmxpcC1aIE5WMTIgdGlsZSBmb3Jt YXQuIEluIHRoZSBlbmQgdGhleSBzaW1wbHkgcHJvdmlkZWQgc3VmZmljaWVudAo+IGRvY3VtZW50 YXRpb24gc28gd2UgY291bGQgZG9jdW1lbnQgaXQgYW5kIGltcGxlbWVudCBzb2Z0d2FyZSBjb252 ZXJ0ZXJzCj4gZm9yIHRlc3QgYW5kIHZhbGlkYXRpb24gcHVycG9zZS4KPiAKPiByZWdhcmRzLAo+ IE5pY29sYXMKCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18KTGludXgtbWVkaWF0ZWsgbWFpbGluZyBsaXN0CkxpbnV4LW1lZGlhdGVrQGxpc3RzLmluZnJh ZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51 eC1tZWRpYXRlawo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: tiffany.lin@mediatek.com (tiffany lin) Date: Wed, 13 Jul 2016 09:50:54 +0800 Subject: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver In-Reply-To: <1468350511.8843.16.camel@gmail.com> References: <1464611363-14936-1-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-2-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-3-git-send-email-tiffany.lin@mediatek.com> <1464611363-14936-4-git-send-email-tiffany.lin@mediatek.com> <5a793171-24a7-4e9e-8bfd-f668c789f8e0@xs4all.nl> <1468205771.3725.8.camel@mtksdaap41> <1468350511.8843.16.camel@gmail.com> Message-ID: <1468374654.2462.17.camel@mtksdaap41> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Nicolas, On Tue, 2016-07-12 at 15:08 -0400, Nicolas Dufresne wrote: > Le mardi 12 juillet 2016 ? 16:16 +0800, Wu-Cheng Li (???) a ?crit : > > Decoder hardware produces MT21 (compressed). Image processor can > > convert it to a format that can be input of display driver. Tiffany. > > When do you plan to upstream image processor (mtk-mdp)? > > > > > > It can be as input format for encoder, MDP and display drivers in > > our > > > platform. > > I remember display driver can only accept uncompressed MT21. Right? > > Basically V4L2_PIX_FMT_MT21 is compressed and is like an opaque > > format. It's not usable until it's decompressed and converted by > > image > > processor. > > Previously it was described as MediaTek block mode, and now as a > MediaTek compressed format. It makes me think you have no idea what > this pixel format really is. Is that right ? > That's not right. Its a compressed format as I document in "[PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver." In MT8173 platform, when using this format, we need Image Processor to cover it to standard format as wucheng mentioned. To prevent this ambiguous, I will change it to V4L2_PIX_FMT_M21C, it means its compressed data. Is it ok? best regards, Tiffany > The main reason why I keep asking, is that we often find similarities > between what vendor like to call their proprietary formats. Doing the > proper research helps not creating a mess like in Android where you > have a lot of formats that all point to the same format. I believe > there was the same concern when Samsung wanted to introduce their Z- > flip-Z NV12 tile format. In the end they simply provided sufficient > documentation so we could document it and implement software converters > for test and validation purpose. > > regards, > Nicolas