[PATCH 08/19] usbnet: smsc95xx: Register MII bus

2019-01-02 Thread Marek Vasut
Register MII bus so that a phydev can attach to it. This works in parallel with the existing MII bus accessors within the driver, however the existing accessors will be removed in subsequent patch. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn

[PATCH 19/19] usbnet: smsc95xx: Use phy bit operations

2019-01-02 Thread Marek Vasut
Use bit operations provided by bit.h instead of reimplementing them. No functional change. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb

[PATCH 13/19] usbnet: smsc95xx: Replace smsc95xx_link_ok_nopm()

2019-01-02 Thread Marek Vasut
Replace the function with genphy_update_link(), since that's really all that it is. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc9

[PATCH 17/19] usbnet: smsc95xx: Replace generic_mii_ioctl()

2019-01-02 Thread Marek Vasut
Replace generic_mii_ioctl() with phydev compatible phy_mii_ioctl(). Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c | 3 ++- 1 file

[PATCH 16/19] usbnet: smsc95xx: Replace mii_check_media()

2019-01-02 Thread Marek Vasut
The mii_check_media() is called to read out the PHY status registers and update the internal status reported by mii_ethtool_gset(), just replace it with phydev compatible phy_read_status(). Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc

[PATCH 11/19] usbnet: smsc95xx: Replace smsc95xx_mdio_write() with phy_write()

2019-01-02 Thread Marek Vasut
Just replace smsc95xx_mdio_write() with generic phy_write() to reduce usage of the ad-hoc accessors. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net

[PATCH 14/19] usbnet: smsc95xx: Replace mii_nway_restart()

2019-01-02 Thread Marek Vasut
This is in fact genphy_restart_aneg(), just use the generic phydev function. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c | 5

[PATCH 10/19] usbnet: smsc95xx: Replace smsc95xx_mdio_read() with phy_read()

2019-01-02 Thread Marek Vasut
Just replace smsc95xx_mdio_read() with generic phy_read() to reduce usage of the ad-hoc accessors. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net

[PATCH 07/19] usbnet: smsc95xx: Split the reset function

2019-01-02 Thread Marek Vasut
preparation of phydev conversion, where the code will connect to PHY between those two halves of the reset function. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org

[PATCH 03/19] usbnet: smsc95xx: Remove smsc95xx_{read,write}_reg_nopm()

2019-01-02 Thread Marek Vasut
These functions are now equal to smsc95xx_{read,write}_reg(), deduplicate them. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c | 78

[PATCH 05/19] usbnet: smsc95xx: Remove smsc95xx_{read,write}_nopm()

2019-01-02 Thread Marek Vasut
These functions are now equal to smsc95xx_mdio_{read,write}(), deduplicate them. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c | 30

[PATCH 04/19] usbnet: smsc95xx: Remove __smsc95xx_{read,write}_reg()

2019-01-02 Thread Marek Vasut
Inline those functions into smsc95xx_{read,write}_reg() to get rid of the wrapper. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c

[PATCH 06/19] usbnet: smsc95xx: Remove __smsc95xx_mdio_{read,write}()

2019-01-02 Thread Marek Vasut
Inline those functions into smsc95xx_mdio_{read,write}() to get rid of the wrapper. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c

[PATCH 01/19] usbnet: smsc95xx: Fix memory leak in smsc95xx_bind

2019-01-02 Thread Marek Vasut
In case reading of PHY register fails in smsc95xx_bind(), the private data allocated earlier are not free()d. Free them before bailing out. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To

[PATCH 00/19] usbnet: smsc95xx: Convert to phydev

2019-01-02 Thread Marek Vasut
Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org Marek Vasut (19): usbnet: smsc95xx: Fix memory leak in smsc95xx_bind usbnet: smsc95xx: Stop propagation of in_pm usbnet: smsc95xx

[PATCH 18/19] usbnet: smsc95xx: Remove all of the carrier checking code

2019-01-02 Thread Marek Vasut
The carrier checking code is completely handled by the internal phydev state machine, remove it. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb

[PATCH 12/19] usbnet: smsc95xx: Replace ad-hoc PHY functions with generic ones

2019-01-02 Thread Marek Vasut
Replace the ad-hoc reimplementation of genphy_soft_reset() and genphy_config_aneg() with the generic functions. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org

[PATCH 09/19] usbnet: smsc95xx: Connect to phydev

2019-01-02 Thread Marek Vasut
breakage. Some of the extra configuration is sorted out by later patches in this series. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb/smsc95xx.c | 29

[PATCH 15/19] usbnet: smsc95xx: Replace mii_ethtool_gset()

2019-01-02 Thread Marek Vasut
Replace mii_ethtool_gset() with a phydev compatible function and update the code to match. Signed-off-by: Marek Vasut Cc: David S. Miller Cc: Nisar Sayed Cc: Woojung Huh Cc: Andrew Lunn Cc: Florian Fainelli Cc: linux-usb@vger.kernel.org To: net...@vger.kernel.org --- drivers/net/usb

Re: [PATCH 10/19] usbnet: smsc95xx: Replace smsc95xx_mdio_read() with phy_read()

2019-01-03 Thread Marek Vasut
} else if (mdix_ctrl == ETH_TP_MDI_AUTO) { >> -buf = smsc95xx_mdio_read(dev->net, dev->mii.phy_id, >> +buf = phy_read(pdata->phydev, >> SPECIAL_CTRL_STS); >> buf &= ~SPECIAL_CTRL_STS_OVR

Re: [PATCH 09/19] usbnet: smsc95xx: Connect to phydev

2019-01-03 Thread Marek Vasut
On 1/3/19 2:22 PM, Andrew Lunn wrote: > On Thu, Jan 03, 2019 at 02:10:30AM +0100, Marek Vasut wrote: >> Add code to detect and connect to PHY. The internal PHY of the SMSC95xx >> is a regular SMSC LAN8700 and the driver only supports the internal PHY, >> so just use t

Re: [PATCH 12/19] usbnet: smsc95xx: Replace ad-hoc PHY functions with generic ones

2019-01-03 Thread Marek Vasut
On 1/3/19 2:28 PM, Andrew Lunn wrote: > On Thu, Jan 03, 2019 at 02:10:33AM +0100, Marek Vasut wrote: >> Replace the ad-hoc reimplementation of genphy_soft_reset() and >> genphy_config_aneg() with the generic functions. > > phylib will either call the phy driver specif

Re: [PATCH 10/19] usbnet: smsc95xx: Replace smsc95xx_mdio_read() with phy_read()

2019-01-04 Thread Marek Vasut
PHY driver has everything you need, > then swap the MAC driver to use phylib. I wonder, if I use the phylib functions instead of the ad-hoc ones in the MAC driver, is there still a problem with synchronization ? -- Best regards, Marek Vasut

Re: [PATCH 02/19] usbnet: smsc95xx: Stop propagation of in_pm

2019-01-07 Thread Marek Vasut
On 1/7/19 12:02 PM, Oliver Neukum wrote: > On Do, 2019-01-03 at 02:10 +0100, Marek Vasut wrote: >> The information whether the SMSC95xx is in_pm or not can be derived from >> the usbdev->suspend_count. First thing called in smsc95xx_suspend() is >> usbnet_suspend(), whi

<    1   2