[U-Boot] [PATCH] sandbox: fix cpu property in test.dts for pytest

Simon Glass sjg at chromium.org
Tue Sep 17 05:47:55 UTC 2019


On Tue, 27 Aug 2019 at 02:43, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Tue, Aug 27, 2019 at 4:15 PM AKASHI Takahiro
> <takahiro.akashi at linaro.org> wrote:
> >
> > When I tried to run some new efi tests with pytest, efi_smbios_register()
> > triggered a segmentation fault. Here is the location where it happened:
> > efi_init_obj_list()
> >   efi_smbios_register()
> >     write_smbios_table()
> >       smbios_write_type4()
> >         smbios_write_type4_dm()
> >
> > where dev_get_parent_platdata() should return a pointer to struct
> > cpu_platdata, but it is actually NULL because any cpu device on
> > sandbox is attached to "root_driver."
> >
> > With this patch, this issue will be fixed by moving all the definitions
> > of cpus under "cpus" node so that they have a "cpu_bus" parent.
> >
> > Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> > ---
> >  arch/sandbox/dts/test.dts | 24 +++++++++++++-----------
> >  1 file changed, 13 insertions(+), 11 deletions(-)
> >
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

Reviewed-by: Simon Glass <sjg at chromium.org>

I assume that all tests still pass?


- Simon


More information about the U-Boot mailing list