]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
at86rf230: fix __at86rf230_read_subreg function
authorAlexander Aring <alex.aring@gmail.com>
Mon, 14 Apr 2014 16:48:02 +0000 (18:48 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 15 Apr 2014 04:08:22 +0000 (00:08 -0400)
The __at86rf230_read_subreg function don't mask and shift register
contents which it should do. This patch adds the necessary masks and
shift operations in this function.

Since we have csma support this can make some trouble on state changes.
Since CSMA support turned on some bits in the TRX_STATUS register that
used to be zero, not masking broke checking of the TRX_STATUS field
after commanding a state change.

Signed-off-by: Alexander Aring <alex.aring@gmail.com>
Reviewed-by: Werner Almesberger <werner@almesberger.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ieee802154/at86rf230.c

index e102eef0b332b3d7d9548dce6b9e94aa3d882e0d..e36f194673a45a2035a15830571e4e2c02039839 100644 (file)
@@ -365,7 +365,7 @@ __at86rf230_read_subreg(struct at86rf230_local *lp,
        dev_vdbg(&lp->spi->dev, "buf[1] = %02x\n", buf[1]);
 
        if (status == 0)
-               *data = buf[1];
+               *data = (buf[1] & mask) >> shift;
 
        return status;
 }