Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-10 Thread Tom Rini
On Mon, Feb 10, 2020 at 12:03:26PM -0700, Simon Glass wrote: > Hi Stephen. > > On Mon, 10 Feb 2020 at 11:28, Stephen Warren wrote: > > > > On 2/8/20 8:21 AM, Bin Meng wrote: > > > On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: > > >> > > >> Some compilers produce a warning about 'child' bein

Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-10 Thread Simon Glass
Hi Stephen. On Mon, 10 Feb 2020 at 11:28, Stephen Warren wrote: > > On 2/8/20 8:21 AM, Bin Meng wrote: > > On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: > >> > >> Some compilers produce a warning about 'child' being used before init. > >> Silence this by setting to NULL at the start. > > >

Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-10 Thread Tom Rini
On Mon, Feb 10, 2020 at 11:27:58AM -0700, Stephen Warren wrote: > On 2/8/20 8:21 AM, Bin Meng wrote: > > On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: > > > > > > Some compilers produce a warning about 'child' being used before init. > > > Silence this by setting to NULL at the start. > > >

Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-10 Thread Stephen Warren
On 2/8/20 8:21 AM, Bin Meng wrote: On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: Some compilers produce a warning about 'child' being used before init. Silence this by setting to NULL at the start. Should be a compiler bug I think. Which compiler has such issue? gcc 7.2.1 (Linaro buil

Re: [PATCH] sandbox: p2sb: Silence compiler warning

2020-02-08 Thread Bin Meng
On Sat, Feb 8, 2020 at 10:53 PM Simon Glass wrote: > > Some compilers produce a warning about 'child' being used before init. > Silence this by setting to NULL at the start. Should be a compiler bug I think. Which compiler has such issue? > > Signed-off-by: Simon Glass > --- > > drivers/misc/p

[PATCH] sandbox: p2sb: Silence compiler warning

2020-02-08 Thread Simon Glass
Some compilers produce a warning about 'child' being used before init. Silence this by setting to NULL at the start. Signed-off-by: Simon Glass --- drivers/misc/p2sb_emul.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/p2sb_emul.c b/drivers/misc/p2sb_emul.c in