Message ID | 20221011122516.32135-2-aakarsh.jain@samsung.com (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Hans Verkuil |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1oiSie-006vAO-5s; Wed, 12 Oct 2022 03:47:45 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229561AbiJLDrg (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 11 Oct 2022 23:47:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46356 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229527AbiJLDrd (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 11 Oct 2022 23:47:33 -0400 Received: from mailout1.samsung.com (mailout1.samsung.com [203.254.224.24]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F0CB2796BA for <linux-media@vger.kernel.org>; Tue, 11 Oct 2022 20:47:31 -0700 (PDT) Received: from epcas5p3.samsung.com (unknown [182.195.41.41]) by mailout1.samsung.com (KnoxPortal) with ESMTP id 20221012034730epoutp011c80f4929cf08e17001c156274926504~dNawcF3yM2161421614epoutp01w for <linux-media@vger.kernel.org>; Wed, 12 Oct 2022 03:47:30 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.samsung.com 20221012034730epoutp011c80f4929cf08e17001c156274926504~dNawcF3yM2161421614epoutp01w DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1665546450; bh=KQA2k/ooP0X6HqqDcCu+dMgG5d+FuA71B4Z/y2XwyDY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=BHIrkPI64do7jHzAZ8zImRabISNbgrWQlJnusYdxOG6yh4RMGU2a65Ruq647XmODc PFsXdDX15SqG0DoTQ+hoEK7u6/3zdWSNZoNw0+hhdbgrUH8mBW83njBf/7PnePccWa gJRcIFoPTAQWAOZkIjG7OV8mBiU0zhqEyeY/teFM= Received: from epsnrtp1.localdomain (unknown [182.195.42.162]) by epcas5p2.samsung.com (KnoxPortal) with ESMTP id 20221012034729epcas5p207f37b5c9bd8158c601dbe57033f8976~dNavmEVGk1526715267epcas5p2B; Wed, 12 Oct 2022 03:47:29 +0000 (GMT) Received: from epsmges5p1new.samsung.com (unknown [182.195.38.178]) by epsnrtp1.localdomain (Postfix) with ESMTP id 4MnJWM13lbz4x9Q0; Wed, 12 Oct 2022 03:47:27 +0000 (GMT) Received: from epcas5p2.samsung.com ( [182.195.41.40]) by epsmges5p1new.samsung.com (Symantec Messaging Gateway) with SMTP id 05.1E.26992.EC836436; Wed, 12 Oct 2022 12:47:26 +0900 (KST) Received: from epsmtrp1.samsung.com (unknown [182.195.40.13]) by epcas5p1.samsung.com (KnoxPortal) with ESMTPA id 20221011125142epcas5p13c858a5f27830fb1de50fa51e9730eca~dBMnQmnWt3163231632epcas5p1X; Tue, 11 Oct 2022 12:51:42 +0000 (GMT) Received: from epsmgms1p1new.samsung.com (unknown [182.195.42.41]) by epsmtrp1.samsung.com (KnoxPortal) with ESMTP id 20221011125142epsmtrp1d972403c617995345b6bc46e2b9bca85~dBMnPheY32654526545epsmtrp1D; Tue, 11 Oct 2022 12:51:42 +0000 (GMT) X-AuditID: b6c32a49-0c7ff70000016970-12-634638ce625f Received: from epsmtip1.samsung.com ( [182.195.34.30]) by epsmgms1p1new.samsung.com (Symantec Messaging Gateway) with SMTP id 58.0D.14392.DD665436; Tue, 11 Oct 2022 21:51:41 +0900 (KST) Received: from cheetah.sa.corp.samsungelectronics.net (unknown [107.109.115.53]) by epsmtip1.samsung.com (KnoxPortal) with ESMTPA id 20221011125138epsmtip1e7c289d1b3c2c222ad1ce0cdc6f672f0~dBMkIdScf2177321773epsmtip11; Tue, 11 Oct 2022 12:51:38 +0000 (GMT) From: aakarsh jain <aakarsh.jain@samsung.com> To: linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Cc: m.szyprowski@samsung.com, andrzej.hajda@intel.com, mchehab@kernel.org, hverkuil-cisco@xs4all.nl, ezequiel@vanguardiasur.com.ar, jernej.skrabec@gmail.com, benjamin.gaignard@collabora.com, stanimir.varbanov@linaro.org, dillon.minfei@gmail.com, david.plowman@raspberrypi.com, mark.rutland@arm.com, robh+dt@kernel.org, krzk+dt@kernel.org, andi@etezian.org, alim.akhtar@samsung.com, aswani.reddy@samsung.com, pankaj.dubey@samsung.com, linux-fsd@tesla.com, smitha.t@samsung.com, aakarsh.jain@samsung.com Subject: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT schema for MFC Date: Tue, 11 Oct 2022 17:55:02 +0530 Message-Id: <20221011122516.32135-2-aakarsh.jain@samsung.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20221011122516.32135-1-aakarsh.jain@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA0WTe0zTVxTHd39tfy0a9Ac4vUNAUsMEDNDO0l1YwQ2Z+y1uSyfZXMQIpfxs GaXt2iLMZJuCgKIIuC1CB5RRGAHksdIyYDB5l6lgyJxNJuVVFh5bmLwcDqNrKWz/fe73fM89 5557L4vmPoF7spLkGkolF8nY+DZ6S2+Af9AwOirmFN0ToN9bixlooqwFR/q1GQyN65fpqKfZ xETG+9/QULW5i4F0fcMM9EP3FB19P2uPjhRb6WhG1wDQnHYMRwbbAwaanD+BfmkvwdHVJhMD 1fdZmajKMoKh7wxPMVRhWmGirM4+JrJ2tAB0MbsPex2SN8tuArLVWglIS+USjWzTWpmkvmMO Iw21l3Fy9EEHTjZXfkFm9f9DJ68ZawH5LKOUSeb2WXBy2eBD3l5dZgp3nEwWSClRIqXypeRi RWKSXBLBPhYTdyQulM/hBnHD0KtsX7kohYpgR78jDDqaJLOPge17ViRLtUtCkVrNDokUqBSp GspXqlBrItiUMlGm5CmD1aIUdapcEiynNOFcDueVULsxPll6rXsBKE1h6eX3apnnQUZALnBh QYIH+y/dALlgG8ud+BFAQ2MO07lYAnDIXI85XO7EYwBvLQm2MoZ1DZjT1AngotaGO01ZGCxY 3eNgnAiCt6q6NvRdxAUAbZc0jgQakUmH1rvrTEfAgzgOK0Yu0BxMJ/zgU9so3cGuRARczc+j O6vtg3VNXRseFyISLlYN4I6NIJHjAgdnf6M5TdFQn5mHO9kDzpuNTCd7wrn87E0WQ1vF3KZf Bhs7vtoscBh23S+xM8veXQBsbA9xyt7w69sNG6enETtg3vo05tRdYWvZFr8MS0bXGE72gr11 VcDJJLySP09zTqgQwD/L6+gFwEf7f4lyAGrBS5RSnSKh1KFKrpxK++/WxIoUA9h46oFvtwLr xKPgHoCxQA+ALBp7lyvQRYndXRNFn56jVIo4VaqMUveAUPsAC2meL4oV9r8i18RxeWEcHp/P 54Ud4nPZe1z1RYFid0Ii0lDJFKWkVFt5GMvF8zxWOjvgzzozPe42Iy2zdxl58tTe8Z7a1nTO w7feeLjgfcTvfV1LS97BDyzPx2pEU+9+pFvZzgILjJlOiVeRdqLgRMHO08nZ1UZO5rmI4qZT MXMvXN2POofe27/yc4NftKBx7Vv/u+Gmj5t3B/Iah3wSCn8Svjb5yLf0oOzKHAx6fGZo+zN9 PYH9Ybw8nSCMPxBvWx+ejJWaDz15U+vmzTO63TGVDfZWZ+yeOmA+ljHe5jF90dS703L2r3Wv 9sjR+vG9nKiE6/7PeWncsTrBr8LZYLcPeZbu0zeaP8MXY/XeaUWebbzrkk/6I/fFPok5bv4y /HB7VkhN+kDA34TLYE0UP+fO52y6WiriBtJUatG/DxerbnMEAAA= X-Brightmail-Tracker: H4sIAAAAAAAAAzWRfUyMcQDH+z333PM8nZVnl9avEnbDlIkm+pk6r62HMZm3eVm51XNXc9fl uZJaXnOsQ6xlqah0xRw1ruuFXlznIkYydJa7lCJFxVWyauFq/vtsn32/3z++FE+YjXtRsXEJ LBcnkYsIAV75WDRniU0aGrUsuykQfa7O4aOO/EoCaX/3YOijdghHpvIKEhne5vHQradGPiow N/NRVcMnHN3/+s++zrHhqKegDKDe3HYC6bta+aizbw968/AagS7cq+CjUrONRCWW1xi6qZ/A UFHFMInUdWYS2WorATpz1oythczd/LuAqbYVA8ZSbOcxD3JtJKOt7cUYvS6dYKyttQRTXnyC UTeO4UyGQQeYydPXSUZjthDMkH4O83xkiAx33ScIjmblsUdYbqn4oCAmo2EAxFesOlr4Skee BKd9NcCZgnQgbC4owzRAQAnpGgDbhsaIaTEb/jn7hJxmN3h7smeKhXQaBtVvQxxM0EvgoxIj 4QjPotUAWm6nTzXx6Dwc/s408jWAotzocFje5+YI4PQCONFlxR3sQofAkUsX8emBufDOPSPP wc60GP4seUJMj4XAU4NXsMvAtRA46YAnG69SyBSqgPiAODbJXyVRqBLjZP5RSoUeTH3n51sN qnQ//E0Ao4AJQIonmuUCCtZHCV2iJckpLKeM5BLlrMoEvClc5OHSonkWKaRlkgT2EMvGs9x/ i1HOXicxo/i9qW6RQhsaqVwM2vrrJ9Qr8sQan63nYGORcvMW1bVBY5jo0v4P0WPdwx6lYYFc Zu671EHteFDj47zjW86sWuvUtiH71pX2lvym0f7opHE23XvHnazelCO7L1+t8SqVWKuz/FZe d/+SuMs0kLbY7u8TUWY+WjI8LBBrug0D9U7bQqg9ReGu9esiTKKgjp0GakRtt6ctl0rHM3V7 rd6BGaNhskl3/rrtycHwpdsbbyn9nfv6q3Rivnymcl5nqNSz8oDFnn1e841jV1yYIW1LsrKu CxuOpaSeWxPKdRnUnEfhi4aNq9Xt3TmSbfmbzst+7VSkmW90Bfs2bz7MJstFuCpGEuDH41SS v4BPiMEqAwAA X-CMS-MailID: 20221011125142epcas5p13c858a5f27830fb1de50fa51e9730eca X-Msg-Generator: CA Content-Type: text/plain; charset="utf-8" X-Sendblock-Type: REQ_APPROVE CMS-TYPE: 105P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20221011125142epcas5p13c858a5f27830fb1de50fa51e9730eca References: <20221011122516.32135-1-aakarsh.jain@samsung.com> <CGME20221011125142epcas5p13c858a5f27830fb1de50fa51e9730eca@epcas5p1.samsung.com> X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.5 (--) X-LSpam-Report: No, score=-2.5 required=5.0 tests=BAYES_00=-1.9,DKIMWL_WL_HIGH=0.001,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
Add MFC v12 support.
|
|
Commit Message
Aakarsh Jain
Oct. 11, 2022, 12:25 p.m. UTC
From: Smitha T Murthy <smitha.t@samsung.com> Convert DT schema for s5p-mfc in yaml format Cc: linux-fsd@tesla.com Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> --- .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ 2 files changed, 163 insertions(+), 75 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml
Comments
On 11/10/2022 08:25, aakarsh jain wrote: > From: Smitha T Murthy <smitha.t@samsung.com> > > Convert DT schema for s5p-mfc in yaml format Convert S5P-MFC bindings to DT schema. You also missed full-stop. My second comment was ignored here. > > Cc: linux-fsd@tesla.com > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > --- > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ > 2 files changed, 163 insertions(+), 75 deletions(-) > create mode 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > > diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt b/Documentation/devicetree/bindings/media/s5p-mfc.txt > index aa54c8159d9f..8b137891791f 100644 > --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt > +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt This is a friendly reminder during the review process. It seems my previous comments were not fully addressed. Maybe my feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. Best regards, Krzysztof
On Tue, 11 Oct 2022 17:55:02 +0530, aakarsh jain wrote: > From: Smitha T Murthy <smitha.t@samsung.com> > > Convert DT schema for s5p-mfc in yaml format > > Cc: linux-fsd@tesla.com > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > --- > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ > 2 files changed, 163 insertions(+), 75 deletions(-) > create mode 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings. Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future. Full log is available here: https://patchwork.ozlabs.org/patch/ codec@13400000: clock-names: ['mfc', 'sclk_mfc'] is too long arch/arm/boot/dts/exynos3250-artik5-eval.dtb arch/arm/boot/dts/exynos3250-monk.dtb arch/arm/boot/dts/exynos3250-rinato.dtb codec@13400000: clocks: [[7, 178], [7, 228]] is too long arch/arm/boot/dts/exynos3250-artik5-eval.dtb arch/arm/boot/dts/exynos3250-monk.dtb arch/arm/boot/dts/exynos3250-rinato.dtb codec@13400000: iommus: [[36]] is too short arch/arm/boot/dts/exynos3250-monk.dtb arch/arm/boot/dts/exynos3250-monk.dtb codec@13400000: iommus: [[40]] is too short arch/arm/boot/dts/exynos3250-artik5-eval.dtb arch/arm/boot/dts/exynos3250-artik5-eval.dtb codec@13400000: iommus: [[47]] is too short arch/arm/boot/dts/exynos3250-rinato.dtb arch/arm/boot/dts/exynos3250-rinato.dtb codec@13400000: memory-region: [[31], [32]] is too long arch/arm/boot/dts/exynos4210-smdkv310.dtb codec@13400000: memory-region: [[37], [38]] is too long arch/arm/boot/dts/exynos4210-origen.dtb codec@13400000: memory-region: [[41], [42]] is too long arch/arm/boot/dts/exynos4412-smdk4412.dtb codec@13400000: memory-region: [[43], [44]] is too long arch/arm/boot/dts/exynos4412-origen.dtb codec@13400000: memory-region: [[47], [48]] is too long arch/arm/boot/dts/exynos4412-itop-elite.dtb arch/arm/boot/dts/exynos4412-odroidx2.dtb arch/arm/boot/dts/exynos4412-odroidx.dtb codec@13400000: memory-region: [[48], [49]] is too long arch/arm/boot/dts/exynos4412-odroidu3.dtb codec@f1700000: clock-names:0: 'mfc' was expected arch/arm/boot/dts/s5pv210-aquila.dtb arch/arm/boot/dts/s5pv210-fascinate4g.dtb arch/arm/boot/dts/s5pv210-galaxys.dtb arch/arm/boot/dts/s5pv210-goni.dtb arch/arm/boot/dts/s5pv210-smdkc110.dtb arch/arm/boot/dts/s5pv210-smdkv210.dtb arch/arm/boot/dts/s5pv210-torbreck.dtb codec@f1700000: clock-names:1: 'sclk_mfc' was expected arch/arm/boot/dts/s5pv210-aquila.dtb arch/arm/boot/dts/s5pv210-fascinate4g.dtb arch/arm/boot/dts/s5pv210-galaxys.dtb arch/arm/boot/dts/s5pv210-goni.dtb arch/arm/boot/dts/s5pv210-smdkc110.dtb arch/arm/boot/dts/s5pv210-smdkv210.dtb arch/arm/boot/dts/s5pv210-torbreck.dtb codec@f1700000: memory-region: [[51], [52]] is too long arch/arm/boot/dts/s5pv210-fascinate4g.dtb codec@f1700000: memory-region: [[55], [56]] is too long arch/arm/boot/dts/s5pv210-galaxys.dtb
On 12/10/2022 09:19, Rob Herring wrote: > On Tue, 11 Oct 2022 17:55:02 +0530, aakarsh jain wrote: >> From: Smitha T Murthy <smitha.t@samsung.com> >> >> Convert DT schema for s5p-mfc in yaml format >> >> Cc: linux-fsd@tesla.com >> Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> >> Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> >> --- >> .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- >> .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ >> 2 files changed, 163 insertions(+), 75 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml >> > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for dtbs_check. > This will change in the future. > > Full log is available here: https://patchwork.ozlabs.org/patch/ > All these must be fixed before schema can be applied. Best regards, Krzysztof
> -----Original Message----- > From: Krzysztof Kozlowski [mailto:krzk@kernel.org] > Sent: 12 October 2022 18:27 > To: aakarsh jain <aakarsh.jain@samsung.com>; linux-arm- > kernel@lists.infradead.org; linux-media@vger.kernel.org; linux- > kernel@vger.kernel.org; devicetree@vger.kernel.org > Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; > mchehab@kernel.org; hverkuil-cisco@xs4all.nl; > ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; > benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; > dillon.minfei@gmail.com; david.plowman@raspberrypi.com; > mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; > andi@etezian.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; > pankaj.dubey@samsung.com; linux-fsd@tesla.com; smitha.t@samsung.com > Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > schema for MFC > > On 11/10/2022 08:25, aakarsh jain wrote: > > From: Smitha T Murthy <smitha.t@samsung.com> > > > > Convert DT schema for s5p-mfc in yaml format > > Convert S5P-MFC bindings to DT schema. > > You also missed full-stop. > Ok will address it in next series. > My second comment was ignored here. > I missed one review comment which was related to change in commit message. Will address it in next series. > > > > Cc: linux-fsd@tesla.com > > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > > --- > > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > > .../bindings/media/samsung,s5p-mfc.yaml | 163 > ++++++++++++++++++ > > 2 files changed, 163 insertions(+), 75 deletions(-) create mode > > 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > > > > diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt > > b/Documentation/devicetree/bindings/media/s5p-mfc.txt > > index aa54c8159d9f..8b137891791f 100644 > > --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt > > +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt > > This is a friendly reminder during the review process. > > It seems my previous comments were not fully addressed. Maybe my > feedback got lost between the quotes, maybe you just forgot to apply it. > Please go back to the previous discussion and either implement all requested > changes or keep discussing them. > > Thank you. > Apart from your second review comment I guess we have addressed all your review comments in the current patch. Please review other changes as well and let us know if any changes required. Thanks for the review. > Best regards, > Krzysztof
> -----Original Message----- > From: Rob Herring [mailto:robh@kernel.org] > Sent: 12 October 2022 18:50 > To: aakarsh jain <aakarsh.jain@samsung.com> > Cc: linux-fsd@tesla.com; linux-media@vger.kernel.org; > pankaj.dubey@samsung.com; linux-arm-kernel@lists.infradead.org; > dillon.minfei@gmail.com; devicetree@vger.kernel.org; krzk+dt@kernel.org; > smitha.t@samsung.com; benjamin.gaignard@collabora.com; > stanimir.varbanov@linaro.org; jernej.skrabec@gmail.com; > robh+dt@kernel.org; aswani.reddy@samsung.com; mchehab@kernel.org; > hverkuil-cisco@xs4all.nl; mark.rutland@arm.com; > m.szyprowski@samsung.com; linux-kernel@vger.kernel.org; > alim.akhtar@samsung.com; andi@etezian.org; andrzej.hajda@intel.com; > ezequiel@vanguardiasur.com.ar; david.plowman@raspberrypi.com > Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > schema for MFC > > On Tue, 11 Oct 2022 17:55:02 +0530, aakarsh jain wrote: > > From: Smitha T Murthy <smitha.t@samsung.com> > > > > Convert DT schema for s5p-mfc in yaml format > > > > Cc: linux-fsd@tesla.com > > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > > --- > > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > > .../bindings/media/samsung,s5p-mfc.yaml | 163 > ++++++++++++++++++ > > 2 files changed, 163 insertions(+), 75 deletions(-) create mode > > 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > > > > Running 'make dtbs_check' with the schema in this patch gives the following > warnings. Consider if they are expected or the schema is incorrect. These > may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for dtbs_check. > This will change in the future. > > Full log is available here: https://protect2.fireeye.com/v1/url?k=b651482e- > e9ca7131-b650c361-000babdfecba-bb28230f82534754&q=1&e=25a7ee78- > 697f-4371-880e- > dcf4e3072c99&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fpatch%2F > > > codec@13400000: clock-names: ['mfc', 'sclk_mfc'] is too long > arch/arm/boot/dts/exynos3250-artik5-eval.dtb > arch/arm/boot/dts/exynos3250-monk.dtb > arch/arm/boot/dts/exynos3250-rinato.dtb > > codec@13400000: clocks: [[7, 178], [7, 228]] is too long > arch/arm/boot/dts/exynos3250-artik5-eval.dtb > arch/arm/boot/dts/exynos3250-monk.dtb > arch/arm/boot/dts/exynos3250-rinato.dtb > > codec@13400000: iommus: [[36]] is too short > arch/arm/boot/dts/exynos3250-monk.dtb > arch/arm/boot/dts/exynos3250-monk.dtb > > codec@13400000: iommus: [[40]] is too short > arch/arm/boot/dts/exynos3250-artik5-eval.dtb > arch/arm/boot/dts/exynos3250-artik5-eval.dtb > > codec@13400000: iommus: [[47]] is too short > arch/arm/boot/dts/exynos3250-rinato.dtb > arch/arm/boot/dts/exynos3250-rinato.dtb > > codec@13400000: memory-region: [[31], [32]] is too long > arch/arm/boot/dts/exynos4210-smdkv310.dtb > > codec@13400000: memory-region: [[37], [38]] is too long > arch/arm/boot/dts/exynos4210-origen.dtb > > codec@13400000: memory-region: [[41], [42]] is too long > arch/arm/boot/dts/exynos4412-smdk4412.dtb > > codec@13400000: memory-region: [[43], [44]] is too long > arch/arm/boot/dts/exynos4412-origen.dtb > > codec@13400000: memory-region: [[47], [48]] is too long > arch/arm/boot/dts/exynos4412-itop-elite.dtb > arch/arm/boot/dts/exynos4412-odroidx2.dtb > arch/arm/boot/dts/exynos4412-odroidx.dtb > > codec@13400000: memory-region: [[48], [49]] is too long > arch/arm/boot/dts/exynos4412-odroidu3.dtb > > codec@f1700000: clock-names:0: 'mfc' was expected > arch/arm/boot/dts/s5pv210-aquila.dtb > arch/arm/boot/dts/s5pv210-fascinate4g.dtb > arch/arm/boot/dts/s5pv210-galaxys.dtb > arch/arm/boot/dts/s5pv210-goni.dtb > arch/arm/boot/dts/s5pv210-smdkc110.dtb > arch/arm/boot/dts/s5pv210-smdkv210.dtb > arch/arm/boot/dts/s5pv210-torbreck.dtb > > codec@f1700000: clock-names:1: 'sclk_mfc' was expected > arch/arm/boot/dts/s5pv210-aquila.dtb > arch/arm/boot/dts/s5pv210-fascinate4g.dtb > arch/arm/boot/dts/s5pv210-galaxys.dtb > arch/arm/boot/dts/s5pv210-goni.dtb > arch/arm/boot/dts/s5pv210-smdkc110.dtb > arch/arm/boot/dts/s5pv210-smdkv210.dtb > arch/arm/boot/dts/s5pv210-torbreck.dtb > > codec@f1700000: memory-region: [[51], [52]] is too long > arch/arm/boot/dts/s5pv210-fascinate4g.dtb > > codec@f1700000: memory-region: [[55], [56]] is too long > arch/arm/boot/dts/s5pv210-galaxys.dtb Hi Rob, We tried reproducing warnings as reported above, but I am not able to see these warnings after running make dtbs_check & make DT_CHECKER_FLAGS=-m dt_binding_check. Packages used- yamllint 1.10.0 $ dt-mk-schema --version 2022.9 Am I missing anything? Do let me know. Thanks, Aakarsh
On 17/10/2022 10:16, Aakarsh Jain wrote: > > >> -----Original Message----- >> From: Krzysztof Kozlowski [mailto:krzk@kernel.org] >> Sent: 12 October 2022 18:27 >> To: aakarsh jain <aakarsh.jain@samsung.com>; linux-arm- >> kernel@lists.infradead.org; linux-media@vger.kernel.org; linux- >> kernel@vger.kernel.org; devicetree@vger.kernel.org >> Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; >> mchehab@kernel.org; hverkuil-cisco@xs4all.nl; >> ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; >> benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; >> dillon.minfei@gmail.com; david.plowman@raspberrypi.com; >> mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; >> andi@etezian.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; >> pankaj.dubey@samsung.com; linux-fsd@tesla.com; smitha.t@samsung.com >> Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT >> schema for MFC >> >> On 11/10/2022 08:25, aakarsh jain wrote: >>> From: Smitha T Murthy <smitha.t@samsung.com> >>> >>> Convert DT schema for s5p-mfc in yaml format >> >> Convert S5P-MFC bindings to DT schema. >> >> You also missed full-stop. >> > Ok will address it in next series. > >> My second comment was ignored here. >> > I missed one review comment which was related to change in commit message. Will address it in next series. >>> >>> Cc: linux-fsd@tesla.com >>> Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> >>> Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> >>> --- >>> .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- >>> .../bindings/media/samsung,s5p-mfc.yaml | 163 >> ++++++++++++++++++ >>> 2 files changed, 163 insertions(+), 75 deletions(-) create mode >>> 100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt >>> b/Documentation/devicetree/bindings/media/s5p-mfc.txt >>> index aa54c8159d9f..8b137891791f 100644 >>> --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt >>> +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt >> >> This is a friendly reminder during the review process. >> >> It seems my previous comments were not fully addressed. Maybe my >> feedback got lost between the quotes, maybe you just forgot to apply it. >> Please go back to the previous discussion and either implement all requested >> changes or keep discussing them. >> >> Thank you. >> > Apart from your second review comment I guess we have addressed all your review comments in the current patch. > Please review other changes as well and let us know if any changes required. No, as I said you did not. Instead of bouncing back immediately, go through previous emails and through reviews. I already reviewed this patch and it did not apply all my comments. Asking me to review second time is not the solution here. The solution here is for you to go through the comments and implement them, or answer and keep discussion going. Best regards, Krzysztof
On 18/10/2022 06:48, Aakarsh Jain wrote: >> >> codec@f1700000: clock-names:1: 'sclk_mfc' was expected >> arch/arm/boot/dts/s5pv210-aquila.dtb >> arch/arm/boot/dts/s5pv210-fascinate4g.dtb >> arch/arm/boot/dts/s5pv210-galaxys.dtb >> arch/arm/boot/dts/s5pv210-goni.dtb >> arch/arm/boot/dts/s5pv210-smdkc110.dtb >> arch/arm/boot/dts/s5pv210-smdkv210.dtb >> arch/arm/boot/dts/s5pv210-torbreck.dtb >> >> codec@f1700000: memory-region: [[51], [52]] is too long >> arch/arm/boot/dts/s5pv210-fascinate4g.dtb >> >> codec@f1700000: memory-region: [[55], [56]] is too long >> arch/arm/boot/dts/s5pv210-galaxys.dtb > > > > Hi Rob, > > We tried reproducing warnings as reported above, but I am not able to see > these warnings after running make dtbs_check & make DT_CHECKER_FLAGS=-m > dt_binding_check. > Packages used- > yamllint 1.10.0 > $ dt-mk-schema --version > 2022.9 Are you sure you are running these commands on proper config and arch? Best regards, Krzysztof
> -----Original Message----- > From: Krzysztof Kozlowski [mailto:krzk@kernel.org] > Sent: 18 October 2022 18:40 > To: Aakarsh Jain <aakarsh.jain@samsung.com>; 'Rob Herring' > <robh@kernel.org> > Cc: linux-fsd@tesla.com; linux-media@vger.kernel.org; > pankaj.dubey@samsung.com; linux-arm-kernel@lists.infradead.org; > dillon.minfei@gmail.com; devicetree@vger.kernel.org; krzk+dt@kernel.org; > smitha.t@samsung.com; benjamin.gaignard@collabora.com; > stanimir.varbanov@linaro.org; jernej.skrabec@gmail.com; > robh+dt@kernel.org; aswani.reddy@samsung.com; mchehab@kernel.org; > hverkuil-cisco@xs4all.nl; mark.rutland@arm.com; > m.szyprowski@samsung.com; linux-kernel@vger.kernel.org; > alim.akhtar@samsung.com; andi@etezian.org; andrzej.hajda@intel.com; > ezequiel@vanguardiasur.com.ar; david.plowman@raspberrypi.com > Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > schema for MFC > > On 18/10/2022 06:48, Aakarsh Jain wrote: > >> > >> codec@f1700000: clock-names:1: 'sclk_mfc' was expected > >> arch/arm/boot/dts/s5pv210-aquila.dtb > >> arch/arm/boot/dts/s5pv210-fascinate4g.dtb > >> arch/arm/boot/dts/s5pv210-galaxys.dtb > >> arch/arm/boot/dts/s5pv210-goni.dtb > >> arch/arm/boot/dts/s5pv210-smdkc110.dtb > >> arch/arm/boot/dts/s5pv210-smdkv210.dtb > >> arch/arm/boot/dts/s5pv210-torbreck.dtb > >> > >> codec@f1700000: memory-region: [[51], [52]] is too long > >> arch/arm/boot/dts/s5pv210-fascinate4g.dtb > >> > >> codec@f1700000: memory-region: [[55], [56]] is too long > >> arch/arm/boot/dts/s5pv210-galaxys.dtb > > > > > > > > Hi Rob, > > > > We tried reproducing warnings as reported above, but I am not able to > > see these warnings after running make dtbs_check & make > > DT_CHECKER_FLAGS=-m dt_binding_check. > > Packages used- > > yamllint 1.10.0 > > $ dt-mk-schema --version > > 2022.9 > > > Are you sure you are running these commands on proper config and arch? > yes. arm64/defconfig . > Best regards, > Krzysztof
On 18/10/2022 09:21, Aakarsh Jain wrote: > > >> -----Original Message----- >> From: Krzysztof Kozlowski [mailto:krzk@kernel.org] >> Sent: 18 October 2022 18:40 >> To: Aakarsh Jain <aakarsh.jain@samsung.com>; 'Rob Herring' >> <robh@kernel.org> >> Cc: linux-fsd@tesla.com; linux-media@vger.kernel.org; >> pankaj.dubey@samsung.com; linux-arm-kernel@lists.infradead.org; >> dillon.minfei@gmail.com; devicetree@vger.kernel.org; krzk+dt@kernel.org; >> smitha.t@samsung.com; benjamin.gaignard@collabora.com; >> stanimir.varbanov@linaro.org; jernej.skrabec@gmail.com; >> robh+dt@kernel.org; aswani.reddy@samsung.com; mchehab@kernel.org; >> hverkuil-cisco@xs4all.nl; mark.rutland@arm.com; >> m.szyprowski@samsung.com; linux-kernel@vger.kernel.org; >> alim.akhtar@samsung.com; andi@etezian.org; andrzej.hajda@intel.com; >> ezequiel@vanguardiasur.com.ar; david.plowman@raspberrypi.com >> Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT >> schema for MFC >> >> On 18/10/2022 06:48, Aakarsh Jain wrote: >>>> >>>> codec@f1700000: clock-names:1: 'sclk_mfc' was expected >>>> arch/arm/boot/dts/s5pv210-aquila.dtb >>>> arch/arm/boot/dts/s5pv210-fascinate4g.dtb >>>> arch/arm/boot/dts/s5pv210-galaxys.dtb >>>> arch/arm/boot/dts/s5pv210-goni.dtb >>>> arch/arm/boot/dts/s5pv210-smdkc110.dtb >>>> arch/arm/boot/dts/s5pv210-smdkv210.dtb >>>> arch/arm/boot/dts/s5pv210-torbreck.dtb >>>> >>>> codec@f1700000: memory-region: [[51], [52]] is too long >>>> arch/arm/boot/dts/s5pv210-fascinate4g.dtb >>>> >>>> codec@f1700000: memory-region: [[55], [56]] is too long >>>> arch/arm/boot/dts/s5pv210-galaxys.dtb >>> >>> >>> >>> Hi Rob, >>> >>> We tried reproducing warnings as reported above, but I am not able to >>> see these warnings after running make dtbs_check & make >>> DT_CHECKER_FLAGS=-m dt_binding_check. >>> Packages used- >>> yamllint 1.10.0 >>> $ dt-mk-schema --version >>> 2022.9 >> >> >> Are you sure you are running these commands on proper config and arch? >> > yes. arm64/defconfig . Which does not look like proper arch. Look at the warnings Rob robot provided. Best regards, Krzysztof
Hi Aakarsh Thanks for the patch. >-----Original Message----- >From: aakarsh jain [mailto:aakarsh.jain@samsung.com] >Sent: Tuesday, October 11, 2022 5:55 PM >To: linux-arm-kernel@lists.infradead.org; linux-media@vger.kernel.org; linux- >kernel@vger.kernel.org; devicetree@vger.kernel.org >Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; >mchehab@kernel.org; hverkuil-cisco@xs4all.nl; >ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; >benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; >dillon.minfei@gmail.com; david.plowman@raspberrypi.com; >mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; >andi@etezian.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; >pankaj.dubey@samsung.com; linux-fsd@tesla.com; smitha.t@samsung.com; >aakarsh.jain@samsung.com >Subject: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT schema for >MFC > >From: Smitha T Murthy <smitha.t@samsung.com> > >Convert DT schema for s5p-mfc in yaml format > >Cc: linux-fsd@tesla.com >Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> >Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> >--- > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ > 2 files changed, 163 insertions(+), 75 deletions(-) create mode 100644 >Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > >diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt >b/Documentation/devicetree/bindings/media/s5p-mfc.txt >index aa54c8159d9f..8b137891791f 100644 >--- a/Documentation/devicetree/bindings/media/s5p-mfc.txt >+++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt >@@ -1,76 +1 @@ >-* Samsung Multi Format Codec (MFC) > >-Multi Format Codec (MFC) is the IP present in Samsung SoCs which -supports >high resolution decoding and encoding functionalities. >-The MFC device driver is a v4l2 driver which can encode/decode -video >raw/elementary streams and has support for all popular -video codecs. >- >-Required properties: >- - compatible : value should be either one among the following >- (a) "samsung,mfc-v5" for MFC v5 present in Exynos4 SoCs >- (b) "samsung,mfc-v6" for MFC v6 present in Exynos5 SoCs >- (c) "samsung,mfc-v7" for MFC v7 present in Exynos5420 SoC >- (d) "samsung,mfc-v8" for MFC v8 present in Exynos5800 SoC >- (e) "samsung,exynos5433-mfc" for MFC v8 present in Exynos5433 SoC >- (f) "samsung,mfc-v10" for MFC v10 present in Exynos7880 SoC >- >- - reg : Physical base address of the IP registers and length of memory >- mapped region. >- >- - interrupts : MFC interrupt number to the CPU. >- - clocks : from common clock binding: handle to mfc clock. >- - clock-names : from common clock binding: must contain "mfc", >- corresponding to entry in the clocks property. >- >-Optional properties: >- - power-domains : power-domain property defined with a phandle >- to respective power domain. >- - memory-region : from reserved memory binding: phandles to two >reserved >- memory regions, first is for "left" mfc memory bus interfaces, >- second if for the "right" mfc memory bus, used when no SYSMMU >- support is available; used only by MFC v5 present in Exynos4 SoCs >- >-Obsolete properties: >- - samsung,mfc-r, samsung,mfc-l : support removed, please use memory- >region >- property instead >- >- >-Example: >-SoC specific DT entry: >- >-mfc: codec@13400000 { >- compatible = "samsung,mfc-v5"; >- reg = <0x13400000 0x10000>; >- interrupts = <0 94 0>; >- power-domains = <&pd_mfc>; >- clocks = <&clock 273>; >- clock-names = "mfc"; >-}; >- >-Reserved memory specific DT entry for given board (see reserved memory >binding -for more information): >- >-reserved-memory { >- #address-cells = <1>; >- #size-cells = <1>; >- ranges; >- >- mfc_left: region@51000000 { >- compatible = "shared-dma-pool"; >- no-map; >- reg = <0x51000000 0x800000>; >- }; >- >- mfc_right: region@43000000 { >- compatible = "shared-dma-pool"; >- no-map; >- reg = <0x43000000 0x800000>; >- }; >-}; >- >-Board specific DT entry: >- >-codec@13400000 { >- memory-region = <&mfc_left>, <&mfc_right>; >-}; >diff --git a/Documentation/devicetree/bindings/media/samsung,s5p- >mfc.yaml b/Documentation/devicetree/bindings/media/samsung,s5p- >mfc.yaml >new file mode 100644 >index 000000000000..ad61b509846f >--- /dev/null >+++ b/Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml >@@ -0,0 +1,163 @@ >+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause %YAML 1.2 >+--- >+$id: http://devicetree.org/schemas/media/samsung,s5p-mfc.yaml# >+$schema: http://devicetree.org/meta-schemas/core.yaml# >+ >+title: Samsung Exynos Multi Format Codec (MFC) >+ >+maintainers: >+ - Marek Szyprowski <m.szyprowski@samsung.com> >+ - Smitha T Murthy <smitha.t@samsung.com> >+ - Aakarsh Jain <aakarsh.jain@samsung.com> >+ >+description: >+ Multi Format Codec (MFC) is the IP present in Samsung SoCs which >+ supports high resolution decoding and encoding functionalities. >+ >+properties: >+ compatible: >+ enum: >+ - samsung,mfc-v5 # Exynos4 >+ - samsung,mfc-v6 # Exynos5 >+ - samsung,mfc-v7 # Exynos5420 >+ - samsung,mfc-v8 # Exynos5800 >+ - samsung,exynos5433-mfc # Exynos5433 >+ - samsung,mfc-v10 # Exynos7880 >+ >+ reg: >+ maxItems: 1 >+ >+ clocks: >+ minItems: 1 >+ maxItems: 3 >+ >+ clock-names: >+ minItems: 1 >+ maxItems: 3 >+ >+ interrupts: >+ maxItems: 1 >+ >+ iommus: >+ maxItems: 2 >+ >+ iommu-names: >+ maxItems: 2 >+ >+ power-domains: >+ maxItems: 1 >+ >+ memory-region: >+ maxItems: 1 >+ >+required: >+ - compatible >+ - reg >+ - clocks >+ - clock-names >+ - interrupts >+ >+allOf: >+ - if: >+ properties: >+ compatible: >+ contains: >+ enum: >+ - samsung,mfc-v6 >+ - samsung,mfc-v7 >+ - samsung,mfc-v8 >+ - tesla,fsd-mfc You have not introduce "tesla,fsd-mfc" compatible yet, so this should be part of the patch which adds fsd-mfc compatible >+ then: >+ properties: >+ clocks: >+ maxItems: 1 >+ clock-names: >+ items: >+ - const: mfc >+ >+ - if: >+ properties: >+ compatible: >+ contains: >+ enum: >+ - samsung,mfc-v5 >+ then: >+ properties: >+ clocks: >+ minItems: 2 >+ maxItems: 2 >+ clock-names: >+ items: >+ - const: mfc >+ - const: sclk_mfc >+ >+ - if: >+ properties: >+ compatible: >+ contains: >+ enum: >+ - samsung,exynos5433-mfc >+ then: >+ properties: >+ clocks: >+ minItems: 3 >+ maxItems: 3 >+ clock-names: >+ items: >+ - const: pclk >+ - const: aclk >+ - const: aclk_xiu >+ >+ - if: >+ properties: >+ compatible: >+ contains: >+ enum: >+ - samsung,mfc-v5 >+ - samsung,mfc-v6 >+ - samsung,mfc-v7 >+ - samsung,mfc-v8 >+ - samsung,exynos5433-mfc >+ >+ then: >+ properties: >+ iommus: >+ minItems: 2 >+ maxItems: 2 >+ iommu-names: >+ items: >+ - const: left >+ - const: right >+ >+ - if: >+ properties: >+ compatible: >+ contains: >+ enum: >+ - tesla,fsd-mfc Same comments as above. My suggestion, first just convert the current s5p-mfc.txt to yaml format and then add fsd-mfc support. That way it will be easy for reviewer to understand the changes. While reviewing and code walk through, I also noticed that exynos3250.dtsi and exynos5420.dtsi are using same compatible as "samsung,mfc-v7" but there "clocks" property is different. That doesn't look correct. This will cause issues when you convert s5p-mfc.txt to schema format. I don’t know why that was done that way, but surely this need to be _fixed_ first as h/w itself are different. So my suggestion is to fix exynos3250.dtsi and exynos5420.dtsi compatible and mfc driver for it first (as a separate patch series). You can use the same approach what was done for mfc-v8 and mfc-v8-5433. >+ then: >+ properties: >+ memory-region: >+ maxItems: 1 >+ >+additionalProperties: false >+ >+examples: >+ - | >+ #include <dt-bindings/clock/exynos4.h> >+ #include <dt-bindings/clock/exynos-audss-clk.h> >+ #include <dt-bindings/interrupt-controller/arm-gic.h> >+ #include <dt-bindings/interrupt-controller/irq.h> >+ >+ codec@13400000 { >+ compatible = "samsung,mfc-v5"; >+ reg = <0x13400000 0x10000>; >+ interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>; >+ power-domains = <&pd_mfc>; >+ clocks = <&clock CLK_MFC>, <&clock CLK_SCLK_MFC>; >+ clock-names = "mfc", "sclk_mfc"; >+ iommus = <&sysmmu_mfc_l>, <&sysmmu_mfc_r>; >+ iommu-names = "left", "right"; >+ >+ }; >-- >2.17.1
> -----Original Message----- > From: Alim Akhtar [mailto:alim.akhtar@samsung.com] > Sent: 27 October 2022 16:10 > To: 'aakarsh jain' <aakarsh.jain@samsung.com>; linux-arm- > kernel@lists.infradead.org; linux-media@vger.kernel.org; linux- > kernel@vger.kernel.org; devicetree@vger.kernel.org > Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; > mchehab@kernel.org; hverkuil-cisco@xs4all.nl; > ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; > benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; > dillon.minfei@gmail.com; david.plowman@raspberrypi.com; > mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; > andi@etezian.org; aswani.reddy@samsung.com; > pankaj.dubey@samsung.com; linux-fsd@tesla.com; smitha.t@samsung.com > Subject: RE: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > schema for MFC > > Hi Aakarsh > Thanks for the patch. > > >-----Original Message----- > >From: aakarsh jain [mailto:aakarsh.jain@samsung.com] > >Sent: Tuesday, October 11, 2022 5:55 PM > >To: linux-arm-kernel@lists.infradead.org; linux-media@vger.kernel.org; > >linux- kernel@vger.kernel.org; devicetree@vger.kernel.org > >Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; > >mchehab@kernel.org; hverkuil-cisco@xs4all.nl; > >ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; > >benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; > >dillon.minfei@gmail.com; david.plowman@raspberrypi.com; > >mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; > >andi@etezian.org; alim.akhtar@samsung.com; > aswani.reddy@samsung.com; > >pankaj.dubey@samsung.com; linux-fsd@tesla.com; > smitha.t@samsung.com; > >aakarsh.jain@samsung.com > >Subject: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > >schema for MFC > > > >From: Smitha T Murthy <smitha.t@samsung.com> > > > >Convert DT schema for s5p-mfc in yaml format > > > >Cc: linux-fsd@tesla.com > >Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > >Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > >--- > > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > > .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ > > 2 files changed, 163 insertions(+), 75 deletions(-) create mode > >100644 Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > > > >diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt > >b/Documentation/devicetree/bindings/media/s5p-mfc.txt > >index aa54c8159d9f..8b137891791f 100644 > >--- a/Documentation/devicetree/bindings/media/s5p-mfc.txt > >+++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt > >@@ -1,76 +1 @@ > >-* Samsung Multi Format Codec (MFC) > > > >-Multi Format Codec (MFC) is the IP present in Samsung SoCs which > >-supports high resolution decoding and encoding functionalities. > >-The MFC device driver is a v4l2 driver which can encode/decode -video > >raw/elementary streams and has support for all popular -video codecs. > >- > >-Required properties: > >- - compatible : value should be either one among the following > >- (a) "samsung,mfc-v5" for MFC v5 present in Exynos4 SoCs > >- (b) "samsung,mfc-v6" for MFC v6 present in Exynos5 SoCs > >- (c) "samsung,mfc-v7" for MFC v7 present in Exynos5420 SoC > >- (d) "samsung,mfc-v8" for MFC v8 present in Exynos5800 SoC > >- (e) "samsung,exynos5433-mfc" for MFC v8 present in Exynos5433 > SoC > >- (f) "samsung,mfc-v10" for MFC v10 present in Exynos7880 SoC > >- > >- - reg : Physical base address of the IP registers and length of memory > >- mapped region. > >- > >- - interrupts : MFC interrupt number to the CPU. > >- - clocks : from common clock binding: handle to mfc clock. > >- - clock-names : from common clock binding: must contain "mfc", > >- corresponding to entry in the clocks property. > >- > >-Optional properties: > >- - power-domains : power-domain property defined with a phandle > >- to respective power domain. > >- - memory-region : from reserved memory binding: phandles to two > >reserved > >- memory regions, first is for "left" mfc memory bus interfaces, > >- second if for the "right" mfc memory bus, used when no SYSMMU > >- support is available; used only by MFC v5 present in Exynos4 SoCs > >- > >-Obsolete properties: > >- - samsung,mfc-r, samsung,mfc-l : support removed, please use memory- > >region > >- property instead > >- > >- > >-Example: > >-SoC specific DT entry: > >- > >-mfc: codec@13400000 { > >- compatible = "samsung,mfc-v5"; > >- reg = <0x13400000 0x10000>; > >- interrupts = <0 94 0>; > >- power-domains = <&pd_mfc>; > >- clocks = <&clock 273>; > >- clock-names = "mfc"; > >-}; > >- > >-Reserved memory specific DT entry for given board (see reserved memory > >binding -for more information): > >- > >-reserved-memory { > >- #address-cells = <1>; > >- #size-cells = <1>; > >- ranges; > >- > >- mfc_left: region@51000000 { > >- compatible = "shared-dma-pool"; > >- no-map; > >- reg = <0x51000000 0x800000>; > >- }; > >- > >- mfc_right: region@43000000 { > >- compatible = "shared-dma-pool"; > >- no-map; > >- reg = <0x43000000 0x800000>; > >- }; > >-}; > >- > >-Board specific DT entry: > >- > >-codec@13400000 { > >- memory-region = <&mfc_left>, <&mfc_right>; > >-}; > >diff --git a/Documentation/devicetree/bindings/media/samsung,s5p- > >mfc.yaml b/Documentation/devicetree/bindings/media/samsung,s5p- > >mfc.yaml > >new file mode 100644 > >index 000000000000..ad61b509846f > >--- /dev/null > >+++ b/Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml > >@@ -0,0 +1,163 @@ > >+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause %YAML 1.2 > >+--- > >+$id: http://devicetree.org/schemas/media/samsung,s5p-mfc.yaml# > >+$schema: http://devicetree.org/meta-schemas/core.yaml# > >+ > >+title: Samsung Exynos Multi Format Codec (MFC) > >+ > >+maintainers: > >+ - Marek Szyprowski <m.szyprowski@samsung.com> > >+ - Smitha T Murthy <smitha.t@samsung.com> > >+ - Aakarsh Jain <aakarsh.jain@samsung.com> > >+ > >+description: > >+ Multi Format Codec (MFC) is the IP present in Samsung SoCs which > >+ supports high resolution decoding and encoding functionalities. > >+ > >+properties: > >+ compatible: > >+ enum: > >+ - samsung,mfc-v5 # Exynos4 > >+ - samsung,mfc-v6 # Exynos5 > >+ - samsung,mfc-v7 # Exynos5420 > >+ - samsung,mfc-v8 # Exynos5800 > >+ - samsung,exynos5433-mfc # Exynos5433 > >+ - samsung,mfc-v10 # Exynos7880 > >+ > >+ reg: > >+ maxItems: 1 > >+ > >+ clocks: > >+ minItems: 1 > >+ maxItems: 3 > >+ > >+ clock-names: > >+ minItems: 1 > >+ maxItems: 3 > >+ > >+ interrupts: > >+ maxItems: 1 > >+ > >+ iommus: > >+ maxItems: 2 > >+ > >+ iommu-names: > >+ maxItems: 2 > >+ > >+ power-domains: > >+ maxItems: 1 > >+ > >+ memory-region: > >+ maxItems: 1 > >+ > >+required: > >+ - compatible > >+ - reg > >+ - clocks > >+ - clock-names > >+ - interrupts > >+ > >+allOf: > >+ - if: > >+ properties: > >+ compatible: > >+ contains: > >+ enum: > >+ - samsung,mfc-v6 > >+ - samsung,mfc-v7 > >+ - samsung,mfc-v8 > >+ - tesla,fsd-mfc > You have not introduce "tesla,fsd-mfc" compatible yet, so this should be part > of the patch which adds fsd-mfc compatible > ok. > >+ then: > >+ properties: > >+ clocks: > >+ maxItems: 1 > >+ clock-names: > >+ items: > >+ - const: mfc > >+ > >+ - if: > >+ properties: > >+ compatible: > >+ contains: > >+ enum: > >+ - samsung,mfc-v5 > >+ then: > >+ properties: > >+ clocks: > >+ minItems: 2 > >+ maxItems: 2 > >+ clock-names: > >+ items: > >+ - const: mfc > >+ - const: sclk_mfc > >+ > >+ - if: > >+ properties: > >+ compatible: > >+ contains: > >+ enum: > >+ - samsung,exynos5433-mfc > >+ then: > >+ properties: > >+ clocks: > >+ minItems: 3 > >+ maxItems: 3 > >+ clock-names: > >+ items: > >+ - const: pclk > >+ - const: aclk > >+ - const: aclk_xiu > >+ > >+ - if: > >+ properties: > >+ compatible: > >+ contains: > >+ enum: > >+ - samsung,mfc-v5 > >+ - samsung,mfc-v6 > >+ - samsung,mfc-v7 > >+ - samsung,mfc-v8 > >+ - samsung,exynos5433-mfc > >+ > >+ then: > >+ properties: > >+ iommus: > >+ minItems: 2 > >+ maxItems: 2 > >+ iommu-names: > >+ items: > >+ - const: left > >+ - const: right > >+ > >+ - if: > >+ properties: > >+ compatible: > >+ contains: > >+ enum: > >+ - tesla,fsd-mfc > > Same comments as above. > > My suggestion, first just convert the current s5p-mfc.txt to yaml format and > then add fsd-mfc support. > That way it will be easy for reviewer to understand the changes. > > While reviewing and code walk through, I also noticed that exynos3250.dtsi > and exynos5420.dtsi are using same compatible as "samsung,mfc-v7" but > there "clocks" property is different. That doesn't look correct. > This will cause issues when you convert s5p-mfc.txt to schema format. > I don’t know why that was done that way, but surely this need to be _fixed_ > first as h/w itself are different. > > So my suggestion is to fix exynos3250.dtsi and exynos5420.dtsi compatible > and mfc driver for it first (as a separate patch series). > You can use the same approach what was done for mfc-v8 and mfc-v8-5433. > > ok will fix this issue . > >+ then: > >+ properties: > >+ memory-region: > >+ maxItems: 1 > >+ > >+additionalProperties: false > >+ > >+examples: > >+ - | > >+ #include <dt-bindings/clock/exynos4.h> > >+ #include <dt-bindings/clock/exynos-audss-clk.h> > >+ #include <dt-bindings/interrupt-controller/arm-gic.h> > >+ #include <dt-bindings/interrupt-controller/irq.h> > >+ > >+ codec@13400000 { > >+ compatible = "samsung,mfc-v5"; > >+ reg = <0x13400000 0x10000>; > >+ interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>; > >+ power-domains = <&pd_mfc>; > >+ clocks = <&clock CLK_MFC>, <&clock CLK_SCLK_MFC>; > >+ clock-names = "mfc", "sclk_mfc"; > >+ iommus = <&sysmmu_mfc_l>, <&sysmmu_mfc_r>; > >+ iommu-names = "left", "right"; > >+ > >+ }; > >-- > >2.17.1 > > Thanks for the review.
On 11/10/2022 14:25, aakarsh jain wrote: > From: Smitha T Murthy <smitha.t@samsung.com> > > Convert DT schema for s5p-mfc in yaml format > > Cc: linux-fsd@tesla.com > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > --- > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > .../bindings/media/samsung,s5p-mfc.yaml | 163 ++++++++++++++++++ Hi Aakarsh, Any plans for new version of this DT schema conversion? The Samsung MFC is one of last remaining bigger pieces for conversion. We already have 100% bindings compliance for ARM64 but the MFC is missing for ARMv7: https://pastebin.com/D8jXnjEe (that's on my pending branch with all patches from LKML collected). Best regards, Krzysztof
> -----Original Message----- > From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org] > Sent: 21 February 2023 21:56 > To: aakarsh jain <aakarsh.jain@samsung.com>; linux-arm- > kernel@lists.infradead.org; linux-media@vger.kernel.org; linux- > kernel@vger.kernel.org; devicetree@vger.kernel.org > Cc: m.szyprowski@samsung.com; andrzej.hajda@intel.com; > mchehab@kernel.org; hverkuil-cisco@xs4all.nl; > ezequiel@vanguardiasur.com.ar; jernej.skrabec@gmail.com; > benjamin.gaignard@collabora.com; stanimir.varbanov@linaro.org; > dillon.minfei@gmail.com; david.plowman@raspberrypi.com; > mark.rutland@arm.com; robh+dt@kernel.org; krzk+dt@kernel.org; > andi@etezian.org; alim.akhtar@samsung.com; aswani.reddy@samsung.com; > pankaj.dubey@samsung.com; linux-fsd@tesla.com; smitha.t@samsung.com > Subject: Re: [Patch v3 01/15] dt-bindings: media: s5p-mfc: Add new DT > schema for MFC > > On 11/10/2022 14:25, aakarsh jain wrote: > > From: Smitha T Murthy <smitha.t@samsung.com> > > > > Convert DT schema for s5p-mfc in yaml format > > > > Cc: linux-fsd@tesla.com > > Signed-off-by: Smitha T Murthy <smitha.t@samsung.com> > > Signed-off-by: Aakarsh Jain <aakarsh.jain@samsung.com> > > --- > > .../devicetree/bindings/media/s5p-mfc.txt | 75 -------- > > .../bindings/media/samsung,s5p-mfc.yaml | 163 > ++++++++++++++++++ > > Hi Aakarsh, > > Any plans for new version of this DT schema conversion? The Samsung MFC > is one of last remaining bigger pieces for conversion. We already have 100% > bindings compliance for ARM64 but the MFC is missing for ARMv7: > > https://protect2.fireeye.com/v1/url?k=9661b377-f7eaa63c-96603838- > 74fe485fb305-bc4c89dbd0204557&q=1&e=1485ea1d-43e8-4541-8078- > 4d13ff6cc42f&u=https%3A%2F%2Fpastebin.com%2FD8jXnjEe > (that's on my pending branch with all patches from LKML collected). > > Best regards, > Krzysztof Hi Krzysztof, I was little occupied with other stuff. I will fix all warnings and errors. Will post the patch soon. Thanks, Aakarsh
diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt b/Documentation/devicetree/bindings/media/s5p-mfc.txt index aa54c8159d9f..8b137891791f 100644 --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt @@ -1,76 +1 @@ -* Samsung Multi Format Codec (MFC) -Multi Format Codec (MFC) is the IP present in Samsung SoCs which -supports high resolution decoding and encoding functionalities. -The MFC device driver is a v4l2 driver which can encode/decode -video raw/elementary streams and has support for all popular -video codecs. - -Required properties: - - compatible : value should be either one among the following - (a) "samsung,mfc-v5" for MFC v5 present in Exynos4 SoCs - (b) "samsung,mfc-v6" for MFC v6 present in Exynos5 SoCs - (c) "samsung,mfc-v7" for MFC v7 present in Exynos5420 SoC - (d) "samsung,mfc-v8" for MFC v8 present in Exynos5800 SoC - (e) "samsung,exynos5433-mfc" for MFC v8 present in Exynos5433 SoC - (f) "samsung,mfc-v10" for MFC v10 present in Exynos7880 SoC - - - reg : Physical base address of the IP registers and length of memory - mapped region. - - - interrupts : MFC interrupt number to the CPU. - - clocks : from common clock binding: handle to mfc clock. - - clock-names : from common clock binding: must contain "mfc", - corresponding to entry in the clocks property. - -Optional properties: - - power-domains : power-domain property defined with a phandle - to respective power domain. - - memory-region : from reserved memory binding: phandles to two reserved - memory regions, first is for "left" mfc memory bus interfaces, - second if for the "right" mfc memory bus, used when no SYSMMU - support is available; used only by MFC v5 present in Exynos4 SoCs - -Obsolete properties: - - samsung,mfc-r, samsung,mfc-l : support removed, please use memory-region - property instead - - -Example: -SoC specific DT entry: - -mfc: codec@13400000 { - compatible = "samsung,mfc-v5"; - reg = <0x13400000 0x10000>; - interrupts = <0 94 0>; - power-domains = <&pd_mfc>; - clocks = <&clock 273>; - clock-names = "mfc"; -}; - -Reserved memory specific DT entry for given board (see reserved memory binding -for more information): - -reserved-memory { - #address-cells = <1>; - #size-cells = <1>; - ranges; - - mfc_left: region@51000000 { - compatible = "shared-dma-pool"; - no-map; - reg = <0x51000000 0x800000>; - }; - - mfc_right: region@43000000 { - compatible = "shared-dma-pool"; - no-map; - reg = <0x43000000 0x800000>; - }; -}; - -Board specific DT entry: - -codec@13400000 { - memory-region = <&mfc_left>, <&mfc_right>; -}; diff --git a/Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml b/Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml new file mode 100644 index 000000000000..ad61b509846f --- /dev/null +++ b/Documentation/devicetree/bindings/media/samsung,s5p-mfc.yaml @@ -0,0 +1,163 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/media/samsung,s5p-mfc.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Samsung Exynos Multi Format Codec (MFC) + +maintainers: + - Marek Szyprowski <m.szyprowski@samsung.com> + - Smitha T Murthy <smitha.t@samsung.com> + - Aakarsh Jain <aakarsh.jain@samsung.com> + +description: + Multi Format Codec (MFC) is the IP present in Samsung SoCs which + supports high resolution decoding and encoding functionalities. + +properties: + compatible: + enum: + - samsung,mfc-v5 # Exynos4 + - samsung,mfc-v6 # Exynos5 + - samsung,mfc-v7 # Exynos5420 + - samsung,mfc-v8 # Exynos5800 + - samsung,exynos5433-mfc # Exynos5433 + - samsung,mfc-v10 # Exynos7880 + + reg: + maxItems: 1 + + clocks: + minItems: 1 + maxItems: 3 + + clock-names: + minItems: 1 + maxItems: 3 + + interrupts: + maxItems: 1 + + iommus: + maxItems: 2 + + iommu-names: + maxItems: 2 + + power-domains: + maxItems: 1 + + memory-region: + maxItems: 1 + +required: + - compatible + - reg + - clocks + - clock-names + - interrupts + +allOf: + - if: + properties: + compatible: + contains: + enum: + - samsung,mfc-v6 + - samsung,mfc-v7 + - samsung,mfc-v8 + - tesla,fsd-mfc + then: + properties: + clocks: + maxItems: 1 + clock-names: + items: + - const: mfc + + - if: + properties: + compatible: + contains: + enum: + - samsung,mfc-v5 + then: + properties: + clocks: + minItems: 2 + maxItems: 2 + clock-names: + items: + - const: mfc + - const: sclk_mfc + + - if: + properties: + compatible: + contains: + enum: + - samsung,exynos5433-mfc + then: + properties: + clocks: + minItems: 3 + maxItems: 3 + clock-names: + items: + - const: pclk + - const: aclk + - const: aclk_xiu + + - if: + properties: + compatible: + contains: + enum: + - samsung,mfc-v5 + - samsung,mfc-v6 + - samsung,mfc-v7 + - samsung,mfc-v8 + - samsung,exynos5433-mfc + + then: + properties: + iommus: + minItems: 2 + maxItems: 2 + iommu-names: + items: + - const: left + - const: right + + - if: + properties: + compatible: + contains: + enum: + - tesla,fsd-mfc + then: + properties: + memory-region: + maxItems: 1 + +additionalProperties: false + +examples: + - | + #include <dt-bindings/clock/exynos4.h> + #include <dt-bindings/clock/exynos-audss-clk.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + #include <dt-bindings/interrupt-controller/irq.h> + + codec@13400000 { + compatible = "samsung,mfc-v5"; + reg = <0x13400000 0x10000>; + interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>; + power-domains = <&pd_mfc>; + clocks = <&clock CLK_MFC>, <&clock CLK_SCLK_MFC>; + clock-names = "mfc", "sclk_mfc"; + iommus = <&sysmmu_mfc_l>, <&sysmmu_mfc_r>; + iommu-names = "left", "right"; + + };