| Message ID | 20190423104505.38778-4-Jerry-Ch.chen@mediatek.com (mailing list archive) |
|---|---|
| State | RFC |
| Headers | show |
| Series | media: platform: Add support for Face Detection (FD) on mt8183 SoC | expand |
On Tue, Apr 23, 2019 at 06:45:02PM +0800, Jerry-ch Chen wrote: > From: Jerry-ch Chen <jerry-ch.chen@mediatek.com> > > This patch adds DT binding documentation for the Face Detection (FD) > unit of the camera system on Mediatek's SoCs. > > Signed-off-by: Jerry-ch Chen <jerry-ch.chen@mediatek.com> > --- > .../bindings/media/mediatek,mt8183-fd.txt | 34 +++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > > diff --git a/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > new file mode 100644 > index 000000000000..97c12fd93e7e > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > @@ -0,0 +1,34 @@ > +* Mediatek Face Detection Unit (FD) > + > +Face Detection (FD) unit is a typical memory-to-memory HW device. > +It provides hardware accelerated face detection function, and it > +is able to detect different poses of faces. FD will writre result > +of detected face into memory as output. > + > +Required properties: > +- compatible: "mediatek,mt8183-fd" > +- reg: Physical base address and length of the function block register space > +- interrupts: interrupt number to the cpu. > +- iommus: should point to the respective IOMMU block with master port as > + argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt > + for details. How many entries? Looks like 3 from the example. > +- mediatek,larb: must contain the local arbiters in the current Socs, see > + Documentation/devicetree/bindings/memory-controllers/mediatek,smi-larb.txt > + for details. > +- clocks : must contain the FDVT clock > +- clock-names: must contain FD_CLK_IMG_FD > + > +Example: > + fd:fd@1502b000 { ^ space needed > + compatible = "mediatek,mt8183-fd"; > + mediatek,larb = <&larb5>; > + mediatek,scp = <&scp>; > + iommus = <&iommu M4U_PORT_CAM_FDVT_RP>, > + <&iommu M4U_PORT_CAM_FDVT_WR>, > + <&iommu M4U_PORT_CAM_FDVT_RB>; > + reg = <0 0x1502b000 0 0x1000>; > + interrupts = <GIC_SPI 269 IRQ_TYPE_LEVEL_LOW>; > + clocks = <&imgsys CLK_IMG_FDVT>; > + clock-names = "FD_CLK_IMG_FD"; > + }; > + > -- > 2.18.0 >
Hi Rob, Thank you for your comments. On Wed, 2019-05-01 at 17:46 -0500, Rob Herring wrote: > On Tue, Apr 23, 2019 at 06:45:02PM +0800, Jerry-ch Chen wrote: > > From: Jerry-ch Chen <jerry-ch.chen@mediatek.com> > > > > This patch adds DT binding documentation for the Face Detection (FD) > > unit of the camera system on Mediatek's SoCs. > > > > Signed-off-by: Jerry-ch Chen <jerry-ch.chen@mediatek.com> > > --- > > .../bindings/media/mediatek,mt8183-fd.txt | 34 +++++++++++++++++++ > > 1 file changed, 34 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > > > > diff --git a/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > > new file mode 100644 > > index 000000000000..97c12fd93e7e > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt > > @@ -0,0 +1,34 @@ > > +* Mediatek Face Detection Unit (FD) > > + > > +Face Detection (FD) unit is a typical memory-to-memory HW device. > > +It provides hardware accelerated face detection function, and it > > +is able to detect different poses of faces. FD will writre result > > +of detected face into memory as output. > > + > > +Required properties: > > +- compatible: "mediatek,mt8183-fd" > > +- reg: Physical base address and length of the function block register space > > +- interrupts: interrupt number to the cpu. > > +- iommus: should point to the respective IOMMU block with master port as > > + argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.txt > > + for details. > > How many entries? Looks like 3 from the example. > Yes, we have 3 entries. I would like to update this part as following: iommus: should point to the 3 entries: M4U_PORT_CAM_FDVT_RP, M4U_PORT_CAM_FDVT_WR and M4U_PORT_CAM_FDVT_RB. (Please see Documentation/devicetree/bindings/iommu/mediatek,iommu.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. > > +- clocks : must contain the FDVT clock > > +- clock-names: must contain FD_CLK_IMG_FD > > + > > +Example: > > + fd:fd@1502b000 { > ^ space needed > We will fix it in next version of the patch. > > + compatible = "mediatek,mt8183-fd"; > > + mediatek,larb = <&larb5>; > > + mediatek,scp = <&scp>; > > + iommus = <&iommu M4U_PORT_CAM_FDVT_RP>, > > + <&iommu M4U_PORT_CAM_FDVT_WR>, > > + <&iommu M4U_PORT_CAM_FDVT_RB>; > > + reg = <0 0x1502b000 0 0x1000>; > > + interrupts = <GIC_SPI 269 IRQ_TYPE_LEVEL_LOW>; > > + clocks = <&imgsys CLK_IMG_FDVT>; > > + clock-names = "FD_CLK_IMG_FD"; > > + }; > > + > > -- > > 2.18.0 > > Sincerely, Jerry
diff --git a/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt new file mode 100644 index 000000000000..97c12fd93e7e --- /dev/null +++ b/Documentation/devicetree/bindings/media/mediatek,mt8183-fd.txt @@ -0,0 +1,34 @@ +* Mediatek Face Detection Unit (FD) + +Face Detection (FD) unit is a typical memory-to-memory HW device. +It provides hardware accelerated face detection function, and it +is able to detect different poses of faces. FD will writre result +of detected face into memory as output. + +Required properties: +- compatible: "mediatek,mt8183-fd" +- reg: Physical base address and length of the function block register space +- interrupts: interrupt number to the cpu. +- iommus: should point to the respective IOMMU block with master port as + argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.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. +- clocks : must contain the FDVT clock +- clock-names: must contain FD_CLK_IMG_FD + +Example: + fd:fd@1502b000 { + compatible = "mediatek,mt8183-fd"; + mediatek,larb = <&larb5>; + mediatek,scp = <&scp>; + iommus = <&iommu M4U_PORT_CAM_FDVT_RP>, + <&iommu M4U_PORT_CAM_FDVT_WR>, + <&iommu M4U_PORT_CAM_FDVT_RB>; + reg = <0 0x1502b000 0 0x1000>; + interrupts = <GIC_SPI 269 IRQ_TYPE_LEVEL_LOW>; + clocks = <&imgsys CLK_IMG_FDVT>; + clock-names = "FD_CLK_IMG_FD"; + }; +