[media] r820t: split the function that read cached regs
authorMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 10 Apr 2013 13:50:50 +0000 (10:50 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 17 Apr 2013 00:31:01 +0000 (21:31 -0300)
As we'll need to retrieve cached registers, make this
function explicit.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Tested-by: Antti Palosaari <crope@iki.fi>
drivers/media/tuners/r820t.c

index d5686e8a8bc650d576af1e5639e709a4a0ddbdc2..ef100ab3564da67616049ede7f8be5b8e18aeb0c 100644 (file)
@@ -402,15 +402,25 @@ static int r820t_write_reg(struct r820t_priv *priv, u8 reg, u8 val)
        return r820t_write(priv, reg, &val, 1);
 }
 
-static int r820t_write_reg_mask(struct r820t_priv *priv, u8 reg, u8 val,
-                               u8 bit_mask)
+static int r820t_read_cache_reg(struct r820t_priv *priv, int reg)
 {
-       int r = reg - REG_SHADOW_START;
+       reg -= REG_SHADOW_START;
 
-       if (r >= 0 && r < NUM_REGS)
-               val = (priv->regs[r] & ~bit_mask) | (val & bit_mask);
+       if (reg >= 0 && reg < NUM_REGS)
+               return priv->regs[reg];
        else
                return -EINVAL;
+}
+
+static int r820t_write_reg_mask(struct r820t_priv *priv, u8 reg, u8 val,
+                               u8 bit_mask)
+{
+       int rc = r820t_read_cache_reg(priv, reg);
+
+       if (rc < 0)
+               return rc;
+
+       val = (rc & ~bit_mask) | (val & bit_mask);
 
        return r820t_write(priv, reg, &val, 1);
 }