Re: [PATCH v2 1/2] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-08-27 Thread Ivan Mikhaylov
On Wed, 5 Aug 2015 15:01:53 +0400 Ivan Mikhaylov wrote: ping > * do the redefinition of emac_regs struct from driver structure > perspective and passing size from actual struct size, not from memory > area variable which set in dts file. > > * passing variable from dts opt

Re: [PATCH] net/ibm/emac: bump version numbers for correct work with ethtool

2015-09-24 Thread Ivan Mikhaylov
On Wed, 23 Sep 2015 11:05:49 -0700 (PDT) "David Miller" wrote: > From: Ivan Mikhaylov > Date: Wed, 23 Sep 2015 14:42:22 +0400 > > > Register dump out work preventing with > > old ethtool + new driver and new ethtool + old driver. > > First of al

Re: [PATCH] ethtool: add new emac_regs struct from driver, add new chip types.

2015-09-24 Thread Ivan Mikhaylov
. * this commit fixing problem with output of MII sections for new driver versions. Signed-off-by: Ivan Mikhaylov --- ibm_emac.c | 161 ++-- 1 file changed, 125 insertions(+), 36 deletions(-) diff --git a/ibm_emac.c b/ibm_emac.c index e128e48

Re: [PATCH] net/ibm/emac: bump version numbers for correct work with ethtool

2015-09-25 Thread Ivan Mikhaylov
On Thu, 24 Sep 2015 22:53:03 -0700 (PDT) "David Miller" wrote: > From: Ivan Mikhaylov > Date: Fri, 25 Sep 2015 08:07:52 +0400 > > > Ben proposed one, is it eligible? > > Need I resubmit patch with sign and detailed description? > > If I genuinely need to a

[PATCH v2] net/ibm/emac: bump version numbers for correct work with ethtool

2015-09-25 Thread Ivan Mikhaylov
tree didn't match the size it assumed. Using the new version numbers indicates unambiguously that the size of the MAC register dump is dependent only on the MAC type. Fixes: 5369c71f7ca2 ("net/ibm/emac: fix size of emac dump memory areas") Signed-off-by: Ivan Mikhaylov --- drive

Re: [PATCH] ethtool: add new emac_regs struct from driver, add new chip types.

2015-10-07 Thread Ivan Mikhaylov
On Fri, 25 Sep 2015 08:15:45 +0400 "Ivan Mikhaylov" wrote: ping > * add new version of emac_regs struct from driver structure > perspective and passing size from actual struct size, not from memory > area variable which set in dts file. > * add three types of networ

[PATCH v2 1/2] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-08-05 Thread Ivan Mikhaylov
h this and 5369c71 commit in kernel. Signed-off-by: Ivan Mikhaylov --- ibm_emac.c | 101 +++- 1 file changed, 73 insertions(+), 28 deletions(-) diff --git a/ibm_emac.c b/ibm_emac.c index e128e48..da6c28e 100644 --- a/ibm_emac.c +++ b/

[PATCH v2 2/2] ethtool: add versioning for emac chips

2015-08-05 Thread Ivan Mikhaylov
* add three types of network chips : emac, emac4, emac4sync * add emac4sync processing in print_emac_regs Signed-off-by: Ivan Mikhaylov --- ibm_emac.c | 56 +--- 1 file changed, 45 insertions(+), 11 deletions(-) diff --git a/ibm_emac.c b

Re: [PATCH v2 1/2] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-09-09 Thread Ivan Mikhaylov
On Wed, 9 Aug 2015 15:10:00 +0400 Ivan Mikhaylov wrote: >On Wed, 2015-08-05 at 15:01 +0400, Ivan Mikhaylov wrote: >> * do the redefinition of emac_regs struct from driver structure >> perspective and passing size from actual struct size, not from memory >> area variable

Re: [PATCH v2 1/2] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-09-09 Thread Ivan Mikhaylov
On Wed, 9 Aug 2015 15:10:00 +0400 Ivan Mikhaylov wrote: >On Wed, 2015-08-05 at 15:01 +0400, Ivan Mikhaylov wrote: >> * do the redefinition of emac_regs struct from driver structure >> perspective and passing size from actual struct size, not from memory >> area variable

