[U-Boot] [PATCH 03/15] rockchip: rk3288: Print reset reason

[Michael Nazzareno Trimarchi]

Hi


On Wednesday, August 14, 2019, Wadim Egorov <w.egorov at phytec.de> wrote:

>
> On 14.08.19 11:40, Jagan Teki wrote:
> > On Mon, Aug 5, 2019 at 6:00 PM Kever Yang <kever.yang at rock-chips.com>
> wrote:
> >>
> >> On 2019/7/29 下午3:46, Jagan Teki wrote:
> >>> Print the reason for reset instead of storing it into
> >>> env variable in rk3288.
> >>>
> >>> This would help to find the reset reason directly
> >>> on U-Boot proper logs.
> >>>
> >>> Cc: Wadim Egorov <w.egorov at phytec.de>
> >>> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
> >>> ---
> >>>   arch/arm/mach-rockchip/rk3288-board.c | 2 +-
> >>>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/arch/arm/mach-rockchip/rk3288-board.c
> b/arch/arm/mach-rockchip/rk3288-board.c
> >>> index d3ec141fea..613264d7ee 100644
> >>> --- a/arch/arm/mach-rockchip/rk3288-board.c
> >>> +++ b/arch/arm/mach-rockchip/rk3288-board.c
> >>> @@ -72,7 +72,7 @@ static void rk3288_detect_reset_reason(void)
> >>>               reason = "unknown reset";
> >>>       }
> >>>
> >>> -     env_set("reset_reason", reason);
> >>> +     printf("Reset cause: %s\n", reason);
> >>
> >> Why this need to set as env before? I didn't touch this code when I
> migrate
> > Don't know the proper reason for this, ie why I have CCed Wadim.
> >
> > Wadim, any comments?
> If we put the reset_reason inside an environment variable we can later
> reuse it from a script, e.g. changing the boot behavior for special
> cases. Right now I have no usecase for it. So from my side you can just
> print the reason instead of setting the environment. AFAIR I copied the
> env-method from another board.
> btw, board/xilinx/zynqmp/zynqmp.c is doing both.


The reason is totally valid and we don't know even is already use in some
deploy. Add the print without remove the environment make more sense to me

Michael


-- 
Sent from Gmail Mobile