lan78xx: Remove BUG_ON()
Removing BUG_ON() Signed-off-by: Woojung Huh <woojung.huh@microchip.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Родитель
81c38e811a
Коммит
d57f6232c1
|
@ -291,8 +291,6 @@ static int lan78xx_read_reg(struct lan78xx_net *dev, u32 index, u32 *data)
|
||||||
u32 *buf = kmalloc(sizeof(u32), GFP_KERNEL);
|
u32 *buf = kmalloc(sizeof(u32), GFP_KERNEL);
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
BUG_ON(!dev);
|
|
||||||
|
|
||||||
if (!buf)
|
if (!buf)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
|
@ -319,8 +317,6 @@ static int lan78xx_write_reg(struct lan78xx_net *dev, u32 index, u32 data)
|
||||||
u32 *buf = kmalloc(sizeof(u32), GFP_KERNEL);
|
u32 *buf = kmalloc(sizeof(u32), GFP_KERNEL);
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
BUG_ON(!dev);
|
|
||||||
|
|
||||||
if (!buf)
|
if (!buf)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
|
@ -351,10 +347,6 @@ static int lan78xx_read_stats(struct lan78xx_net *dev,
|
||||||
u32 *src;
|
u32 *src;
|
||||||
u32 *dst;
|
u32 *dst;
|
||||||
|
|
||||||
BUG_ON(!dev);
|
|
||||||
BUG_ON(!data);
|
|
||||||
BUG_ON(sizeof(struct lan78xx_statstage) != 0xBC);
|
|
||||||
|
|
||||||
stats = kmalloc(sizeof(*stats), GFP_KERNEL);
|
stats = kmalloc(sizeof(*stats), GFP_KERNEL);
|
||||||
if (!stats)
|
if (!stats)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
@ -687,9 +679,6 @@ static int lan78xx_read_raw_eeprom(struct lan78xx_net *dev, u32 offset,
|
||||||
u32 val;
|
u32 val;
|
||||||
int i, ret;
|
int i, ret;
|
||||||
|
|
||||||
BUG_ON(!dev);
|
|
||||||
BUG_ON(!data);
|
|
||||||
|
|
||||||
ret = lan78xx_eeprom_confirm_not_busy(dev);
|
ret = lan78xx_eeprom_confirm_not_busy(dev);
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
@ -737,9 +726,6 @@ static int lan78xx_write_raw_eeprom(struct lan78xx_net *dev, u32 offset,
|
||||||
u32 val;
|
u32 val;
|
||||||
int i, ret;
|
int i, ret;
|
||||||
|
|
||||||
BUG_ON(!dev);
|
|
||||||
BUG_ON(!data);
|
|
||||||
|
|
||||||
ret = lan78xx_eeprom_confirm_not_busy(dev);
|
ret = lan78xx_eeprom_confirm_not_busy(dev);
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
@ -2220,20 +2206,10 @@ static enum skb_state defer_bh(struct lan78xx_net *dev, struct sk_buff *skb,
|
||||||
spin_lock_irqsave(&list->lock, flags);
|
spin_lock_irqsave(&list->lock, flags);
|
||||||
old_state = entry->state;
|
old_state = entry->state;
|
||||||
entry->state = state;
|
entry->state = state;
|
||||||
if (!list->prev)
|
|
||||||
BUG_ON(!list->prev);
|
|
||||||
if (!list->next)
|
|
||||||
BUG_ON(!list->next);
|
|
||||||
if (!skb->prev || !skb->next)
|
|
||||||
BUG_ON(true);
|
|
||||||
|
|
||||||
__skb_unlink(skb, list);
|
__skb_unlink(skb, list);
|
||||||
spin_unlock(&list->lock);
|
spin_unlock(&list->lock);
|
||||||
spin_lock(&dev->done.lock);
|
spin_lock(&dev->done.lock);
|
||||||
if (!dev->done.prev)
|
|
||||||
BUG_ON(!dev->done.prev);
|
|
||||||
if (!dev->done.next)
|
|
||||||
BUG_ON(!dev->done.next);
|
|
||||||
|
|
||||||
__skb_queue_tail(&dev->done, skb);
|
__skb_queue_tail(&dev->done, skb);
|
||||||
if (skb_queue_len(&dev->done) == 1)
|
if (skb_queue_len(&dev->done) == 1)
|
||||||
|
@ -2749,8 +2725,6 @@ static void lan78xx_tx_bh(struct lan78xx_net *dev)
|
||||||
memcpy(skb->data + pos, skb2->data, skb2->len);
|
memcpy(skb->data + pos, skb2->data, skb2->len);
|
||||||
pos += roundup(skb2->len, sizeof(u32));
|
pos += roundup(skb2->len, sizeof(u32));
|
||||||
dev_kfree_skb(skb2);
|
dev_kfree_skb(skb2);
|
||||||
} else {
|
|
||||||
BUG_ON(true);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2859,11 +2833,6 @@ static void lan78xx_bh(unsigned long param)
|
||||||
struct sk_buff *skb;
|
struct sk_buff *skb;
|
||||||
struct skb_data *entry;
|
struct skb_data *entry;
|
||||||
|
|
||||||
if (!dev->done.prev)
|
|
||||||
BUG_ON(!dev->done.prev);
|
|
||||||
if (!dev->done.next)
|
|
||||||
BUG_ON(!dev->done.next);
|
|
||||||
|
|
||||||
while ((skb = skb_dequeue(&dev->done))) {
|
while ((skb = skb_dequeue(&dev->done))) {
|
||||||
entry = (struct skb_data *)(skb->cb);
|
entry = (struct skb_data *)(skb->cb);
|
||||||
switch (entry->state) {
|
switch (entry->state) {
|
||||||
|
|
Загрузка…
Ссылка в новой задаче