On Fri, 2022-02-04 at 23:25 -0300, Fabio Estevam wrote:
> Hi Marcel,
> 
> On Fri, Feb 4, 2022 at 7:17 PM Marcel Ziswiler <[email protected]> wrote:
> 
> > +#if 0
> > +int board_phys_sdram_size(phys_size_t *bank1_size, phys_size_t *bank2_size)
> 
> Please remove this unused function.

Yes, this is left for after basic support got applied and I can work on 
properly supporting dynamic memory
sizing on the i.MX 8M Plus as well.

> > diff --git a/doc/board/toradex/verdin-imx8mm.rst 
> > b/doc/board/toradex/verdin-imx8mm.rst
> > index a11c82da2f..917bea82b7 100644
> > --- a/doc/board/toradex/verdin-imx8mm.rst
> > +++ b/doc/board/toradex/verdin-imx8mm.rst
> 
> The changes in this file should better be in a separate patch.

Yes, good catch. That one really slipped in. I will remove it again in a v4.

> Reviewed-by: Fabio Estevam <[email protected]>

Thanks, Fabio.

Reply via email to