lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 04 May 2011 15:35:55 +0400 From: Sergei Shtylyov <sshtylyov@...sta.com> To: Joe Perches <joe@...ches.com> CC: Oliver Neukum <oneukum@...e.de>, David Brownell <dbrownell@...rs.sourceforge.net>, Greg Kroah-Hartman <gregkh@...e.de>, netdev@...r.kernel.org, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 2/2] usbnet: Convert dbg to dev_dbg and neatening Hello. On 03-05-2011 22:17, Joe Perches wrote: > Use the more standard logging form. > Add a bit more tidying style. Style changes look rather doubtful to me... > Signed-off-by: Joe Perches<joe@...ches.com> > --- > drivers/net/usb/usbnet.c | 23 +++++++++++------------ > 1 files changed, 11 insertions(+), 12 deletions(-) > diff --git a/drivers/net/usb/usbnet.c b/drivers/net/usb/usbnet.c > index 28aecbb..b803085 100644 > --- a/drivers/net/usb/usbnet.c > +++ b/drivers/net/usb/usbnet.c > @@ -192,8 +192,8 @@ static int init_status(struct usbnet *dev, struct usb_interface *intf) > return 0; > > pipe = usb_rcvintpipe(dev->udev, > - dev->status->desc.bEndpointAddress > - & USB_ENDPOINT_NUMBER_MASK); > + (dev->status->desc.bEndpointAddress > + & USB_ENDPOINT_NUMBER_MASK)); Why add parens? > @@ -1345,8 +1346,9 @@ usbnet_probe(struct usb_interface *udev, const struct usb_device_id *prod) > dev->intf = udev; > dev->driver_info = info; > dev->driver_name = name; > - dev->msg_enable = netif_msg_init(msg_level, NETIF_MSG_DRV > - | NETIF_MSG_PROBE | NETIF_MSG_LINK); > + dev->msg_enable = netif_msg_init(msg_level, (NETIF_MSG_DRV | > + NETIF_MSG_PROBE | > + NETIF_MSG_LINK)); Why add parens? > @@ -1466,8 +1468,7 @@ EXPORT_SYMBOL_GPL(usbnet_probe); > > /*-------------------------------------------------------------------------*/ > > -/* > - * suspend the whole driver as soon as the first interface is suspended > +/* suspend the whole driver as soon as the first interface is suspended Why? This already corresponded to the preferred multi-line comment style... > * resume only when the last interface is resumed > */ > > @@ -1485,16 +1486,14 @@ int usbnet_suspend(struct usb_interface *intf, pm_message_t message) > set_bit(EVENT_DEV_ASLEEP,&dev->flags); > spin_unlock_irq(&dev->txq.lock); > } > - /* > - * accelerate emptying of the rx and queues, to avoid > + /* accelerate emptying of the rx and queues, to avoid Why? > * having everything error out. > */ > netif_device_detach(dev->net); > usbnet_terminate_urbs(dev); > usb_kill_urb(dev->interrupt); > > - /* > - * reattach so runtime management can use and > + /* reattach so runtime management can use and Why? WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists