[PATCH 0/3] more cleanups

All of lore.kernel.org
 help / color / mirror / Atom feed

* [PATCH 0/3] more cleanups
@ 2012-01-06  5:10 Stephen Hemminger
  2012-01-06  5:10 ` [PATCH 1/3] usbnet: make ethtool_ops const Stephen Hemminger
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Stephen Hemminger @ 2012-01-06  5:10 UTC (permalink / raw)
  To: David Miller; +Cc: netdev

Just found more places where function tables should be const.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/3] usbnet: make ethtool_ops const
  2012-01-06  5:10 [PATCH 0/3] more cleanups Stephen Hemminger
@ 2012-01-06  5:10 ` Stephen Hemminger
       [not found]   ` <20120106051050.936477418-ZtmgI6mnKB3QT0dZR+AlfA@public.gmane.org>
  2012-01-06  5:10 ` [PATCH 2/3] bcm63xx: " Stephen Hemminger
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 6+ messages in thread
From: Stephen Hemminger @ 2012-01-06  5:10 UTC (permalink / raw)
  To: David Miller, Greg Kroah-Hartman, Oliver Neukum; +Cc: netdev, linux-usb

[-- Attachment #1: usb-ethtool-ops.patch --]
[-- Type: text/plain, Size: 2541 bytes --]

The ethtool_ops table of function pointers should be const.
Fix all the usb network drivers.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>

---
Not sure if you want these through net-next or usb-next?

 drivers/net/usb/asix.c       |    2 +-
 drivers/net/usb/cdc_ncm.c    |    4 ++--
 drivers/net/usb/ipheth.c     |    2 +-
 drivers/net/usb/sierra_net.c |    2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

--- a/drivers/net/usb/asix.c	2011-12-23 21:06:15.724436897 -0800
+++ b/drivers/net/usb/asix.c	2012-01-04 14:12:38.832942290 -0800
@@ -1152,7 +1152,7 @@ static int ax88772_bind(struct usbnet *d
 	return 0;
 }
 
-static struct ethtool_ops ax88178_ethtool_ops = {
+static const struct ethtool_ops ax88178_ethtool_ops = {
 	.get_drvinfo		= asix_get_drvinfo,
 	.get_link		= asix_get_link,
 	.get_msglevel		= usbnet_get_msglevel,
--- a/drivers/net/usb/cdc_ncm.c	2011-12-07 10:54:18.960280313 -0800
+++ b/drivers/net/usb/cdc_ncm.c	2012-01-04 14:12:17.648667526 -0800
@@ -138,7 +138,7 @@ struct cdc_ncm_ctx {
 static void cdc_ncm_tx_timeout(unsigned long arg);
 static const struct driver_info cdc_ncm_info;
 static struct usb_driver cdc_ncm_driver;
-static struct ethtool_ops cdc_ncm_ethtool_ops;
+static const struct ethtool_ops cdc_ncm_ethtool_ops;
 
 static const struct usb_device_id cdc_devs[] = {
 	{ USB_INTERFACE_INFO(USB_CLASS_COMM,
@@ -1220,7 +1220,7 @@ static struct usb_driver cdc_ncm_driver
 	.supports_autosuspend = 1,
 };
 
-static struct ethtool_ops cdc_ncm_ethtool_ops = {
+static const struct ethtool_ops cdc_ncm_ethtool_ops = {
 	.get_drvinfo = cdc_ncm_get_drvinfo,
 	.get_link = usbnet_get_link,
 	.get_msglevel = usbnet_get_msglevel,
--- a/drivers/net/usb/ipheth.c	2011-12-07 10:54:18.964280345 -0800
+++ b/drivers/net/usb/ipheth.c	2012-01-04 14:12:01.008460107 -0800
@@ -420,7 +420,7 @@ static u32 ipheth_ethtool_op_get_link(st
 	return netif_carrier_ok(dev->net);
 }
 
-static struct ethtool_ops ops = {
+static const struct ethtool_ops ops = {
 	.get_link = ipheth_ethtool_op_get_link
 };
 
--- a/drivers/net/usb/sierra_net.c	2011-12-07 10:54:18.968280377 -0800
+++ b/drivers/net/usb/sierra_net.c	2012-01-04 14:11:19.368146426 -0800
@@ -618,7 +618,7 @@ static u32 sierra_net_get_link(struct ne
 	return sierra_net_get_private(dev)->link_up && netif_running(net);
 }
 
-static struct ethtool_ops sierra_net_ethtool_ops = {
+static const struct ethtool_ops sierra_net_ethtool_ops = {
 	.get_drvinfo = sierra_net_get_drvinfo,
 	.get_link = sierra_net_get_link,
 	.get_msglevel = usbnet_get_msglevel,

^ permalink raw reply	[flat|nested] 6+ messages in thread

[parent not found: <20120106051050.936477418-ZtmgI6mnKB3QT0dZR+AlfA@public.gmane.org>]

* Re: [PATCH 1/3] usbnet: make ethtool_ops const
       [not found]   ` <20120106051050.936477418-ZtmgI6mnKB3QT0dZR+AlfA@public.gmane.org>
@ 2012-01-06 13:34     ` Greg KH
  0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2012-01-06 13:34 UTC (permalink / raw)
  To: Stephen Hemminger
  Cc: David Miller, Oliver Neukum, netdev-u79uwXL29TY76Z2rM5mHXA,
	linux-usb-u79uwXL29TY76Z2rM5mHXA

On Thu, Jan 05, 2012 at 09:10:23PM -0800, Stephen Hemminger wrote:
> The ethtool_ops table of function pointers should be const.
> Fix all the usb network drivers.
> 
> Signed-off-by: Stephen Hemminger <shemminger-ZtmgI6mnKB3QT0dZR+AlfA@public.gmane.org>
> 
> ---
> Not sure if you want these through net-next or usb-next?
> 
>  drivers/net/usb/asix.c       |    2 +-
>  drivers/net/usb/cdc_ncm.c    |    4 ++--
>  drivers/net/usb/ipheth.c     |    2 +-
>  drivers/net/usb/sierra_net.c |    2 +-
>  4 files changed, 5 insertions(+), 5 deletions(-)

drivers/net/usb/ patches go through David and net-next, not me.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 2/3] bcm63xx: make ethtool_ops const
  2012-01-06  5:10 [PATCH 0/3] more cleanups Stephen Hemminger
  2012-01-06  5:10 ` [PATCH 1/3] usbnet: make ethtool_ops const Stephen Hemminger
@ 2012-01-06  5:10 ` Stephen Hemminger
  2012-01-06  5:10 ` [PATCH 3/3] netdev: make net_device_ops const Stephen Hemminger
  2012-01-09 22:05 ` [PATCH 0/3] more cleanups David Miller
  3 siblings, 0 replies; 6+ messages in thread
From: Stephen Hemminger @ 2012-01-06  5:10 UTC (permalink / raw)
  To: David Miller; +Cc: netdev

[-- Attachment #1: broadcom-ethtool-ops.patch --]
[-- Type: text/plain, Size: 564 bytes --]

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>

--- a/drivers/net/ethernet/broadcom/bcm63xx_enet.c	2011-12-07 10:54:18.688278133 -0800
+++ b/drivers/net/ethernet/broadcom/bcm63xx_enet.c	2012-01-04 14:19:01.945391195 -0800
@@ -1469,7 +1469,7 @@ static int bcm_enet_set_pauseparam(struc
 	return 0;
 }
 
-static struct ethtool_ops bcm_enet_ethtool_ops = {
+static const struct ethtool_ops bcm_enet_ethtool_ops = {
 	.get_strings		= bcm_enet_get_strings,
 	.get_sset_count		= bcm_enet_get_sset_count,
 	.get_ethtool_stats      = bcm_enet_get_ethtool_stats,

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 3/3] netdev: make net_device_ops const
  2012-01-06  5:10 [PATCH 0/3] more cleanups Stephen Hemminger
  2012-01-06  5:10 ` [PATCH 1/3] usbnet: make ethtool_ops const Stephen Hemminger
  2012-01-06  5:10 ` [PATCH 2/3] bcm63xx: " Stephen Hemminger
@ 2012-01-06  5:10 ` Stephen Hemminger
  2012-01-09 22:05 ` [PATCH 0/3] more cleanups David Miller
  3 siblings, 0 replies; 6+ messages in thread
From: Stephen Hemminger @ 2012-01-06  5:10 UTC (permalink / raw)
  To: David Miller, Sathya Perla, Chris Metcalf; +Cc: netdev

[-- Attachment #1: netdev-ops-const3.patch --]
[-- Type: text/plain, Size: 1111 bytes --]

More drivers where net_device_ops should be const.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>

---
 drivers/net/ethernet/emulex/benet/be_main.c |    2 +-
 drivers/net/ethernet/tile/tilepro.c         |    3 +--
 3 files changed, 3 insertions(+), 4 deletions(-)

--- a/drivers/net/ethernet/emulex/benet/be_main.c	2012-01-03 08:06:38.418232216 -0800
+++ b/drivers/net/ethernet/emulex/benet/be_main.c	2012-01-05 08:17:04.567161286 -0800
@@ -3080,7 +3080,7 @@ fw_exit:
 	return status;
 }
 
-static struct net_device_ops be_netdev_ops = {
+static const struct net_device_ops be_netdev_ops = {
 	.ndo_open		= be_open,
 	.ndo_stop		= be_close,
 	.ndo_start_xmit		= be_xmit,
--- a/drivers/net/ethernet/tile/tilepro.c	2011-12-23 21:06:15.724436897 -0800
+++ b/drivers/net/ethernet/tile/tilepro.c	2012-01-05 08:16:03.870511570 -0800
@@ -2260,8 +2260,7 @@ static int tile_net_get_mac(struct net_d
 	return 0;
 }
 
-
-static struct net_device_ops tile_net_ops = {
+static const struct net_device_ops tile_net_ops = {
 	.ndo_open = tile_net_open,
 	.ndo_stop = tile_net_stop,
 	.ndo_start_xmit = tile_net_tx,

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 0/3] more cleanups
  2012-01-06  5:10 [PATCH 0/3] more cleanups Stephen Hemminger
                   ` (2 preceding siblings ...)
  2012-01-06  5:10 ` [PATCH 3/3] netdev: make net_device_ops const Stephen Hemminger
@ 2012-01-09 22:05 ` David Miller
  3 siblings, 0 replies; 6+ messages in thread
From: David Miller @ 2012-01-09 22:05 UTC (permalink / raw)
  To: shemminger; +Cc: netdev

From: Stephen Hemminger <shemminger@vyatta.com>
Date: Thu, 05 Jan 2012 21:10:22 -0800

> Just found more places where function tables should be const.

All applied, thanks.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-01-09 22:05 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-06  5:10 [PATCH 0/3] more cleanups Stephen Hemminger
2012-01-06  5:10 ` [PATCH 1/3] usbnet: make ethtool_ops const Stephen Hemminger
     [not found]   ` <20120106051050.936477418-ZtmgI6mnKB3QT0dZR+AlfA@public.gmane.org>
2012-01-06 13:34     ` Greg KH
2012-01-06  5:10 ` [PATCH 2/3] bcm63xx: " Stephen Hemminger
2012-01-06  5:10 ` [PATCH 3/3] netdev: make net_device_ops const Stephen Hemminger
2012-01-09 22:05 ` [PATCH 0/3] more cleanups David Miller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.