[media] smiapp: Allow generic quirk registers
authorSakari Ailus <sakari.ailus@maxwell.research.nokia.com>
Thu, 26 Apr 2012 16:39:44 +0000 (13:39 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 20 May 2012 15:47:25 +0000 (12:47 -0300)
Implement more generic quirk registers than just limit and capability
registers. This comes with the expense of a little bit more access time so
these should be only used when really needed.

Signed-off-by: Sakari Ailus <sakari.ailus@maxwell.research.nokia.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/smiapp/smiapp-quirk.c
drivers/media/video/smiapp/smiapp-quirk.h
drivers/media/video/smiapp/smiapp-regs.c

index 81c2be3d47cb9eccdf06dff1ca3edb580e6a4cb8..55e87950dcea5ca23d9c482f3f3c93a3ffaf3ed9 100644 (file)
@@ -81,6 +81,52 @@ int smiapp_replace_limit_at(struct smiapp_sensor *sensor,
        return -EINVAL;
 }
 
+bool smiapp_quirk_reg(struct smiapp_sensor *sensor,
+                     u32 reg, u32 *val)
+{
+       struct i2c_client *client = v4l2_get_subdevdata(&sensor->src->sd);
+       const struct smia_reg *sreg;
+
+       if (!sensor->minfo.quirk)
+               return false;
+
+       sreg = sensor->minfo.quirk->regs;
+
+       if (!sreg)
+               return false;
+
+       while (sreg->type) {
+               u16 type = reg >> 16;
+               u16 reg16 = reg;
+
+               if (sreg->type != type || sreg->reg != reg16) {
+                       sreg++;
+                       continue;
+               }
+
+               switch ((u8)type) {
+               case SMIA_REG_8BIT:
+                       dev_dbg(&client->dev, "quirk: 0x%8.8x: 0x%2.2x\n",
+                               reg, sreg->val);
+                       break;
+               case SMIA_REG_16BIT:
+                       dev_dbg(&client->dev, "quirk: 0x%8.8x: 0x%4.4x\n",
+                               reg, sreg->val);
+                       break;
+               case SMIA_REG_32BIT:
+                       dev_dbg(&client->dev, "quirk: 0x%8.8x: 0x%8.8x\n",
+                               reg, sreg->val);
+                       break;
+               }
+
+               *val = sreg->val;
+
+               return true;
+       }
+
+       return false;
+}
+
 static int jt8ew9_limits(struct smiapp_sensor *sensor)
 {
        if (sensor->minfo.revision_number_major < 0x03)
index de82cdf43a89b1edf5c6649248b62f637c4a2bbe..f4dcaabaefe7066f99cd8d10ed22c5f4bfb62625 100644 (file)
@@ -41,6 +41,7 @@ struct smiapp_quirk {
        int (*post_poweron)(struct smiapp_sensor *sensor);
        int (*pre_streamon)(struct smiapp_sensor *sensor);
        int (*post_streamoff)(struct smiapp_sensor *sensor);
+       const struct smia_reg *regs;
        unsigned long flags;
 };
 
@@ -55,6 +56,15 @@ struct smiapp_reg_8 {
 
 void smiapp_replace_limit(struct smiapp_sensor *sensor,
                          u32 limit, u32 val);
+bool smiapp_quirk_reg(struct smiapp_sensor *sensor,
+                     u32 reg, u32 *val);
+
+#define SMIAPP_MK_QUIRK_REG(_reg, _val) \
+       {                               \
+               .type = (_reg >> 16),   \
+               .reg = (u16)_reg,       \
+               .val = _val,            \
+       }
 
 #define smiapp_call_quirk(_sensor, _quirk, ...)                                \
        (_sensor->minfo.quirk &&                                        \
index 9c430647b0476b4d8b53b1e3adb21baf5371b31a..b1812b17a4075150365a5ff7c6c1cd6b8512b69a 100644 (file)
@@ -172,6 +172,9 @@ static int __smiapp_read(struct smiapp_sensor *sensor, u32 reg, u32 *val,
            && len != SMIA_REG_32BIT)
                return -EINVAL;
 
+       if (smiapp_quirk_reg(sensor, reg, val))
+               goto found_quirk;
+
        if (len == SMIA_REG_8BIT && !only8)
                rval = ____smiapp_read(sensor, (u16)reg, len, val);
        else
@@ -179,6 +182,7 @@ static int __smiapp_read(struct smiapp_sensor *sensor, u32 reg, u32 *val,
        if (rval < 0)
                return rval;
 
+found_quirk:
        if (reg & SMIA_REG_FLAG_FLOAT)
                *val = float_to_u32_mul_1000000(client, *val);