Message ID | 3bf865c2-2c1b-e419-2f8f-44295e495197@gmail.com |
---|---|
State | Superseded |
Headers | show |
Series | i2c: i801: Series with improvements | expand |
On Sun, 01 Aug 2021 16:19:24 +0200, Heiner Kallweit wrote: > If a user is interested in such details he can enable smbus tracing. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > --- > drivers/i2c/busses/i2c-i801.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c > index 504f02e1e..d971ee20c 100644 > --- a/drivers/i2c/busses/i2c-i801.c > +++ b/drivers/i2c/busses/i2c-i801.c > @@ -558,10 +558,6 @@ static void i801_isr_byte_done(struct i801_priv *priv) > priv->len); > /* FIXME: Recover */ > priv->len = I2C_SMBUS_BLOCK_MAX; > - } else { > - dev_dbg(&priv->pci_dev->dev, > - "SMBus block read size is %d\n", > - priv->len); > } > priv->data[-1] = priv->len; > } I think this is a leftover from the development phase, when we were just adding support for this feature and needed to verify that the interrupt callback was being called when and only when needed. I agree it no longer has any value and can be removed now. Reviewed-by: Jean Delvare <jdelvare@suse.de> Tested-by: Jean Delvare <jdelvare@suse.de>
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c index 504f02e1e..d971ee20c 100644 --- a/drivers/i2c/busses/i2c-i801.c +++ b/drivers/i2c/busses/i2c-i801.c @@ -558,10 +558,6 @@ static void i801_isr_byte_done(struct i801_priv *priv) priv->len); /* FIXME: Recover */ priv->len = I2C_SMBUS_BLOCK_MAX; - } else { - dev_dbg(&priv->pci_dev->dev, - "SMBus block read size is %d\n", - priv->len); } priv->data[-1] = priv->len; }
If a user is interested in such details he can enable smbus tracing. Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> --- drivers/i2c/busses/i2c-i801.c | 4 ---- 1 file changed, 4 deletions(-)