linux-toradex.git/drivers/net/phy/dp83867.c, branch v4.10

net: phy: dp83867: allow RGMII_TXID/RGMII_RXID interface types

2017-01-16T18:29:51+00:00

Currently dp83867 driver returns error if phy interface type
PHY_INTERFACE_MODE_RGMII_RXID is used to set the rx only internal
delay. Similarly issue happens for PHY_INTERFACE_MODE_RGMII_TXID.
Fix this by checking also the interface type if a particular delay
value is missing in the phy dt bindings. Also update the DT document
accordingly.

Signed-off-by: Murali Karicheri 
Signed-off-by: Sekhar Nori 
Signed-off-by: David S. Miller

net: phy: dp83867: fix irq generation

2017-01-06T20:52:43+00:00

For proper IRQ generation by DP83867 phy the INT/PWDN pin has to be
programmed as an interrupt output instead of a Powerdown input in
Configuration Register 3 (CFG3), Address 0x001E, bit 7 INT_OE = 1. The
current driver doesn't do this and as result IRQs will not be generated by
DP83867 phy even if they are properly configured in DT.

Hence, fix IRQ generation by properly configuring CFG3.INT_OE bit and
ensure that Link Status Change (LINK_STATUS_CHNG_INT) and Auto-Negotiation
Complete (AUTONEG_COMP_INT) interrupt are enabled. After this the DP83867
driver will work properly in interrupt enabled mode.

Signed-off-by: Grygorii Strashko 
Reviewed-by: Florian Fainelli 
Signed-off-by: David S. Miller

net: phy: dp83867: add support for MAC impedance configuration

2016-10-18T14:42:16+00:00

Add support for programmable MAC impedance configuration

Signed-off-by: Mugunthan V N 
Signed-off-by: David S. Miller

net: phy: dp83867: Fix initialization of PHYCR register

2016-07-02T18:48:58+00:00

When initializing the PHY control register, the FIFO depth bits are
written without reading the previous register value, i.e. all other
bits are overwritten with zero. This disables automatic MDI-X
configuration, which is enabled by default. Fix initialization by doing
a read/modify/write operation.

Signed-off-by: Stefan Hauser 
Reviewed-by: Florian Fainelli 
Signed-off-by: David S. Miller

phy: Centralize setting driver module owner

2016-01-07T19:31:27+00:00

Rather than have each driver set the driver owner field, do it once in
the core code. This will also help with later changes, when the device
structure will move.

Signed-off-by: Andrew Lunn 
Reviewed-by: Florian Fainelli 
Signed-off-by: David S. Miller

phy: Add an mdio_device structure

2016-01-07T19:31:26+00:00

Not all devices attached to an MDIO bus are phys. So add an
mdio_device structure to represent the generic parts of an mdio
device, and place this structure into the phy_device.

Signed-off-by: Andrew Lunn 
Signed-off-by: David S. Miller

phy: phy_{read|write}_mmd_indirect: get addr from phydev

2016-01-07T19:31:25+00:00

The address of the device can be determined from the phydev structure,
rather than passing it as a parameter.

Signed-off-by: Andrew Lunn 
Reviewed-by: Florian Fainelli 
Signed-off-by: David S. Miller

PHY: DP83867: Remove looking in parent device for OF properties

2015-12-07T04:05:56+00:00

Device tree properties for a phy device are expected to be in the phy
node. The current code for the DP83867 also tries to look in the
parent node. The devices binding documentation does not mention this,
no current device tree file makes use of this, and it is not behaviour
we want. So remove looking in the parent device.

Signed-off-by: Andrew Lunn 
Acked-by: Florian Fainelli 
Signed-off-by: David S. Miller

net: phy: dp83867: fix simple_return.cocci warnings

2015-07-27T07:59:16+00:00

drivers/net/phy/dp83867.c:126:1-4: WARNING: end returns can be simpified
drivers/net/phy/dp83867.c:74:5-8: WARNING: end returns can be simpified if tested value is negative or 0

 Simplify a trivial if-return sequence.  Possibly combine with a
 preceding function call.

Generated by: scripts/coccinelle/misc/simple_return.cocci

CC: Dan Murphy 
Signed-off-by: Fengguang Wu 
Signed-off-by: David S. Miller

net: phy: dp83867: Fix warning check for setting the internal delay

2015-07-22T05:29:31+00:00

Fix warning: logical ‘or’ of collectively exhaustive tests is always true

Change the internal delay check from an 'or' condition to an 'and'
condition.

Reported-by: David Binderman 
Signed-off-by: Dan Murphy 
Acked-by: Florian Fainelli 
Signed-off-by: David S. Miller