[PATCH] net/ibm/emac: bump version numbers for correct work with ethtool

2015-09-23 Thread Ivan Mikhaylov
Register dump out work preventing with old ethtool + new driver and new ethtool + old driver. --- drivers/net/ethernet/ibm/emac/core.h |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/ibm/emac/core.h b/drivers/net/ethernet/ibm/emac/core.h index 2

[PATCH] ethtool: add new emac_regs struct from driver, add new chip types.

2015-09-23 Thread Ivan Mikhaylov
* add new version of emac_regs struct from driver structure perspective and passing size from actual struct size, not from memory area variable which set in dts file. * add three types of network chips for new struct : emac, emac4, emac4sync. * add emac4sync processing in print_emac_regs. * thi

Re: [PATCH] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-06-23 Thread Ivan Mikhaylov
On Wed, 3 Jun 2015 18:28:37 +0400 Ivan Mikhaylov wrote: > On Mon, 1 Jun 2015 22:11:00 +0400 > Ben Hutchings wrote: > > >On Mon, 2015-06-01 at 16:30 +0400, Ivan Mikhaylov wrote: > >> On Mon, 1 June 2015 12:57 +0400 > >> Ben Hutchings wrote: > >> >

[PATCH] net/ibm/emac: fix size of emac dump memory areas

2015-05-21 Thread Ivan Mikhaylov
Fix in send of emac regs dump to ethtool which causing in wrong data interpretation on ethtool layer for MII and EMAC. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/core.c | 16 ++-- drivers/net/ethernet/ibm/emac/core.h |7 ++- 2 files changed, 8

[PATCH] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-05-21 Thread Ivan Mikhaylov
In ibm_emac.c in ethtool size of emac structure which passing through to driver is nailed down and not correlating with current emac_regs structure. Signed-off-by: Ivan Mikhaylov --- ibm_emac.c | 143 +--- 1 file changed, 107 insertions

Re: [PATCH] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-06-01 Thread Ivan Mikhaylov
On Mon, 1 June 2015 12:57 +0400 Ben Hutchings wrote: >On Thu, 2015-05-21 at 19:09 +0400, Ivan Mikhaylov wrote: >> In ibm_emac.c in ethtool size of emac structure which passing through >> to driver is nailed down and not correlating with current emac_regs >> structure. >&

Re: [PATCH] ethtool: changes of emac_regs structure accordingly within driver emac_regs structure.

2015-06-03 Thread Ivan Mikhaylov
On Mon, 1 Jun 2015 22:11:00 +0400 Ben Hutchings wrote: >On Mon, 2015-06-01 at 16:30 +0400, Ivan Mikhaylov wrote: >> On Mon, 1 June 2015 12:57 +0400 >> Ben Hutchings wrote: >> >> >On Thu, 2015-05-21 at 19:09 +0400, Ivan Mikhaylov wrote: >> >> In ib

[PATCH 2/2] net/ibm/emac: wrong bit is used for STA control register write

2018-01-22 Thread Ivan Mikhaylov
/MDIO mode with 19 bit and write operation is set into 18 bit which is mode selection, not a write operation. To correlate write with read we set it into 20 bit. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/emac.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) di

[PATCH 1/2] net/ibm/emac: add 8192 rx/tx fifo size

2018-01-22 Thread Ivan Mikhaylov
emac4syn chips has availability to use 8192 rx/tx fifo buffer sizes, in current state if we set it up in dts 8192 as example, we will get only 2048 which may impact on network speed. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/core.c |6 ++ drivers/net/ethernet/ibm

Re: [PATCH 2/2] net/ibm/emac: wrong bit is used for STA control register write

2018-01-22 Thread Ivan Mikhaylov
>Something looks wrong here?! The commit message talks about bit 18, 19 and 20. >However, 0x0800, 0x1000, 0x2000 and are like bit 11, 12 and 13? Furthermore, >what about the EMAC_STACR_STAC_MASK? shouldn't it be 0x1800 now (or delete it >since it doesn't look like it's used anywhere?). Christian, n

[PATCH v2 2/2] net/ibm/emac: wrong bit is used for STA control register write

