Re: [2.6 patch] usbnet.c: remove an unused function

[David Brownell]

On Thursday 28 October 2004 17:28, Adrian Bunk wrote:
> [ this time without the problems due to a digital signature... ]
> 
> The patch below removes an unused function from drivers/usb/net/usbnet.c

Actually in this case I'd rather leave the function there;
the documentation on this chip is hard to find, and this
function will be needed when someone finally spends the time
to fix some of the init/reset handshake issues for these
PL2301/2302 chips.  The current init code cloned some
very ancient code from about the 2.2.10 (!) or so kernel.

Speaking of which ... anyone motivated to solve that can
probably find answers in a file transfer package I saw
a while back (but lost!) for file transferring files
to/from Linux, especialy for Playstation 2 boxes.  That
used a very different handshaking scheme... possibly one
that's incompatible with Linux 2.2, but evidently one
that worked reliably and noticed connect change events.
(Neither is true of the current PL2301/2302 code!)

- Dave

p.s. Last I looked, GCC ignored unused inlines; no code
     generated, no warnings.  Did that change?

> 
> diffstat output:
>  drivers/usb/net/usbnet.c |    6 ------
>  1 files changed, 6 deletions(-)
> 
> 
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>
> 
> --- linux-2.6.10-rc1-mm1-full/drivers/usb/net/usbnet.c.old	2004-10-28 
23:32:50.000000000 +0200
> +++ linux-2.6.10-rc1-mm1-full/drivers/usb/net/usbnet.c	2004-10-28 
23:33:23.000000000 +0200
> @@ -2127,12 +2127,6 @@
>  }
>  
>  static inline int
> -pl_clear_QuickLink_features (struct usbnet *dev, int val)
> -{
> -	return pl_vendor_req (dev, 1, (u8) val, 0);
> -}
> -
> -static inline int
>  pl_set_QuickLink_features (struct usbnet *dev, int val)
>  {
>  	return pl_vendor_req (dev, 3, (u8) val, 0);
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/