Re: [PATCH v10 1/3] test: fix test/dm/regmap.c

2021-04-20 Thread Tom Rini
On Sun, Apr 11, 2021 at 11:21:56AM +0200, Heinrich Schuchardt wrote: > regmap_read() only fills the first two bytes of val. The last two bytes are > random data from the stack. This means the test will fail randomly. > > For low endian systems we could simply initialize val to 0 and get correct >

Re: [PATCH v10 1/3] test: fix test/dm/regmap.c

2021-04-14 Thread Simon Glass
On Sun, 11 Apr 2021 at 10:23, Heinrich Schuchardt wrote: > > regmap_read() only fills the first two bytes of val. The last two bytes are > random data from the stack. This means the test will fail randomly. > > For low endian systems we could simply initialize val to 0 and get correct > results. B

[PATCH v10 1/3] test: fix test/dm/regmap.c

2021-04-11 Thread Heinrich Schuchardt
regmap_read() only fills the first two bytes of val. The last two bytes are random data from the stack. This means the test will fail randomly. For low endian systems we could simply initialize val to 0 and get correct results. But tests should not depend on endianness. So let's use a pointer conv