Dear Stefano Babic,
> On 29/08/2012 03:14, Marek Vasut wrote:
> > This functionality allows configuring SCRATCH0 and SCRATCH1 registers
> > to special values, which make the SPL hang after the CPU was properly
> > initialized.
> >
> > This is for bootstrap purposes only and MUST BE DISABLED for normal
> > operation.
> >
> > Signed-off-by: Marek Vasut <ma...@denx.de>
> > Cc: Wolfgang Denk <w...@denx.de>
> > Cc: Stefano Babic <sba...@denx.de>
> > Cc: Fabio Estevam <fabio.este...@freescale.com>
> > ---
>
> Hi Marek,
>
> > arch/arm/cpu/arm926ejs/mxs/Makefile | 3 ++
> > arch/arm/cpu/arm926ejs/mxs/mxs_init.h | 6 +++
> > arch/arm/cpu/arm926ejs/mxs/spl_boot.c | 2 +
> > arch/arm/cpu/arm926ejs/mxs/spl_debug.c | 63
> > ++++++++++++++++++++++++++++++++ 4 files changed, 74 insertions(+)
> > create mode 100644 arch/arm/cpu/arm926ejs/mxs/spl_debug.c
> >
> > diff --git a/arch/arm/cpu/arm926ejs/mxs/Makefile
> > b/arch/arm/cpu/arm926ejs/mxs/Makefile index eeecf89..e75eabd 100644
> > --- a/arch/arm/cpu/arm926ejs/mxs/Makefile
> > +++ b/arch/arm/cpu/arm926ejs/mxs/Makefile
> > @@ -29,6 +29,9 @@ COBJS = clock.o mxs.o iomux.o timer.o
> >
> > ifdef CONFIG_SPL_BUILD
> > COBJS += spl_boot.o spl_lradc_init.o spl_mem_init.o spl_power_init.o
> >
> > +ifdef CONFIG_MX28_DEBUG
> > +COBJS += spl_debug.o
> > +endif
> >
> > endif
> >
> > SRCS := $(START:.o=.S) $(COBJS:.o=.c)
> >
> > diff --git a/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
> > b/arch/arm/cpu/arm926ejs/mxs/mxs_init.h index 2ddc5bc..e6f837c 100644
> > --- a/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
> > +++ b/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
> > @@ -42,4 +42,10 @@ uint32_t mxs_mem_get_size(void);
> >
> > void mxs_lradc_init(void);
> > void mxs_lradc_enable_batt_measurement(void);
> >
> > +#ifdef CONFIG_MX28_DEBUG
> > +void mx28_common_spl_debug_halt(void);
> > +#else
> > +static inline void mx28_common_spl_debug_halt(void) {}
> > +#endif
> > +
> >
> > #endif /* __M28_INIT_H__ */
> >
> > diff --git a/arch/arm/cpu/arm926ejs/mxs/spl_boot.c
> > b/arch/arm/cpu/arm926ejs/mxs/spl_boot.c index ad66c57..f4f0c09 100644
> > --- a/arch/arm/cpu/arm926ejs/mxs/spl_boot.c
> > +++ b/arch/arm/cpu/arm926ejs/mxs/spl_boot.c
> > @@ -108,6 +108,8 @@ void mxs_common_spl_init(const iomux_cfg_t
> > *iomux_setup,
> >
> > data->boot_mode_idx = bootmode;
> >
> > + mx28_common_spl_debug_halt();
> > +
> >
> > mxs_power_wait_pswitch();
> >
> > }
>
> I admit I use the same trick when the first initialization fails and I
> want to check what happens. Make the SOC hanging helps, sure.
Yea ... so either way if it's merged or not, I hope it might help someone :)
> However, this is part of the porting / development. Why should we
> introduce this code into mainline ? In the official configuration file
> CONFIG_MX28_DEBUG must be off, and then all this stuff is dead code.
Indeed, I'll shift this decision on you ;-)
>
> Regards,
> Stefano
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
