All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/radeon: use NULL instead of zero
@ 2014-05-03 13:55 Michele CURTI
  2014-05-05 12:53 ` Deucher, Alexander
  0 siblings, 1 reply; 6+ messages in thread
From: Michele CURTI @ 2014-05-03 13:55 UTC (permalink / raw)
  To: alexander.deucher, christian.koenig, airlied; +Cc: linux-next


Use NULL instead of zero for NULL pointers

Signed-off-by: Michele Curti <michele.curti@gmail.com>
---
 drivers/gpu/drm/radeon/clearstate_cayman.h | 8 ++++----
 drivers/gpu/drm/radeon/clearstate_ci.h     | 4 ++--
 drivers/gpu/drm/radeon/clearstate_si.h     | 4 ++--
 drivers/gpu/drm/radeon/radeon_object.c     | 2 +-
 drivers/gpu/drm/radeon/radeon_object.h     | 2 +-
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/radeon/clearstate_cayman.h b/drivers/gpu/drm/radeon/clearstate_cayman.h
index aa908c5..ee428ac 100644
--- a/drivers/gpu/drm/radeon/clearstate_cayman.h
+++ b/drivers/gpu/drm/radeon/clearstate_cayman.h
@@ -1050,7 +1050,7 @@ static const struct cs_extent_def SECT_CONTEXT_defs[] =
     {SECT_CONTEXT_def_5, 0x0000a29e, 5 },
     {SECT_CONTEXT_def_6, 0x0000a2a5, 56 },
     {SECT_CONTEXT_def_7, 0x0000a2de, 290 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const u32 SECT_CLEAR_def_1[] =
 {
@@ -1061,7 +1061,7 @@ static const u32 SECT_CLEAR_def_1[] =
 static const struct cs_extent_def SECT_CLEAR_defs[] =
 {
     {SECT_CLEAR_def_1, 0x0000ffc0, 3 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const u32 SECT_CTRLCONST_def_1[] =
 {
@@ -1071,11 +1071,11 @@ static const u32 SECT_CTRLCONST_def_1[] =
 static const struct cs_extent_def SECT_CTRLCONST_defs[] =
 {
     {SECT_CTRLCONST_def_1, 0x0000f3fc, 2 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def cayman_cs_data[] = {
     { SECT_CONTEXT_defs, SECT_CONTEXT },
     { SECT_CLEAR_defs, SECT_CLEAR },
     { SECT_CTRLCONST_defs, SECT_CTRLCONST },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
diff --git a/drivers/gpu/drm/radeon/clearstate_ci.h b/drivers/gpu/drm/radeon/clearstate_ci.h
index c3982f9..1673c34 100644
--- a/drivers/gpu/drm/radeon/clearstate_ci.h
+++ b/drivers/gpu/drm/radeon/clearstate_ci.h
@@ -936,9 +936,9 @@ static const struct cs_extent_def ci_SECT_CONTEXT_defs[] =
     {ci_SECT_CONTEXT_def_5, 0x0000a2a0, 2 },
     {ci_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
     {ci_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def ci_cs_data[] = {
     { ci_SECT_CONTEXT_defs, SECT_CONTEXT },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
diff --git a/drivers/gpu/drm/radeon/clearstate_si.h b/drivers/gpu/drm/radeon/clearstate_si.h
index b994cb2..60474b8 100644
--- a/drivers/gpu/drm/radeon/clearstate_si.h
+++ b/drivers/gpu/drm/radeon/clearstate_si.h
@@ -933,9 +933,9 @@ static const struct cs_extent_def si_SECT_CONTEXT_defs[] =
     {si_SECT_CONTEXT_def_5, 0x0000a2a1, 1 },
     {si_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
     {si_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def si_cs_data[] = {
     { si_SECT_CONTEXT_defs, SECT_CONTEXT },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
index 19bec0d..95197aa 100644
--- a/drivers/gpu/drm/radeon/radeon_object.c
+++ b/drivers/gpu/drm/radeon/radeon_object.c
@@ -722,7 +722,7 @@ int radeon_bo_wait(struct radeon_bo *bo, u32 *mem_type, bool no_wait)
 {
 	int r;
 
-	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, 0);
+	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, NULL);
 	if (unlikely(r != 0))
 		return r;
 	spin_lock(&bo->tbo.bdev->fence_lock);
diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h
index 9e7b25a..5a873f3 100644
--- a/drivers/gpu/drm/radeon/radeon_object.h
+++ b/drivers/gpu/drm/radeon/radeon_object.h
@@ -65,7 +65,7 @@ static inline int radeon_bo_reserve(struct radeon_bo *bo, bool no_intr)
 {
 	int r;
 
-	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, 0);
+	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, NULL);
 	if (unlikely(r != 0)) {
 		if (r != -ERESTARTSYS)
 			dev_err(bo->rdev->dev, "%p reserve failed\n", bo);
-- 
1.9.2

Regards,
Michele

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [PATCH] drm/radeon: use NULL instead of zero
  2014-05-03 13:55 [PATCH] drm/radeon: use NULL instead of zero Michele CURTI
@ 2014-05-05 12:53 ` Deucher, Alexander
  2014-05-05 20:26   ` [PATCH v2 0/2] " Michele CURTI
                     ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Deucher, Alexander @ 2014-05-05 12:53 UTC (permalink / raw)
  To: Michele CURTI, Koenig, Christian, airlied@linux.ie
  Cc: linux-next@vger.kernel.org

> -----Original Message-----
> From: Michele CURTI [mailto:michele.curti@gmail.com]
> Sent: Saturday, May 03, 2014 9:55 AM
> To: Deucher, Alexander; Koenig, Christian; airlied@linux.ie
> Cc: linux-next@vger.kernel.org
> Subject: [PATCH] drm/radeon: use NULL instead of zero
> 
> 
> Use NULL instead of zero for NULL pointers
> 
> Signed-off-by: Michele Curti <michele.curti@gmail.com>

Can you break this into two patches?  One that changes that clearstate headers and one that changes object functions?

Thanks,

Alex

> ---
>  drivers/gpu/drm/radeon/clearstate_cayman.h | 8 ++++----
>  drivers/gpu/drm/radeon/clearstate_ci.h     | 4 ++--
>  drivers/gpu/drm/radeon/clearstate_si.h     | 4 ++--
>  drivers/gpu/drm/radeon/radeon_object.c     | 2 +-
>  drivers/gpu/drm/radeon/radeon_object.h     | 2 +-
>  5 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/gpu/drm/radeon/clearstate_cayman.h
> b/drivers/gpu/drm/radeon/clearstate_cayman.h
> index aa908c5..ee428ac 100644
> --- a/drivers/gpu/drm/radeon/clearstate_cayman.h
> +++ b/drivers/gpu/drm/radeon/clearstate_cayman.h
> @@ -1050,7 +1050,7 @@ static const struct cs_extent_def
> SECT_CONTEXT_defs[] =
>      {SECT_CONTEXT_def_5, 0x0000a29e, 5 },
>      {SECT_CONTEXT_def_6, 0x0000a2a5, 56 },
>      {SECT_CONTEXT_def_7, 0x0000a2de, 290 },
> -    { 0, 0, 0 }
> +    {NULL, 0, 0 }
>  };
>  static const u32 SECT_CLEAR_def_1[] =
>  {
> @@ -1061,7 +1061,7 @@ static const u32 SECT_CLEAR_def_1[] =
>  static const struct cs_extent_def SECT_CLEAR_defs[] =
>  {
>      {SECT_CLEAR_def_1, 0x0000ffc0, 3 },
> -    { 0, 0, 0 }
> +    {NULL, 0, 0 }
>  };
>  static const u32 SECT_CTRLCONST_def_1[] =
>  {
> @@ -1071,11 +1071,11 @@ static const u32 SECT_CTRLCONST_def_1[] =
>  static const struct cs_extent_def SECT_CTRLCONST_defs[] =
>  {
>      {SECT_CTRLCONST_def_1, 0x0000f3fc, 2 },
> -    { 0, 0, 0 }
> +    {NULL, 0, 0 }
>  };
>  static const struct cs_section_def cayman_cs_data[] = {
>      { SECT_CONTEXT_defs, SECT_CONTEXT },
>      { SECT_CLEAR_defs, SECT_CLEAR },
>      { SECT_CTRLCONST_defs, SECT_CTRLCONST },
> -    { 0, SECT_NONE }
> +    { NULL, SECT_NONE }
>  };
> diff --git a/drivers/gpu/drm/radeon/clearstate_ci.h
> b/drivers/gpu/drm/radeon/clearstate_ci.h
> index c3982f9..1673c34 100644
> --- a/drivers/gpu/drm/radeon/clearstate_ci.h
> +++ b/drivers/gpu/drm/radeon/clearstate_ci.h
> @@ -936,9 +936,9 @@ static const struct cs_extent_def
> ci_SECT_CONTEXT_defs[] =
>      {ci_SECT_CONTEXT_def_5, 0x0000a2a0, 2 },
>      {ci_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
>      {ci_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
> -    { 0, 0, 0 }
> +    {NULL, 0, 0 }
>  };
>  static const struct cs_section_def ci_cs_data[] = {
>      { ci_SECT_CONTEXT_defs, SECT_CONTEXT },
> -    { 0, SECT_NONE }
> +    { NULL, SECT_NONE }
>  };
> diff --git a/drivers/gpu/drm/radeon/clearstate_si.h
> b/drivers/gpu/drm/radeon/clearstate_si.h
> index b994cb2..60474b8 100644
> --- a/drivers/gpu/drm/radeon/clearstate_si.h
> +++ b/drivers/gpu/drm/radeon/clearstate_si.h
> @@ -933,9 +933,9 @@ static const struct cs_extent_def
> si_SECT_CONTEXT_defs[] =
>      {si_SECT_CONTEXT_def_5, 0x0000a2a1, 1 },
>      {si_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
>      {si_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
> -    { 0, 0, 0 }
> +    {NULL, 0, 0 }
>  };
>  static const struct cs_section_def si_cs_data[] = {
>      { si_SECT_CONTEXT_defs, SECT_CONTEXT },
> -    { 0, SECT_NONE }
> +    { NULL, SECT_NONE }
>  };
> diff --git a/drivers/gpu/drm/radeon/radeon_object.c
> b/drivers/gpu/drm/radeon/radeon_object.c
> index 19bec0d..95197aa 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.c
> +++ b/drivers/gpu/drm/radeon/radeon_object.c
> @@ -722,7 +722,7 @@ int radeon_bo_wait(struct radeon_bo *bo, u32
> *mem_type, bool no_wait)
>  {
>  	int r;
> 
> -	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, 0);
> +	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, NULL);
>  	if (unlikely(r != 0))
>  		return r;
>  	spin_lock(&bo->tbo.bdev->fence_lock);
> diff --git a/drivers/gpu/drm/radeon/radeon_object.h
> b/drivers/gpu/drm/radeon/radeon_object.h
> index 9e7b25a..5a873f3 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.h
> +++ b/drivers/gpu/drm/radeon/radeon_object.h
> @@ -65,7 +65,7 @@ static inline int radeon_bo_reserve(struct radeon_bo
> *bo, bool no_intr)
>  {
>  	int r;
> 
> -	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, 0);
> +	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, NULL);
>  	if (unlikely(r != 0)) {
>  		if (r != -ERESTARTSYS)
>  			dev_err(bo->rdev->dev, "%p reserve failed\n", bo);
> --
> 1.9.2
> 
> Regards,
> Michele

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2 0/2] drm/radeon: use NULL instead of zero
  2014-05-05 12:53 ` Deucher, Alexander
