Message ID | 20240531141837.13189-21-linux.amoon@gmail.com |
---|---|
State | RFC |
Delegated to: | Kever Yang |
Headers | show |
Series | Enable display cpuinfo to get the reset cause | expand |
Hello Anand, On 2024-05-31 16:18, Anand Moon wrote: > From: Anand Moon <anand@edgeble.ai> > > Read the reset cause from clock reset unit for RV1126 SoC. > > Cc: Jagan Teki <jagan@edgeble.ai> > Signed-off-by: Anand Moon <anand@edgeble.ai> > Signed-off-by: Anand Moon <linux.amoon@gmail.com> > --- > arch/arm/include/asm/arch-rockchip/cru.h | 2 ++ > arch/arm/mach-rockchip/cpu-info.c | 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 e886adf515..e1a8d8adad 100644 > --- a/arch/arm/include/asm/arch-rockchip/cru.h > +++ b/arch/arm/include/asm/arch-rockchip/cru.h > @@ -29,6 +29,8 @@ > #include <asm/arch-rockchip/cru_rk3368.h> Although not related to this patch directly, a single space character should be added between "#" and "include", for consistency. > #elif defined(CONFIG_ROCKCHIP_RV1108) > #include <asm/arch-rockchip/cru_rv1108.h> The same comment as above. > +#elif defined(CONFIG_ROCKCHIP_RV1126) > +#include <asm/arch-rockchip/cru_rv1126.h> The same comment as above, but this time related to this patch. > #endif > > /* CRU_GLB_RST_ST */ > diff --git a/arch/arm/mach-rockchip/cpu-info.c > b/arch/arm/mach-rockchip/cpu-info.c > index b2cfdade13..cb4ac39884 100644 > --- a/arch/arm/mach-rockchip/cpu-info.c > +++ b/arch/arm/mach-rockchip/cpu-info.c > @@ -32,6 +32,8 @@ char *get_reset_cause(void) > struct rk3368_cru *cru = rockchip_get_cru(); > #elif IS_ENABLED(CONFIG_ROCKCHIP_RV1108) > struct rv1108_cru *cru = rockchip_get_cru(); > +#elif IS_ENABLED(CONFIG_ROCKCHIP_RV1126) > + struct rv1126_cru *cru = rockchip_get_cru(); > #endif > char *cause = NULL;
Hi Dragan, On Sat, 1 Jun 2024 at 03:23, Dragan Simic <dsimic@manjaro.org> wrote: > > Hello Anand, > > On 2024-05-31 16:18, Anand Moon wrote: > > From: Anand Moon <anand@edgeble.ai> > > > > Read the reset cause from clock reset unit for RV1126 SoC. > > > > Cc: Jagan Teki <jagan@edgeble.ai> > > Signed-off-by: Anand Moon <anand@edgeble.ai> > > Signed-off-by: Anand Moon <linux.amoon@gmail.com> > > --- > > arch/arm/include/asm/arch-rockchip/cru.h | 2 ++ > > arch/arm/mach-rockchip/cpu-info.c | 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 e886adf515..e1a8d8adad 100644 > > --- a/arch/arm/include/asm/arch-rockchip/cru.h > > +++ b/arch/arm/include/asm/arch-rockchip/cru.h > > @@ -29,6 +29,8 @@ > > #include <asm/arch-rockchip/cru_rk3368.h> > > Although not related to this patch directly, a single space character > should be added between "#" and "include", for consistency. > Not sure of this. > > #elif defined(CONFIG_ROCKCHIP_RV1108) > > #include <asm/arch-rockchip/cru_rv1108.h> > > The same comment as above. > > > +#elif defined(CONFIG_ROCKCHIP_RV1126) > > +#include <asm/arch-rockchip/cru_rv1126.h> > > The same comment as above, but this time related to this patch. > > > #endif Let me check this again if needed. I will update in the next version. Thanks -Anand
diff --git a/arch/arm/include/asm/arch-rockchip/cru.h b/arch/arm/include/asm/arch-rockchip/cru.h index e886adf515..e1a8d8adad 100644 --- a/arch/arm/include/asm/arch-rockchip/cru.h +++ b/arch/arm/include/asm/arch-rockchip/cru.h @@ -29,6 +29,8 @@ #include <asm/arch-rockchip/cru_rk3368.h> #elif defined(CONFIG_ROCKCHIP_RV1108) #include <asm/arch-rockchip/cru_rv1108.h> +#elif defined(CONFIG_ROCKCHIP_RV1126) +#include <asm/arch-rockchip/cru_rv1126.h> #endif /* CRU_GLB_RST_ST */ diff --git a/arch/arm/mach-rockchip/cpu-info.c b/arch/arm/mach-rockchip/cpu-info.c index b2cfdade13..cb4ac39884 100644 --- a/arch/arm/mach-rockchip/cpu-info.c +++ b/arch/arm/mach-rockchip/cpu-info.c @@ -32,6 +32,8 @@ char *get_reset_cause(void) struct rk3368_cru *cru = rockchip_get_cru(); #elif IS_ENABLED(CONFIG_ROCKCHIP_RV1108) struct rv1108_cru *cru = rockchip_get_cru(); +#elif IS_ENABLED(CONFIG_ROCKCHIP_RV1126) + struct rv1126_cru *cru = rockchip_get_cru(); #endif char *cause = NULL;