Re: [Mesa-dev] [PATCH mesa] intel/tools: add missing variable initialisation

2018-08-08 Thread Eric Engestrom
On Wednesday, 2018-08-08 15:29:49 +0100, Lionel Landwerlin wrote: > Reviewed-by: Lionel Landwerlin > > bo_addr doesn't have a warning? Nope; I guess bo_type guards it perfectly ;) > > On 08/08/18 15:27, Eric Engestrom wrote: > > Cc: Lionel Landwerlin > > Signed-off-by: Eric Engestrom > > ---

Re: [Mesa-dev] [PATCH mesa] intel/tools: add missing variable initialisation

2018-08-08 Thread Eric Engestrom
On Wednesday, 2018-08-08 15:27:13 +0100, Eric Engestrom wrote: > Cc: Lionel Landwerlin Actually: Cc: Jason Ekstrand Fixes: 6a60beba4089315685b8 "intel/tools: Add an error state to aub translator" > Signed-off-by: Eric Engestrom > --- > src/intel/tools/error2aub.c | 2 +- > 1 file changed, 1 i

Re: [Mesa-dev] [PATCH mesa] intel/tools: add missing variable initialisation

2018-08-08 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin bo_addr doesn't have a warning? On 08/08/18 15:27, Eric Engestrom wrote: Cc: Lionel Landwerlin Signed-off-by: Eric Engestrom --- src/intel/tools/error2aub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/tools/error2aub.c b/src

[Mesa-dev] [PATCH mesa] intel/tools: add missing variable initialisation

2018-08-08 Thread Eric Engestrom
Cc: Lionel Landwerlin Signed-off-by: Eric Engestrom --- src/intel/tools/error2aub.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/tools/error2aub.c b/src/intel/tools/error2aub.c index 3407dcec0b795168c4c8..2030593691cc4ba6dba0 100644 --- a/src/intel/tools/error2au