@ 2014-05-05 20:26   ` Michele CURTI
  2014-05-06 16:28     ` Deucher, Alexander
  2014-05-05 20:29   ` [PATCH v2 1/2] drm/radeon: use NULL instead of zero in object functions Michele CURTI
  2014-05-05 20:32   ` [PATCH v2 2/2] drm/radeon: use NULL instead of zero in clearstate headers Michele CURTI
  2 siblings, 1 reply; 6+ messages in thread
From: Michele CURTI @ 2014-05-05 20:26 UTC (permalink / raw)
  To: Deucher, Alexander
  Cc: Koenig, Christian, airlied@linux.ie, linux-next@vger.kernel.org

This patches replace 0 with NULL when NULL pointers in radeon driver.
The first in object functions.
the second in clearstate headers.

Changes from v1:
- splitted in two separated patches

On Mon, May 05, 2014 at 12:53:58PM +0000, Deucher, Alexander wrote:
> > -----Original Message-----
> > From: Michele CURTI [mailto:michele.curti@gmail.com]
> > Sent: Saturday, May 03, 2014 9:55 AM
> > To: Deucher, Alexander; Koenig, Christian; airlied@linux.ie
> > Cc: linux-next@vger.kernel.org
> > Subject: [PATCH] drm/radeon: use NULL instead of zero
> > 
> > 
> > Use NULL instead of zero for NULL pointers
> > 
> > Signed-off-by: Michele Curti <michele.curti@gmail.com>
> 
> Can you break this into two patches?  One that changes that clearstate headers and one that changes object functions?
> 
> Thanks,
> 
> Alex
>

