From: Antti Palosaari <crope@iki.fi>
To: Nibble Max <nibble.max@gmail.com>, Olli Salonen <olli.salonen@iki.fi>
Cc: linux-media <linux-media@vger.kernel.org>
Subject: Re: [PATCHv2 1/4] sp2: Add I2C driver for CIMaX SP2 common interface module
Date: Sun, 28 Sep 2014 17:14:44 +0300 [thread overview]
Message-ID: <542817D4.9040303@iki.fi> (raw)
In-Reply-To: <201409282127356407018@gmail.com>
Yes it is clearly wrong and as it is trivial and clear bug I recommended
just send patch instead.
Antti
On 09/28/2014 04:27 PM, Nibble Max wrote:
> Hello,
>
> static int sp2_remove(struct i2c_client *client)
> {
> struct si2157 *s = i2c_get_clientdata(client);
> ......
>
> wrong "struct si2157", it should be "struct sp2".
> Could you check it?
>
> BR,
> Max
>
>> Driver for the CIMaX SP2 common interface chip. It is very much based on
>> the existing cimax2 driver for cx23885, but should be more reusable. The
>> product has been sold with name Atmel T90FJR as well and the data sheets
>> for that chip seem to be publicly available.
>>
>> It seems that the USB device that I have and the cx23885 based devices will
>> need to interact differently with the chip for the CAM operations. Thus
>> there is one callback function that is passed on to the sp2 driver
>> (see function sp2_ci_op_cam for that one).
>>
>> IRQ functionality is not included currently (not needed by USB devices
>> and I don't have a PCIe device for development).
>>
>> This is the second version of the patch series after review by Antti
>> Palosaari.
>>
>> Signed-off-by: Olli Salonen <olli.salonen@iki.fi>
>> ---
>> drivers/media/dvb-frontends/Kconfig | 7 +
>> drivers/media/dvb-frontends/Makefile | 1 +
>> drivers/media/dvb-frontends/sp2.c | 441 +++++++++++++++++++++++++++++++++
>> drivers/media/dvb-frontends/sp2.h | 53 ++++
>> drivers/media/dvb-frontends/sp2_priv.h | 50 ++++
>> 5 files changed, 552 insertions(+)
>> create mode 100644 drivers/media/dvb-frontends/sp2.c
>> create mode 100644 drivers/media/dvb-frontends/sp2.h
>> create mode 100644 drivers/media/dvb-frontends/sp2_priv.h
>>
>> diff --git a/drivers/media/dvb-frontends/Kconfig b/drivers/media/dvb-frontends/Kconfig
>> index fe0ddcc..c38c936 100644
>> --- a/drivers/media/dvb-frontends/Kconfig
>> +++ b/drivers/media/dvb-frontends/Kconfig
>> @@ -720,6 +720,13 @@ config DVB_A8293
>> depends on DVB_CORE && I2C
>> default m if !MEDIA_SUBDRV_AUTOSELECT
>>
>> +config DVB_SP2
>> + tristate "CIMaX SP2"
>> + depends on DVB_CORE && I2C
>> + default m if !MEDIA_SUBDRV_AUTOSELECT
>> + help
>> + CIMaX SP2/SP2HF Common Interface module.
>> +
>> config DVB_LGS8GL5
>> tristate "Silicon Legend LGS-8GL5 demodulator (OFDM)"
>> depends on DVB_CORE && I2C
>> diff --git a/drivers/media/dvb-frontends/Makefile b/drivers/media/dvb-frontends/Makefile
>> index edf103d..3498b95 100644
>> --- a/drivers/media/dvb-frontends/Makefile
>> +++ b/drivers/media/dvb-frontends/Makefile
>> @@ -107,6 +107,7 @@ obj-$(CONFIG_DVB_DRXK) += drxk.o
>> obj-$(CONFIG_DVB_TDA18271C2DD) += tda18271c2dd.o
>> obj-$(CONFIG_DVB_SI2165) += si2165.o
>> obj-$(CONFIG_DVB_A8293) += a8293.o
>> +obj-$(CONFIG_DVB_SP2) += sp2.o
>> obj-$(CONFIG_DVB_TDA10071) += tda10071.o
>> obj-$(CONFIG_DVB_RTL2830) += rtl2830.o
>> obj-$(CONFIG_DVB_RTL2832) += rtl2832.o
>> diff --git a/drivers/media/dvb-frontends/sp2.c b/drivers/media/dvb-frontends/sp2.c
>> new file mode 100644
>> index 0000000..9b684d5
>> --- /dev/null
>> +++ b/drivers/media/dvb-frontends/sp2.c
>> @@ -0,0 +1,441 @@
>> +/*
>> + * CIMaX SP2/SP2HF (Atmel T90FJR) CI driver
>> + *
>> + * Copyright (C) 2014 Olli Salonen <olli.salonen@iki.fi>
>> + *
>> + * Heavily based on CIMax2(R) SP2 driver in conjunction with NetUp Dual
>> + * DVB-S2 CI card (cimax2) with following copyrights:
>> + *
>> + * Copyright (C) 2009 NetUP Inc.
>> + * Copyright (C) 2009 Igor M. Liplianin <liplianin@netup.ru>
>> + * Copyright (C) 2009 Abylay Ospan <aospan@netup.ru>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#include "sp2_priv.h"
>> +
>> +static int sp2_read_i2c(struct sp2 *s, u8 reg, u8 *buf, int len)
>> +{
>> + int ret;
>> + struct i2c_client *client = s->client;
>> + struct i2c_adapter *adap = client->adapter;
>> + struct i2c_msg msg[] = {
>> + {
>> + .addr = client->addr,
>> + .flags = 0,
>> + .buf = ®,
>> + .len = 1
>> + }, {
>> + .addr = client->addr,
>> + .flags = I2C_M_RD,
>> + .buf = buf,
>> + .len = len
>> + }
>> + };
>> +
>> + ret = i2c_transfer(adap, msg, 2);
>> +
>> + if (ret != 2) {
>> + dev_err(&client->dev, "i2c read error, reg = 0x%02x, status = %d\n",
>> + reg, ret);
>> + if (ret < 0)
>> + return ret;
>> + else
>> + return -EIO;
>> + }
>> +
>> + dev_dbg(&s->client->dev, "addr=0x%04x, reg = 0x%02x, data = %02x\n",
>> + client->addr, reg, buf[0]);
>> +
>> + return 0;
>> +}
>> +
>> +static int sp2_write_i2c(struct sp2 *s, u8 reg, u8 *buf, int len)
>> +{
>> + int ret;
>> + u8 buffer[35];
>> + struct i2c_client *client = s->client;
>> + struct i2c_adapter *adap = client->adapter;
>> + struct i2c_msg msg = {
>> + .addr = client->addr,
>> + .flags = 0,
>> + .buf = &buffer[0],
>> + .len = len + 1
>> + };
>> +
>> + if ((len + 1) > sizeof(buffer)) {
>> + dev_err(&client->dev, "i2c wr reg=%02x: len=%d is too big!\n",
>> + reg, len);
>> + return -EINVAL;
>> + }
>> +
>> + buffer[0] = reg;
>> + memcpy(&buffer[1], buf, len);
>> +
>> + ret = i2c_transfer(adap, &msg, 1);
>> +
>> + if (ret != 1) {
>> + dev_err(&client->dev, "i2c write error, reg = 0x%02x, status = %d\n",
>> + reg, ret);
>> + if (ret < 0)
>> + return ret;
>> + else
>> + return -EIO;
>> + }
>> +
>> + return 0;
>> +}
>> +
>> +static int sp2_ci_op_cam(struct dvb_ca_en50221 *en50221, int slot, u8 acs,
>> + u8 read, int addr, u8 data)
>> +{
>> + struct sp2 *s = en50221->data;
>> + u8 store;
>> + int mem, ret;
>> + int (*ci_op_cam)(void*, u8, int, u8, int*) = s->ci_control;
>> +
>> + dev_dbg(&s->client->dev, "slot=%d, acs=0x%02x, addr=0x%04x, data = 0x%02x",
>> + slot, acs, addr, data);
>> +
>> + if (slot != 0)
>> + return -EINVAL;
>> +
>> + /*
>> + * change module access type between IO space and attribute memory
>> + * when needed
>> + */
>> + if (s->module_access_type != acs) {
>> + ret = sp2_read_i2c(s, 0x00, &store, 1);
>> +
>> + if (ret)
>> + return ret;
>> +
>> + store &= ~(SP2_MOD_CTL_ACS1 | SP2_MOD_CTL_ACS0);
>> + store |= acs;
>> +
>> + ret = sp2_write_i2c(s, 0x00, &store, 1);
>> + if (ret)
>> + return ret;
>> + }
>> +
>> + s->module_access_type = acs;
>> +
>> + /* implementation of ci_op_cam is device specific */
>> + if (ci_op_cam) {
>> + ret = ci_op_cam(s->priv, read, addr, data, &mem);
>> + } else {
>> + dev_err(&s->client->dev, "callback not defined");
>> + return -EINVAL;
>> + }
>> +
>> + if (ret)
>> + return ret;
>> +
>> + if (read) {
>> + dev_dbg(&s->client->dev, "cam read, addr=0x%04x, data = 0x%04x",
>> + addr, mem);
>> + return mem;
>> + } else {
>> + return 0;
>> + }
>> +}
>> +
>> +int sp2_ci_read_attribute_mem(struct dvb_ca_en50221 *en50221,
>> + int slot, int addr)
>> +{
>> + return sp2_ci_op_cam(en50221, slot, SP2_CI_ATTR_ACS,
>> + SP2_CI_RD, addr, 0);
>> +}
>> +
>> +int sp2_ci_write_attribute_mem(struct dvb_ca_en50221 *en50221,
>> + int slot, int addr, u8 data)
>> +{
>> + return sp2_ci_op_cam(en50221, slot, SP2_CI_ATTR_ACS,
>> + SP2_CI_WR, addr, data);
>> +}
>> +
>> +int sp2_ci_read_cam_control(struct dvb_ca_en50221 *en50221,
>> + int slot, u8 addr)
>> +{
>> + return sp2_ci_op_cam(en50221, slot, SP2_CI_IO_ACS,
>> + SP2_CI_RD, addr, 0);
>> +}
>> +
>> +int sp2_ci_write_cam_control(struct dvb_ca_en50221 *en50221,
>> + int slot, u8 addr, u8 data)
>> +{
>> + return sp2_ci_op_cam(en50221, slot, SP2_CI_IO_ACS,
>> + SP2_CI_WR, addr, data);
>> +}
>> +
>> +int sp2_ci_slot_reset(struct dvb_ca_en50221 *en50221, int slot)
>> +{
>> + struct sp2 *s = en50221->data;
>> + u8 buf;
>> + int ret;
>> +
>> + dev_dbg(&s->client->dev, "slot: %d\n", slot);
>> +
>> + if (slot != 0)
>> + return -EINVAL;
>> +
>> + /* RST on */
>> + buf = SP2_MOD_CTL_RST;
>> + ret = sp2_write_i2c(s, 0x00, &buf, 1);
>> +
>> + if (ret)
>> + return ret;
>> +
>> + usleep_range(500, 600);
>> +
>> + /* RST off */
>> + buf = 0x00;
>> + ret = sp2_write_i2c(s, 0x00, &buf, 1);
>> +
>> + if (ret)
>> + return ret;
>> +
>> + msleep(1000);
>> +
>> + return 0;
>> +}
>> +
>> +int sp2_ci_slot_shutdown(struct dvb_ca_en50221 *en50221, int slot)
>> +{
>> + struct sp2 *s = en50221->data;
>> +
>> + dev_dbg(&s->client->dev, "slot:%d\n", slot);
>> +
>> + /* not implemented */
>> + return 0;
>> +}
>> +
>> +int sp2_ci_slot_ts_enable(struct dvb_ca_en50221 *en50221, int slot)
>> +{
>> + struct sp2 *s = en50221->data;
>> + u8 buf;
>> +
>> + dev_dbg(&s->client->dev, "slot:%d\n", slot);
>> +
>> + if (slot != 0)
>> + return -EINVAL;
>> +
>> + sp2_read_i2c(s, 0x00, &buf, 1);
>> +
>> + /* disable bypass and enable TS */
>> + buf |= (SP2_MOD_CTL_TSOEN | SP2_MOD_CTL_TSIEN);
>> + return sp2_write_i2c(s, 0, &buf, 1);
>> +}
>> +
>> +int sp2_ci_poll_slot_status(struct dvb_ca_en50221 *en50221,
>> + int slot, int open)
>> +{
>> + struct sp2 *s = en50221->data;
>> + u8 buf[2];
>> + int ret;
>> +
>> + dev_dbg(&s->client->dev, "slot:%d open:%d\n", slot, open);
>> +
>> + /*
>> + * CAM module INSERT/REMOVE processing. Slow operation because of i2c
>> + * transfers. Throttle read to one per sec.
>> + */
>> + if (time_after(jiffies, s->next_status_checked_time)) {
>> + ret = sp2_read_i2c(s, 0x00, buf, 1);
>> + s->next_status_checked_time = jiffies + msecs_to_jiffies(1000);
>> +
>> + if (ret)
>> + return 0;
>> +
>> + if (buf[0] & SP2_MOD_CTL_DET)
>> + s->status = DVB_CA_EN50221_POLL_CAM_PRESENT |
>> + DVB_CA_EN50221_POLL_CAM_READY;
>> + else
>> + s->status = 0;
>> + }
>> +
>> + return s->status;
>> +}
>> +
>> +int sp2_init(struct sp2 *s)
>> +{
>> + int ret = 0;
>> + u8 buf;
>> + u8 cimax_init[34] = {
>> + 0x00, /* module A control*/
>> + 0x00, /* auto select mask high A */
>> + 0x00, /* auto select mask low A */
>> + 0x00, /* auto select pattern high A */
>> + 0x00, /* auto select pattern low A */
>> + 0x44, /* memory access time A, 600 ns */
>> + 0x00, /* invert input A */
>> + 0x00, /* RFU */
>> + 0x00, /* RFU */
>> + 0x00, /* module B control*/
>> + 0x00, /* auto select mask high B */
>> + 0x00, /* auto select mask low B */
>> + 0x00, /* auto select pattern high B */
>> + 0x00, /* auto select pattern low B */
>> + 0x44, /* memory access time B, 600 ns */
>> + 0x00, /* invert input B */
>> + 0x00, /* RFU */
>> + 0x00, /* RFU */
>> + 0x00, /* auto select mask high Ext */
>> + 0x00, /* auto select mask low Ext */
>> + 0x00, /* auto select pattern high Ext */
>> + 0x00, /* auto select pattern low Ext */
>> + 0x00, /* RFU */
>> + 0x02, /* destination - module A */
>> + 0x01, /* power control reg, VCC power on */
>> + 0x00, /* RFU */
>> + 0x00, /* int status read only */
>> + 0x00, /* Interrupt Mask Register */
>> + 0x05, /* EXTINT=active-high, INT=push-pull */
>> + 0x00, /* USCG1 */
>> + 0x04, /* ack active low */
>> + 0x00, /* LOCK = 0 */
>> + 0x22, /* unknown */
>> + 0x00, /* synchronization? */
>> + };
>> +
>> + dev_dbg(&s->client->dev, "\n");
>> +
>> + s->ca.owner = THIS_MODULE;
>> + s->ca.read_attribute_mem = sp2_ci_read_attribute_mem;
>> + s->ca.write_attribute_mem = sp2_ci_write_attribute_mem;
>> + s->ca.read_cam_control = sp2_ci_read_cam_control;
>> + s->ca.write_cam_control = sp2_ci_write_cam_control;
>> + s->ca.slot_reset = sp2_ci_slot_reset;
>> + s->ca.slot_shutdown = sp2_ci_slot_shutdown;
>> + s->ca.slot_ts_enable = sp2_ci_slot_ts_enable;
>> + s->ca.poll_slot_status = sp2_ci_poll_slot_status;
>> + s->ca.data = s;
>> + s->module_access_type = 0;
>> +
>> + /* initialize all regs */
>> + ret = sp2_write_i2c(s, 0x00, &cimax_init[0], 34);
>> + if (ret)
>> + goto err;
>> +
>> + /* lock registers */
>> + buf = 1;
>> + ret = sp2_write_i2c(s, 0x1f, &buf, 1);
>> + if (ret)
>> + goto err;
>> +
>> + /* power on slots */
>> + ret = sp2_write_i2c(s, 0x18, &buf, 1);
>> + if (ret)
>> + goto err;
>> +
>> + ret = dvb_ca_en50221_init(s->dvb_adap, &s->ca, 0, 1);
>> + if (ret)
>> + goto err;
>> +
>> + return 0;
>> +
>> +err:
>> + dev_dbg(&s->client->dev, "init failed=%d\n", ret);
>> + return ret;
>> +}
>> +
>> +int sp2_exit(struct i2c_client *client)
>> +{
>> + struct sp2 *s;
>> +
>> + dev_dbg(&client->dev, "\n");
>> +
>> + if (client == NULL)
>> + return 0;
>> +
>> + s = i2c_get_clientdata(client);
>> + if (s == NULL)
>> + return 0;
>> +
>> + if (s->ca.data == NULL)
>> + return 0;
>> +
>> + dvb_ca_en50221_release(&s->ca);
>> +
>> + return 0;
>> +}
>> +
>> +static int sp2_probe(struct i2c_client *client,
>> + const struct i2c_device_id *id)
>> +{
>> + struct sp2_config *cfg = client->dev.platform_data;
>> + struct sp2 *s;
>> + int ret;
>> +
>> + dev_dbg(&client->dev, "\n");
>> +
>> + s = kzalloc(sizeof(struct sp2), GFP_KERNEL);
>> + if (!s) {
>> + ret = -ENOMEM;
>> + dev_err(&client->dev, "kzalloc() failed\n");
>> + goto err;
>> + }
>> +
>> + s->client = client;
>> + s->dvb_adap = cfg->dvb_adap;
>> + s->priv = cfg->priv;
>> + s->ci_control = cfg->ci_control;
>> +
>> + i2c_set_clientdata(client, s);
>> +
>> + ret = sp2_init(s);
>> + if (ret)
>> + goto err;
>> +
>> + dev_info(&s->client->dev, "CIMaX SP2 successfully attached\n");
>> + return 0;
>> +err:
>> + dev_dbg(&client->dev, "init failed=%d\n", ret);
>> + kfree(s);
>> +
>> + return ret;
>> +}
>> +
>> +static int sp2_remove(struct i2c_client *client)
>> +{
>> + struct si2157 *s = i2c_get_clientdata(client);
> wrong "struct si2157", it should be "struct sp2".
>> +
>> + dev_dbg(&client->dev, "\n");
>> +
>> + sp2_exit(client);
>> + if (s != NULL)
>> + kfree(s);
>> +
>> + return 0;
>> +}
>> +
>> +static const struct i2c_device_id sp2_id[] = {
>> + {"sp2", 0},
>> + {}
>> +};
>> +MODULE_DEVICE_TABLE(i2c, sp2_id);
>> +
>> +static struct i2c_driver sp2_driver = {
>> + .driver = {
>> + .owner = THIS_MODULE,
>> + .name = "sp2",
>> + },
>> + .probe = sp2_probe,
>> + .remove = sp2_remove,
>> + .id_table = sp2_id,
>> +};
>> +
>> +module_i2c_driver(sp2_driver);
>> +
>> +MODULE_DESCRIPTION("CIMaX SP2/HF CI driver");
>> +MODULE_AUTHOR("Olli Salonen <olli.salonen@iki.fi>");
>> +MODULE_LICENSE("GPL");
>> diff --git a/drivers/media/dvb-frontends/sp2.h b/drivers/media/dvb-frontends/sp2.h
>> new file mode 100644
>> index 0000000..6cceea0
>> --- /dev/null
>> +++ b/drivers/media/dvb-frontends/sp2.h
>> @@ -0,0 +1,53 @@
>> +/*
>> + * CIMaX SP2/HF CI driver
>> + *
>> + * Copyright (C) 2014 Olli Salonen <olli.salonen@iki.fi>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#ifndef SP2_H
>> +#define SP2_H
>> +
>> +#include <linux/kconfig.h>
>> +#include "dvb_ca_en50221.h"
>> +
>> +/*
>> + * I2C address
>> + * 0x40 (port 0)
>> + * 0x41 (port 1)
>> + */
>> +struct sp2_config {
>> + /* dvb_adapter to attach the ci to */
>> + struct dvb_adapter *dvb_adap;
>> +
>> + /* function ci_control handles the device specific ci ops */
>> + void *ci_control;
>> +
>> + /* priv is passed back to function ci_control */
>> + void *priv;
>> +};
>> +
>> +extern int sp2_ci_read_attribute_mem(struct dvb_ca_en50221 *en50221,
>> + int slot, int addr);
>> +extern int sp2_ci_write_attribute_mem(struct dvb_ca_en50221 *en50221,
>> + int slot, int addr, u8 data);
>> +extern int sp2_ci_read_cam_control(struct dvb_ca_en50221 *en50221,
>> + int slot, u8 addr);
>> +extern int sp2_ci_write_cam_control(struct dvb_ca_en50221 *en50221,
>> + int slot, u8 addr, u8 data);
>> +extern int sp2_ci_slot_reset(struct dvb_ca_en50221 *en50221, int slot);
>> +extern int sp2_ci_slot_shutdown(struct dvb_ca_en50221 *en50221, int slot);
>> +extern int sp2_ci_slot_ts_enable(struct dvb_ca_en50221 *en50221, int slot);
>> +extern int sp2_ci_poll_slot_status(struct dvb_ca_en50221 *en50221,
>> + int slot, int open);
>> +
>> +#endif
>> diff --git a/drivers/media/dvb-frontends/sp2_priv.h b/drivers/media/dvb-frontends/sp2_priv.h
>> new file mode 100644
>> index 0000000..37fef7b
>> --- /dev/null
>> +++ b/drivers/media/dvb-frontends/sp2_priv.h
>> @@ -0,0 +1,50 @@
>> +/*
>> + * CIMaX SP2/HF CI driver
>> + *
>> + * Copyright (C) 2014 Olli Salonen <olli.salonen@iki.fi>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#ifndef SP2_PRIV_H
>> +#define SP2_PRIV_H
>> +
>> +#include "sp2.h"
>> +#include "dvb_frontend.h"
>> +
>> +/* state struct */
>> +struct sp2 {
>> + int status;
>> + struct i2c_client *client;
>> + struct dvb_adapter *dvb_adap;
>> + struct dvb_ca_en50221 ca;
>> + int module_access_type;
>> + unsigned long next_status_checked_time;
>> + void *priv;
>> + void *ci_control;
>> +};
>> +
>> +#define SP2_CI_ATTR_ACS 0x00
>> +#define SP2_CI_IO_ACS 0x04
>> +#define SP2_CI_WR 0
>> +#define SP2_CI_RD 1
>> +
>> +/* Module control register (0x00 module A, 0x09 module B) bits */
>> +#define SP2_MOD_CTL_DET 0x01
>> +#define SP2_MOD_CTL_AUTO 0x02
>> +#define SP2_MOD_CTL_ACS0 0x04
>> +#define SP2_MOD_CTL_ACS1 0x08
>> +#define SP2_MOD_CTL_HAD 0x10
>> +#define SP2_MOD_CTL_TSIEN 0x20
>> +#define SP2_MOD_CTL_TSOEN 0x40
>> +#define SP2_MOD_CTL_RST 0x80
>> +
>> +#endif
>> --
>> 1.9.1
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>> .
>
--
http://palosaari.fi/
next prev parent reply other threads:[~2014-09-28 14:14 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-28 13:27 [PATCHv2 1/4] sp2: Add I2C driver for CIMaX SP2 common interface module Nibble Max
2014-09-28 14:14 ` Antti Palosaari [this message]
-- strict thread matches above, loose matches on Subject: below --
2014-09-27 15:37 Nibble Max
2014-08-08 7:06 Olli Salonen
2014-08-09 22:47 ` Antti Palosaari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=542817D4.9040303@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=nibble.max@gmail.com \
--cc=olli.salonen@iki.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).