Re: [PATCH 0/9] rockchip: Remove unneeded syscon driver for RK35xx

Simon Glass <[email protected]>
Newsgroups org.u-boot-project.lists.u-boot
Message-ID <CAFLszTjBiTtKmjgknvXGFBrphh6kpBzRg8iroWjFQ0EDvuXTyQ@mail.gmail.com>
Hi Jonas,

On Tue, 4 Aug 2026 at 04:57, Jonas Karlman <[email protected]> wrote:
>
> This series takes the initial steps required to try and remove use of
> syscon_get_first_range() or syscon_get_regmap_by_driver_data() to get
> GRF base addresses at runtime for the Rockchip platform.
>
> The GRF base addresses are fixed and known at compile time, yet drivers
> keep trying to resolve them at runtime delaying some operations by
> several ms.

I suppose all addresses are known at compile-time, if you have a
single device tree. I have always hoped that we might end up with a
generic U-Boot for rockchip.

I wonder if there is another way to do this, perhaps reading the
address once and caching it?

>
> In this initial part the three current uses of ROCKCHIP_SYSCON_x lookup
> used on RK35xx SoCs is converted to use a constant value or using
> phandle lookup, followed by removing the SoC specific syscon driver and
> finally removing the RK35xx special ROCKCHIP_SYSCON_x enum values.
>
> The changes included in this series have no intended behavior change for
> the affected RK35xx SoCs.
>
> Future series will continue to convert uses of ROCKCHIP_SYSCON_x for
> remaining older RK SoCs if this initial part is accepted.
>
> Jonas Karlman (9):
>   ram: rockchip: rk3568: Simplify get_info() ops
>   ram: rockchip: rk3588: Simplify get_info() ops
>   video: rockchip: dw_mipi_dsi: Get GRF base address from phandle
>   rockchip: rk3568: Remove unneeded syscon driver
>   rockchip: rk3588: Remove unneeded syscon driver
>   rockchip: rk3576: Remove unneeded syscon driver
>   rockchip: rk3528: Remove unneeded syscon driver
>   rockchip: rk3506: Remove unneeded syscon driver
>   rockchip: include: Remove unused ROCKCHIP_SYSCON_x enum values
>
>  arch/arm/include/asm/arch-rockchip/clock.h    |  8 -----
>  arch/arm/mach-rockchip/rk3506/Makefile        |  1 -
>  arch/arm/mach-rockchip/rk3506/syscon_rk3506.c | 19 ------------
>  arch/arm/mach-rockchip/rk3528/Makefile        |  1 -
>  arch/arm/mach-rockchip/rk3528/syscon_rk3528.c | 19 ------------
>  arch/arm/mach-rockchip/rk3568/Makefile        |  1 -
>  arch/arm/mach-rockchip/rk3568/syscon_rk3568.c | 23 --------------
>  arch/arm/mach-rockchip/rk3576/Makefile        |  1 -
>  arch/arm/mach-rockchip/rk3576/syscon_rk3576.c | 22 -------------
>  arch/arm/mach-rockchip/rk3588/Makefile        |  1 -
>  arch/arm/mach-rockchip/rk3588/syscon_rk3588.c | 31 -------------------
>  drivers/ram/rockchip/sdram_rk3568.c           | 29 +++--------------
>  drivers/ram/rockchip/sdram_rk3588.c           | 31 ++++---------------
>  drivers/video/rockchip/dw_mipi_dsi_rockchip.c | 16 ++++++----
>  14 files changed, 21 insertions(+), 182 deletions(-)
>  delete mode 100644 arch/arm/mach-rockchip/rk3506/syscon_rk3506.c
>  delete mode 100644 arch/arm/mach-rockchip/rk3528/syscon_rk3528.c
>  delete mode 100644 arch/arm/mach-rockchip/rk3568/syscon_rk3568.c
>  delete mode 100644 arch/arm/mach-rockchip/rk3576/syscon_rk3576.c
>  delete mode 100644 arch/arm/mach-rockchip/rk3588/syscon_rk3588.c
>
> --
> 2.54.0
>

Regards,
Simon
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.