linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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 = &reg,
>> +			.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/

  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).