[PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil hans.verk...@cisco.com

drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain integer 
as NULL pointer
drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using plain integer 
as NULL pointer

Signed-off-by: Hans Verkuil hans.verk...@cisco.com
Cc: Antti Palosaari cr...@iki.fi
---
 drivers/media/dvb-frontends/drxk_hard.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/dvb-frontends/drxk_hard.c 
b/drivers/media/dvb-frontends/drxk_hard.c
index 082014d..d416c15 100644
--- a/drivers/media/dvb-frontends/drxk_hard.c
+++ b/drivers/media/dvb-frontends/drxk_hard.c
@@ -1083,7 +1083,7 @@ static int hi_cfg_command(struct drxk_state *state)
 SIO_HI_RA_RAM_PAR_1_PAR1_SEC_KEY);
if (status  0)
goto error;
-   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, 0);
+   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, NULL);
if (status  0)
goto error;
 
@@ -2781,7 +2781,7 @@ static int ConfigureI2CBridge(struct drxk_state *state, 
bool b_enable_bridge)
goto error;
}
 
-   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, 0);
+   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, NULL);
 
 error:
if (status  0)
-- 
1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Antti Palosaari

On 04.10.2013 17:01, Hans Verkuil wrote:

From: Hans Verkuil hans.verk...@cisco.com

drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain integer 
as NULL pointer
drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using plain integer 
as NULL pointer

Signed-off-by: Hans Verkuil hans.verk...@cisco.com


Reviewed-by: Antti Palosaari cr...@iki.fi



Cc: Antti Palosaari cr...@iki.fi
---
  drivers/media/dvb-frontends/drxk_hard.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/dvb-frontends/drxk_hard.c 
b/drivers/media/dvb-frontends/drxk_hard.c
index 082014d..d416c15 100644
--- a/drivers/media/dvb-frontends/drxk_hard.c
+++ b/drivers/media/dvb-frontends/drxk_hard.c
@@ -1083,7 +1083,7 @@ static int hi_cfg_command(struct drxk_state *state)
 SIO_HI_RA_RAM_PAR_1_PAR1_SEC_KEY);
if (status  0)
goto error;
-   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, 0);
+   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, NULL);
if (status  0)
goto error;

@@ -2781,7 +2781,7 @@ static int ConfigureI2CBridge(struct drxk_state *state, 
bool b_enable_bridge)
goto error;
}

-   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, 0);
+   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, NULL);

  error:
if (status  0)




--
http://palosaari.fi/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Michael Krufky
On Fri, Oct 4, 2013 at 10:40 AM, Antti Palosaari cr...@iki.fi wrote:
 On 04.10.2013 17:01, Hans Verkuil wrote:

 From: Hans Verkuil hans.verk...@cisco.com

 drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain
 integer as NULL pointer
 drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using plain
 integer as NULL pointer

 Signed-off-by: Hans Verkuil hans.verk...@cisco.com


 Reviewed-by: Antti Palosaari cr...@iki.fi



 Cc: Antti Palosaari cr...@iki.fi
 ---
   drivers/media/dvb-frontends/drxk_hard.c | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

 diff --git a/drivers/media/dvb-frontends/drxk_hard.c
 b/drivers/media/dvb-frontends/drxk_hard.c
 index 082014d..d416c15 100644
 --- a/drivers/media/dvb-frontends/drxk_hard.c
 +++ b/drivers/media/dvb-frontends/drxk_hard.c
 @@ -1083,7 +1083,7 @@ static int hi_cfg_command(struct drxk_state *state)
  SIO_HI_RA_RAM_PAR_1_PAR1_SEC_KEY);
 if (status  0)
 goto error;
 -   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, 0);
 +   status = hi_command(state, SIO_HI_RA_RAM_CMD_CONFIG, NULL);
 if (status  0)
 goto error;

 @@ -2781,7 +2781,7 @@ static int ConfigureI2CBridge(struct drxk_state
 *state, bool b_enable_bridge)
 goto error;
 }

 -   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, 0);
 +   status = hi_command(state, SIO_HI_RA_RAM_CMD_BRDCTRL, NULL);

   error:
 if (status  0)



 --
 http://palosaari.fi/

 --
 To unsubscribe from this list: send the line unsubscribe linux-media in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reviewed-by: Michael Krufky mkru...@linuxtv.org
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html