Commit f11d32df authored by Harvey Harrison's avatar Harvey Harrison Committed by David S. Miller

mISDN: endian annotations for struct zt

Found two possible bugs where the z1 value was used directly without
byteswapping.
Signed-off-by: default avatarHarvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4d5392cc
...@@ -183,8 +183,8 @@ ...@@ -183,8 +183,8 @@
#define D_FREG_MASK 0xF #define D_FREG_MASK 0xF
struct zt { struct zt {
unsigned short z1; /* Z1 pointer 16 Bit */ __le16 z1; /* Z1 pointer 16 Bit */
unsigned short z2; /* Z2 pointer 16 Bit */ __le16 z2; /* Z2 pointer 16 Bit */
}; };
struct dfifo { struct dfifo {
......
...@@ -366,8 +366,7 @@ static void hfcpci_clear_fifo_tx(struct hfc_pci *hc, int fifo) ...@@ -366,8 +366,7 @@ static void hfcpci_clear_fifo_tx(struct hfc_pci *hc, int fifo)
bzt->f2 = MAX_B_FRAMES; bzt->f2 = MAX_B_FRAMES;
bzt->f1 = bzt->f2; /* init F pointers to remain constant */ bzt->f1 = bzt->f2; /* init F pointers to remain constant */
bzt->za[MAX_B_FRAMES].z1 = cpu_to_le16(B_FIFO_SIZE + B_SUB_VAL - 1); bzt->za[MAX_B_FRAMES].z1 = cpu_to_le16(B_FIFO_SIZE + B_SUB_VAL - 1);
bzt->za[MAX_B_FRAMES].z2 = cpu_to_le16( bzt->za[MAX_B_FRAMES].z2 = cpu_to_le16(B_FIFO_SIZE + B_SUB_VAL - 2);
le16_to_cpu(bzt->za[MAX_B_FRAMES].z1 - 1));
if (fifo_state) if (fifo_state)
hc->hw.fifo_en |= fifo_state; hc->hw.fifo_en |= fifo_state;
Write_hfc(hc, HFCPCI_FIFO_EN, hc->hw.fifo_en); Write_hfc(hc, HFCPCI_FIFO_EN, hc->hw.fifo_en);
...@@ -482,7 +481,7 @@ receive_dmsg(struct hfc_pci *hc) ...@@ -482,7 +481,7 @@ receive_dmsg(struct hfc_pci *hc)
df->f2 = ((df->f2 + 1) & MAX_D_FRAMES) | df->f2 = ((df->f2 + 1) & MAX_D_FRAMES) |
(MAX_D_FRAMES + 1); /* next buffer */ (MAX_D_FRAMES + 1); /* next buffer */
df->za[df->f2 & D_FREG_MASK].z2 = df->za[df->f2 & D_FREG_MASK].z2 =
cpu_to_le16((zp->z2 + rcnt) & (D_FIFO_SIZE - 1)); cpu_to_le16((le16_to_cpu(zp->z2) + rcnt) & (D_FIFO_SIZE - 1));
} else { } else {
dch->rx_skb = mI_alloc_skb(rcnt - 3, GFP_ATOMIC); dch->rx_skb = mI_alloc_skb(rcnt - 3, GFP_ATOMIC);
if (!dch->rx_skb) { if (!dch->rx_skb) {
...@@ -526,7 +525,7 @@ receive_dmsg(struct hfc_pci *hc) ...@@ -526,7 +525,7 @@ receive_dmsg(struct hfc_pci *hc)
int int
hfcpci_empty_fifo_trans(struct bchannel *bch, struct bzfifo *bz, u_char *bdata) hfcpci_empty_fifo_trans(struct bchannel *bch, struct bzfifo *bz, u_char *bdata)
{ {
unsigned short *z1r, *z2r; __le16 *z1r, *z2r;
int new_z2, fcnt, maxlen; int new_z2, fcnt, maxlen;
u_char *ptr, *ptr1; u_char *ptr, *ptr1;
...@@ -724,7 +723,7 @@ hfcpci_fill_fifo(struct bchannel *bch) ...@@ -724,7 +723,7 @@ hfcpci_fill_fifo(struct bchannel *bch)
struct bzfifo *bz; struct bzfifo *bz;
u_char *bdata; u_char *bdata;
u_char new_f1, *src, *dst; u_char new_f1, *src, *dst;
unsigned short *z1t, *z2t; __le16 *z1t, *z2t;
if ((bch->debug & DEBUG_HW_BCHANNEL) && !(bch->debug & DEBUG_HW_BFIFO)) if ((bch->debug & DEBUG_HW_BCHANNEL) && !(bch->debug & DEBUG_HW_BFIFO))
printk(KERN_DEBUG "%s\n", __func__); printk(KERN_DEBUG "%s\n", __func__);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment