]> git.neil.brown.name Git - history.git/commitdiff
[PATCH] remove redundant check in de2104x ->get_regs()
authorChris Wright <chrisw@osdl.org>
Mon, 19 Apr 2004 08:38:01 +0000 (04:38 -0400)
committerJeff Garzik <jgarzik@redhat.com>
Mon, 19 Apr 2004 08:38:01 +0000 (04:38 -0400)
Trivial patchlet...ethtool core already caps regs.len at a max of
->get_regs_len():

reglen = ops->get_regs_len(dev);
if (regs.len > reglen)
regs.len = reglen;

So doing the same in the in de2104x driver ->get_regs() is redundant.
Patch below simply removes it to clarify the guarantee of the API.

drivers/net/tulip/de2104x.c

index 494f0a8410c2f3f4d3624d7e4b09679d8fce9e27..be797b5372bda0df16114b8fe6b1a33c99d0ae15 100644 (file)
@@ -1676,8 +1676,6 @@ static void de_get_regs(struct net_device *dev, struct ethtool_regs *regs,
 {
        struct de_private *de = dev->priv;
 
-       if (regs->len > DE_REGS_SIZE)
-               regs->len = DE_REGS_SIZE;
        regs->version = (DE_REGS_VER << 2) | de->de21040;
 
        spin_lock_irq(&de->lock);