diff options
| author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2007-08-10 18:25:40 -0400 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2007-08-14 01:52:12 -0400 |
| commit | 48e3eeb1a1a89a3ea256cf6d3c898e753787cfaa (patch) | |
| tree | 4f607c078f1a4a23becd140298888f846886261f | |
| parent | 1ee325438cdbe374ebea6e3d2f19204072c15038 (diff) | |
[IRDA] irda-usb.c: mostly kmalloc + memset conversion to k[cz]alloc
drivers/net/irda/irda-usb.c | 59694 -> 59541 (-153 bytes)
drivers/net/irda/irda-usb.o | 170588 -> 169256 (-1332 bytes)
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/irda/irda-usb.c | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/drivers/net/irda/irda-usb.c b/drivers/net/irda/irda-usb.c index 0ac240ca905b..3b0fd83fa266 100644 --- a/drivers/net/irda/irda-usb.c +++ b/drivers/net/irda/irda-usb.c | |||
| @@ -1561,10 +1561,9 @@ static inline struct irda_class_desc *irda_usb_find_class_desc(struct usb_interf | |||
| 1561 | struct irda_class_desc *desc; | 1561 | struct irda_class_desc *desc; |
| 1562 | int ret; | 1562 | int ret; |
| 1563 | 1563 | ||
| 1564 | desc = kmalloc(sizeof (*desc), GFP_KERNEL); | 1564 | desc = kzalloc(sizeof(*desc), GFP_KERNEL); |
| 1565 | if (desc == NULL) | 1565 | if (!desc) |
| 1566 | return NULL; | 1566 | return NULL; |
| 1567 | memset(desc, 0, sizeof(*desc)); | ||
| 1568 | 1567 | ||
| 1569 | /* USB-IrDA class spec 1.0: | 1568 | /* USB-IrDA class spec 1.0: |
| 1570 | * 6.1.3: Standard "Get Descriptor" Device Request is not | 1569 | * 6.1.3: Standard "Get Descriptor" Device Request is not |
| @@ -1617,7 +1616,7 @@ static int irda_usb_probe(struct usb_interface *intf, | |||
| 1617 | { | 1616 | { |
| 1618 | struct net_device *net; | 1617 | struct net_device *net; |
| 1619 | struct usb_device *dev = interface_to_usbdev(intf); | 1618 | struct usb_device *dev = interface_to_usbdev(intf); |
| 1620 | struct irda_usb_cb *self = NULL; | 1619 | struct irda_usb_cb *self; |
| 1621 | struct usb_host_interface *interface; | 1620 | struct usb_host_interface *interface; |
| 1622 | struct irda_class_desc *irda_desc; | 1621 | struct irda_class_desc *irda_desc; |
| 1623 | int ret = -ENOMEM; | 1622 | int ret = -ENOMEM; |
| @@ -1655,7 +1654,7 @@ static int irda_usb_probe(struct usb_interface *intf, | |||
| 1655 | self->header_length = USB_IRDA_HEADER; | 1654 | self->header_length = USB_IRDA_HEADER; |
| 1656 | } | 1655 | } |
| 1657 | 1656 | ||
| 1658 | self->rx_urb = kzalloc(self->max_rx_urb * sizeof(struct urb *), | 1657 | self->rx_urb = kcalloc(self->max_rx_urb, sizeof(struct urb *), |
| 1659 | GFP_KERNEL); | 1658 | GFP_KERNEL); |
| 1660 | 1659 | ||
| 1661 | for (i = 0; i < self->max_rx_urb; i++) { | 1660 | for (i = 0; i < self->max_rx_urb; i++) { |
| @@ -1715,7 +1714,7 @@ static int irda_usb_probe(struct usb_interface *intf, | |||
| 1715 | /* Find IrDA class descriptor */ | 1714 | /* Find IrDA class descriptor */ |
| 1716 | irda_desc = irda_usb_find_class_desc(intf); | 1715 | irda_desc = irda_usb_find_class_desc(intf); |
| 1717 | ret = -ENODEV; | 1716 | ret = -ENODEV; |
| 1718 | if (irda_desc == NULL) | 1717 | if (!irda_desc) |
| 1719 | goto err_out_3; | 1718 | goto err_out_3; |
| 1720 | 1719 | ||
| 1721 | if (self->needspatch) { | 1720 | if (self->needspatch) { |
| @@ -1738,15 +1737,13 @@ static int irda_usb_probe(struct usb_interface *intf, | |||
| 1738 | /* Don't change this buffer size and allocation without doing | 1737 | /* Don't change this buffer size and allocation without doing |
| 1739 | * some heavy and complete testing. Don't ask why :-( | 1738 | * some heavy and complete testing. Don't ask why :-( |
| 1740 | * Jean II */ | 1739 | * Jean II */ |
| 1741 | self->speed_buff = kmalloc(IRDA_USB_SPEED_MTU, GFP_KERNEL); | 1740 | self->speed_buff = kzalloc(IRDA_USB_SPEED_MTU, GFP_KERNEL); |
| 1742 | if (self->speed_buff == NULL) | 1741 | if (!self->speed_buff) |
| 1743 | goto err_out_3; | 1742 | goto err_out_3; |
| 1744 | 1743 | ||
| 1745 | memset(self->speed_buff, 0, IRDA_USB_SPEED_MTU); | ||
| 1746 | |||
| 1747 | self->tx_buff = kzalloc(IRDA_SKB_MAX_MTU + self->header_length, | 1744 | self->tx_buff = kzalloc(IRDA_SKB_MAX_MTU + self->header_length, |
| 1748 | GFP_KERNEL); | 1745 | GFP_KERNEL); |
| 1749 | if (self->tx_buff == NULL) | 1746 | if (!self->tx_buff) |
| 1750 | goto err_out_4; | 1747 | goto err_out_4; |
| 1751 | 1748 | ||
| 1752 | ret = irda_usb_open(self); | 1749 | ret = irda_usb_open(self); |
| @@ -1767,12 +1764,11 @@ static int irda_usb_probe(struct usb_interface *intf, | |||
| 1767 | 1764 | ||
| 1768 | /* replace IrDA class descriptor with what patched device is now reporting */ | 1765 | /* replace IrDA class descriptor with what patched device is now reporting */ |
| 1769 | irda_desc = irda_usb_find_class_desc (self->usbintf); | 1766 | irda_desc = irda_usb_find_class_desc (self->usbintf); |
| 1770 | if (irda_desc == NULL) { | 1767 | if (!irda_desc) { |
| 1771 | ret = -ENODEV; | 1768 | ret = -ENODEV; |
| 1772 | goto err_out_6; | 1769 | goto err_out_6; |
| 1773 | } | 1770 | } |
| 1774 | if (self->irda_desc) | 1771 | kfree(self->irda_desc); |
| 1775 | kfree (self->irda_desc); | ||
| 1776 | self->irda_desc = irda_desc; | 1772 | self->irda_desc = irda_desc; |
| 1777 | irda_usb_init_qos(self); | 1773 | irda_usb_init_qos(self); |
| 1778 | } | 1774 | } |
