linux-toradex.git/drivers/net/ethernet/netx-eth.c, branch v5.0

net: deprecate eth_change_mtu, remove usage

2016-10-13T13:36:57+00:00

With centralized MTU checking, there's nothing productive done by
eth_change_mtu that isn't already done in dev_set_mtu, so mark it as
deprecated and remove all usage of it in the kernel. All callers have been
audited for calls to alloc_etherdev* or ether_setup directly, which means
they all have a valid dev->min_mtu and dev->max_mtu. Now eth_change_mtu
prints out a netdev_warn about being deprecated, for the benefit of
out-of-tree drivers that might be utilizing it.

Of note, dvb_net.c actually had dev->mtu = 4096, while using
eth_change_mtu, meaning that if you ever tried changing it's mtu, you
couldn't set it above 1500 anymore. It's now getting dev->max_mtu also set
to 4096 to remedy that.

v2: fix up lantiq_etop, missed breakage due to drive not compiling on x86

CC: netdev@vger.kernel.org
Signed-off-by: Jarod Wilson 
Signed-off-by: David S. Miller

drivers: net: Don't print unpopulated net_device name

2016-05-17T16:30:19+00:00

For ethernet devices, net_device.name will be eth%d before
register_netdev() is called. Don't print the net_device name until
the format string is replaced.

Signed-off-by: Harvey Hunt 
Cc: Marcel Ziswiler 
Cc: Robert Jarzmik 
Cc: Barry Song 
Cc: netdev@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: David S. Miller

net: ethernet: drop owner assignment from platform_drivers

2014-10-20T14:21:02+00:00

A platform_driver does not need to set an owner, it will be populated by the
driver core.

Signed-off-by: Wolfram Sang

net: ethernet: Remove superfluous ether_setup after alloc_etherdev

2014-10-03T22:31:40+00:00

There is no need to call ether_setup after alloc_ethdev since it was
already called there.

Follow commits c706471b2601 ("net: axienet: remove unnecessary
ether_setup after alloc_etherdev") and 3c87dcbfb36c ("net: ll_temac:
Remove unnecessary ether_setup after alloc_etherdev") and fix the
pattern in all remaining ethernet drivers.

Signed-off-by: Tobias Klauser 
Acked-by: Nicolas Ferre 
Signed-off-by: David S. Miller

ethernet: Fix FSF address in file headers

2013-12-06T17:37:55+00:00

Several files refer to an old address for the Free Software Foundation
in the file header comment.  Resolve by replacing the address with
the URL  so that we do not have to keep
updating the header comments anytime the address changes.

CC: Santosh Raspatur 
CC: Dimitris Michailidis 
CC: Michael Chan 
CC: Santiago Leon 
CC: Sebastian Hesselbarth 
CC: Olof Johansson 
CC: Manish Chopra 
CC: Sony Chacko 
CC: Rajesh Borundia 
CC: Nicolas Pitre 
CC: Steve Glendinning 
Signed-off-by: Jeff Kirsher 
Signed-off-by: David S. Miller

net: netx-eth: remove unnecessary casting

2013-09-04T04:27:27+00:00

Casting from 'void *' is unnecessary, because casting from 'void *'
to any pointer type is automatic.

Reported-by: Sergei Shtylyov 
Signed-off-by: Jingoo Han 
Signed-off-by: David S. Miller

net: netx-eth: use dev_get_platdata()

2013-08-30T21:43:36+00:00

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han 
Signed-off-by: David S. Miller

net: ethernet: remove unnecessary platform_set_drvdata()

2013-05-28T05:34:51+00:00

The driver core clears the driver data to NULL after device_release
or on probe failure, since commit 0998d0631001288a5974afc0b2a5f568bcdecb4d
(device-core: Ensure drvdata = NULL when no driver is bound).
Thus, it is not needed to manually clear the device driver data to NULL.

Signed-off-by: Jingoo Han 
Acked-by: Nicolas Ferre 
Acked-by: Rob Herring 
Acked-by: Roland Stigge 
Acked-by: Mugunthan V N 
Reviewed-by: H Hartley Sweeten 
Tested-by: Roland Stigge 
Signed-off-by: David S. Miller

net: ethernet: use platform_{get,set}_drvdata()

2013-05-26T04:27:58+00:00

Use the wrapper functions for getting and setting the driver data using
platform_device instead of using dev_{get,set}_drvdata() with &pdev->dev,
so we can directly pass a struct platform_device.

Also, unnecessary dev_set_drvdata() is removed, because the driver core
clears the driver data to NULL after device_release or on probe failure.

Signed-off-by: Jingoo Han 
Signed-off-by: David S. Miller

drivers:net: Remove unnecessary OOM messages after netdev_alloc_skb

2013-03-09T21:09:19+00:00

Emitting netdev_alloc_skb and netdev_alloc_skb_ip_align OOM
messages is unnecessary as there is already a dump_stack
after allocation failures.

Other trivial changes around these removals:

Convert a few comparisons of pointer to 0 to !pointer.
Change flow to remove unnecessary label.
Remove now unused variable.
Hoist assignment from if.

Signed-off-by: Joe Perches 
Signed-off-by: David S. Miller