2018-01-23 Thread Ivan Mikhaylov
/MDIO mode with 19 bit and write operation is set into 18 bit which is mode selection, not a write operation. To correlate write with read we set it into 20 bit. All those bit operations are MSB 0 based. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/emac.h |2 +- 1 files

Re: [PATCH 2/2] net/ibm/emac: wrong bit is used for STA control register write

2018-01-23 Thread Ivan Mikhaylov
> So if someone tries to #define EMAC_STACR_STAC_WRITE BIT(18) it would be > 0x4 instead. This is where the confusion is coming from. Can you please > at least mention this somewhere that all the bits in the commit message are > in "MSB 0" format? It's confusing enough as it is ;). Yeap, sure,

[PATCH v2 1/2] net/ibm/emac: add 8192 rx/tx fifo size

2018-01-24 Thread Ivan Mikhaylov
emac4syn chips has availability to use 8192 rx/tx fifo buffer sizes, in current state if we set it up in dts 8192 as example, we will get only 2048 which may impact on network speed. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/core.c |6 ++ drivers/net/ethernet/ibm

[PATCH v2 2/2] net/ibm/emac: wrong bit is used for STA control register write

2018-01-24 Thread Ivan Mikhaylov
/MDIO mode with 19 bit and write operation is set into 18 bit which is mode selection, not a write operation. To correlate write with read we set it into 20 bit. All those bit operations are MSB 0 based. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/emac.h |2 +- 1 files

[PATCH 1/2] net/ibm/emac: add set mac addr callback

2016-09-14 Thread Ivan Mikhaylov
add realization for mac address set and remove dummy callback. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/core.c | 31 +-- 1 file changed, 29 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/ibm/emac/core.c b/drivers/net/ethernet

[PATCH 2/2] net/ibm/emac: add mutex to 'set multicast list'

2016-09-14 Thread Ivan Mikhaylov
for preventing race conditions within ioctl calls. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/ibm/emac/core.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/ethernet/ibm/emac/core.c b/drivers/net/ethernet/ibm/emac/core.c index 2dfc603..7af09cb 100644 --- a

[PATCH 0/2] Add LED mode behavior/select properties and handle

2020-12-09 Thread Ivan Mikhaylov
e properly? Ivan Mikhaylov (2): net: phy: micrel: add LED control on KSZ9131 dt-bindings: net: phy: micrel: add LED mode behavior and select properties .../devicetree/bindings/net/micrel.txt| 7 ++ drivers/net/phy/micrel.c | 69 ++- 2 files c

[PATCH 1/2] net: phy: micrel: add LED control on KSZ9131

2020-12-09 Thread Ivan Mikhaylov
Add the possibility to read the LED configuration via DTS properties from KSZ9131 PHY node. Add the new proprties and handle for them: micrel,led-mode-behavior micrel,led-mode-select Signed-off-by: Ivan Mikhaylov --- drivers/net/phy/micrel.c | 69 +++- 1 file

[PATCH 2/2] dt-bindings: net: phy: micrel: add LED mode behavior and select properties

2020-12-09 Thread Ivan Mikhaylov
Add LED mode behavior and LED mode select properties which can be used in KSZ9131 PHY. Signed-off-by: Ivan Mikhaylov --- Documentation/devicetree/bindings/net/micrel.txt | 7 +++ 1 file changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/net/micrel.txt b/Documentation

[PATCH 0/1] add ast2400/2500 phy-handle support

2020-10-13 Thread Ivan Mikhaylov
#address-cells = <1>; #size-cells = <0>; phy: ethernet-phy@0 { compatible = "ethernet-phy-id."; reg = <0>; }; }; }; The patch fixes it and gets possible PHYs a

[PATCH 1/1] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-13 Thread Ivan Mikhaylov
phy-handle can't be handled well for ast2400/2500 which has an embedded MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and initialize PHYs from mdio child node with of_mdiobus_register. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c

Re: [PATCH 1/1] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-14 Thread Ivan Mikhaylov
On Wed, 2020-10-14 at 05:23 +, Joel Stanley wrote: > Hi Ivan, > > On Tue, 13 Oct 2020 at 12:38, Ivan Mikhaylov wrote: > > phy-handle can't be handled well for ast2400/2500 which has an embedded > > MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and ini

