linux.git/drivers/net/dsa, branch v4.16-rc2 Linux kernel source tree net: dsa: mv88e6xxx: Free ATU/VTU irq only when there is chip irq 2018-01-19T20:57:02+00:00 Andrew Lunn andrew@lunn.ch 2018-01-18T16:42:50+00:00 ae14cafc933b977f368ea7510d3b50b9ad5e1fc6 We only register the ATU and VTU irq when we have a chip level IRQ. In the error path, we should only attempt to remove the ATU and VTU irq if we also have a chip level IRQ. Signed-off-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net> 
We only register the ATU and VTU irq when we have a chip level IRQ.
In the error path, we should only attempt to remove the ATU and VTU
irq if we also have a chip level IRQ.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
net: dsa: mv88e6xxx: Return error from irq_find_mapping() 2018-01-19T20:57:02+00:00 Andrew Lunn andrew@lunn.ch 2018-01-18T16:42:49+00:00 9b662a3ec21221f2500e8009c849abf9c0136b99 Fix a cut/paste error. When irq_find_mapping() returns an error for the ATU or VTU interrupt, return that error, not the value of chip->device_irq. Signed-off-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net> 
Fix a cut/paste error. When irq_find_mapping() returns an error for
the ATU or VTU interrupt, return that error, not the value of
chip->device_irq.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
net: dsa: lan9303: check error value from devm_gpiod_get_optional() 2018-01-15T18:18:02+00:00 Phil Reid preid@electromag.com.au 2018-01-10T07:39:33+00:00 f16891326c1c6e6b0f17f60cb6921a145a13e017 devm_gpiod_get_optional() can return an error in addition to a NULL ptr. Check for error and propagate that to the probe function. Check return value in probe. This will now handle EPROBE_DEFER for the reset gpio. Signed-off-by: Phil Reid <preid@electromag.com.au> Signed-off-by: David S. Miller <davem@davemloft.net> 
devm_gpiod_get_optional() can return an error in addition to a NULL ptr.
Check for error and propagate that to the probe function. Check return
value in probe. This will now handle EPROBE_DEFER for the reset gpio.

Signed-off-by: Phil Reid <preid@electromag.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net: dsa: lan9303: make lan9303_handle_reset() a void function 2018-01-15T18:18:02+00:00 Phil Reid preid@electromag.com.au 2018-01-10T07:39:32+00:00 a57d476d9707a470bbdcbd1ab00df73b55650bf7 lan9303_handle_reset never returns anything other than success. So there's not need for it to return an error code. Signed-off-by: Phil Reid <preid@electromag.com.au> Signed-off-by: David S. Miller <davem@davemloft.net> 
lan9303_handle_reset never returns anything other than success.
So there's not need for it to return an error code.

Signed-off-by: Phil Reid <preid@electromag.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net: dsa: mv88e6xxx: Decode VTU problem interrupt 2018-01-14T17:08:45+00:00 Andrew Lunn andrew@lunn.ch 2018-01-14T01:32:45+00:00 62eb1162ec3a65f12d0e0c383f4f57de814a5e51 When there is a problem with the VTU, an interrupt can be generated. Trap this interrupt and decode the registers to determine what the problem was, then log the error. Signed-off-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net> 
When there is a problem with the VTU, an interrupt can be
generated. Trap this interrupt and decode the registers to determine
what the problem was, then log the error.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
net: dsa: mv88e6xxx: Decode ATU problem interrupt 2018-01-14T17:08:45+00:00 Andrew Lunn andrew@lunn.ch 2018-01-14T01:32:44+00:00 0977644c5005ca2d186b353d7236deca6aa2933e When there is a problem with the ATU, an interrupt can be generated. Trap this interrupt and decode the registers to determine what the problem was, then log the error. Signed-off-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net> 
When there is a problem with the ATU, an interrupt can be
generated. Trap this interrupt and decode the registers to determine
what the problem was, then log the error.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net 2018-01-09T15:37:00+00:00 David S. Miller davem@davemloft.net 2018-01-09T15:37:00+00:00 a0ce093180f2bbb832b3f5583adc640ad67ea568 






net: dsa: lan9303: Fix error return code in lan9303_check_device()
2018-01-05T16:24:54+00:00

Wei Yongjun
weiyongjun1@huawei.com

2018-01-04T07:30:28+00:00

a31e795a3b4d8808724c7ea80f382b0fda85be0f

Fix to return error code -ENODEV from the chip not found error handling
case instead of 0(ret have been overwritten to 0 by lan9303_read()), as
done elsewhere in this function.

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Reviewed-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>





Fix to return error code -ENODEV from the chip not found error handling
case instead of 0(ret have been overwritten to 0 by lan9303_read()), as
done elsewhere in this function.

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Reviewed-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>







net: dsa: lan9303: Adjust phy_addr_base expressions
2018-01-04T18:35:07+00:00

Egil Hjelmeland
privat@egil-hjelmeland.no

2017-12-29T12:38:24+00:00

589d197660e529dab0933261b402e079326815f8

Simplify calculation of chip->phy_addr_base in lan9303_detect_phy_setup().

Use GENMASK to calculate phys_mii_mask from LAN9303_NUM_PORTS and
phy_addr_base.

Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>





Simplify calculation of chip->phy_addr_base in lan9303_detect_phy_setup().

Use GENMASK to calculate phys_mii_mask from LAN9303_NUM_PORTS and
phy_addr_base.

Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>







net: dsa: lan9303: phy_addr_sel_strap rename and retype
2018-01-04T18:35:07+00:00

Egil Hjelmeland
privat@egil-hjelmeland.no

2017-12-29T12:38:23+00:00

b17c6b1f4538b66e9bac6710b57b343c2aafadea

chip->phy_addr_sel_strap is declared as a bool, but is also used as an
integer address base.

Rename 'phy_addr_sel_strap' to 'phy_addr_base', and change type to int.

Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>





chip->phy_addr_sel_strap is declared as a bool, but is also used as an
integer address base.

Rename 'phy_addr_sel_strap' to 'phy_addr_base', and change type to int.

Signed-off-by: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: David S. Miller <davem@davemloft.net>