Message ID | 20240805164342.211857-9-biju.das.jz@bp.renesas.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | Add support for RZ/{G2L,G2LC,G2UL,V2L} Camera Data Receiving Unit | expand |
Hi! > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > commit a18004173a087573628ddc9910f0665a3829f48f upstream. > > Document the CRU block found on Renesas RZ/G2L (and alike) SoCs. > create mode 100644 Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > > diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > new file mode 100644 > index 000000000000..7dde7967c886 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml ... > +description: > + The CRU image processing module is a data conversion module equipped with pixel > + color space conversion, LUT, pixel format conversion, etc. An MIPI CSI-2 input and > + parallel (including ITU-R BT.656) input are provided as the image sensor interface. input -> inputs? > + clocks: > + items: > + - description: CRU Main clock Main -> main? > + - description: CRU Register access clock Register -> register? Best regards, Pavel
Hi Pavel, Thanks for the feedback. > -----Original Message----- > From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On Behalf Of Pavel Machek via > lists.cip-project.org > Subject: Re: [cip-dev] [PATCH 6.1.y-cip 08/30] media: dt-bindings: Document Renesas RZ/G2L CRU block > > Hi! > > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > commit a18004173a087573628ddc9910f0665a3829f48f upstream. > > > > Document the CRU block found on Renesas RZ/G2L (and alike) SoCs. > > > create mode 100644 Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > > > > diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > > new file mode 100644 > > index 000000000000..7dde7967c886 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml > ... > > +description: > > + The CRU image processing module is a data conversion module equipped with pixel > > + color space conversion, LUT, pixel format conversion, etc. An MIPI CSI-2 input and > > + parallel (including ITU-R BT.656) input are provided as the image sensor interface. > > input -> inputs? OK. > > > + clocks: > > + items: > > + - description: CRU Main clock > > Main -> main? OK. > > > + - description: CRU Register access clock > > Register -> register? Ok. Will fix this in mainline. Cheers, Biju
Hi! > > > +description: > > > + The CRU image processing module is a data conversion module equipped with pixel > > > + color space conversion, LUT, pixel format conversion, etc. An MIPI CSI-2 input and > > > + parallel (including ITU-R BT.656) input are provided as the image sensor interface. > > > > input -> inputs? > > OK. > > > > > > + clocks: > > > + items: > > > + - description: CRU Main clock > > > > Main -> main? > > OK. > > > > > + - description: CRU Register access clock > > > > Register -> register? > > Ok. > > Will fix this in mainline. Thank you. These are just nitpicks I noticed during review, and then had them in "I should post this". Best regards, Pavel
Hi Pavel and All, RZ/G2UL display support patches are now in linux-rc1 [1]. Previously as CIP policy, we are not allowed to backport anything from linux next, but ok to backport from rc series. Is any CIP policy is changed for back porting recently? If yes, for new feature/device support patches like [1], without any fixes tag or dependencies, how many rc cycles we need to wait? [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.12-rc1&id=cc49fcd0bc2db23489a87f6fa17119a76b70ec6b Cheers, Biju
Hi! > RZ/G2UL display support patches are now in linux-rc1 [1]. > > Previously as CIP policy, we are not allowed to backport anything from linux next, but > ok to backport from rc series. > > Is any CIP policy is changed for back porting recently? If yes, for new feature/device support > patches like [1], without any fixes tag or dependencies, how many rc cycles we need to wait? > I don't believe we have strict/firm policies here. It really depends on many factors. If you have 3-line patch that enables driver on newer hardware, we can likely take it directly. If you have 3000-line beast that includes changes to mm/slab.c, well, that's different :-). If the patches are very simple, very safe and well tested, feel free to send them directly. Best regards, Pavel
Hi Pavel, > -----Original Message----- > From: Pavel Machek <pavel@denx.de> > Sent: Tuesday, October 1, 2024 3:38 PM > Subject: Re: [cip-dev] Backporting query > > Hi! > > > RZ/G2UL display support patches are now in linux-rc1 [1]. > > > > Previously as CIP policy, we are not allowed to backport anything from > > linux next, but ok to backport from rc series. > > > > Is any CIP policy is changed for back porting recently? If yes, for > > new feature/device support patches like [1], without any fixes tag or dependencies, how many rc > cycles we need to wait? > > > > I don't believe we have strict/firm policies here. It really depends on many factors. If you have 3- > line patch that enables driver on newer hardware, we can likely take it directly. If you have 3000- > line beast that includes changes to mm/slab.c, well, that's different :-). That make sense. > > If the patches are very simple, very safe and well tested, feel free to send them directly. OK. Thanks for clarification. Cheers, Biju
diff --git a/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml new file mode 100644 index 000000000000..7dde7967c886 --- /dev/null +++ b/Documentation/devicetree/bindings/media/renesas,rzg2l-cru.yaml @@ -0,0 +1,157 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +# Copyright (C) 2022 Renesas Electronics Corp. +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/media/renesas,rzg2l-cru.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Renesas RZ/G2L (and alike SoC's) Camera Data Receiving Unit (CRU) Image processing + +maintainers: + - Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> + +description: + The CRU image processing module is a data conversion module equipped with pixel + color space conversion, LUT, pixel format conversion, etc. An MIPI CSI-2 input and + parallel (including ITU-R BT.656) input are provided as the image sensor interface. + +properties: + compatible: + items: + - enum: + - renesas,r9a07g044-cru # RZ/G2{L,LC} + - renesas,r9a07g054-cru # RZ/V2L + - const: renesas,rzg2l-cru + + reg: + maxItems: 1 + + interrupts: + maxItems: 3 + + interrupt-names: + items: + - const: image_conv + - const: image_conv_err + - const: axi_mst_err + + clocks: + items: + - description: CRU Main clock + - description: CRU Register access clock + - description: CRU image transfer clock + + clock-names: + items: + - const: video + - const: apb + - const: axi + + power-domains: + maxItems: 1 + + resets: + items: + - description: CRU_PRESETN reset terminal + - description: CRU_ARESETN reset terminal + + reset-names: + items: + - const: presetn + - const: aresetn + + ports: + $ref: /schemas/graph.yaml#/properties/ports + + properties: + port@0: + $ref: /schemas/graph.yaml#/$defs/port-base + unevaluatedProperties: false + description: + Input port node, single endpoint describing a parallel input source. + + properties: + endpoint: + $ref: video-interfaces.yaml# + unevaluatedProperties: false + + properties: + hsync-active: true + vsync-active: true + bus-width: true + data-shift: true + + port@1: + $ref: /schemas/graph.yaml#/properties/port + description: + Input port node, describing the Image Processing module connected to the + CSI-2 receiver. + + required: + - port@0 + - port@1 + +required: + - compatible + - reg + - interrupts + - interrupt-names + - clocks + - clock-names + - resets + - reset-names + - power-domains + +additionalProperties: false + +examples: + # Device node example with CSI-2 + - | + #include <dt-bindings/clock/r9a07g044-cpg.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + + cru: video@10830000 { + compatible = "renesas,r9a07g044-cru", "renesas,rzg2l-cru"; + reg = <0x10830000 0x400>; + interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>; + interrupt-names = "image_conv", "image_conv_err", "axi_mst_err"; + clocks = <&cpg CPG_MOD R9A07G044_CRU_VCLK>, + <&cpg CPG_MOD R9A07G044_CRU_PCLK>, + <&cpg CPG_MOD R9A07G044_CRU_ACLK>; + clock-names = "video", "apb", "axi"; + power-domains = <&cpg>; + resets = <&cpg R9A07G044_CRU_PRESETN>, + <&cpg R9A07G044_CRU_ARESETN>; + reset-names = "presetn", "aresetn"; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + #address-cells = <1>; + #size-cells = <0>; + reg = <0>; + + cru_parallel_in: endpoint@0 { + reg = <0>; + remote-endpoint= <&ov5642>; + hsync-active = <1>; + vsync-active = <1>; + }; + }; + + port@1 { + #address-cells = <1>; + #size-cells = <0>; + reg = <1>; + + cru_csi_in: endpoint@0 { + reg = <0>; + remote-endpoint= <&csi_cru_in>; + }; + }; + }; + };