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=-8.2 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,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 8BB84C433E0 for ; Thu, 18 Jun 2020 03:43:12 +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 5C40621707 for ; Thu, 18 Jun 2020 03:43:12 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Af1gXNv4"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="uwL64kzb" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5C40621707 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.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date: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=tOwzSxyS1TKsIbZ0poY57EZFZJRAp1t65DtGauhYQfE=; b=Af1gXNv4VtuthZ tXRkqA1eZ7+dwzcV7BNSEhWUCjs/cmlfSJWEkN3p3AxOfkQVdbqPKYiVqm33zsluouzqMmMelHHu6 YYzdAL8D2HPxzu60l3HbkJeI0p2rQp9Qp0aQDlAdahHr17cstBOwC5pebqpbPmmkcuY5Pzs3jvSl6 xb4XDqLR08HDTLMmqg7yqRhhpygD9BnYxenJwgwxve7iqElKLb+y41L1mK0x4q5NYqNISa2pOjQFM WjRyoUOZrvs7A5UrcaP4EFZpHG0ogvxvslDgy1noFIisGc6lntMxVhV4PJyBlt6AzOyK4ZvZgLAhp OpLjrKwIWClrIueLO+EQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jllS9-0002dB-Ua; Thu, 18 Jun 2020 03:43:01 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jllS5-0002c4-UL; Thu, 18 Jun 2020 03:42:59 +0000 X-UUID: f028bdd505e34a80973fcb25899d2a27-20200617 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=i4jSTDSTh0GaOSOIPc+y2DdxkaaCQiUZ9e4P/5TV8Tc=; b=uwL64kzbKtCqafMZJaBJO8Dg+HDeUyDHDTrUpi6lT6q0IVvr9OXI2lFFS+87Ms+NfaOvfTD6ZIS0l4p90+mQv8+bKJrxXYuHg/3fTcT0eYmJ1DoeRgU7VcxLDxThlK/r6K3Sii1s9H4FZ3p4wBfGeEr4rwPYNm2enP7Tz4tuwkI=; X-UUID: f028bdd505e34a80973fcb25899d2a27-20200617 Received: from mtkcas67.mediatek.inc [(172.29.193.45)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLS) with ESMTP id 1946055112; Wed, 17 Jun 2020 19:42:49 -0800 Received: from MTKMBS31N1.mediatek.inc (172.27.4.69) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Wed, 17 Jun 2020 20:42:49 -0700 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31N1.mediatek.inc (172.27.4.69) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 18 Jun 2020 11:42:47 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 18 Jun 2020 11:42:45 +0800 Message-ID: <1592451616.15355.25.camel@mhfsdcap03> Subject: Re: [PATCH v8 11/14] media: dt-bindings: Add jpeg enc device tree node document From: Xia Jiang To: Tomasz Figa Date: Thu, 18 Jun 2020 11:40:16 +0800 In-Reply-To: <20200521160046.GJ209565@chromium.org> References: <20200403094033.8288-1-xia.jiang@mediatek.com> <20200403094033.8288-12-xia.jiang@mediatek.com> <20200521160046.GJ209565@chromium.org> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: E100C0680B0A813F09A9DC10EAB829B13E2B394890EB73037F1C971D096F1A762000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200617_204257_981281_AE2E2B3A X-CRM114-Status: GOOD ( 21.98 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: drinkcat@chromium.org, devicetree@vger.kernel.org, mojahsu@chromium.org, srv_heupstream@mediatek.com, Rick Chang , senozhatsky@chromium.org, linux-kernel@vger.kernel.org, maoguang.meng@mediatek.com, Mauro Carvalho Chehab , sj.huang@mediatek.com, Rob Herring , Matthias Brugger , yong.wu@mediatek.com, Hans Verkuil , linux-mediatek@lists.infradead.org, Marek Szyprowski , linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org 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 Thu, 2020-05-21 at 16:00 +0000, Tomasz Figa wrote: > Hi Xia, > > On Fri, Apr 03, 2020 at 05:40:30PM +0800, Xia Jiang wrote: > > Add jpeg enc device tree node document > > > > Reviewed-by: Rob Herring > > Signed-off-by: Xia Jiang > > --- > > v8: no changes > > > > v7: no changes > > > > v6: no changes > > > > v5: no changes > > > > v4: no changes > > > > v3: change compatible to SoC specific compatible > > > > v2: no changes > > --- > > .../bindings/media/mediatek-jpeg-encoder.txt | 37 +++++++++++++++++++ > > 1 file changed, 37 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > > > Thank you for the patch. Please see my comments inline. Dear Tomasz, Sorry for missing this message. Replied below. > > > diff --git a/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > new file mode 100644 > > index 000000000000..fa8da699493b > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > @@ -0,0 +1,37 @@ > > +* MediaTek JPEG Encoder > > + > > +MediaTek JPEG Encoder is the JPEG encode hardware present in MediaTek SoCs > > + > > +Required properties: > > +- compatible : should be one of: > > + "mediatek,mt2701-jpgenc" > > + ... > > What does this "..." mean? "..." means that compatible name is not just "mediatek,mt2701-jpgenc", different project has different compatible name(for example the MT8173's compatible name may be "mediatek,mt8173-jpgenc"). > > > + followed by "mediatek,mtk-jpgenc" > > +- reg : physical base address of the JPEG encoder registers and length of > > + memory mapped region. > > +- interrupts : interrupt number to the interrupt controller. > > +- clocks: device clocks, see > > + Documentation/devicetree/bindings/clock/clock-bindings.txt for details. > > +- clock-names: must contain "jpgenc". It is the clock of JPEG encoder. > > nit: In principle the clocks should be named after the function the clock > performs on the consumer side, i.e. the JPEG block in this case, I guess > here it's just a generic clock that does everything, but I guess it comes > from somewhere. Is it the AHB clock or something? In that case it would > normally be called "ahb". I have confirmed with hardware designer that the jpeg clock is not AHB clock,it follows subsys clock(because 2701 is the old IC,I didn't get the source name).It has the same source with venc clock.We can see that the clocks = , Should I name it "venc" or the orignal "jpgenc"? > > > +- power-domains: a phandle to the power domain, see > > + Documentation/devicetree/bindings/power/power_domain.txt for details. > > +- mediatek,larb: must contain the local arbiters in the current SoCs, see > > + Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > + for details. > > I believe this isn't necessary anymore, because larbs are added > automatically by the MTK IOMMU driver using device links. +CC Yong who > worked on that. Yes,I have confirmed with Yong that he will help me to modify this.Is it ok that I keep the orignal larb code? Best Regards, Xia Jiang > > > +- iommus: should point to the respective IOMMU block with master port as > > + argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt > > + for details. > > + > > +Example: > > + jpegenc: jpegenc@1500a000 { > > + compatible = "mediatek,mt2701-jpgenc", > > + "mediatek,mtk-jpgenc"; > > + reg = <0 0x1500a000 0 0x1000>; > > + interrupts = ; > > + clocks = <&imgsys CLK_IMG_VENC>; > > + clock-names = "jpgenc"; > > + power-domains = <&scpsys MT2701_POWER_DOMAIN_ISP>; > > + mediatek,larb = <&larb2>; > > Ditto. > > Best regards, > Tomasz _______________________________________________ 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=-8.2 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,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 904F4C433DF for ; Thu, 18 Jun 2020 03:43:03 +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 50BF121707 for ; Thu, 18 Jun 2020 03:43:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="kt3iO/JS"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="uwL64kzb" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 50BF121707 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+infradead-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.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date: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=I8ZzIMhBLjFTlLcGrGOIeT/xUoc5Ni90Yxx9wzhqgfo=; b=kt3iO/JS7906dx D3woavDiQFSRlTpQ4/74rKuBhDi0k8mXwE4GC+G6IthMYlEmQkyub6oTSP1c3CQ3VvriGnV594Lou 6L/7lSafQA1WTgsL0t6Fd9nGCHbnhcF7xRxFWBVy+xxgxzLAhZHK00f7efK2AR1b2mmp892rtuzD8 fGyDOC7N3RAx+ARtmmNDcyhMiNO7xbBL0Eczw88XqQK97yA0SEdordztM7k0d7+pyRrHvBONZy7+E KPfGA3K0Zn5KzFop4cxdTW+MQ2D2mJw717Pd/gyeZJkNb+q0SNToTid1uR0yH+iP5hRV2kJi/jM9d 6FzlCmCusNg10Kt1RCEw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jllSA-0002dz-Pq; Thu, 18 Jun 2020 03:43:02 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jllS5-0002c4-UL; Thu, 18 Jun 2020 03:42:59 +0000 X-UUID: f028bdd505e34a80973fcb25899d2a27-20200617 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=i4jSTDSTh0GaOSOIPc+y2DdxkaaCQiUZ9e4P/5TV8Tc=; b=uwL64kzbKtCqafMZJaBJO8Dg+HDeUyDHDTrUpi6lT6q0IVvr9OXI2lFFS+87Ms+NfaOvfTD6ZIS0l4p90+mQv8+bKJrxXYuHg/3fTcT0eYmJ1DoeRgU7VcxLDxThlK/r6K3Sii1s9H4FZ3p4wBfGeEr4rwPYNm2enP7Tz4tuwkI=; X-UUID: f028bdd505e34a80973fcb25899d2a27-20200617 Received: from mtkcas67.mediatek.inc [(172.29.193.45)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLS) with ESMTP id 1946055112; Wed, 17 Jun 2020 19:42:49 -0800 Received: from MTKMBS31N1.mediatek.inc (172.27.4.69) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Wed, 17 Jun 2020 20:42:49 -0700 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31N1.mediatek.inc (172.27.4.69) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 18 Jun 2020 11:42:47 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 18 Jun 2020 11:42:45 +0800 Message-ID: <1592451616.15355.25.camel@mhfsdcap03> Subject: Re: [PATCH v8 11/14] media: dt-bindings: Add jpeg enc device tree node document From: Xia Jiang To: Tomasz Figa Date: Thu, 18 Jun 2020 11:40:16 +0800 In-Reply-To: <20200521160046.GJ209565@chromium.org> References: <20200403094033.8288-1-xia.jiang@mediatek.com> <20200403094033.8288-12-xia.jiang@mediatek.com> <20200521160046.GJ209565@chromium.org> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: E100C0680B0A813F09A9DC10EAB829B13E2B394890EB73037F1C971D096F1A762000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200617_204257_981281_AE2E2B3A X-CRM114-Status: GOOD ( 21.98 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: drinkcat@chromium.org, devicetree@vger.kernel.org, mojahsu@chromium.org, srv_heupstream@mediatek.com, Rick Chang , senozhatsky@chromium.org, linux-kernel@vger.kernel.org, maoguang.meng@mediatek.com, Mauro Carvalho Chehab , sj.huang@mediatek.com, Rob Herring , Matthias Brugger , yong.wu@mediatek.com, Hans Verkuil , linux-mediatek@lists.infradead.org, Marek Szyprowski , linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, 2020-05-21 at 16:00 +0000, Tomasz Figa wrote: > Hi Xia, > > On Fri, Apr 03, 2020 at 05:40:30PM +0800, Xia Jiang wrote: > > Add jpeg enc device tree node document > > > > Reviewed-by: Rob Herring > > Signed-off-by: Xia Jiang > > --- > > v8: no changes > > > > v7: no changes > > > > v6: no changes > > > > v5: no changes > > > > v4: no changes > > > > v3: change compatible to SoC specific compatible > > > > v2: no changes > > --- > > .../bindings/media/mediatek-jpeg-encoder.txt | 37 +++++++++++++++++++ > > 1 file changed, 37 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > > > Thank you for the patch. Please see my comments inline. Dear Tomasz, Sorry for missing this message. Replied below. > > > diff --git a/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > new file mode 100644 > > index 000000000000..fa8da699493b > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt > > @@ -0,0 +1,37 @@ > > +* MediaTek JPEG Encoder > > + > > +MediaTek JPEG Encoder is the JPEG encode hardware present in MediaTek SoCs > > + > > +Required properties: > > +- compatible : should be one of: > > + "mediatek,mt2701-jpgenc" > > + ... > > What does this "..." mean? "..." means that compatible name is not just "mediatek,mt2701-jpgenc", different project has different compatible name(for example the MT8173's compatible name may be "mediatek,mt8173-jpgenc"). > > > + followed by "mediatek,mtk-jpgenc" > > +- reg : physical base address of the JPEG encoder registers and length of > > + memory mapped region. > > +- interrupts : interrupt number to the interrupt controller. > > +- clocks: device clocks, see > > + Documentation/devicetree/bindings/clock/clock-bindings.txt for details. > > +- clock-names: must contain "jpgenc". It is the clock of JPEG encoder. > > nit: In principle the clocks should be named after the function the clock > performs on the consumer side, i.e. the JPEG block in this case, I guess > here it's just a generic clock that does everything, but I guess it comes > from somewhere. Is it the AHB clock or something? In that case it would > normally be called "ahb". I have confirmed with hardware designer that the jpeg clock is not AHB clock,it follows subsys clock(because 2701 is the old IC,I didn't get the source name).It has the same source with venc clock.We can see that the clocks = , Should I name it "venc" or the orignal "jpgenc"? > > > +- power-domains: a phandle to the power domain, see > > + Documentation/devicetree/bindings/power/power_domain.txt for details. > > +- mediatek,larb: must contain the local arbiters in the current SoCs, see > > + Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > > + for details. > > I believe this isn't necessary anymore, because larbs are added > automatically by the MTK IOMMU driver using device links. +CC Yong who > worked on that. Yes,I have confirmed with Yong that he will help me to modify this.Is it ok that I keep the orignal larb code? Best Regards, Xia Jiang > > > +- iommus: should point to the respective IOMMU block with master port as > > + argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt > > + for details. > > + > > +Example: > > + jpegenc: jpegenc@1500a000 { > > + compatible = "mediatek,mt2701-jpgenc", > > + "mediatek,mtk-jpgenc"; > > + reg = <0 0x1500a000 0 0x1000>; > > + interrupts = ; > > + clocks = <&imgsys CLK_IMG_VENC>; > > + clock-names = "jpgenc"; > > + power-domains = <&scpsys MT2701_POWER_DOMAIN_ISP>; > > + mediatek,larb = <&larb2>; > > Ditto. > > Best regards, > Tomasz _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel 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=-8.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,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 7FD87C433E0 for ; Thu, 18 Jun 2020 03:42:55 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5257721841 for ; Thu, 18 Jun 2020 03:42:55 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=mediatek.com header.i=@mediatek.com header.b="uwL64kzb" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726966AbgFRDmy (ORCPT ); Wed, 17 Jun 2020 23:42:54 -0400 Received: from Mailgw01.mediatek.com ([1.203.163.78]:28455 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1726853AbgFRDmx (ORCPT ); Wed, 17 Jun 2020 23:42:53 -0400 X-UUID: 9c0c628e6d1147de9a55d7b9b2d44b79-20200618 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=i4jSTDSTh0GaOSOIPc+y2DdxkaaCQiUZ9e4P/5TV8Tc=; b=uwL64kzbKtCqafMZJaBJO8Dg+HDeUyDHDTrUpi6lT6q0IVvr9OXI2lFFS+87Ms+NfaOvfTD6ZIS0l4p90+mQv8+bKJrxXYuHg/3fTcT0eYmJ1DoeRgU7VcxLDxThlK/r6K3Sii1s9H4FZ3p4wBfGeEr4rwPYNm2enP7Tz4tuwkI=; X-UUID: 9c0c628e6d1147de9a55d7b9b2d44b79-20200618 Received: from mtkcas34.mediatek.inc [(172.27.4.253)] by mailgw01.mediatek.com (envelope-from ) (mailgw01.mediatek.com ESMTP with TLS) with ESMTP id 37253424; Thu, 18 Jun 2020 11:42:48 +0800 Received: from MTKCAS32.mediatek.inc (172.27.4.184) by MTKMBS31N1.mediatek.inc (172.27.4.69) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 18 Jun 2020 11:42:47 +0800 Received: from [10.17.3.153] (10.17.3.153) by MTKCAS32.mediatek.inc (172.27.4.170) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 18 Jun 2020 11:42:45 +0800 Message-ID: <1592451616.15355.25.camel@mhfsdcap03> Subject: Re: [PATCH v8 11/14] media: dt-bindings: Add jpeg enc device tree node document From: Xia Jiang To: Tomasz Figa CC: Hans Verkuil , Mauro Carvalho Chehab , Rob Herring , "Matthias Brugger" , Rick Chang , , , , , , Marek Szyprowski , , , , , , , Date: Thu, 18 Jun 2020 11:40:16 +0800 In-Reply-To: <20200521160046.GJ209565@chromium.org> References: <20200403094033.8288-1-xia.jiang@mediatek.com> <20200403094033.8288-12-xia.jiang@mediatek.com> <20200521160046.GJ209565@chromium.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: E100C0680B0A813F09A9DC10EAB829B13E2B394890EB73037F1C971D096F1A762000:8 X-MTK: N Content-Transfer-Encoding: base64 Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org T24gVGh1LCAyMDIwLTA1LTIxIGF0IDE2OjAwICswMDAwLCBUb21hc3ogRmlnYSB3cm90ZToNCj4g SGkgWGlhLA0KPiANCj4gT24gRnJpLCBBcHIgMDMsIDIwMjAgYXQgMDU6NDA6MzBQTSArMDgwMCwg WGlhIEppYW5nIHdyb3RlOg0KPiA+IEFkZCBqcGVnIGVuYyBkZXZpY2UgdHJlZSBub2RlIGRvY3Vt ZW50DQo+ID4gDQo+ID4gUmV2aWV3ZWQtYnk6IFJvYiBIZXJyaW5nIDxyb2JoQGtlcm5lbC5vcmc+ DQo+ID4gU2lnbmVkLW9mZi1ieTogWGlhIEppYW5nIDx4aWEuamlhbmdAbWVkaWF0ZWsuY29tPg0K PiA+IC0tLQ0KPiA+IHY4OiBubyBjaGFuZ2VzDQo+ID4gDQo+ID4gdjc6IG5vIGNoYW5nZXMNCj4g PiANCj4gPiB2Njogbm8gY2hhbmdlcw0KPiA+IA0KPiA+IHY1OiBubyBjaGFuZ2VzDQo+ID4gDQo+ ID4gdjQ6IG5vIGNoYW5nZXMNCj4gPiANCj4gPiB2MzogY2hhbmdlIGNvbXBhdGlibGUgdG8gU29D IHNwZWNpZmljIGNvbXBhdGlibGUNCj4gPiANCj4gPiB2Mjogbm8gY2hhbmdlcw0KPiA+IC0tLQ0K PiA+ICAuLi4vYmluZGluZ3MvbWVkaWEvbWVkaWF0ZWstanBlZy1lbmNvZGVyLnR4dCAgfCAzNyAr KysrKysrKysrKysrKysrKysrDQo+ID4gIDEgZmlsZSBjaGFuZ2VkLCAzNyBpbnNlcnRpb25zKCsp DQo+ID4gIGNyZWF0ZSBtb2RlIDEwMDY0NCBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGlu Z3MvbWVkaWEvbWVkaWF0ZWstanBlZy1lbmNvZGVyLnR4dA0KPiA+IA0KPiANCj4gVGhhbmsgeW91 IGZvciB0aGUgcGF0Y2guIFBsZWFzZSBzZWUgbXkgY29tbWVudHMgaW5saW5lLg0KRGVhciBUb21h c3osDQoNClNvcnJ5IGZvciBtaXNzaW5nIHRoaXMgbWVzc2FnZS4gUmVwbGllZCBiZWxvdy4NCj4g DQo+ID4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZWRp YS9tZWRpYXRlay1qcGVnLWVuY29kZXIudHh0IGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2Jp bmRpbmdzL21lZGlhL21lZGlhdGVrLWpwZWctZW5jb2Rlci50eHQNCj4gPiBuZXcgZmlsZSBtb2Rl IDEwMDY0NA0KPiA+IGluZGV4IDAwMDAwMDAwMDAwMC4uZmE4ZGE2OTk0OTNiDQo+ID4gLS0tIC9k ZXYvbnVsbA0KPiA+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZWRp YS9tZWRpYXRlay1qcGVnLWVuY29kZXIudHh0DQo+ID4gQEAgLTAsMCArMSwzNyBAQA0KPiA+ICsq IE1lZGlhVGVrIEpQRUcgRW5jb2Rlcg0KPiA+ICsNCj4gPiArTWVkaWFUZWsgSlBFRyBFbmNvZGVy IGlzIHRoZSBKUEVHIGVuY29kZSBoYXJkd2FyZSBwcmVzZW50IGluIE1lZGlhVGVrIFNvQ3MNCj4g PiArDQo+ID4gK1JlcXVpcmVkIHByb3BlcnRpZXM6DQo+ID4gKy0gY29tcGF0aWJsZSA6IHNob3Vs ZCBiZSBvbmUgb2Y6DQo+ID4gKyAgICAgICAgICAgICAgICJtZWRpYXRlayxtdDI3MDEtanBnZW5j Ig0KPiA+ICsgICAgICAgICAgICAgICAuLi4NCj4gDQo+IFdoYXQgZG9lcyB0aGlzICIuLi4iIG1l YW4/DQoiLi4uIiBtZWFucyB0aGF0IGNvbXBhdGlibGUgbmFtZSBpcyBub3QganVzdCAibWVkaWF0 ZWssbXQyNzAxLWpwZ2VuYyIsDQpkaWZmZXJlbnQgcHJvamVjdCBoYXMgZGlmZmVyZW50IGNvbXBh dGlibGUgbmFtZShmb3IgZXhhbXBsZSB0aGUgTVQ4MTczJ3MNCmNvbXBhdGlibGUgbmFtZSBtYXkg YmUgIm1lZGlhdGVrLG10ODE3My1qcGdlbmMiKS4NCj4gDQo+ID4gKyAgICAgICAgICAgICAgIGZv bGxvd2VkIGJ5ICJtZWRpYXRlayxtdGstanBnZW5jIg0KPiA+ICstIHJlZyA6IHBoeXNpY2FsIGJh c2UgYWRkcmVzcyBvZiB0aGUgSlBFRyBlbmNvZGVyIHJlZ2lzdGVycyBhbmQgbGVuZ3RoIG9mDQo+ ID4gKyAgbWVtb3J5IG1hcHBlZCByZWdpb24uDQo+ID4gKy0gaW50ZXJydXB0cyA6IGludGVycnVw dCBudW1iZXIgdG8gdGhlIGludGVycnVwdCBjb250cm9sbGVyLg0KPiA+ICstIGNsb2NrczogZGV2 aWNlIGNsb2Nrcywgc2VlDQo+ID4gKyAgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdz L2Nsb2NrL2Nsb2NrLWJpbmRpbmdzLnR4dCBmb3IgZGV0YWlscy4NCj4gPiArLSBjbG9jay1uYW1l czogbXVzdCBjb250YWluICJqcGdlbmMiLiBJdCBpcyB0aGUgY2xvY2sgb2YgSlBFRyBlbmNvZGVy Lg0KPiANCj4gbml0OiBJbiBwcmluY2lwbGUgdGhlIGNsb2NrcyBzaG91bGQgYmUgbmFtZWQgYWZ0 ZXIgdGhlIGZ1bmN0aW9uIHRoZSBjbG9jaw0KPiBwZXJmb3JtcyBvbiB0aGUgY29uc3VtZXIgc2lk ZSwgaS5lLiB0aGUgSlBFRyBibG9jayBpbiB0aGlzIGNhc2UsIEkgZ3Vlc3MNCj4gaGVyZSBpdCdz IGp1c3QgYSBnZW5lcmljIGNsb2NrIHRoYXQgZG9lcyBldmVyeXRoaW5nLCBidXQgSSBndWVzcyBp dCBjb21lcw0KPiBmcm9tIHNvbWV3aGVyZS4gSXMgaXQgdGhlIEFIQiBjbG9jayBvciBzb21ldGhp bmc/IEluIHRoYXQgY2FzZSBpdCB3b3VsZA0KPiBub3JtYWxseSBiZSBjYWxsZWQgImFoYiIuDQpJ IGhhdmUgY29uZmlybWVkIHdpdGggaGFyZHdhcmUgZGVzaWduZXIgdGhhdCB0aGUganBlZyBjbG9j ayBpcyBub3QgQUhCDQpjbG9jayxpdCBmb2xsb3dzIHN1YnN5cyBjbG9jayhiZWNhdXNlIDI3MDEg aXMgdGhlIG9sZCBJQyxJIGRpZG4ndCBnZXQNCnRoZSBzb3VyY2UgbmFtZSkuSXQgaGFzIHRoZSBz YW1lIHNvdXJjZSB3aXRoIHZlbmMgY2xvY2suV2UgY2FuIHNlZSB0aGF0DQp0aGUgY2xvY2tzID0g PGltZ3N5cyBDTEtfSU1HX1ZFTkM+LCBTaG91bGQgSSBuYW1lIGl0ICJ2ZW5jIiBvciB0aGUNCm9y aWduYWwgImpwZ2VuYyI/DQo+IA0KPiA+ICstIHBvd2VyLWRvbWFpbnM6IGEgcGhhbmRsZSB0byB0 aGUgcG93ZXIgZG9tYWluLCBzZWUNCj4gPiArICBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmlu ZGluZ3MvcG93ZXIvcG93ZXJfZG9tYWluLnR4dCBmb3IgZGV0YWlscy4NCj4gPiArLSBtZWRpYXRl ayxsYXJiOiBtdXN0IGNvbnRhaW4gdGhlIGxvY2FsIGFyYml0ZXJzIGluIHRoZSBjdXJyZW50IFNv Q3MsIHNlZQ0KPiA+ICsgIERvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZW1vcnkt Y29udHJvbGxlcnMvbWVkaWF0ZWssc21pLWxhcmIudHh0DQo+ID4gKyAgZm9yIGRldGFpbHMuDQo+ IA0KPiBJIGJlbGlldmUgdGhpcyBpc24ndCBuZWNlc3NhcnkgYW55bW9yZSwgYmVjYXVzZSBsYXJi cyBhcmUgYWRkZWQNCj4gYXV0b21hdGljYWxseSBieSB0aGUgTVRLIElPTU1VIGRyaXZlciB1c2lu ZyBkZXZpY2UgbGlua3MuICtDQyBZb25nIHdobw0KPiB3b3JrZWQgb24gdGhhdC4NClllcyxJIGhh dmUgY29uZmlybWVkIHdpdGggWW9uZyB0aGF0IGhlIHdpbGwgaGVscCBtZSB0byBtb2RpZnkgdGhp cy5JcyBpdA0Kb2sgdGhhdCBJIGtlZXAgdGhlIG9yaWduYWwgbGFyYiBjb2RlPw0KDQpCZXN0IFJl Z2FyZHMsDQpYaWEgSmlhbmcNCj4gDQo+ID4gKy0gaW9tbXVzOiBzaG91bGQgcG9pbnQgdG8gdGhl IHJlc3BlY3RpdmUgSU9NTVUgYmxvY2sgd2l0aCBtYXN0ZXIgcG9ydCBhcw0KPiA+ICsgIGFyZ3Vt ZW50LCBzZWUgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2lvbW11L21lZGlhdGVr LGlvbW11LnR4dA0KPiA+ICsgIGZvciBkZXRhaWxzLg0KPiA+ICsNCj4gPiArRXhhbXBsZToNCj4g PiArCWpwZWdlbmM6IGpwZWdlbmNAMTUwMGEwMDAgew0KPiA+ICsJCWNvbXBhdGlibGUgPSAibWVk aWF0ZWssbXQyNzAxLWpwZ2VuYyIsDQo+ID4gKwkJCSAgICAgIm1lZGlhdGVrLG10ay1qcGdlbmMi Ow0KPiA+ICsJCXJlZyA9IDwwIDB4MTUwMGEwMDAgMCAweDEwMDA+Ow0KPiA+ICsJCWludGVycnVw dHMgPSA8R0lDX1NQSSAxNDEgSVJRX1RZUEVfTEVWRUxfTE9XPjsNCj4gPiArCQljbG9ja3MgPSAg PCZpbWdzeXMgQ0xLX0lNR19WRU5DPjsNCj4gPiArCQljbG9jay1uYW1lcyA9ICJqcGdlbmMiOw0K PiA+ICsJCXBvd2VyLWRvbWFpbnMgPSA8JnNjcHN5cyBNVDI3MDFfUE9XRVJfRE9NQUlOX0lTUD47 DQo+ID4gKwkJbWVkaWF0ZWssbGFyYiA9IDwmbGFyYjI+Ow0KPiANCj4gRGl0dG8uDQo+IA0KPiBC ZXN0IHJlZ2FyZHMsDQo+IFRvbWFzeg0KDQo=