From: Mickey Stein <yekkim@pacbell.net>
To: Greg K-H <greg@kroah.com>
Cc: linux-kernel@vger.kernel.org, sensors@Stimpy.netroedge.com
Subject: Re: [PATCH] I2C: Fix some gcc 4.0 compile failures and warnings
Date: Fri, 04 Mar 2005 14:55:11 -0800 [thread overview]
Message-ID: <4228E74F.9070706@pacbell.net> (raw)
In-Reply-To: <11099685961021@kroah.com>
[-- Attachment #1: Type: text/plain, Size: 16316 bytes --]
Greg KH wrote:
>ChangeSet 1.2108, 2005/03/02 15:02:27-08:00, yekkim@pacbell.net
>
>[PATCH] I2C: Fix some gcc 4.0 compile failures and warnings
>
>gcc 4.0.x cvs seems to dislike "include/linux/i2c.h file" and others due
>to a current gcc 4.0.x change having to do with array declarations.
>
>Example error msg: include/linux/i2c.h:{55,194} error: array type has
>incomplete element type
>
>A. Daplas has recently done a workaround for this on another header
>file. A thread discussing this can be found by following the link below:
>
>http://gcc.gnu.org/ml/gcc/2005-02/msg00053.html
>
>The patch changes the array(struct i2c_msg) declaration used by
>*i2c_transfer and *master_xfer from "struct i2c_msg msg[]" format to
>"struct i2c_msg *msg".
>
>After some grepping, I came up with about a dozen files that used the
>format disliked by gcc4 that're addressed by the attached patch.
>Tested on gcc 3.x & gcc 4.x by configuring kernel with all i2c switches
>enabled as module, and saw no errors or warnings in i2c.
>
>Signed-off-by: Mickey Stein <yekkim@pacbell.net>
>Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
>
>
> Documentation/i2c/writing-clients | 2 +-
> drivers/i2c/algos/i2c-algo-ite.c | 4 ++--
> drivers/i2c/algos/i2c-algo-pca.c | 2 +-
> drivers/i2c/algos/i2c-algo-pcf.c | 2 +-
> drivers/i2c/algos/i2c-algo-sgi.c | 2 +-
> drivers/i2c/busses/i2c-au1550.c | 2 +-
> drivers/i2c/busses/i2c-ibm_iic.c | 2 +-
> drivers/i2c/busses/i2c-iop3xx.c | 2 +-
> drivers/i2c/busses/i2c-keywest.c | 2 +-
> drivers/i2c/busses/i2c-mpc.c | 2 +-
> drivers/i2c/busses/i2c-s3c2410.c | 4 ++--
> drivers/i2c/i2c-core.c | 2 +-
> drivers/media/common/saa7146_i2c.c | 8 ++++----
> drivers/media/dvb/b2c2/skystar2.c | 2 +-
> drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c | 2 +-
> drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c | 2 +-
> drivers/media/video/bttv-i2c.c | 2 +-
> drivers/media/video/saa7134/saa7134-i2c.c | 2 +-
> include/linux/i2c.h | 4 ++--
> include/media/saa7146.h | 2 +-
> 20 files changed, 26 insertions(+), 26 deletions(-)
>
>
>diff -Nru a/Documentation/i2c/writing-clients b/Documentation/i2c/writing-clients
>--- a/Documentation/i2c/writing-clients 2005-03-04 12:23:35 -08:00
>+++ b/Documentation/i2c/writing-clients 2005-03-04 12:23:35 -08:00
>@@ -638,7 +638,7 @@
> parameter contains the bytes the read/write, the third the length of the
> buffer. Returned is the actual number of bytes read/written.
>
>- extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg msg[],
>+ extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msg,
> int num);
>
> This sends a series of messages. Each message can be a read or write,
>diff -Nru a/drivers/i2c/algos/i2c-algo-ite.c b/drivers/i2c/algos/i2c-algo-ite.c
>--- a/drivers/i2c/algos/i2c-algo-ite.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/algos/i2c-algo-ite.c 2005-03-04 12:23:35 -08:00
>@@ -490,7 +490,7 @@
> * condition.
> */
> #if 0
>-static int iic_combined_transaction(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
>+static int iic_combined_transaction(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
> {
> int i;
> struct i2c_msg *pmsg;
>@@ -600,7 +600,7 @@
> * verify that the bus is not busy or in some unknown state.
> */
> static int iic_xfer(struct i2c_adapter *i2c_adap,
>- struct i2c_msg msgs[],
>+ struct i2c_msg *msgs,
> int num)
> {
> struct i2c_algo_iic_data *adap = i2c_adap->algo_data;
>diff -Nru a/drivers/i2c/algos/i2c-algo-pca.c b/drivers/i2c/algos/i2c-algo-pca.c
>--- a/drivers/i2c/algos/i2c-algo-pca.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/algos/i2c-algo-pca.c 2005-03-04 12:23:35 -08:00
>@@ -178,7 +178,7 @@
> }
>
> static int pca_xfer(struct i2c_adapter *i2c_adap,
>- struct i2c_msg msgs[],
>+ struct i2c_msg *msgs,
> int num)
> {
> struct i2c_algo_pca_data *adap = i2c_adap->algo_data;
>diff -Nru a/drivers/i2c/algos/i2c-algo-pcf.c b/drivers/i2c/algos/i2c-algo-pcf.c
>--- a/drivers/i2c/algos/i2c-algo-pcf.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/algos/i2c-algo-pcf.c 2005-03-04 12:23:35 -08:00
>@@ -332,7 +332,7 @@
> }
>
> static int pcf_xfer(struct i2c_adapter *i2c_adap,
>- struct i2c_msg msgs[],
>+ struct i2c_msg *msgs,
> int num)
> {
> struct i2c_algo_pcf_data *adap = i2c_adap->algo_data;
>diff -Nru a/drivers/i2c/algos/i2c-algo-sgi.c b/drivers/i2c/algos/i2c-algo-sgi.c
>--- a/drivers/i2c/algos/i2c-algo-sgi.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/algos/i2c-algo-sgi.c 2005-03-04 12:23:35 -08:00
>@@ -131,7 +131,7 @@
> return 0;
> }
>
>-static int sgi_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[],
>+static int sgi_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs,
> int num)
> {
> struct i2c_algo_sgi_data *adap = i2c_adap->algo_data;
>diff -Nru a/drivers/i2c/busses/i2c-au1550.c b/drivers/i2c/busses/i2c-au1550.c
>--- a/drivers/i2c/busses/i2c-au1550.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-au1550.c 2005-03-04 12:23:35 -08:00
>@@ -253,7 +253,7 @@
> }
>
> static int
>-au1550_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
>+au1550_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
> {
> struct i2c_au1550_data *adap = i2c_adap->algo_data;
> struct i2c_msg *p;
>diff -Nru a/drivers/i2c/busses/i2c-ibm_iic.c b/drivers/i2c/busses/i2c-ibm_iic.c
>--- a/drivers/i2c/busses/i2c-ibm_iic.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-ibm_iic.c 2005-03-04 12:23:35 -08:00
>@@ -549,7 +549,7 @@
> * Generic master transfer entrypoint.
> * Returns the number of processed messages or error (<0)
> */
>-static int iic_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
>+static int iic_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num)
> {
> struct ibm_iic_private* dev = (struct ibm_iic_private*)(i2c_get_adapdata(adap));
> volatile struct iic_regs __iomem *iic = dev->vaddr;
>diff -Nru a/drivers/i2c/busses/i2c-iop3xx.c b/drivers/i2c/busses/i2c-iop3xx.c
>--- a/drivers/i2c/busses/i2c-iop3xx.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-iop3xx.c 2005-03-04 12:23:35 -08:00
>@@ -361,7 +361,7 @@
> * master_xfer() - main read/write entry
> */
> static int
>-iop3xx_i2c_master_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[],
>+iop3xx_i2c_master_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs,
> int num)
> {
> struct i2c_algo_iop3xx_data *iop3xx_adap = i2c_adap->algo_data;
>diff -Nru a/drivers/i2c/busses/i2c-keywest.c b/drivers/i2c/busses/i2c-keywest.c
>--- a/drivers/i2c/busses/i2c-keywest.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-keywest.c 2005-03-04 12:23:35 -08:00
>@@ -399,7 +399,7 @@
> */
> static int
> keywest_xfer( struct i2c_adapter *adap,
>- struct i2c_msg msgs[],
>+ struct i2c_msg *msgs,
> int num)
> {
> struct keywest_chan* chan = i2c_get_adapdata(adap);
>diff -Nru a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
>--- a/drivers/i2c/busses/i2c-mpc.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-mpc.c 2005-03-04 12:23:35 -08:00
>@@ -233,7 +233,7 @@
> return length;
> }
>
>-static int mpc_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
>+static int mpc_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num)
> {
> struct i2c_msg *pmsg;
> int i;
>diff -Nru a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
>--- a/drivers/i2c/busses/i2c-s3c2410.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/busses/i2c-s3c2410.c 2005-03-04 12:23:35 -08:00
>@@ -483,7 +483,7 @@
> * this starts an i2c transfer
> */
>
>-static int s3c24xx_i2c_doxfer(struct s3c24xx_i2c *i2c, struct i2c_msg msgs[], int num)
>+static int s3c24xx_i2c_doxfer(struct s3c24xx_i2c *i2c, struct i2c_msg *msgs, int num)
> {
> unsigned long timeout;
> int ret;
>@@ -534,7 +534,7 @@
> */
>
> static int s3c24xx_i2c_xfer(struct i2c_adapter *adap,
>- struct i2c_msg msgs[], int num)
>+ struct i2c_msg *msgs, int num)
> {
> struct s3c24xx_i2c *i2c = (struct s3c24xx_i2c *)adap->algo_data;
> int retry;
>diff -Nru a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c
>--- a/drivers/i2c/i2c-core.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/i2c/i2c-core.c 2005-03-04 12:23:35 -08:00
>@@ -582,7 +582,7 @@
> * ----------------------------------------------------
> */
>
>-int i2c_transfer(struct i2c_adapter * adap, struct i2c_msg msgs[],int num)
>+int i2c_transfer(struct i2c_adapter * adap, struct i2c_msg *msgs, int num)
> {
> int ret;
>
>diff -Nru a/drivers/media/common/saa7146_i2c.c b/drivers/media/common/saa7146_i2c.c
>--- a/drivers/media/common/saa7146_i2c.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/common/saa7146_i2c.c 2005-03-04 12:23:35 -08:00
>@@ -25,7 +25,7 @@
> sent through the saa7146. have a look at the specifications p. 122 ff
> to understand this. it returns the number of u32s to send, or -1
> in case of an error. */
>-static int saa7146_i2c_msg_prepare(const struct i2c_msg m[], int num, u32 *op)
>+static int saa7146_i2c_msg_prepare(const struct i2c_msg *m, int num, u32 *op)
> {
> int h1, h2;
> int i, j, addr;
>@@ -89,7 +89,7 @@
> which bytes were read through the adapter and write them back to the corresponding
> i2c-message. but instead, we simply write back all bytes.
> fixme: this could be improved. */
>-static int saa7146_i2c_msg_cleanup(const struct i2c_msg m[], int num, u32 *op)
>+static int saa7146_i2c_msg_cleanup(const struct i2c_msg *m, int num, u32 *op)
> {
> int i, j;
> int op_count = 0;
>@@ -272,7 +272,7 @@
> return 0;
> }
>
>-int saa7146_i2c_transfer(struct saa7146_dev *dev, const struct i2c_msg msgs[], int num, int retries)
>+int saa7146_i2c_transfer(struct saa7146_dev *dev, const struct i2c_msg *msgs, int num, int retries)
> {
> int i = 0, count = 0;
> u32* buffer = dev->d_i2c.cpu_addr;
>@@ -372,7 +372,7 @@
> }
>
> /* utility functions */
>-static int saa7146_i2c_xfer(struct i2c_adapter* adapter, struct i2c_msg msg[], int num)
>+static int saa7146_i2c_xfer(struct i2c_adapter* adapter, struct i2c_msg *msg, int num)
> {
> struct saa7146_dev* dev = i2c_get_adapdata(adapter);
>
>diff -Nru a/drivers/media/dvb/b2c2/skystar2.c b/drivers/media/dvb/b2c2/skystar2.c
>--- a/drivers/media/dvb/b2c2/skystar2.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/dvb/b2c2/skystar2.c 2005-03-04 12:23:35 -08:00
>@@ -293,7 +293,7 @@
> return buf - start;
> }
>
>-static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg msgs[], int num)
>+static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg *msgs, int num)
> {
> struct adapter *tmp = i2c_get_adapdata(adapter);
> int i, ret = 0;
>diff -Nru a/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c b/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c
>--- a/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c 2005-03-04 12:23:35 -08:00
>@@ -38,7 +38,7 @@
> /*
> * I2C master xfer function
> */
>-static int dibusb_i2c_xfer(struct i2c_adapter *adap,struct i2c_msg msg[],int num)
>+static int dibusb_i2c_xfer(struct i2c_adapter *adap,struct i2c_msg *msg,int num)
> {
> struct usb_dibusb *dib = i2c_get_adapdata(adap);
> int i;
>diff -Nru a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c
>--- a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c 2005-03-04 12:23:35 -08:00
>@@ -252,7 +252,7 @@
> return rcv_len;
> }
>
>-static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg msg[], int num)
>+static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg *msg, int num)
> {
> struct ttusb *ttusb = i2c_get_adapdata(adapter);
> int i = 0;
>diff -Nru a/drivers/media/video/bttv-i2c.c b/drivers/media/video/bttv-i2c.c
>--- a/drivers/media/video/bttv-i2c.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/video/bttv-i2c.c 2005-03-04 12:23:35 -08:00
>@@ -245,7 +245,7 @@
> return retval;
> }
>
>-static int bttv_i2c_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
>+static int bttv_i2c_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
> {
> struct bttv *btv = i2c_get_adapdata(i2c_adap);
> int retval = 0;
>diff -Nru a/drivers/media/video/saa7134/saa7134-i2c.c b/drivers/media/video/saa7134/saa7134-i2c.c
>--- a/drivers/media/video/saa7134/saa7134-i2c.c 2005-03-04 12:23:35 -08:00
>+++ b/drivers/media/video/saa7134/saa7134-i2c.c 2005-03-04 12:23:35 -08:00
>@@ -236,7 +236,7 @@
> }
>
> static int saa7134_i2c_xfer(struct i2c_adapter *i2c_adap,
>- struct i2c_msg msgs[], int num)
>+ struct i2c_msg *msgs, int num)
> {
> struct saa7134_dev *dev = i2c_adap->algo_data;
> enum i2c_status status;
>diff -Nru a/include/linux/i2c.h b/include/linux/i2c.h
>--- a/include/linux/i2c.h 2005-03-04 12:23:35 -08:00
>+++ b/include/linux/i2c.h 2005-03-04 12:23:35 -08:00
>@@ -55,7 +55,7 @@
>
> /* Transfer num messages.
> */
>-extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg msg[],int num);
>+extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num);
>
> /*
> * Some adapter types (i.e. PCF 8584 based ones) may support slave behaviuor.
>@@ -191,7 +191,7 @@
> to NULL. If an adapter algorithm can do SMBus access, set
> smbus_xfer. If set to NULL, the SMBus protocol is simulated
> using common I2C messages */
>- int (*master_xfer)(struct i2c_adapter *adap,struct i2c_msg msgs[],
>+ int (*master_xfer)(struct i2c_adapter *adap,struct i2c_msg *msgs,
> int num);
> int (*smbus_xfer) (struct i2c_adapter *adap, u16 addr,
> unsigned short flags, char read_write,
>diff -Nru a/include/media/saa7146.h b/include/media/saa7146.h
>--- a/include/media/saa7146.h 2005-03-04 12:23:35 -08:00
>+++ b/include/media/saa7146.h 2005-03-04 12:23:35 -08:00
>@@ -157,7 +157,7 @@
>
> /* from saa7146_i2c.c */
> int saa7146_i2c_adapter_prepare(struct saa7146_dev *dev, struct i2c_adapter *i2c_adapter, u32 bitrate);
>-int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct i2c_msg msgs[], int num, int retries);
>+int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct *i2c_msg msgs, int num, int retries);
>
> /* from saa7146_core.c */
> extern struct list_head saa7146_devices;
>
>
>
>
Greg,
I was just scanning this email and it looks like you possibly grabbed
the first of my patches with a typo because this last little bit I
corrected in a prior email to you. It got into the *mm* tree ok.
So I'm not sure where this took a wrong turn again.
Its the very last bit of the above patch on saa7146.h, where I'd at
first mistakenly managed to
change struct i2c_msg msgs[] to struct *i2c_msg msgs, when it should
have been changed to
struct i2c_msg *msgs as below. (Its just a cut and paste below, so it
might be easier if you just typed it into whatever version you've got now.
If its left uncorrected it'd give some sort of compiler error (to say
the least).
Thanks very much,
Mick
---
diff -Nru a/include/media/saa7146.h b/include/media/saa7146.h
--- a/include/media/saa7146.h 2005-03-04 12:23:35 -08:00
+++ b/include/media/saa7146.h 2005-03-04 12:23:35 -08:00
@@ -157,7 +157,7 @@
/* from saa7146_i2c.c */
int saa7146_i2c_adapter_prepare(struct saa7146_dev *dev, struct i2c_adapter *i2c_adapter, u32 bitrate);
-int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct i2c_msg msgs[], int num, int retries);
+int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct i2c_msg *msgs, int num, int retries);
/* from saa7146_core.c */
extern struct list_head saa7146_devices;
PS: I'm headed out of town for a week, and the above is only a
cut&paste, so the correct patch is attached here once more.
[-- Attachment #2: i2c_xfer.patch --]
[-- Type: text/plain, Size: 10959 bytes --]
--- ./include/linux/i2c.h.sav 2005-02-23 10:35:36.000000000 -0800
+++ ./include/linux/i2c.h 2005-02-23 10:46:23.000000000 -0800
@@ -55,7 +55,7 @@
/* Transfer num messages.
*/
-extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg msg[],int num);
+extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num);
/*
* Some adapter types (i.e. PCF 8584 based ones) may support slave behaviuor.
@@ -194,7 +194,7 @@
to NULL. If an adapter algorithm can do SMBus access, set
smbus_xfer. If set to NULL, the SMBus protocol is simulated
using common I2C messages */
- int (*master_xfer)(struct i2c_adapter *adap,struct i2c_msg msgs[],
+ int (*master_xfer)(struct i2c_adapter *adap,struct i2c_msg *msgs,
int num);
int (*smbus_xfer) (struct i2c_adapter *adap, u16 addr,
unsigned short flags, char read_write,
--- ./drivers/i2c/busses/i2c-iop3xx.c.sav 2005-02-23 10:36:21.000000000 -0800
+++ ./drivers/i2c/busses/i2c-iop3xx.c 2005-02-23 10:47:36.000000000 -0800
@@ -361,7 +361,7 @@
* master_xfer() - main read/write entry
*/
static int
-iop3xx_i2c_master_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[],
+iop3xx_i2c_master_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs,
int num)
{
struct i2c_algo_iop3xx_data *iop3xx_adap = i2c_adap->algo_data;
--- ./drivers/i2c/i2c-core.c.sav 2005-02-23 10:34:59.000000000 -0800
+++ ./drivers/i2c/i2c-core.c 2005-02-23 10:49:18.000000000 -0800
@@ -583,7 +583,7 @@
* ----------------------------------------------------
*/
-int i2c_transfer(struct i2c_adapter * adap, struct i2c_msg msgs[],int num)
+int i2c_transfer(struct i2c_adapter * adap, struct i2c_msg *msgs, int num)
{
int ret;
--- ./drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c.sav 2005-02-23 10:39:53.000000000 -0800
+++ ./drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c 2005-02-23 10:50:07.000000000 -0800
@@ -252,7 +252,7 @@
return rcv_len;
}
-static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg msg[], int num)
+static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg *msg, int num)
{
struct ttusb *ttusb = i2c_get_adapdata(adapter);
int i = 0;
--- ./drivers/media/dvb/b2c2/skystar2.c.sav 2005-02-23 10:40:33.000000000 -0800
+++ ./drivers/media/dvb/b2c2/skystar2.c 2005-02-23 10:50:42.000000000 -0800
@@ -293,7 +293,7 @@
return buf - start;
}
-static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg msgs[], int num)
+static int master_xfer(struct i2c_adapter* adapter, struct i2c_msg *msgs, int num)
{
struct adapter *tmp = i2c_get_adapdata(adapter);
int i, ret = 0;
--- ./Documentation/i2c/writing-clients.sav 2005-02-23 10:42:00.000000000 -0800
+++ ./Documentation/i2c/writing-clients 2005-02-23 10:42:30.000000000 -0800
@@ -642,7 +642,7 @@
parameter contains the bytes the read/write, the third the length of the
buffer. Returned is the actual number of bytes read/written.
- extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg msg[],
+ extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msg,
int num);
This sends a series of messages. Each message can be a read or write,
--- ./include/media/saa7146.h.sav 2005-02-23 12:01:51.000000000 -0800
+++ ./include/media/saa7146.h 2005-02-23 12:19:38.000000000 -0800
@@ -157,7 +157,7 @@
/* from saa7146_i2c.c */
int saa7146_i2c_adapter_prepare(struct saa7146_dev *dev, struct i2c_adapter *i2c_adapter, u32 bitrate);
-int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct i2c_msg msgs[], int num, int retries);
+int saa7146_i2c_transfer(struct saa7146_dev *saa, const struct i2c_msg *msgs, int num, int retries);
/* from saa7146_core.c */
extern struct list_head saa7146_devices;
--- ./drivers/i2c/busses/i2c-s3c2410.c.sav 2005-02-23 12:03:29.000000000 -0800
+++ ./drivers/i2c/busses/i2c-s3c2410.c 2005-02-23 12:20:31.000000000 -0800
@@ -483,7 +483,7 @@
* this starts an i2c transfer
*/
-static int s3c24xx_i2c_doxfer(struct s3c24xx_i2c *i2c, struct i2c_msg msgs[], int num)
+static int s3c24xx_i2c_doxfer(struct s3c24xx_i2c *i2c, struct i2c_msg *msgs, int num)
{
unsigned long timeout;
int ret;
@@ -534,7 +534,7 @@
*/
static int s3c24xx_i2c_xfer(struct i2c_adapter *adap,
- struct i2c_msg msgs[], int num)
+ struct i2c_msg *msgs, int num)
{
struct s3c24xx_i2c *i2c = (struct s3c24xx_i2c *)adap->algo_data;
int retry;
--- ./drivers/i2c/busses/i2c-mpc.c.sav 2005-02-23 12:04:11.000000000 -0800
+++ ./drivers/i2c/busses/i2c-mpc.c 2005-02-23 12:17:28.000000000 -0800
@@ -233,7 +233,7 @@
return length;
}
-static int mpc_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
+static int mpc_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num)
{
struct i2c_msg *pmsg;
int i;
--- ./drivers/i2c/busses/i2c-ibm_iic.c.sav 2005-02-23 12:04:42.000000000 -0800
+++ ./drivers/i2c/busses/i2c-ibm_iic.c 2005-02-23 12:21:08.000000000 -0800
@@ -549,7 +549,7 @@
* Generic master transfer entrypoint.
* Returns the number of processed messages or error (<0)
*/
-static int iic_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
+static int iic_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num)
{
struct ibm_iic_private* dev = (struct ibm_iic_private*)(i2c_get_adapdata(adap));
volatile struct iic_regs __iomem *iic = dev->vaddr;
--- ./drivers/i2c/busses/i2c-keywest.c.sav 2005-02-23 12:05:10.000000000 -0800
+++ ./drivers/i2c/busses/i2c-keywest.c 2005-02-23 12:26:47.000000000 -0800
@@ -399,7 +399,7 @@
*/
static int
keywest_xfer( struct i2c_adapter *adap,
- struct i2c_msg msgs[],
+ struct i2c_msg *msgs,
int num)
{
struct keywest_chan* chan = i2c_get_adapdata(adap);
--- ./drivers/i2c/busses/i2c-au1550.c.sav 2005-02-23 12:06:00.000000000 -0800
+++ ./drivers/i2c/busses/i2c-au1550.c 2005-02-23 12:27:39.000000000 -0800
@@ -253,7 +253,7 @@
}
static int
-au1550_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
+au1550_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
{
struct i2c_au1550_data *adap = i2c_adap->algo_data;
struct i2c_msg *p;
--- ./drivers/i2c/algos/i2c-algo-pca.c.sav 2005-02-23 12:06:21.000000000 -0800
+++ ./drivers/i2c/algos/i2c-algo-pca.c 2005-02-23 12:28:09.000000000 -0800
@@ -178,7 +178,7 @@
}
static int pca_xfer(struct i2c_adapter *i2c_adap,
- struct i2c_msg msgs[],
+ struct i2c_msg *msgs,
int num)
{
struct i2c_algo_pca_data *adap = i2c_adap->algo_data;
--- ./drivers/i2c/algos/i2c-algo-pcf.c.sav 2005-02-23 12:07:04.000000000 -0800
+++ ./drivers/i2c/algos/i2c-algo-pcf.c 2005-02-23 12:28:49.000000000 -0800
@@ -332,7 +332,7 @@
}
static int pcf_xfer(struct i2c_adapter *i2c_adap,
- struct i2c_msg msgs[],
+ struct i2c_msg *msgs,
int num)
{
struct i2c_algo_pcf_data *adap = i2c_adap->algo_data;
--- ./drivers/i2c/algos/i2c-algo-ite.c.sav 2005-02-23 12:07:17.000000000 -0800
+++ ./drivers/i2c/algos/i2c-algo-ite.c 2005-02-23 12:29:31.000000000 -0800
@@ -490,7 +490,7 @@
* condition.
*/
#if 0
-static int iic_combined_transaction(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
+static int iic_combined_transaction(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
{
int i;
struct i2c_msg *pmsg;
@@ -600,7 +600,7 @@
* verify that the bus is not busy or in some unknown state.
*/
static int iic_xfer(struct i2c_adapter *i2c_adap,
- struct i2c_msg msgs[],
+ struct i2c_msg *msgs,
int num)
{
struct i2c_algo_iic_data *adap = i2c_adap->algo_data;
--- ./drivers/i2c/algos/i2c-algo-sgi.c.sav 2005-02-23 12:07:40.000000000 -0800
+++ ./drivers/i2c/algos/i2c-algo-sgi.c 2005-02-23 12:29:58.000000000 -0800
@@ -131,7 +131,7 @@
return 0;
}
-static int sgi_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[],
+static int sgi_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs,
int num)
{
struct i2c_algo_sgi_data *adap = i2c_adap->algo_data;
--- ./drivers/media/video/saa7134/saa7134-i2c.c.sav 2005-02-23 12:08:12.000000000 -0800
+++ ./drivers/media/video/saa7134/saa7134-i2c.c 2005-02-23 12:30:20.000000000 -0800
@@ -236,7 +236,7 @@
}
static int saa7134_i2c_xfer(struct i2c_adapter *i2c_adap,
- struct i2c_msg msgs[], int num)
+ struct i2c_msg *msgs, int num)
{
struct saa7134_dev *dev = i2c_adap->algo_data;
enum i2c_status status;
--- ./drivers/media/video/bttv-i2c.c.sav 2005-02-23 12:08:29.000000000 -0800
+++ ./drivers/media/video/bttv-i2c.c 2005-02-23 12:30:56.000000000 -0800
@@ -245,7 +245,7 @@
return retval;
}
-static int bttv_i2c_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg msgs[], int num)
+static int bttv_i2c_xfer(struct i2c_adapter *i2c_adap, struct i2c_msg *msgs, int num)
{
struct bttv *btv = i2c_get_adapdata(i2c_adap);
int retval = 0;
--- ./drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c.sav 2005-02-23 12:09:43.000000000 -0800
+++ ./drivers/media/dvb/dibusb/dvb-dibusb-fe-i2c.c 2005-02-23 12:31:38.000000000 -0800
@@ -38,7 +38,7 @@
/*
* I2C master xfer function
*/
-static int dibusb_i2c_xfer(struct i2c_adapter *adap,struct i2c_msg msg[],int num)
+static int dibusb_i2c_xfer(struct i2c_adapter *adap,struct i2c_msg *msg,int num)
{
struct usb_dibusb *dib = i2c_get_adapdata(adap);
int i;
--- ./drivers/media/common/saa7146_i2c.c.sav 2005-02-23 12:10:06.000000000 -0800
+++ ./drivers/media/common/saa7146_i2c.c 2005-02-23 12:32:45.000000000 -0800
@@ -25,7 +25,7 @@
sent through the saa7146. have a look at the specifications p. 122 ff
to understand this. it returns the number of u32s to send, or -1
in case of an error. */
-static int saa7146_i2c_msg_prepare(const struct i2c_msg m[], int num, u32 *op)
+static int saa7146_i2c_msg_prepare(const struct i2c_msg *m, int num, u32 *op)
{
int h1, h2;
int i, j, addr;
@@ -89,7 +89,7 @@
which bytes were read through the adapter and write them back to the corresponding
i2c-message. but instead, we simply write back all bytes.
fixme: this could be improved. */
-static int saa7146_i2c_msg_cleanup(const struct i2c_msg m[], int num, u32 *op)
+static int saa7146_i2c_msg_cleanup(const struct i2c_msg *m, int num, u32 *op)
{
int i, j;
int op_count = 0;
@@ -272,7 +272,7 @@
return 0;
}
-int saa7146_i2c_transfer(struct saa7146_dev *dev, const struct i2c_msg msgs[], int num, int retries)
+int saa7146_i2c_transfer(struct saa7146_dev *dev, const struct i2c_msg *msgs, int num, int retries)
{
int i = 0, count = 0;
u32* buffer = dev->d_i2c.cpu_addr;
@@ -372,7 +372,7 @@
}
/* utility functions */
-static int saa7146_i2c_xfer(struct i2c_adapter* adapter, struct i2c_msg msg[], int num)
+static int saa7146_i2c_xfer(struct i2c_adapter* adapter, struct i2c_msg *msg, int num)
{
struct saa7146_dev* dev = i2c_get_adapdata(adapter);
next prev parent reply other threads:[~2005-03-05 0:00 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-03-04 20:35 [BK PATCH] I2C patches for 2.6.11 Greg KH
2005-03-04 20:36 ` [PATCH] I2C: add fscpos chip driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Allow it87 pwm reconfiguration Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Fix up some build warnings in the fscpos driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill i2c_client.id (1/5) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill i2c_client.id (2/5) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill i2c_client.id (3/5) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill i2c_client.id (4/5) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill i2c_client.id (5/5) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: just delete the id field, let's not delay it any longer Greg KH
2005-03-04 20:36 ` [PATCH] I2C: fix for fscpos voltage values Greg KH
2005-03-04 20:36 ` [PATCH] I2C: i2c-dev namespace cleanup Greg KH
2005-03-04 20:36 ` [PATCH] I2C: lm78 driver improvement Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Enable w83781d and w83627hf temperature channels Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Kill unused includes in i2c-sensor-detect.c Greg KH
2005-03-04 20:36 ` [PATCH] I2C: unnecessary #includes in asb100.c Greg KH
2005-03-04 20:36 ` [PATCH] I2C: lm80 driver improvement Greg KH
2005-03-04 20:36 ` [PATCH] i2c-core.c: make some code static Greg KH
2005-03-04 20:36 ` [PATCH] I2C: use time_after instead of comparing jiffies Greg KH
2005-03-04 20:36 ` [PATCH] I2C: add ST M41T00 I2C RTC chip driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: New chip driver: sis5595 Greg KH
2005-03-04 20:36 ` [PATCH] I2C: add Marvell mv64xxx i2c driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: add GL520SM Sensor Chip driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: improve debugging output Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Enable I2C_PIIX4 for 64-bit platforms Greg KH
2005-03-04 20:36 ` [PATCH] I2C: fix typo in drivers/i2c/busses/i2c-ixp4xx.c Greg KH
2005-03-04 20:36 ` [PATCH] I2C i2c-nforce2: add support for nForce4 (patch against 2.6.11-rc4) Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Remove NULL client checks in rtc8564 driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Make i2c list terminators explicitely unsigned Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Fix some gcc 4.0 compile failures and warnings Greg KH
2005-03-04 20:36 ` [PATCH] I2C: S3C2410 missing I2C_CLASS_HWMON Greg KH
2005-03-04 20:36 ` [PATCH] I2C: minor I2C cleanups Greg KH
2005-03-04 20:36 ` [PATCH] Add class definition to the elektor bus driver Greg KH
2005-03-04 20:36 ` [PATCH] I2C: saa7146 build fix Greg KH
2005-03-04 20:36 ` [PATCH] I2C: w83627hf needs i2c-isa Greg KH
2005-03-04 20:36 ` [PATCH] I2C: fixed up the i2c-id.h algo ids Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Change of i2c co-maintainer Greg KH
2005-03-04 20:36 ` [PATCH] I2C: Trivial indentation fix in i2c/chips/Kconfig Greg KH
2005-03-04 22:55 ` Mickey Stein [this message]
2005-03-04 23:02 ` [PATCH] I2C: Fix some gcc 4.0 compile failures and warnings Greg KH
2005-03-05 5:57 ` [RFQ] Rules for accepting patches into the linux-releases tree Shawn Starr
2005-03-05 6:11 ` Randy.Dunlap
2005-03-05 16:33 ` Greg KH
2005-03-24 6:39 ` [2.6.11.5][BUILD] i2c.h breakage in 2.6.12-rc1 + -mm only Shawn Starr
2005-03-24 8:03 ` Greg KH
2005-03-05 11:59 ` [BK PATCH] I2C patches for 2.6.11 Jean Delvare
2005-03-06 6:55 ` Greg KH
2005-03-07 8:50 ` Adrian Bunk
2005-03-08 8:41 ` Domen Puncer
-- strict thread matches above, loose matches on Subject: below --
2005-02-23 22:12 [PATCH] [i2c]: Fix some gcc 4.0 compile failures and warnings Mickey Stein
2005-02-23 22:57 ` linux-os
2005-02-23 23:17 ` Greg KH
2005-02-25 20:07 ` Greg KH
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=4228E74F.9070706@pacbell.net \
--to=yekkim@pacbell.net \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sensors@Stimpy.netroedge.com \
/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