Re: [PATCH] udoo_neo: fix udoo neo UNDEFINED

[Tommaso Merciai <tomm.merciai@gmail.com>]

On Sun, Jan 02, 2022 at 04:28:35PM -0300, Fabio Estevam wrote:
> Hi Tommaso,
> 
> On Sun, Jan 2, 2022 at 3:27 PM Tommaso Merciai <tomm.merciai@gmail.com> wrote:
> >
> > get_board_value function fails to get the right board configuration on
> > second time that is call. This patch move get_board_value function at
> > spl level, once is call, store the right configuration into gd static
> > variable gd->board_type and fix the previous error.
> > get_board_value is now call only one time.
> >
> > Signed-off-by: Tommaso Merciai <tomm.merciai@gmail.com>
> 
> I tested your patch on my udoo neo full, but it reports "BASIC" now:
> 
> U-Boot SPL 2022.01-rc4-00031-gc5c4ced998e3-dirty (Jan 02 2022 - 16:22:05 -0300)
> Trying to boot from MMC1
> 
> 
> U-Boot 2022.01-rc4-00031-gc5c4ced998e3-dirty (Jan 02 2022 - 16:22:05 -0300)
> 
> CPU:   Freescale i.MX6SX rev1.2 996 MHz (running at 792 MHz)
> CPU:   Extended Commercial temperature grade (-20C to 105C) at 59C
> Reset cause: WDOG
> Model: UDOO Neo Basic
> Board: UDOO Neo BASIC
> I2C:   ready
> DRAM:  1 GiB
> 
> (The size is correctly detected as 'full' though')

Hi Fabio,
Sent v2, I think this issue is fixed. Can you test it please.
The ploblem is that after spl, gd->board_type is re-initialized.
Using get_board_value inside board_early_init_f seems solve.

Let me know.
Thanks,
Tommaso Merciai