From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Dreier Subject: Re: [PATCH Resubmission] drivers/net/usb: Add new driver ipheth Date: Wed, 07 Apr 2010 15:37:15 -0700 Message-ID: References: <1269984864-28159-1-git-send-email-agimenez@sysvalve.es> <1270678281-20750-1-git-send-email-agimenez@sysvalve.es> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, dgiagio@gmail.com, dborca@yahoo.com, Greg Kroah-Hartman , "David S. Miller" , Jonas =?utf-8?Q?Sj=C3=B6quist?= , Steve Glendinning , Torgny Johansson , David Brownell , Omar Laazimani , linux-usb@vger.kernel.org, netdev@vger.kernel.org To: "L. Alberto =?utf-8?Q?Gim=C3=A9nez?=" Return-path: Received: from sj-iport-5.cisco.com ([171.68.10.87]:41833 "EHLO sj-iport-5.cisco.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751929Ab0DGWh3 (ORCPT ); Wed, 7 Apr 2010 18:37:29 -0400 In-Reply-To: <1270678281-20750-1-git-send-email-agimenez@sysvalve.es> ("L. Alberto =?utf-8?Q?Gim=C3=A9nez=22's?= message of "Thu, 8 Apr 2010 00:11:20 +0200") Sender: netdev-owner@vger.kernel.org List-ID: These are a few of the cosmetic issues alluded to, and by no means merge blockers, but: > + schedule_delayed_work(&dev->carrier_work, IPHETH_CARRIER_CHECK_TIMEOUT); Seems this might as well be round_jiffies_relative(IPHETH_CARRIER_CHECK_TIMEOUT) to avoid extra wakeups. > + netdev = alloc_etherdev(sizeof(struct ipheth_device)); This means that the interface will get an ethX name and look to networkmanager et al like an ethernet device. Seems we would maybe want to make this a "wwan" type device (cf drivers/net/usb/usbnet.c and how it handles FLAG_WWAN)? - R. -- Roland Dreier || For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/index.html