chromebook_coral.dts: multiple FSP-S entries for same parameters

Simon Glass sjg at chromium.org
Mon Sep 28 06:24:37 CEST 2020


Hi Wolfgang,

On Thu, 10 Sep 2020 at 08:00, Wolfgang Wallner
<wolfgang.wallner at br-automation.com> wrote:
>
> Hi Simon,
>
> chromebook_coral.dts contains multiple entries of fsps,pcie-root-port-en
> and fsps,pcie-rp-hot-plug.
>
> I don't think this is intentional, as the code parsing the FSP-S settings will
> only use one of those settings.
>
> regards, Wolfgang
>
>
> ------------------ 8< ---------------------------------
> chromebook_coral.dts:
>
> &fsp_s {
>         u-boot,dm-pre-proper;
>
>         fsps,ish-enable = <0>;
>         fsps,enable-sata = <0>;
>         fsps,pcie-root-port-en = [00 00 00 00 00 01];
>         fsps,pcie-rp-hot-plug = [00 00 00 00 00 01];
>
>         // ...
>
>     /* Enable WiFi */
>         fsps,pcie-root-port-en = [01 00 00 00 00 00];
>         fsps,pcie-rp-hot-plug = [00 00 00 00 00 00];
>
>         // ...
> };

Thanks for reporting this. I'll send a patch.

Regards,
SImon


More information about the U-Boot mailing list