From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH v3 2/5] i2c: i2c-mpc: use dev based printout function Date: Tue, 7 Apr 2009 08:37:05 -0700 Message-ID: References: <20090407082052.477328750@denx.de> <20090407082231.335327816@denx.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20090407082231.335327816-ynQEQJNshbs@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Wolfgang Grandegger Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Ben Dooks , linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org, devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.org, Kumar Gala List-Id: linux-i2c@vger.kernel.org On Tue, Apr 7, 2009 at 1:20 AM, Wolfgang Grandegger = wrote: > This patch used the dev_dbg, dev_err, etc. functions for debug > and error output instead of printk and pr_debug. > > Signed-off-by: Wolfgang Grandegger Acked-by: Grant Likely > --- > =A0drivers/i2c/busses/i2c-mpc.c | =A0 31 +++++++++++++++++-----------= --- > =A01 file changed, 17 insertions(+), 14 deletions(-) > > Index: linux-2.6-galak/drivers/i2c/busses/i2c-mpc.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > --- linux-2.6-galak.orig/drivers/i2c/busses/i2c-mpc.c =A0 2009-04-07 = 10:09:13.271719088 +0200 > +++ linux-2.6-galak/drivers/i2c/busses/i2c-mpc.c =A0 =A0 =A0 =A02009-= 04-07 10:09:14.012720756 +0200 > @@ -50,6 +50,7 @@ > =A0#define CSR_RXAK 0x01 > > =A0struct mpc_i2c { > + =A0 =A0 =A0 struct device *dev; > =A0 =A0 =A0 =A0void __iomem *base; > =A0 =A0 =A0 =A0u32 interrupt; > =A0 =A0 =A0 =A0wait_queue_head_t queue; > @@ -104,7 +105,7 @@ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0while (!(readb(i2c->base + MPC_I2C_SR)= & CSR_MIF)) { > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0schedule(); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (time_after(jiffies= , orig_jiffies + timeout)) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debu= g("I2C: timeout\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg= (i2c->dev, "timeout\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0writec= cr(i2c, 0); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result= =3D -EIO; > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0break; > @@ -118,7 +119,7 @@ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(i2c->interrupt & CSR_= MIF), timeout); > > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (unlikely(!(i2c->interrupt & CSR_MI= =46))) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: wait tim= eout\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "wait= timeout\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0writeccr(i2c, 0); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result =3D -ETIMEDOUT; > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > @@ -131,17 +132,17 @@ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return result; > > =A0 =A0 =A0 =A0if (!(x & CSR_MCF)) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: unfinished\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "unfinished\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -EIO; > =A0 =A0 =A0 =A0} > > =A0 =A0 =A0 =A0if (x & CSR_MAL) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: MAL\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "MAL\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -EIO; > =A0 =A0 =A0 =A0} > > =A0 =A0 =A0 =A0if (writing && (x & CSR_RXAK)) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: No RXAK\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "No RXAK\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0/* generate stop */ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0writeccr(i2c, CCR_MEN); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -EIO; > @@ -263,12 +264,12 @@ > =A0 =A0 =A0 =A0/* Allow bus up to 1s to become not busy */ > =A0 =A0 =A0 =A0while (readb(i2c->base + MPC_I2C_SR) & CSR_MBB) { > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (signal_pending(current)) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: Interrup= ted\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "Inte= rrupted\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0writeccr(i2c, 0); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -EINTR; > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (time_after(jiffies, orig_jiffies += HZ)) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("I2C: timeout\= n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, "time= out\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (readb(i2c->base + = MPC_I2C_SR) =3D=3D > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(CSR_MCF | CSR= _MBB | CSR_RXAK)) > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0mpc_i2= c_fixup(i2c); > @@ -279,9 +280,10 @@ > > =A0 =A0 =A0 =A0for (i =3D 0; ret >=3D 0 && i < num; i++) { > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0pmsg =3D &msgs[i]; > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 pr_debug("Doing %s %d bytes to 0x%02x -= %d of %d messages\n", > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0pmsg->flags & I2C_M_= RD ? "read" : "write", > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0pmsg->len, pmsg->add= r, i + 1, num); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_dbg(i2c->dev, > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 "Doing %s %d bytes to 0= x%02x - %d of %d messages\n", > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pmsg->flags & I2C_M_RD = ? "read" : "write", > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 pmsg->len, pmsg->addr, = i + 1, num); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (pmsg->flags & I2C_M_RD) > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0ret =3D > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0mpc_read(i2c, = pmsg->addr, pmsg->buf, pmsg->len, i); > @@ -320,6 +322,8 @@ > =A0 =A0 =A0 =A0if (!i2c) > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0return -ENOMEM; > > + =A0 =A0 =A0 i2c->dev =3D &op->dev; /* for debug and error output */ > + > =A0 =A0 =A0 =A0if (of_get_property(op->node, "dfsrr", NULL)) > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0i2c->flags |=3D FSL_I2C_DEV_SEPARATE_D= =46SRR; > > @@ -331,7 +335,7 @@ > > =A0 =A0 =A0 =A0i2c->base =3D of_iomap(op->node, 0); > =A0 =A0 =A0 =A0if (!i2c->base) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 printk(KERN_ERR "i2c-mpc - failed to ma= p controller\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_err(i2c->dev, "failed to map contro= ller\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result =3D -ENOMEM; > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0goto fail_map; > =A0 =A0 =A0 =A0} > @@ -341,8 +345,7 @@ > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0result =3D request_irq(i2c->irq, mpc_i= 2c_isr, > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= IRQF_SHARED, "i2c-mpc", i2c); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (result < 0) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 printk(KERN_ERR > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0"i2c-mpc= - failed to attach interrupt\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_err(i2c->dev, "fail= ed to attach interrupt\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0goto fail_request; > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0} > =A0 =A0 =A0 =A0} > @@ -357,7 +360,7 @@ > > =A0 =A0 =A0 =A0result =3D i2c_add_adapter(&i2c->adap); > =A0 =A0 =A0 =A0if (result < 0) { > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 printk(KERN_ERR "i2c-mpc - failed to ad= d adapter\n"); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 dev_err(i2c->dev, "failed to add adapte= r\n"); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0goto fail_add; > =A0 =A0 =A0 =A0} > =A0 =A0 =A0 =A0of_register_i2c_devices(&i2c->adap, op->node); > > --=20 Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd.