[PATCH v2] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
the variable chan is dereferenced in line 190, so it is no reason to check null again in line 198. Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/nouveau_fence.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b

[PATCH] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
On Tue, Jan 15, 2013 at 10:39:23AM +, David Howells wrote: > Cong Ding wrote: > > > the variable sender is dereferenced in line 190, so it is no reason to check > > null again in line 198. > > Did you mean "The variable 'chan'"? sorry, my fault. s

[PATCH v2] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
the variable chan is dereferenced in line 190, so it is no reason to check null again in line 198. Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/nouveau_fence.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b

Re: [PATCH] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
On Tue, Jan 15, 2013 at 10:39:23AM +, David Howells wrote: > Cong Ding wrote: > > > the variable sender is dereferenced in line 190, so it is no reason to check > > null again in line 198. > > Did you mean "The variable 'chan'"? sorry, my fault. s

[PATCH] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
the variable sender is dereferenced in line 190, so it is no reason to check null again in line 198. Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/nouveau_fence.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b

[PATCH] gpu: drm/gma500/mdfld_dsi_pkg_sender.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
the variable sender is dereferenced in line 536, so it is no reason to check null again in line 541. Signed-off-by: Cong Ding --- drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c b

[PATCH] gpu: drm/gma500/mdfld_dsi_output.c: remove unnecessary null pointer check

2013-01-15 Thread Cong Ding
the variable sender is dereferenced in line 95, so it is no reason to check null again in line 99. Signed-off-by: Cong Ding --- drivers/gpu/drm/gma500/mdfld_dsi_output.c |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_output.c b/drivers/gpu/drm

[PATCH] gpu: drm/nouveau/nouveau_fence.c: remove unnecessary null pointer check

2013-01-14 Thread Cong Ding
the variable sender is dereferenced in line 190, so it is no reason to check null again in line 198. Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/nouveau_fence.c |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b

[PATCH] gpu: drm/gma500/mdfld_dsi_pkg_sender.c: remove unnecessary null pointer check

2013-01-14 Thread Cong Ding
the variable sender is dereferenced in line 536, so it is no reason to check null again in line 541. Signed-off-by: Cong Ding --- drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_pkg_sender.c b

[PATCH] gpu: drm/gma500/mdfld_dsi_output.c: remove unnecessary null pointer check

2013-01-14 Thread Cong Ding
the variable sender is dereferenced in line 95, so it is no reason to check null again in line 99. Signed-off-by: Cong Ding --- drivers/gpu/drm/gma500/mdfld_dsi_output.c |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_output.c b/drivers/gpu/drm

[PATCH] gpu: drm/nouveau/core/engine/disp/dacnv50.c: fix var uninit issue

2012-12-07 Thread Cong Ding
the variable "ret" might be uninitialized in the "default" branch of "switch" Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/core/engine/disp/dacnv50.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/core/en

[PATCH] gpu: drm/nouveau/core/engine/disp/dacnv50.c: fix var uninit issue

2012-12-06 Thread Cong Ding
the variable "ret" might be uninitialized in the "default" branch of "switch" Signed-off-by: Cong Ding --- drivers/gpu/drm/nouveau/core/engine/disp/dacnv50.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/nouveau/core/en