Thread (9 messages) flat view 9 messages, 5 authors, 2016-02-18

Re: [PATCH net-next 2/2] phy: marvell: Add support for phy packet generator

From: Lino Sanfilippo <LinoSanfilippo@gmx.de>
Date: 2016-02-17 22:33:22

Hi,

On 17.02.2016 21:32, Andrew Lunn wrote:
+
+	oldpage = phy_read(phydev, MII_MARVELL_PHY_PAGE);
+	if (oldpage < 0) {
+		err = oldpage;
+		goto out;
+	}
+
shouldn't this return immediately? Jump to out label and writing an
error value to the phy does not seem to be correct.

Regards,
Lino
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help