* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-25 16:19 [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU Dragan Simic
@ 2024-03-26 19:11 ` Rob Herring
2024-03-26 19:27 ` Dragan Simic
2024-03-26 19:54 ` Sebastian Reichel
2024-03-27 10:29 ` Heiko Stübner
2 siblings, 1 reply; 7+ messages in thread
From: Rob Herring @ 2024-03-26 19:11 UTC (permalink / raw)
To: Dragan Simic
Cc: conor+dt, kernel, devicetree, sebastian.reichel, heiko,
linux-arm-kernel, boris.brezillon, linux-kernel, linux-rockchip,
robh+dt, krzysztof.kozlowski+dt
On Mon, 25 Mar 2024 17:19:04 +0100, Dragan Simic wrote:
> Following the approach used to enable the Mali GPU on the rk3588-evb1, [1]
> do the same for the Pine64 QuartzPro64, which uses nearly identical hardware
> design as the RK3588 EVB1.
>
> The slight disadvantage is that the regulator coupling logic requires the
> regulators to be always on, which is also noted in the comments. This is
> obviously something to be improved at some point in the future, but should
> be fine for now, especially because the QuartzPro64 isn't a battery-powered
> board, so low power consumption isn't paramount.
>
> [1] https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
> .../arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
My bot found new DTB warnings on the .dts files added or changed in this
series.
Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.
If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:
pip3 install dtschema --upgrade
New warnings running 'make CHECK_DTBS=y rockchip/rk3588-quartzpro64.dtb' for 0f3759ee390f245dac447bbee038445ddfecbec0.1711383286.git.dsimic@manjaro.org:
Error: arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts:288.1-5 Label or path gpu not found
FATAL ERROR: Syntax error parsing input tree
make[3]: *** [scripts/Makefile.lib:427: arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dtb] Error 1
make[2]: *** [scripts/Makefile.build:485: arch/arm64/boot/dts/rockchip] Error 2
make[2]: Target 'arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dtb' not remade because of errors.
make[1]: *** [/home/rob/proj/linux-dt-testing/Makefile:1387: rockchip/rk3588-quartzpro64.dtb] Error 2
make: *** [Makefile:240: __sub-make] Error 2
make: Target 'rockchip/rk3588-quartzpro64.dtb' not remade because of errors.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-26 19:11 ` Rob Herring
@ 2024-03-26 19:27 ` Dragan Simic
0 siblings, 0 replies; 7+ messages in thread
From: Dragan Simic @ 2024-03-26 19:27 UTC (permalink / raw)
To: Rob Herring
Cc: conor+dt, kernel, devicetree, sebastian.reichel, heiko,
linux-arm-kernel, boris.brezillon, linux-kernel, linux-rockchip,
robh+dt, krzysztof.kozlowski+dt
On 2024-03-26 20:11, Rob Herring wrote:
> On Mon, 25 Mar 2024 17:19:04 +0100, Dragan Simic wrote:
>> Following the approach used to enable the Mali GPU on the rk3588-evb1,
>> [1]
>> do the same for the Pine64 QuartzPro64, which uses nearly identical
>> hardware
>> design as the RK3588 EVB1.
>>
>> The slight disadvantage is that the regulator coupling logic requires
>> the
>> regulators to be always on, which is also noted in the comments. This
>> is
>> obviously something to be improved at some point in the future, but
>> should
>> be fine for now, especially because the QuartzPro64 isn't a
>> battery-powered
>> board, so low power consumption isn't paramount.
>>
>> [1]
>> https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>> .../arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 14
>> ++++++++++++++
>> 1 file changed, 14 insertions(+)
>
> My bot found new DTB warnings on the .dts files added or changed in
> this
> series.
>
> Some warnings may be from an existing SoC .dtsi. Or perhaps the
> warnings
> are fixed by another series. Ultimately, it is up to the platform
> maintainer whether these warnings are acceptable or not. No need to
> reply
> unless the platform maintainer has comments.
>
> If you already ran DT checks and didn't see these error(s), then
> make sure dt-schema is up to date:
>
> pip3 install dtschema --upgrade
>
>
> New warnings running 'make CHECK_DTBS=y
> rockchip/rk3588-quartzpro64.dtb' for
> 0f3759ee390f245dac447bbee038445ddfecbec0.1711383286.git.dsimic@manjaro.org:
>
> Error: arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts:288.1-5
> Label or path gpu not found
> FATAL ERROR: Syntax error parsing input tree
> make[3]: *** [scripts/Makefile.lib:427:
> arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dtb] Error 1
> make[2]: *** [scripts/Makefile.build:485: arch/arm64/boot/dts/rockchip]
> Error 2
> make[2]: Target 'arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dtb'
> not remade because of errors.
> make[1]: *** [/home/rob/proj/linux-dt-testing/Makefile:1387:
> rockchip/rk3588-quartzpro64.dtb] Error 2
> make: *** [Makefile:240: __sub-make] Error 2
> make: Target 'rockchip/rk3588-quartzpro64.dtb' not remade because of
> errors.
This error can be safely ignored, because this patch is supposed
to be applied together with a recent patch series, [1] or in more
detail, after one of the patches in that series [2] that adds the
missing "gpu" node.
[1]
https://lore.kernel.org/linux-rockchip/20240326165232.73585-1-sebastian.reichel@collabora.com/T/#u
[2]
https://lore.kernel.org/linux-rockchip/20240326165232.73585-3-sebastian.reichel@collabora.com/
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-25 16:19 [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU Dragan Simic
2024-03-26 19:11 ` Rob Herring
@ 2024-03-26 19:54 ` Sebastian Reichel
2024-03-26 19:57 ` Dragan Simic
2024-03-27 10:29 ` Heiko Stübner
2 siblings, 1 reply; 7+ messages in thread
From: Sebastian Reichel @ 2024-03-26 19:54 UTC (permalink / raw)
To: Dragan Simic
Cc: linux-rockchip, heiko, linux-arm-kernel, devicetree, robh+dt,
krzysztof.kozlowski+dt, conor+dt, boris.brezillon, linux-kernel,
kernel
[-- Attachment #1.1: Type: text/plain, Size: 2699 bytes --]
Hi,
On Mon, Mar 25, 2024 at 05:19:04PM +0100, Dragan Simic wrote:
> Following the approach used to enable the Mali GPU on the rk3588-evb1, [1]
> do the same for the Pine64 QuartzPro64, which uses nearly identical hardware
> design as the RK3588 EVB1.
>
> The slight disadvantage is that the regulator coupling logic requires the
> regulators to be always on, which is also noted in the comments. This is
> obviously something to be improved at some point in the future, but should
> be fine for now, especially because the QuartzPro64 isn't a battery-powered
> board, so low power consumption isn't paramount.
>
> [1] https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
> ---
FWIW
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>
-- Sebastian
> .../arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
> index 67414d72e2b6..68d432c61ea5 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
> @@ -285,6 +285,12 @@ &gmac0_rgmii_clk
> status = "okay";
> };
>
> +&gpu {
> + mali-supply = <&vdd_gpu_s0>;
> + sram-supply = <&vdd_gpu_mem_s0>;
> + status = "okay";
> +};
> +
> &i2c2 {
> status = "okay";
>
> @@ -491,11 +497,15 @@ rk806_dvs3_null: dvs3-null-pins {
> regulators {
> vdd_gpu_s0: dcdc-reg1 {
> regulator-name = "vdd_gpu_s0";
> + /* regulator coupling requires always-on */
> + regulator-always-on;
> regulator-boot-on;
> regulator-enable-ramp-delay = <400>;
> regulator-min-microvolt = <550000>;
> regulator-max-microvolt = <950000>;
> regulator-ramp-delay = <12500>;
> + regulator-coupled-with = <&vdd_gpu_mem_s0>;
> + regulator-coupled-max-spread = <10000>;
>
> regulator-state-mem {
> regulator-off-in-suspend;
> @@ -545,11 +555,15 @@ regulator-state-mem {
>
> vdd_gpu_mem_s0: dcdc-reg5 {
> regulator-name = "vdd_gpu_mem_s0";
> + /* regulator coupling requires always-on */
> + regulator-always-on;
> regulator-boot-on;
> regulator-enable-ramp-delay = <400>;
> regulator-min-microvolt = <675000>;
> regulator-max-microvolt = <950000>;
> regulator-ramp-delay = <12500>;
> + regulator-coupled-with = <&vdd_gpu_s0>;
> + regulator-coupled-max-spread = <10000>;
>
> regulator-state-mem {
> regulator-off-in-suspend;
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-26 19:54 ` Sebastian Reichel
@ 2024-03-26 19:57 ` Dragan Simic
0 siblings, 0 replies; 7+ messages in thread
From: Dragan Simic @ 2024-03-26 19:57 UTC (permalink / raw)
To: Sebastian Reichel
Cc: linux-rockchip, heiko, linux-arm-kernel, devicetree, robh+dt,
krzysztof.kozlowski+dt, conor+dt, boris.brezillon, linux-kernel,
kernel
Hello Sebastian,
On 2024-03-26 20:54, Sebastian Reichel wrote:
> On Mon, Mar 25, 2024 at 05:19:04PM +0100, Dragan Simic wrote:
>> Following the approach used to enable the Mali GPU on the rk3588-evb1,
>> [1]
>> do the same for the Pine64 QuartzPro64, which uses nearly identical
>> hardware
>> design as the RK3588 EVB1.
>>
>> The slight disadvantage is that the regulator coupling logic requires
>> the
>> regulators to be always on, which is also noted in the comments. This
>> is
>> obviously something to be improved at some point in the future, but
>> should
>> be fine for now, especially because the QuartzPro64 isn't a
>> battery-powered
>> board, so low power consumption isn't paramount.
>>
>> [1]
>> https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>> ---
>
> FWIW
>
> Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>
Great, thanks for your review!
>> .../arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 14
>> ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
>> b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
>> index 67414d72e2b6..68d432c61ea5 100644
>> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
>> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
>> @@ -285,6 +285,12 @@ &gmac0_rgmii_clk
>> status = "okay";
>> };
>>
>> +&gpu {
>> + mali-supply = <&vdd_gpu_s0>;
>> + sram-supply = <&vdd_gpu_mem_s0>;
>> + status = "okay";
>> +};
>> +
>> &i2c2 {
>> status = "okay";
>>
>> @@ -491,11 +497,15 @@ rk806_dvs3_null: dvs3-null-pins {
>> regulators {
>> vdd_gpu_s0: dcdc-reg1 {
>> regulator-name = "vdd_gpu_s0";
>> + /* regulator coupling requires always-on */
>> + regulator-always-on;
>> regulator-boot-on;
>> regulator-enable-ramp-delay = <400>;
>> regulator-min-microvolt = <550000>;
>> regulator-max-microvolt = <950000>;
>> regulator-ramp-delay = <12500>;
>> + regulator-coupled-with = <&vdd_gpu_mem_s0>;
>> + regulator-coupled-max-spread = <10000>;
>>
>> regulator-state-mem {
>> regulator-off-in-suspend;
>> @@ -545,11 +555,15 @@ regulator-state-mem {
>>
>> vdd_gpu_mem_s0: dcdc-reg5 {
>> regulator-name = "vdd_gpu_mem_s0";
>> + /* regulator coupling requires always-on */
>> + regulator-always-on;
>> regulator-boot-on;
>> regulator-enable-ramp-delay = <400>;
>> regulator-min-microvolt = <675000>;
>> regulator-max-microvolt = <950000>;
>> regulator-ramp-delay = <12500>;
>> + regulator-coupled-with = <&vdd_gpu_s0>;
>> + regulator-coupled-max-spread = <10000>;
>>
>> regulator-state-mem {
>> regulator-off-in-suspend;
>
> _______________________________________________
> Linux-rockchip mailing list
> Linux-rockchip@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-rockchip
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-25 16:19 [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU Dragan Simic
2024-03-26 19:11 ` Rob Herring
2024-03-26 19:54 ` Sebastian Reichel
@ 2024-03-27 10:29 ` Heiko Stübner
2024-03-27 20:28 ` Dragan Simic
2 siblings, 1 reply; 7+ messages in thread
From: Heiko Stübner @ 2024-03-27 10:29 UTC (permalink / raw)
To: linux-rockchip, Dragan Simic
Cc: linux-arm-kernel, devicetree, robh+dt, krzysztof.kozlowski+dt,
conor+dt, boris.brezillon, linux-kernel, kernel,
sebastian.reichel
Hi,
Am Montag, 25. März 2024, 17:19:04 CET schrieb Dragan Simic:
> Following the approach used to enable the Mali GPU on the rk3588-evb1, [1]
> do the same for the Pine64 QuartzPro64, which uses nearly identical hardware
> design as the RK3588 EVB1.
>
> The slight disadvantage is that the regulator coupling logic requires the
> regulators to be always on, which is also noted in the comments. This is
> obviously something to be improved at some point in the future, but should
> be fine for now, especially because the QuartzPro64 isn't a battery-powered
> board, so low power consumption isn't paramount.
>
> [1] https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>
> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
as lore.kernel.org and therefore b4 seems to be on vacation today, you
get a very personal "applied" message ;-) .
So, applied for 6.10 after the core rk3588-gpu-series from Sebastian.
Heiko
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] arm64: dts: rockchip: quartzpro64: Enable the GPU
2024-03-27 10:29 ` Heiko Stübner
@ 2024-03-27 20:28 ` Dragan Simic
0 siblings, 0 replies; 7+ messages in thread
From: Dragan Simic @ 2024-03-27 20:28 UTC (permalink / raw)
To: Heiko Stübner
Cc: linux-rockchip, linux-arm-kernel, devicetree, robh+dt,
krzysztof.kozlowski+dt, conor+dt, boris.brezillon, linux-kernel,
kernel, sebastian.reichel
Hello Heiko,
On 2024-03-27 11:29, Heiko Stübner wrote:
> Am Montag, 25. März 2024, 17:19:04 CET schrieb Dragan Simic:
>> Following the approach used to enable the Mali GPU on the rk3588-evb1,
>> [1]
>> do the same for the Pine64 QuartzPro64, which uses nearly identical
>> hardware
>> design as the RK3588 EVB1.
>>
>> The slight disadvantage is that the regulator coupling logic requires
>> the
>> regulators to be always on, which is also noted in the comments. This
>> is
>> obviously something to be improved at some point in the future, but
>> should
>> be fine for now, especially because the QuartzPro64 isn't a
>> battery-powered
>> board, so low power consumption isn't paramount.
>>
>> [1]
>> https://lore.kernel.org/linux-rockchip/20240325153850.189128-5-sebastian.reichel@collabora.com/
>>
>> Signed-off-by: Dragan Simic <dsimic@manjaro.org>
>
> as lore.kernel.org and therefore b4 seems to be on vacation today, you
> get a very personal "applied" message ;-) .
>
> So, applied for 6.10 after the core rk3588-gpu-series from Sebastian.
Oh, thank you very much for this special message. :)
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 7+ messages in thread