i2c: pxa: remove some unnecessary debug
authorRussell King <rmk+kernel@armlinux.org.uk>
Mon, 11 May 2020 21:10:42 +0000 (22:10 +0100)
committerWolfram Sang <wsa@kernel.org>
Tue, 12 May 2020 10:32:52 +0000 (12:32 +0200)
Remove unnecessary show_state() in the loop inside
i2c_pxa_pio_set_master(), which can be unnecessarily verbose.

Remove the i2c_pxa_scream_blue_murder() in i2c_pxa_pio_xfer(), which
will trigger if we are probing the I2C bus and a slave does not
respond; this is a normal event, and not something to report.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
drivers/i2c/busses/i2c-pxa.c

index 0becab239476a9f4604d9e27bf3c6ee5f979a346..db739cce93acbe7393d83b338f1fb10041a7adf9 100644 (file)
@@ -1161,10 +1161,8 @@ static int i2c_pxa_pio_set_master(struct pxa_i2c *i2c)
        /*
         * Wait for the bus to become free.
         */
-       while (timeout-- && readl(_ISR(i2c)) & (ISR_IBB | ISR_UB)) {
+       while (timeout-- && readl(_ISR(i2c)) & (ISR_IBB | ISR_UB))
                udelay(1000);
-               show_state(i2c);
-       }
 
        if (timeout < 0) {
                show_state(i2c);