net: dsa: mv88e6xxx: Fix serdes irq setup going recursive

lunn · davem330 · commit 6fb6e6371f8c · 2019-01-27T23:19:19.000-08:00
Duec to a typo, mv88e6390_serdes_irq_setup() calls itself, rather than mv88e6390x_serdes_irq_setup(). It then blows the stack, and shortly after the machine blows up. Fixes: 2defda1 ("net: dsa: mv88e6xxx: Add support for SERDES on ports 2-8 for 6390X") Signed-off-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
diff --git a/drivers/net/dsa/mv88e6xxx/serdes.c b/drivers/net/dsa/mv88e6xxx/serdes.c
@@ -664,7 +664,7 @@ int mv88e6390_serdes_irq_setup(struct mv88e6xxx_chip *chip, int port)
 	if (port < 9)
 		return 0;
 
-	return mv88e6390_serdes_irq_setup(chip, port);
+	return mv88e6390x_serdes_irq_setup(chip, port);
 }
 
 void mv88e6390x_serdes_irq_free(struct mv88e6xxx_chip *chip, int port)

Original file line number	Diff line number	Diff line change
`@@ -664,7 +664,7 @@ int mv88e6390_serdes_irq_setup(struct mv88e6xxx_chip *chip, int port)`
`664`	`664`	`if (port < 9)`
`665`	`665`	`return 0;`
`666`	`666`
`667`		`- return mv88e6390_serdes_irq_setup(chip, port);`
	`667`	`+ return mv88e6390x_serdes_irq_setup(chip, port);`
`668`	`668`	`}`
`669`	`669`
`670`	`670`	`void mv88e6390x_serdes_irq_free(struct mv88e6xxx_chip *chip, int port)`