Sure!
Or at least, I'm going to try..
Michele

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2 1/2] drm/radeon: use NULL instead of zero in object functions
  2014-05-05 12:53 ` Deucher, Alexander
  2014-05-05 20:26   ` [PATCH v2 0/2] " Michele CURTI
@ 2014-05-05 20:29   ` Michele CURTI
  2014-05-05 20:32   ` [PATCH v2 2/2] drm/radeon: use NULL instead of zero in clearstate headers Michele CURTI
  2 siblings, 0 replies; 6+ messages in thread
From: Michele CURTI @ 2014-05-05 20:29 UTC (permalink / raw)
  To: Deucher, Alexander
  Cc: Koenig, Christian, airlied@linux.ie, linux-next@vger.kernel.org

drm/radeon: use NULL instead of zero in object functions

Signed-off-by: Michele Curti <michele.curti@gmail.com>
---
 drivers/gpu/drm/radeon/radeon_object.c | 2 +-
 drivers/gpu/drm/radeon/radeon_object.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
index 19bec0d..95197aa 100644
--- a/drivers/gpu/drm/radeon/radeon_object.c
+++ b/drivers/gpu/drm/radeon/radeon_object.c
@@ -722,7 +722,7 @@ int radeon_bo_wait(struct radeon_bo *bo, u32 *mem_type, bool no_wait)
 {
 	int r;
 
-	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, 0);
+	r = ttm_bo_reserve(&bo->tbo, true, no_wait, false, NULL);
 	if (unlikely(r != 0))
 		return r;
 	spin_lock(&bo->tbo.bdev->fence_lock);
