* [PATCH v4 0/3] Add rga to rk322x device tree @ 2020-02-03 22:30 Justin Swartz 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz ` (2 more replies) 0 siblings, 3 replies; 14+ messages in thread From: Justin Swartz @ 2020-02-03 22:30 UTC (permalink / raw) To: Jacob Chen, Ezequiel Garcia, Mauro Carvalho Chehab, Rob Herring, Mark Rutland, Heiko Stuebner, linux-media, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Cc: Justin Swartz This patchset aims to enable use of Rockchip's RGA, a 2D raster graphic acceleration unit, on rk322x based devices. Changes in v4: - Add a compatible value entry for the rk3228 to the rockchip-rga device tree binding documentation. Changes in v3: - Relocate rga node to the correct position in rk322x.dtsi, as indicated by Johan Jonker. Changes in v2: - Remove unnecessary "rockchip,rk3228-rga" device tree compatibility string patch, as advised by Ezequiel Garcia. - Use both "rockchip,rk3228-rga" and "rockchip,rk3288-rga" in the rga node's compatibility property, as suggested by Heiko Stuebner. Justin Swartz (3): [media] dt-bindings: Add binding for rk3228 rga ARM: dts: rockchip: add rga node for rk322x ARM: dts: rockchip: enable rga for rk3229-xms6 Documentation/devicetree/bindings/media/rockchip-rga.txt | 5 +++-- arch/arm/boot/dts/rk3229-xms6.dts | 4 ++++ arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ 3 files changed, 18 insertions(+), 2 deletions(-) -- 2.11.0 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga 2020-02-03 22:30 [PATCH v4 0/3] Add rga to rk322x device tree Justin Swartz @ 2020-02-03 22:40 ` Justin Swartz 2020-02-06 19:06 ` Rob Herring 2020-03-01 0:29 ` Heiko Stuebner 2020-02-03 22:40 ` [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x Justin Swartz 2020-02-03 22:40 ` [PATCH v4 3/3] ARM: dts: rockchip: enable rga for rk3229-xms6 Justin Swartz 2 siblings, 2 replies; 14+ messages in thread From: Justin Swartz @ 2020-02-03 22:40 UTC (permalink / raw) To: Jacob Chen, Ezequiel Garcia, Mauro Carvalho Chehab, Rob Herring, Mark Rutland, Heiko Stuebner Cc: Justin Swartz, linux-media, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Indicate that the rk3228 rga is compatible with that of the rk3288. But if any rk3228-specific quirks are identified in future that require handling logic that differs from what is provided for the rk3288, then allow for the compatibility string "rockchip,rk3228-rga" to be matched instead of "rockchip,rk3288-rga". Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> --- Documentation/devicetree/bindings/media/rockchip-rga.txt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/media/rockchip-rga.txt b/Documentation/devicetree/bindings/media/rockchip-rga.txt index fd5276abf..c53a8e513 100644 --- a/Documentation/devicetree/bindings/media/rockchip-rga.txt +++ b/Documentation/devicetree/bindings/media/rockchip-rga.txt @@ -6,8 +6,9 @@ BitBLT, alpha blending and image blur/sharpness. Required properties: - compatible: value should be one of the following - "rockchip,rk3288-rga"; - "rockchip,rk3399-rga"; + "rockchip,rk3228-rga", "rockchip,rk3288-rga": for Rockchip RK3228 + "rockchip,rk3288-rga": for Rockchip RK3288 + "rockchip,rk3399-rga": for Rockchip RK3399 - interrupts: RGA interrupt specifier. -- 2.11.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz @ 2020-02-06 19:06 ` Rob Herring 2020-03-01 0:29 ` Heiko Stuebner 1 sibling, 0 replies; 14+ messages in thread From: Rob Herring @ 2020-02-06 19:06 UTC (permalink / raw) To: Justin Swartz Cc: Jacob Chen, Ezequiel Garcia, Mauro Carvalho Chehab, Mark Rutland, Heiko Stuebner, Justin Swartz, linux-media, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel On Mon, 3 Feb 2020 22:40:15 +0000, Justin Swartz wrote: > Indicate that the rk3228 rga is compatible with that of the rk3288. > > But if any rk3228-specific quirks are identified in future that > require handling logic that differs from what is provided for the > rk3288, then allow for the compatibility string "rockchip,rk3228-rga" > to be matched instead of "rockchip,rk3288-rga". > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> > --- > Documentation/devicetree/bindings/media/rockchip-rga.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz 2020-02-06 19:06 ` Rob Herring @ 2020-03-01 0:29 ` Heiko Stuebner 1 sibling, 0 replies; 14+ messages in thread From: Heiko Stuebner @ 2020-03-01 0:29 UTC (permalink / raw) To: Justin Swartz, Mauro Carvalho Chehab Cc: Jacob Chen, Ezequiel Garcia, Rob Herring, Mark Rutland, linux-media, devicetree, linux-arm-kernel, linux-rockchip, linux-kernel Hi Mauro, Am Montag, 3. Februar 2020, 23:40:15 CET schrieb Justin Swartz: > Indicate that the rk3228 rga is compatible with that of the rk3288. > > But if any rk3228-specific quirks are identified in future that > require handling logic that differs from what is provided for the > rk3288, then allow for the compatibility string "rockchip,rk3228-rga" > to be matched instead of "rockchip,rk3288-rga". > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> what is the process in the media-world for this dt-binding only patch? Do you want to apply it, or should I just also apply it with the following 2 dt-patches? There shouldn't be any conflicts with the media tree as this patch really only affects the rga binding document. Thanks Heiko > --- > Documentation/devicetree/bindings/media/rockchip-rga.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/rockchip-rga.txt b/Documentation/devicetree/bindings/media/rockchip-rga.txt > index fd5276abf..c53a8e513 100644 > --- a/Documentation/devicetree/bindings/media/rockchip-rga.txt > +++ b/Documentation/devicetree/bindings/media/rockchip-rga.txt > @@ -6,8 +6,9 @@ BitBLT, alpha blending and image blur/sharpness. > > Required properties: > - compatible: value should be one of the following > - "rockchip,rk3288-rga"; > - "rockchip,rk3399-rga"; > + "rockchip,rk3228-rga", "rockchip,rk3288-rga": for Rockchip RK3228 > + "rockchip,rk3288-rga": for Rockchip RK3288 > + "rockchip,rk3399-rga": for Rockchip RK3399 > > - interrupts: RGA interrupt specifier. > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-02-03 22:30 [PATCH v4 0/3] Add rga to rk322x device tree Justin Swartz 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz @ 2020-02-03 22:40 ` Justin Swartz 2020-04-18 9:15 ` Johan Jonker 2020-02-03 22:40 ` [PATCH v4 3/3] ARM: dts: rockchip: enable rga for rk3229-xms6 Justin Swartz 2 siblings, 1 reply; 14+ messages in thread From: Justin Swartz @ 2020-02-03 22:40 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Mark Rutland Cc: Justin Swartz, linux-arm-kernel, linux-rockchip, devicetree, linux-kernel Add a node to define the presence of RGA, a 2D raster graphic acceleration unit. Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> --- arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/arch/arm/boot/dts/rk322x.dtsi b/arch/arm/boot/dts/rk322x.dtsi index 340ed6ccb..29d50bebc 100644 --- a/arch/arm/boot/dts/rk322x.dtsi +++ b/arch/arm/boot/dts/rk322x.dtsi @@ -621,6 +621,17 @@ status = "disabled"; }; + rga: rga@20060000 { + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; + reg = <0x20060000 0x1000>; + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; + clock-names = "aclk", "hclk", "sclk"; + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru SRST_RGA_H>; + reset-names = "core", "axi", "ahb"; + status = "disabled"; + }; + iep_mmu: iommu@20070800 { compatible = "rockchip,iommu"; reg = <0x20070800 0x100>; -- 2.11.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-02-03 22:40 ` [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x Justin Swartz @ 2020-04-18 9:15 ` Johan Jonker 2020-04-18 9:49 ` Heiko Stuebner 0 siblings, 1 reply; 14+ messages in thread From: Johan Jonker @ 2020-04-18 9:15 UTC (permalink / raw) To: justin.swartz Cc: devicetree, heiko, linux-arm-kernel, linux-kernel, linux-rockchip, mark.rutland, robh+dt Hi Justin, Heiko, In the discussion about "rockchip,px30-rga" it was mentioned that: "no status for soc internal components without any board-specifics". The binding is in mainline, but the dts node is missing I think. Should this also be changed? Johan +&rga { + status = "okay"; +}; + > Add a node to define the presence of RGA, a 2D raster > graphic acceleration unit. > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> > --- > arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm/boot/dts/rk322x.dtsi b/arch/arm/boot/dts/rk322x.dtsi > index 340ed6ccb..29d50bebc 100644 > --- a/arch/arm/boot/dts/rk322x.dtsi > +++ b/arch/arm/boot/dts/rk322x.dtsi > @@ -621,6 +621,17 @@ > status = "disabled"; > }; > > + rga: rga@20060000 { > + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; > + reg = <0x20060000 0x1000>; > + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; > + clock-names = "aclk", "hclk", "sclk"; > + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru SRST_RGA_H>; > + reset-names = "core", "axi", "ahb"; > + status = "disabled"; remove? > + }; > + > iep_mmu: iommu@20070800 { > compatible = "rockchip,iommu"; > reg = <0x20070800 0x100>; > -- > 2.11.0 ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-04-18 9:15 ` Johan Jonker @ 2020-04-18 9:49 ` Heiko Stuebner 2020-04-18 9:56 ` Justin Swartz 0 siblings, 1 reply; 14+ messages in thread From: Heiko Stuebner @ 2020-04-18 9:49 UTC (permalink / raw) To: Johan Jonker Cc: justin.swartz, devicetree, linux-arm-kernel, linux-kernel, linux-rockchip, mark.rutland, robh+dt Am Samstag, 18. April 2020, 11:15:27 CEST schrieb Johan Jonker: > Hi Justin, Heiko, > > In the discussion about "rockchip,px30-rga" it was mentioned that: > "no status for soc internal components without any board-specifics". > The binding is in mainline, but the dts node is missing I think. > Should this also be changed? yep, drop the status disabled from the dtsi and that then results in not needing the board-level enablement (background: rga is a simple mem2mem device without any board-specific hooks) > > Johan > > +&rga { > + status = "okay"; > +}; > + > > > Add a node to define the presence of RGA, a 2D raster > > graphic acceleration unit. > > > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> > > --- > > arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/arch/arm/boot/dts/rk322x.dtsi b/arch/arm/boot/dts/rk322x.dtsi > > index 340ed6ccb..29d50bebc 100644 > > --- a/arch/arm/boot/dts/rk322x.dtsi > > +++ b/arch/arm/boot/dts/rk322x.dtsi > > @@ -621,6 +621,17 @@ > > status = "disabled"; > > }; > > > > + rga: rga@20060000 { > > + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; > > + reg = <0x20060000 0x1000>; > > + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; > > + clock-names = "aclk", "hclk", "sclk"; > > + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru SRST_RGA_H>; > > + reset-names = "core", "axi", "ahb"; > > > + status = "disabled"; > > remove? > > > + }; > > + > > iep_mmu: iommu@20070800 { > > compatible = "rockchip,iommu"; > > reg = <0x20070800 0x100>; > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-04-18 9:49 ` Heiko Stuebner @ 2020-04-18 9:56 ` Justin Swartz 2020-04-18 10:53 ` Heiko Stuebner 0 siblings, 1 reply; 14+ messages in thread From: Justin Swartz @ 2020-04-18 9:56 UTC (permalink / raw) To: Heiko Stuebner Cc: Johan Jonker, devicetree, linux-arm-kernel, linux-kernel, linux-rockchip, mark.rutland, robh+dt Hi Johan, Heiko, On 2020-04-18 11:49, Heiko Stuebner wrote: > Am Samstag, 18. April 2020, 11:15:27 CEST schrieb Johan Jonker: > >> Hi Justin, Heiko, >> >> In the discussion about "rockchip,px30-rga" it was mentioned that: >> "no status for soc internal components without any board-specifics". >> The binding is in mainline, but the dts node is missing I think. >> Should this also be changed? > > yep, drop the status disabled from the dtsi and that then results > in not needing the board-level enablement > > (background: rga is a simple mem2mem device without any > board-specific hooks) Shall I submit version 5 of this patch series? > Johan > > +&rga { > + status = "okay"; > +}; > + > > Add a node to define the presence of RGA, a 2D raster > graphic acceleration unit. > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> > --- > arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm/boot/dts/rk322x.dtsi > b/arch/arm/boot/dts/rk322x.dtsi > index 340ed6ccb..29d50bebc 100644 > --- a/arch/arm/boot/dts/rk322x.dtsi > +++ b/arch/arm/boot/dts/rk322x.dtsi > @@ -621,6 +621,17 @@ > status = "disabled"; > }; > > + rga: rga@20060000 { > + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; > + reg = <0x20060000 0x1000>; > + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; > + clock-names = "aclk", "hclk", "sclk"; > + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru > SRST_RGA_H>; > + reset-names = "core", "axi", "ahb"; > + status = "disabled"; > remove? > > + }; > + > iep_mmu: iommu@20070800 { > compatible = "rockchip,iommu"; > reg = <0x20070800 0x100>; ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-04-18 9:56 ` Justin Swartz @ 2020-04-18 10:53 ` Heiko Stuebner 2020-04-18 12:14 ` Johan Jonker 0 siblings, 1 reply; 14+ messages in thread From: Heiko Stuebner @ 2020-04-18 10:53 UTC (permalink / raw) To: Justin Swartz Cc: Johan Jonker, devicetree, linux-arm-kernel, linux-kernel, linux-rockchip, mark.rutland, robh+dt Am Samstag, 18. April 2020, 11:56:03 CEST schrieb Justin Swartz: > Hi Johan, Heiko, > > On 2020-04-18 11:49, Heiko Stuebner wrote: > > > Am Samstag, 18. April 2020, 11:15:27 CEST schrieb Johan Jonker: > > > >> Hi Justin, Heiko, > >> > >> In the discussion about "rockchip,px30-rga" it was mentioned that: > >> "no status for soc internal components without any board-specifics". > >> The binding is in mainline, but the dts node is missing I think. > >> Should this also be changed? > > > > yep, drop the status disabled from the dtsi and that then results > > in not needing the board-level enablement > > > > (background: rga is a simple mem2mem device without any > > board-specific hooks) > > Shall I submit version 5 of this patch series? I guess so ... especially as we expect patch1 to go through the media tree and sending a new version might serve as a ping :-) and include the Ack by Rob for easy applying Heiko > > > Johan > > > > +&rga { > > + status = "okay"; > > +}; > > + > > > > Add a node to define the presence of RGA, a 2D raster > > graphic acceleration unit. > > > > Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> > > --- > > arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/arch/arm/boot/dts/rk322x.dtsi > > b/arch/arm/boot/dts/rk322x.dtsi > > index 340ed6ccb..29d50bebc 100644 > > --- a/arch/arm/boot/dts/rk322x.dtsi > > +++ b/arch/arm/boot/dts/rk322x.dtsi > > @@ -621,6 +621,17 @@ > > status = "disabled"; > > }; > > > > + rga: rga@20060000 { > > + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; > > + reg = <0x20060000 0x1000>; > > + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; > > + clock-names = "aclk", "hclk", "sclk"; > > + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru > > SRST_RGA_H>; > > + reset-names = "core", "axi", "ahb"; > > + status = "disabled"; > > remove? > > > > + }; > > + > > iep_mmu: iommu@20070800 { > > compatible = "rockchip,iommu"; > > reg = <0x20070800 0x100>; > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x 2020-04-18 10:53 ` Heiko Stuebner @ 2020-04-18 12:14 ` Johan Jonker 2020-04-19 12:51 ` [PATCH v5 0/1] Add rga to rk322x device tree Justin Swartz 2020-04-19 12:51 ` [PATCH v5 1/1] ARM: dts: rockchip: add rga node for rk322x Justin Swartz 0 siblings, 2 replies; 14+ messages in thread From: Johan Jonker @ 2020-04-18 12:14 UTC (permalink / raw) To: Heiko Stuebner, Justin Swartz Cc: devicetree, linux-arm-kernel, linux-kernel, linux-rockchip, mark.rutland, robh+dt On 4/18/20 12:53 PM, Heiko Stuebner wrote: > Am Samstag, 18. April 2020, 11:56:03 CEST schrieb Justin Swartz: >> Hi Johan, Heiko, >> >> On 2020-04-18 11:49, Heiko Stuebner wrote: >> >>> Am Samstag, 18. April 2020, 11:15:27 CEST schrieb Johan Jonker: >>> >>>> Hi Justin, Heiko, >>>> >>>> In the discussion about "rockchip,px30-rga" it was mentioned that: >>>> "no status for soc internal components without any board-specifics". >>>> The binding is in mainline, but the dts node is missing I think. >>>> Should this also be changed? >>> >>> yep, drop the status disabled from the dtsi and that then results >>> in not needing the board-level enablement >>> >>> (background: rga is a simple mem2mem device without any >>> board-specific hooks) >> >> Shall I submit version 5 of this patch series? > > I guess so ... especially as we expect patch1 to go through the > media tree and sending a new version might serve as a ping :-) > and include the Ack by Rob for easy applying The conversion to rockchip-rga.yaml including rockchip,rk3228-rga is already reviewed by robh. > > > Heiko > > >> >>> Johan >>> >>> +&rga { >>> + status = "okay"; >>> +}; >>> + >>> >>> Add a node to define the presence of RGA, a 2D raster >>> graphic acceleration unit. >>> >>> Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> >>> --- >>> arch/arm/boot/dts/rk322x.dtsi | 11 +++++++++++ >>> 1 file changed, 11 insertions(+) >>> >>> diff --git a/arch/arm/boot/dts/rk322x.dtsi >>> b/arch/arm/boot/dts/rk322x.dtsi >>> index 340ed6ccb..29d50bebc 100644 >>> --- a/arch/arm/boot/dts/rk322x.dtsi >>> +++ b/arch/arm/boot/dts/rk322x.dtsi >>> @@ -621,6 +621,17 @@ >>> status = "disabled"; >>> }; >>> >>> + rga: rga@20060000 { >>> + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; >>> + reg = <0x20060000 0x1000>; >>> + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; >>> + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; >>> + clock-names = "aclk", "hclk", "sclk"; >>> + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru >>> SRST_RGA_H>; >>> + reset-names = "core", "axi", "ahb"; >>> + status = "disabled"; >>> remove? >>> >>> + }; >>> + >>> iep_mmu: iommu@20070800 { >>> compatible = "rockchip,iommu"; >>> reg = <0x20070800 0x100>; >> > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v5 0/1] Add rga to rk322x device tree 2020-04-18 12:14 ` Johan Jonker @ 2020-04-19 12:51 ` Justin Swartz 2020-05-18 22:52 ` Heiko Stuebner 2020-04-19 12:51 ` [PATCH v5 1/1] ARM: dts: rockchip: add rga node for rk322x Justin Swartz 1 sibling, 1 reply; 14+ messages in thread From: Justin Swartz @ 2020-04-19 12:51 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Mark Rutland Cc: Justin Swartz, linux-arm-kernel, linux-rockchip, devicetree, linux-kernel This patchset aims to enable use of Rockchip's RGA, a 2D raster graphic acceleration unit, on rk322x based devices. Changes in v5: - Remove the patch to rockchip-rga device tree binding documentation as Johan Jonker has already included mention of "rockchip,rk3228-rga" since the conversion to YAML. - Remove the assignment of "disabled" to the rga "status" attribute in rk322x.dtsi, as Heiko Stuebner has said: "no status for soc internal components without any board-specifics" - Remove the now unnecessary patch to enable rga for the rk3229-xms6 board. Changes in v4: - Add a compatible value entry for the rk3228 to the rockchip-rga device tree binding documentation. Changes in v3: - Relocate rga node to the correct position in rk322x.dtsi, as indicated by Johan Jonker. Changes in v2: - Remove unnecessary "rockchip,rk3228-rga" device tree compatibility string patch, as advised by Ezequiel Garcia. - Use both "rockchip,rk3228-rga" and "rockchip,rk3288-rga" in the rga node's compatibility property, as suggested by Heiko Stuebner. Justin Swartz (1): ARM: dts: rockchip: add rga node for rk322x arch/arm/boot/dts/rk322x.dtsi | 10 ++++++++++ 1 file changed, 10 insertions(+) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v5 0/1] Add rga to rk322x device tree 2020-04-19 12:51 ` [PATCH v5 0/1] Add rga to rk322x device tree Justin Swartz @ 2020-05-18 22:52 ` Heiko Stuebner 0 siblings, 0 replies; 14+ messages in thread From: Heiko Stuebner @ 2020-05-18 22:52 UTC (permalink / raw) To: Mark Rutland, Justin Swartz, Rob Herring Cc: Heiko Stuebner, linux-arm-kernel, linux-kernel, devicetree, linux-rockchip On Sun, 19 Apr 2020 12:51:32 +0000, Justin Swartz wrote: > This patchset aims to enable use of Rockchip's RGA, a 2D raster > graphic acceleration unit, on rk322x based devices. > > Changes in v5: > - Remove the patch to rockchip-rga device tree binding documentation > as Johan Jonker has already included mention of "rockchip,rk3228-rga" > since the conversion to YAML. > > [...] Applied, thanks! [1/1] ARM: dts: rockchip: add rga node for rk322x commit: 54b1a4e070330c3fba5becfb0b619bf360bc2657 Best regards, -- Heiko Stuebner <heiko@sntech.de> ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v5 1/1] ARM: dts: rockchip: add rga node for rk322x 2020-04-18 12:14 ` Johan Jonker 2020-04-19 12:51 ` [PATCH v5 0/1] Add rga to rk322x device tree Justin Swartz @ 2020-04-19 12:51 ` Justin Swartz 1 sibling, 0 replies; 14+ messages in thread From: Justin Swartz @ 2020-04-19 12:51 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Mark Rutland Cc: Justin Swartz, linux-arm-kernel, linux-rockchip, devicetree, linux-kernel Add a node to define the presence of RGA, a 2D raster graphic acceleration unit. Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> --- arch/arm/boot/dts/rk322x.dtsi | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/rk322x.dtsi b/arch/arm/boot/dts/rk322x.dtsi index 4e90efdc9..ce88f5bd6 100644 --- a/arch/arm/boot/dts/rk322x.dtsi +++ b/arch/arm/boot/dts/rk322x.dtsi @@ -621,6 +621,16 @@ status = "disabled"; }; + rga: rga@20060000 { + compatible = "rockchip,rk3228-rga", "rockchip,rk3288-rga"; + reg = <0x20060000 0x1000>; + interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&cru ACLK_RGA>, <&cru HCLK_RGA>, <&cru SCLK_RGA>; + clock-names = "aclk", "hclk", "sclk"; + resets = <&cru SRST_RGA>, <&cru SRST_RGA_A>, <&cru SRST_RGA_H>; + reset-names = "core", "axi", "ahb"; + }; + iep_mmu: iommu@20070800 { compatible = "rockchip,iommu"; reg = <0x20070800 0x100>; -- 2.11.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v4 3/3] ARM: dts: rockchip: enable rga for rk3229-xms6 2020-02-03 22:30 [PATCH v4 0/3] Add rga to rk322x device tree Justin Swartz 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz 2020-02-03 22:40 ` [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x Justin Swartz @ 2020-02-03 22:40 ` Justin Swartz 2 siblings, 0 replies; 14+ messages in thread From: Justin Swartz @ 2020-02-03 22:40 UTC (permalink / raw) To: Heiko Stuebner, Rob Herring, Mark Rutland Cc: Justin Swartz, linux-arm-kernel, linux-rockchip, devicetree, linux-kernel Enable RGA for Mecer Xtreme Mini S6. Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za> --- arch/arm/boot/dts/rk3229-xms6.dts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/rk3229-xms6.dts b/arch/arm/boot/dts/rk3229-xms6.dts index 679fc2b00..894f64a4a 100644 --- a/arch/arm/boot/dts/rk3229-xms6.dts +++ b/arch/arm/boot/dts/rk3229-xms6.dts @@ -202,6 +202,10 @@ status = "okay"; }; +&rga { + status = "okay"; +}; + &sdmmc { cap-mmc-highspeed; disable-wp; -- 2.11.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
end of thread, other threads:[~2020-05-18 22:52 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-02-03 22:30 [PATCH v4 0/3] Add rga to rk322x device tree Justin Swartz 2020-02-03 22:40 ` [PATCH v4 1/3] [media] dt-bindings: Add binding for rk3228 rga Justin Swartz 2020-02-06 19:06 ` Rob Herring 2020-03-01 0:29 ` Heiko Stuebner 2020-02-03 22:40 ` [PATCH v4 2/3] ARM: dts: rockchip: add rga node for rk322x Justin Swartz 2020-04-18 9:15 ` Johan Jonker 2020-04-18 9:49 ` Heiko Stuebner 2020-04-18 9:56 ` Justin Swartz 2020-04-18 10:53 ` Heiko Stuebner 2020-04-18 12:14 ` Johan Jonker 2020-04-19 12:51 ` [PATCH v5 0/1] Add rga to rk322x device tree Justin Swartz 2020-05-18 22:52 ` Heiko Stuebner 2020-04-19 12:51 ` [PATCH v5 1/1] ARM: dts: rockchip: add rga node for rk322x Justin Swartz 2020-02-03 22:40 ` [PATCH v4 3/3] ARM: dts: rockchip: enable rga for rk3229-xms6 Justin Swartz
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).