[PATCH] rockchip: include cru_rk3588.h and define rockchip_cru for RK3588
FUKAUMI Naoki
naoki at radxa.com
Tue Jul 16 04:24:33 CEST 2024
hi
On 6/19/24 05:39, FUKAUMI Naoki wrote:
> sorry,
>
> On 6/19/24 05:29, FUKAUMI Naoki wrote:
>> Hello,
>>
>> this happens only if HOSTCC=clang. I can see several other errors on
>> several other targets with HOSTCC=clang.
>
> ^this happens only if clang is used at make foo-bar_defconfig stage.
>
> it seems
>
> make foo-bar_defconfig
> make HOSTCC=clang CC=clang all
>
> works.
>
>> CC=clang seems to be fine.
>>
>> Best regards,
>>
>> --
>> FUKAUMI Naoki
>> Radxa Computer (Shenzhen) Co., Ltd.
should I check/make patches for other RK3xxx targets?
or should I avoid to use clang for defconfig?
Best regards,
--
FUKAUMI Naoki
Radxa Computer (Shenzhen) Co., Ltd.
On 7/16/24 11:09, Kever Yang wrote:
>
> On 2024/6/19 03:30, FUKAUMI Naoki wrote:
>> fix following error found by clang:
>>
>> CC arch/arm/mach-rockchip/cpu-info.o
>> arch/arm/mach-rockchip/cpu-info.c:23:13: error: incomplete definition
>> of type 'struct rockchip_cru'
>> 23 | switch (cru->glb_rst_st) {
>> | ~~~^
>> ./arch/arm/include/asm/arch-rockchip/clock.h:181:8: note: forward
>> declaration of 'struct rockchip_cru'
>> 181 | struct rockchip_cru;
>> | ^
>> 1 error generated.
>>
>> Signed-off-by: FUKAUMI Naoki <naoki at radxa.com>
> Reviewed-by: Kever Yang <kever.yang at rock-chips.com>
>
> Thanks,
> - Kever
>> ---
>> arch/arm/include/asm/arch-rockchip/cru.h | 2 ++
>> arch/arm/include/asm/arch-rockchip/cru_rk3588.h | 2 ++
>> 2 files changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/include/asm/arch-rockchip/cru.h
>> b/arch/arm/include/asm/arch-rockchip/cru.h
>> index 9778790f34..c3259b8e7c 100644
>> --- a/arch/arm/include/asm/arch-rockchip/cru.h
>> +++ b/arch/arm/include/asm/arch-rockchip/cru.h
>> @@ -17,6 +17,8 @@
>> # include <asm/arch-rockchip/cru_rk3399.h>
>> #elif defined(CONFIG_ROCKCHIP_RK3568)
>> #include <asm/arch-rockchip/cru_rk3568.h>
>> +#elif defined(CONFIG_ROCKCHIP_RK3588)
>> +#include <asm/arch-rockchip/cru_rk3588.h>
>> #endif
>> /* CRU_GLB_RST_ST */
>> diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h
>> b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h
>> index a0e54d3965..dad484813f 100644
>> --- a/arch/arm/include/asm/arch-rockchip/cru_rk3588.h
>> +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3588.h
>> @@ -92,6 +92,8 @@ struct rk3588_cru {
>> unsigned int pmuclkgate_con[9]; /* Address Offset: 0x0100 */
>> };
>> +#define rockchip_cru rk3588_cru
>> +
>> check_member(rk3588_cru, mode_con00, 0x280);
>> check_member(rk3588_cru, pmuclksel_con[1], 0x30304);
>
More information about the U-Boot
mailing list