diff --git a/drivers/gpu/drm/radeon/radeon_object.h b/drivers/gpu/drm/radeon/radeon_object.h
index 9e7b25a..5a873f3 100644
--- a/drivers/gpu/drm/radeon/radeon_object.h
+++ b/drivers/gpu/drm/radeon/radeon_object.h
@@ -65,7 +65,7 @@ static inline int radeon_bo_reserve(struct radeon_bo *bo, bool no_intr)
 {
 	int r;
 
-	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, 0);
+	r = ttm_bo_reserve(&bo->tbo, !no_intr, false, false, NULL);
 	if (unlikely(r != 0)) {
 		if (r != -ERESTARTSYS)
 			dev_err(bo->rdev->dev, "%p reserve failed\n", bo);
-- 
1.9.2

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH v2 2/2] drm/radeon: use NULL instead of zero in clearstate headers
  2014-05-05 12:53 ` Deucher, Alexander
  2014-05-05 20:26   ` [PATCH v2 0/2] " Michele CURTI
  2014-05-05 20:29   ` [PATCH v2 1/2] drm/radeon: use NULL instead of zero in object functions Michele CURTI
@ 2014-05-05 20:32   ` Michele CURTI
  2 siblings, 0 replies; 6+ messages in thread
From: Michele CURTI @ 2014-05-05 20:32 UTC (permalink / raw)
  To: Deucher, Alexander
  Cc: Koenig, Christian, airlied@linux.ie, linux-next@vger.kernel.org

drm/radeon: use NULL instead of zero in clearstate headers

