[PATCH 3/3] ARM: boards: Add MyirTech MYD-YA15XC-T development board support

Alexander Shiyan eagle.alexander923 at gmail.com
Wed Aug 30 23:40:16 PDT 2023


Hello Ahmad.

> Thanks for your patch. A few comments below.
...
> > +ENTRY_FUNCTION(start_stm32mp15x_myir, r0, r1, r2)
> > +{
> > +     void *fdt;
> > +
> > +     stm32mp_cpu_lowlevel_init();
> > +
> > +     if (IS_ENABLED(CONFIG_DEBUG_LL))
> > +             setup_uart();
> > +
> > +     fdt = __dtb_z_stm32mp15x_myirtech_myd_start + get_runtime_offset();
> > +
> > +     stm32mp1_barebox_entry(fdt);
>
> Same question: Do you use an old TF-A or why do you need a specific entry
> point instead of barebox-stm32mp-generic-bl33.img?

Initially, I used the old version of TF-A, to facilitate the first
start-up of the board.
Now I'm using new version and FIP, so yes, this piece of code can be removed.

Thanks for the comments on the patch, I'll rework some parts of the code.

2 Sascha: Is it possible to commit the first two parts of the patch
now to minimize the overall work?



More information about the barebox mailing list