On 23 March 2016 at 01:23, Ezequiel Garcia <ezequiel@xxxxxxxxxxxxxxxxxxxx> wrote: > From: Kevin Fitch <kfitch42@xxxxxxxxx> > > It's been reported that CJC7113 devices are returning > all 1s when reading register 0: > > "1111111111111111" found @ 0x4a (stk1160) > > This new device is apparently compatible with SA7113, so let's > add a quirk to allow its autodetection. Given there isn't > any known differences with SAA7113, this commit does not > introduces a new saa711x_model value. > > Reported-by: Philippe Desrochers <desrochers.philippe@xxxxxxxxx> > Signed-off-by: Kevin Fitch <kfitch42@xxxxxxxxx> > Signed-off-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxxxxxxxxx> > --- > drivers/media/i2c/saa7115.c | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/drivers/media/i2c/saa7115.c b/drivers/media/i2c/saa7115.c > index 24d2b76dbe97..04f266d0a1ef 100644 > --- a/drivers/media/i2c/saa7115.c > +++ b/drivers/media/i2c/saa7115.c > @@ -1794,6 +1794,21 @@ static int saa711x_detect_chip(struct i2c_client *client, > return GM7113C; > } > > + /* Check if it is a CJC7113 */ > + if (!memcmp(name, "1111111111111111", CHIP_VER_SIZE)) { > + strlcpy(name, "cjc7113", CHIP_VER_SIZE); > + > + if (!autodetect && strcmp(name, id->name)) > + return -EINVAL; > + > + v4l_dbg(1, debug, client, > + "It seems to be a %s chip (%*ph) @ 0x%x.\n", > + name, 16, chip_ver, client->addr << 1); > + > + /* CJC7113 seems to be SAA7113-compatible */ > + return SAA7113; > + } > + > /* Chip was not discovered. Return its ID and don't bind */ > v4l_dbg(1, debug, client, "chip %*ph @ 0x%x is unknown.\n", > 16, chip_ver, client->addr << 1); > -- > 2.7.0 > Any feedback on this one? -- Ezequiel García, VanguardiaSur www.vanguardiasur.com.ar -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html