diff options
| author | Julia Lawall <julia@diku.dk> | 2010-01-16 16:59:51 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-03-02 14:54:34 -0800 |
| commit | a45a1e07aa5e4d3dc952c264f87184c5ac8c0a62 (patch) | |
| tree | cd4113243e7a16574176b3c5543609ebea82340b /drivers/usb/serial/cypress_m8.c | |
| parent | aa7907407adf1358ee39be0e98beaf4c129a78d6 (diff) | |
| download | op-kernel-dev-a45a1e07aa5e4d3dc952c264f87184c5ac8c0a62.zip op-kernel-dev-a45a1e07aa5e4d3dc952c264f87184c5ac8c0a62.tar.gz | |
USB: serial: Eliminate useless code
The variables priv and portdata are initialized twice to the same (side
effect-free) expressions. Drop one initialization in each case.
A simplified version of the semantic match that finds this problem is:
(http://coccinelle.lip6.fr/)
// <smpl>
@forall@
idexpression *x;
identifier f!=ERR_PTR;
@@
x = f(...)
... when != x
(
x = f(...,<+...x...+>,...)
|
* x = f(...)
)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/serial/cypress_m8.c')
| -rw-r--r-- | drivers/usb/serial/cypress_m8.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/usb/serial/cypress_m8.c b/drivers/usb/serial/cypress_m8.c index eee8726..c349f79 100644 --- a/drivers/usb/serial/cypress_m8.c +++ b/drivers/usb/serial/cypress_m8.c @@ -691,7 +691,6 @@ static void cypress_dtr_rts(struct usb_serial_port *port, int on) { struct cypress_private *priv = usb_get_serial_port_data(port); /* drop dtr and rts */ - priv = usb_get_serial_port_data(port); spin_lock_irq(&priv->lock); if (on == 0) priv->line_control = 0; |