[PATCH v1 0/2] add ast2400/2500 phy-handle support

2020-10-15 Thread Ivan Mikhaylov
#address-cells = <1>; #size-cells = <0>; phy: ethernet-phy@0 { compatible = "ethernet-phy-id."; reg = <0>; }; }; }; The patch fixes it and gets possible PHYs a

[PATCH v1 1/2] net: ftgmac100: move phy connect out from ftgmac100_setup_mdio

2020-10-15 Thread Ivan Mikhaylov
Split MDIO registration and PHY connect into ftgmac100_setup_mdio and ftgmac100_mii_probe. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c | 92 1 file changed, 47 insertions(+), 45 deletions(-) diff --git a/drivers/net/ethernet/faraday

[PATCH v1 2/2] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-15 Thread Ivan Mikhaylov
phy-handle can't be handled well for ast2400/2500 which has an embedded MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and initialize PHYs from mdio child node with of_mdiobus_register. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c

[PATCH v2 2/2] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-27 Thread Ivan Mikhaylov
phy-handle can't be handled well for ast2400/2500 which has an embedded MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and initialize PHYs from mdio child node with of_mdiobus_register. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c

[PATCH v2 1/2] net: ftgmac100: move phy connect out from ftgmac100_setup_mdio

2020-10-27 Thread Ivan Mikhaylov
Split MDIO registration and PHY connect into ftgmac100_setup_mdio and ftgmac100_mii_probe. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c | 104 --- 1 file changed, 54 insertions(+), 50 deletions(-) diff --git a/drivers/net/ethernet/faraday

[PATCH v2 0/2] add ast2400/2500 phy-handle support

2020-10-27 Thread Ivan Mikhaylov
unneeded mdio_np checks. Changes from v1: 1. split one patch into two. Ivan Mikhaylov (2): net: ftgmac100: move phy connect out from ftgmac100_setup_mdio net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500 drivers/net/ethernet/faraday/ftgmac100.c | 122 +--

Re: [PATCH v2 2/2] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-28 Thread Ivan Mikhaylov
On Tue, 2020-10-27 at 19:23 +0100, Andrew Lunn wrote: > On Tue, Oct 27, 2020 at 05:49:24PM +0300, Ivan Mikhaylov wrote: > > phy-handle can't be handled well for ast2400/2500 which has an embedded > > MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and initialize >

[PATCH v3 1/3] net: ftgmac100: move phy connect out from ftgmac100_setup_mdio

2020-10-30 Thread Ivan Mikhaylov
Split MDIO registration and PHY connect into ftgmac100_setup_mdio and ftgmac100_mii_probe. Reviewed-by: Andrew Lunn Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c | 104 --- 1 file changed, 54 insertions(+), 50 deletions(-) diff --git a/drivers

[PATCH v3 0/3] add ast2400/2500 phy-handle support

2020-10-30 Thread Ivan Mikhaylov
t label. 3. split ftgmac100_destroy_mdio into ftgmac100_phy_disconnect and ftgmac100_destroy_mdio. 4. remove unneeded mdio_np checks. Changes from v1: 1. split one patch into two. Ivan Mikhaylov (3): net: ftgmac100: move phy connect out from ftgmac100_setup_mdio net: ftgmac100: add handling of md

[PATCH v3 3/3] dt-bindings: net: ftgmac100: describe phy-handle and MDIO

2020-10-30 Thread Ivan Mikhaylov
Add the phy-handle and MDIO description and add the example with PHY and MDIO nodes. Signed-off-by: Ivan Mikhaylov --- .../devicetree/bindings/net/ftgmac100.txt | 25 +++ 1 file changed, 25 insertions(+) diff --git a/Documentation/devicetree/bindings/net/ftgmac100.txt b

[PATCH v3 2/3] net: ftgmac100: add handling of mdio/phy nodes for ast2400/2500

2020-10-30 Thread Ivan Mikhaylov
phy-handle can't be handled well for ast2400/2500 which has an embedded MDIO controller. Add ftgmac100_mdio_setup for ast2400/2500 and initialize PHYs from mdio child node with of_mdiobus_register. Signed-off-by: Ivan Mikhaylov --- drivers/net/ethernet/faraday/ftgmac100.c