Hi Arnd,
thanks for the comments and sorry for the large 'To' list. In my v2 I'll
limit to Patrice Chotard (STi maintainer).
On Thu, Jan 28, 2021 at 02:08:51PM +0100, Arnd Bergmann wrote:
> On Thu, Jan 28, 2021 at 8:58 AM Alain Volmat <avolmat@xxxxxx> wrote:
> >
> > 4KOpen (B2264) is a board based on the STMicroelectronics STiH418 soc:
> > - 2GB DDR
> > - HDMI
> > - Ethernet 1000-BaseT
> > - PCIe (mini PCIe connector)
> > - MicroSD slot
> > - USB2 and USB3 connectors
> > - Sata
> > - 40 pins GPIO header
> >
> > Signed-off-by: Alain Volmat <avolmat@xxxxxx>
>
> The changelog text seems fine here.
>
> > +// SPDX-License-Identifier: GPL-2.0-only
> > +/*
> > + * Copyright (C) 2021 STMicroelectronics
> > + * Author: Alain Volmat <avolmat@xxxxxx>
> > + */
> > +/dts-v1/;
> > +#include "stih418.dtsi"
> > +#include <dt-bindings/gpio/gpio.h>
> > +/ {
> > + model = "STiH418 B2264";
> > + compatible = "st,stih418-b2264", "st,stih418";
> > +
> > + chosen {
> > + bootargs = "console=ttyAS0,115200 clk_ignore_unused";
> > + stdout-path = &sbc_serial0;
> > + };
>
> I don't think the bootargs should be there, both are just hacks
> when things don't work correctly. If they don't, then try to fix them
> first, by making sure that all clocks are properly listed, and that the
> stdou-path points the working console.
Indeed, bootargs can be cleaned up. console is not necessary, however
it's going to be hard to get rid of the clk_ignore_unused now and I'd
prefer to keep it now, in the same way as for the other sti DTs.
>
> > +&rng11 {
> > + status = "disabled";
> > +};
>
> Why do you disable the random number generator?
I shouldn't have putted this within the board DT. This rng11 is not
available on the STiH418.
In the v2 serie I'll had a separate commit to fix the stih418.dtsi instead.
Alain
>
> Arnd
