[U-Boot] [PATCH 2/2] libfdt: return correct value if #size-cells property is not present
sjg at chromium.org
Tue Aug 13 09:34:17 UTC 2019
On Thu, 1 Aug 2019 at 05:42, Matthias Brugger <matthias.bgg at gmail.com> wrote:
> Hi all,
> On 26/07/2019 11:13, matthias.bgg at kernel.org wrote:
> > From: Matthias Brugger <mbrugger at suse.com>
> > According to the device tree specification, the default value for
> > was not present.
> > This patch also makes fdt_address_cells() and fdt_size_cells() conform
> > to the behaviour documented in libfdt.h. The defaults are only returned
> > if fdt_getprop() returns -FDT_ERR_NOTFOUND, otherwise the actual error
> > is returned.
> > This is based on upstream commit:
> > aa7254d ("libfdt: return correct value if #size-cells property is not present")
> > but misses the test case part, as we don't implement them in u-boot.
> > Signed-off-by: Matthias Brugger <mbrugger at suse.com>
> After running these two patches through the CI  I realized that three test
> are failing:
> test/py sandbox
> test/py sandbox with clang
> test/py sandbox_flattree
> All three fail dm_test_fdt_translation() in the case "No translation for busses
> with #size-cells == 0" .
> Can anybody with more insight in the test infrastructure and the sandbox
> architecture help me to identify if this is
> a) a bug in the sandbox
> b) a bug in our test
> c) a bug in my patch
> I write this because I'm pretty sure that it is not option c), as we just stick
> to the specs here.
Can you check the test and see? It might well be that the test is wrong.
I hope we don't have tet code relying on this.
More information about the U-Boot