Re: [PATCH 3/3] i2c: octeon: Avoid printk after too long SMBUS message

[Wolfram Sang]

On Wed, Jun 08, 2016 at 08:51:19AM +0200, Jan Glauber wrote:
> Remove the warning about a too long SMBUS message because
> the ipmi_ssif driver triggers this warning too frequently so it
> spams the message log.
> 
> Signed-off-by: Jan Glauber 

Applied to for-current, thanks!



signature.asc
Description: PGP signature

Re: [PATCH 3/3] i2c: octeon: Avoid printk after too long SMBUS message

[Wolfram Sang]

On Wed, Jun 08, 2016 at 08:51:19AM +0200, Jan Glauber wrote:
> Remove the warning about a too long SMBUS message because
> the ipmi_ssif driver triggers this warning too frequently so it
> spams the message log.
> 
> Signed-off-by: Jan Glauber 

Applied to for-current, thanks!



signature.asc
Description: PGP signature

[PATCH 3/3] i2c: octeon: Avoid printk after too long SMBUS message

[Jan Glauber]

Remove the warning about a too long SMBUS message because
the ipmi_ssif driver triggers this warning too frequently so it
spams the message log.

Signed-off-by: Jan Glauber 
---
 drivers/i2c/busses/i2c-octeon.c | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/i2c/busses/i2c-octeon.c b/drivers/i2c/busses/i2c-octeon.c
index 8ade7fb..83fd6d8 100644
--- a/drivers/i2c/busses/i2c-octeon.c
+++ b/drivers/i2c/busses/i2c-octeon.c
@@ -975,12 +975,8 @@ restart:
 
data[i] = octeon_i2c_data_read(i2c);
if (recv_len && i == 0) {
-   if (data[i] > I2C_SMBUS_BLOCK_MAX + 1) {
-   dev_err(i2c->dev,
-   "%s: read len > I2C_SMBUS_BLOCK_MAX 
%d\n",
-   __func__, data[i]);
+   if (data[i] > I2C_SMBUS_BLOCK_MAX + 1)
return -EPROTO;
-   }
length += data[i];
}
 
-- 
2.9.0.rc0.21.g322

[PATCH 3/3] i2c: octeon: Avoid printk after too long SMBUS message

[Jan Glauber]

Remove the warning about a too long SMBUS message because
the ipmi_ssif driver triggers this warning too frequently so it
spams the message log.

Signed-off-by: Jan Glauber 
---
 drivers/i2c/busses/i2c-octeon.c | 6 +-
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/i2c/busses/i2c-octeon.c b/drivers/i2c/busses/i2c-octeon.c
index 8ade7fb..83fd6d8 100644
--- a/drivers/i2c/busses/i2c-octeon.c
+++ b/drivers/i2c/busses/i2c-octeon.c
@@ -975,12 +975,8 @@ restart:
 
data[i] = octeon_i2c_data_read(i2c);
if (recv_len && i == 0) {
-   if (data[i] > I2C_SMBUS_BLOCK_MAX + 1) {
-   dev_err(i2c->dev,
-   "%s: read len > I2C_SMBUS_BLOCK_MAX 
%d\n",
-   __func__, data[i]);
+   if (data[i] > I2C_SMBUS_BLOCK_MAX + 1)
return -EPROTO;
-   }
length += data[i];
}
 
-- 
2.9.0.rc0.21.g322