Signed-off-by: Michele Curti <michele.curti@gmail.com>
---
 drivers/gpu/drm/radeon/clearstate_cayman.h | 8 ++++----
 drivers/gpu/drm/radeon/clearstate_ci.h     | 4 ++--
 drivers/gpu/drm/radeon/clearstate_si.h     | 4 ++--
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/radeon/clearstate_cayman.h b/drivers/gpu/drm/radeon/clearstate_cayman.h
index aa908c5..ee428ac 100644
--- a/drivers/gpu/drm/radeon/clearstate_cayman.h
+++ b/drivers/gpu/drm/radeon/clearstate_cayman.h
@@ -1050,7 +1050,7 @@ static const struct cs_extent_def SECT_CONTEXT_defs[] =
     {SECT_CONTEXT_def_5, 0x0000a29e, 5 },
     {SECT_CONTEXT_def_6, 0x0000a2a5, 56 },
     {SECT_CONTEXT_def_7, 0x0000a2de, 290 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const u32 SECT_CLEAR_def_1[] =
 {
@@ -1061,7 +1061,7 @@ static const u32 SECT_CLEAR_def_1[] =
 static const struct cs_extent_def SECT_CLEAR_defs[] =
 {
     {SECT_CLEAR_def_1, 0x0000ffc0, 3 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const u32 SECT_CTRLCONST_def_1[] =
 {
@@ -1071,11 +1071,11 @@ static const u32 SECT_CTRLCONST_def_1[] =
 static const struct cs_extent_def SECT_CTRLCONST_defs[] =
 {
     {SECT_CTRLCONST_def_1, 0x0000f3fc, 2 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def cayman_cs_data[] = {
     { SECT_CONTEXT_defs, SECT_CONTEXT },
     { SECT_CLEAR_defs, SECT_CLEAR },
     { SECT_CTRLCONST_defs, SECT_CTRLCONST },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
diff --git a/drivers/gpu/drm/radeon/clearstate_ci.h b/drivers/gpu/drm/radeon/clearstate_ci.h
index c3982f9..1673c34 100644
--- a/drivers/gpu/drm/radeon/clearstate_ci.h
+++ b/drivers/gpu/drm/radeon/clearstate_ci.h
@@ -936,9 +936,9 @@ static const struct cs_extent_def ci_SECT_CONTEXT_defs[] =
     {ci_SECT_CONTEXT_def_5, 0x0000a2a0, 2 },
     {ci_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
     {ci_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def ci_cs_data[] = {
     { ci_SECT_CONTEXT_defs, SECT_CONTEXT },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
diff --git a/drivers/gpu/drm/radeon/clearstate_si.h b/drivers/gpu/drm/radeon/clearstate_si.h
index b994cb2..60474b8 100644
--- a/drivers/gpu/drm/radeon/clearstate_si.h
+++ b/drivers/gpu/drm/radeon/clearstate_si.h
@@ -933,9 +933,9 @@ static const struct cs_extent_def si_SECT_CONTEXT_defs[] =
     {si_SECT_CONTEXT_def_5, 0x0000a2a1, 1 },
     {si_SECT_CONTEXT_def_6, 0x0000a2a3, 1 },
     {si_SECT_CONTEXT_def_7, 0x0000a2a5, 233 },
-    { 0, 0, 0 }
+    {NULL, 0, 0 }
 };
 static const struct cs_section_def si_cs_data[] = {
     { si_SECT_CONTEXT_defs, SECT_CONTEXT },
-    { 0, SECT_NONE }
+    { NULL, SECT_NONE }
 };
-- 
1.9.2

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [PATCH v2 0/2] drm/radeon: use NULL instead of zero
  2014-05-05 20:26   ` [PATCH v2 0/2] " Michele CURTI
@ 2014-05-06 16:28     ` Deucher, Alexander
  0 siblings, 0 replies; 6+ messages in thread
From: Deucher, Alexander @ 2014-05-06 16:28 UTC (permalink / raw)
  To: Michele CURTI
  Cc: Koenig, Christian, airlied@linux.ie, linux-next@vger.kernel.org

> -----Original Message-----
> From: Michele CURTI [mailto:michele.curti@gmail.com]
> Sent: Monday, May 05, 2014 4:27 PM
> To: Deucher, Alexander
> Cc: Koenig, Christian; airlied@linux.ie; linux-next@vger.kernel.org
> Subject: [PATCH v2 0/2] drm/radeon: use NULL instead of zero
> 
> This patches replace 0 with NULL when NULL pointers in radeon driver.
> The first in object functions.
> the second in clearstate headers.
> 
> Changes from v1:
> - splitted in two separated patches

For the series:

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

We'll pick these up for 3.16.

Alex

> 
> On Mon, May 05, 2014 at 12:53:58PM +0000, Deucher, Alexander wrote:
> > > -----Original Message-----
> > > From: Michele CURTI [mailto:michele.curti@gmail.com]
> > > Sent: Saturday, May 03, 2014 9:55 AM
> > > To: Deucher, Alexander; Koenig, Christian; airlied@linux.ie
> > > Cc: linux-next@vger.kernel.org
> > > Subject: [PATCH] drm/radeon: use NULL instead of zero
> > >
> > >
> > > Use NULL instead of zero for NULL pointers
> > >
> > > Signed-off-by: Michele Curti <michele.curti@gmail.com>
> >
> > Can you break this into two patches?  One that changes that clearstate
> headers and one that changes object functions?
> >
> > Thanks,
> >
> > Alex
> >
> 
> Sure!
> Or at least, I'm going to try..
> Michele

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-05-06 16:29 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-03 13:55 [PATCH] drm/radeon: use NULL instead of zero Michele CURTI
2014-05-05 12:53 ` Deucher, Alexander
2014-05-05 20:26   ` [PATCH v2 0/2] " Michele CURTI
2014-05-06 16:28     ` Deucher, Alexander
2014-05-05 20:29   ` [PATCH v2 1/2] drm/radeon: use NULL instead of zero in object functions Michele CURTI
2014-05-05 20:32   ` [PATCH v2 2/2] drm/radeon: use NULL instead of zero in clearstate headers Michele CURTI

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.