From: Greg KH <[email protected]>

3.5-stable review patch.  If anyone has any objections, please let me know.

------------------

From: Karsten Keil <[email protected]>

commit b41a9a66f67817f8acd85bd650e012a14da39faa upstream.

The test for the fillempty condition was wrong in one place.
Changed the variable to the right boolean type.

Signed-off-by: Karsten Keil <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>

---
 drivers/isdn/hardware/mISDN/avmfritz.c |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

--- a/drivers/isdn/hardware/mISDN/avmfritz.c
+++ b/drivers/isdn/hardware/mISDN/avmfritz.c
@@ -449,7 +449,8 @@ hdlc_fill_fifo(struct bchannel *bch)
 {
        struct fritzcard *fc = bch->hw;
        struct hdlc_hw *hdlc;
-       int count, fs, cnt = 0, idx, fillempty = 0;
+       int count, fs, cnt = 0, idx;
+       bool fillempty = false;
        u8 *p;
        u32 *ptr, val, addr;
 
@@ -462,7 +463,7 @@ hdlc_fill_fifo(struct bchannel *bch)
                        return;
                count = fs;
                p = bch->fill;
-               fillempty = 1;
+               fillempty = true;
        } else {
                count = bch->tx_skb->len - bch->tx_idx;
                if (count <= 0)
@@ -477,7 +478,7 @@ hdlc_fill_fifo(struct bchannel *bch)
                        hdlc->ctrl.sr.cmd |= HDLC_CMD_XME;
        }
        ptr = (u32 *)p;
-       if (fillempty) {
+       if (!fillempty) {
                pr_debug("%s.B%d: %d/%d/%d", fc->name, bch->nr, count,
                         bch->tx_idx, bch->tx_skb->len);
                bch->tx_idx += count;


--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to