Message ID | 1231949520.4105.97.camel@dhollis-lnx |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Wed, 2009-01-14 at 11:12 -0500, David Hollis wrote: > The attached patch adds support for ASIX AX88772A devices, which is used > in some netbooks, notebooks and embedded devices. It's largely > compatible with the AX88772 family. > > This re-post does not include changes to the ax88772_link_reset() > function that removed the use of the generic MII calls in favor of more > explicit register reads. Those changes were not required for AX88772A > support. > > Thanks to Louis <louis@asix.com.tw> for the initial patch. > > Signed-off-by: David Hollis <dhollis@davehollis.com> This patch, including the context, has had tabs converted to spaces. > --- a/drivers/net/usb/asix.c 2009-01-02 01:06:26.000000000 -0500 > +++ b/drivers/net/usb/asix.c 2009-01-14 10:35:11.000000000 -0500 > @@ -35,7 +35,7 @@ > #include <linux/crc32.h> > #include <linux/usb/usbnet.h> > > -#define DRIVER_VERSION "14-Jun-2006" > +#define DRIVER_VERSION "8-Jan-2009" > static const char driver_name [] = "asix"; > > /* ASIX AX8817X based USB 2.0 Ethernet Devices */ > @@ -93,8 +93,8 @@ static const char driver_name [] = "asix > #define AX_SWRESET_IPPD 0x40 > > #define AX88772_IPG0_DEFAULT 0x15 > -#define AX88772_IPG1_DEFAULT 0x0c > -#define AX88772_IPG2_DEFAULT 0x12 > +#define AX88772_IPG1_DEFAULT 0x16 > +#define AX88772_IPG2_DEFAULT 0x1A Is this change correct for the plain AX88772 as well? > /* AX88772 & AX88178 Medium Mode Register */ > #define AX_MEDIUM_PF 0x0080 > @@ -136,6 +136,13 @@ static const char driver_name [] = "asix > #define AX_DEFAULT_RX_CTL \ > (AX_RX_CTL_SO | AX_RX_CTL_AB ) > > +#define AX_PHYSELECT_PSEL 0x01 > +#define AX_PHYSELECT_ASEL 0x02 > +#define AX_PHYSELECT_SSMII 0x04 > +#define AX_PHYSELECT_SSRMII 0x08 > +#define AX_PHYSELECT_SSRRMII 0x0C > +#define AX_PHYSELECT_SSEN 0x10 > + > /* GPIO 0 .. 2 toggles */ > #define AX_GPIO_GPO0EN 0x01 /* GPIO0 Output enable */ > #define AX_GPIO_GPO_0 0x02 /* GPIO0 Output value */ > @@ -891,7 +898,11 @@ static int ax88772_link_reset(struct usb > if (ecmd.duplex != DUPLEX_FULL) > mode &= ~AX_MEDIUM_FD; > > - devdbg(dev, "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", ecmd.speed, ecmd.duplex, mode); > + devdbg(dev, > + "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", > + ecmd.speed, > + ecmd.duplex, > + mode); This is just a formatting change so it belongs in a separate patch. And if you're going to change it, the format string should be indented consistently with the other arguments. > asix_write_medium_mode(dev, mode); > > @@ -1018,6 +1029,121 @@ out: > return ret; > } > > +static int ax88772a_bind(struct usbnet *dev, struct usb_interface *intf) > +{ > + int ret; > + u16 rx_ctl; > + struct asix_data *data = (struct asix_data *)&dev->data; > + u8 buf[ETH_ALEN]; > + u32 phyid; > + > + data->eeprom_len = AX88772_EEPROM_LEN; > + > + usbnet_get_endpoints(dev,intf); > + > + /* Power up the embedded PHY */ > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > + goto out; Please separate the assignment and the test. I know the existing code combines them and you don't need to fix that, but new code should not do this. > + /* Select the embedded PHY as the active PHY */ > + if ((ret = asix_write_cmd(dev, AX_CMD_SW_PHY_SELECT, > + AX_PHYSELECT_SSEN | AX_PHYSELECT_PSEL, > + 0, 0, NULL)) < 0) { > + dbg("Select PHY #1 failed: %d", ret); > + goto out; > + } > + > + /* Reload the EEPROM and configure the GPIO pins */ > + if ((ret = asix_write_gpio(dev, > + AX_GPIO_RSE | AX_GPIO_GPO_2 | AX_GPIO_GPO2EN, 5)) < 0) > + goto out; "goto out;" should be indented. > + /* Set embedded PHY in power down state */ > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL | AX_SWRESET_IPPD)) < 0) > + goto out; > + > + msleep(10); > + > + /* Set embedded PHY in power up state */ > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > + goto out; > + > + msleep(60); > + > + /* Set embedded PHY in reset state */ > + if ((ret = asix_sw_reset(dev, AX_SWRESET_CLEAR)) < 0) > + goto out; > + > + /* Set embedded PHY in operating state */ > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > + goto out; > + > + /* Stop RX operation */ > + if ((ret = asix_write_rx_ctl(dev, 0)) < 0) > + goto out; > + > + /* Get the MAC address */ > + if ((ret = asix_read_cmd(dev, AX_CMD_READ_NODE_ID, > + 0, 0, ETH_ALEN, buf)) < 0) { > + dbg("Failed to read MAC address: %d", ret); > + goto out; > + } > + memcpy(dev->net->dev_addr, buf, ETH_ALEN); > + > + /* Initialize MII structure */ > + dev->mii.dev = dev->net; > + dev->mii.mdio_read = asix_mdio_read; > + dev->mii.mdio_write = asix_mdio_write; > + dev->mii.phy_id_mask = 0x1f; > + dev->mii.reg_num_mask = 0x1f; > + dev->net->do_ioctl = asix_ioctl; > + dev->mii.phy_id = asix_get_phy_addr(dev); > + > + phyid = asix_get_phyid(dev); > + dbg("PHYID=0x%08x", phyid); > + > + dev->net->set_multicast_list = asix_set_multicast; > + dev->net->ethtool_ops = &ax88772_ethtool_ops; > + > + asix_mdio_write(dev->net, dev->mii.phy_id, MII_BMCR, BMCR_RESET); > + asix_mdio_write(dev->net, dev->mii.phy_id, MII_ADVERTISE, > + ADVERTISE_ALL | ADVERTISE_CSMA); > + mii_nway_restart(&dev->mii); > + > + if ((ret = asix_write_medium_mode(dev, AX88772_MEDIUM_DEFAULT)) < 0) > + goto out; > + > + if ((ret = asix_write_cmd(dev, AX_CMD_WRITE_IPG0, > + AX88772_IPG0_DEFAULT | AX88772_IPG1_DEFAULT, > + AX88772_IPG2_DEFAULT, 0, NULL)) < 0) { > + dbg("Write IPG,IPG1,IPG2 failed: %d", ret); > + goto out; > + } > + > + msleep(1); > + > + /* Set RX_CTL to default values with 2k buffer, and enable cactus */ > + if ((ret = asix_write_rx_ctl(dev, AX_DEFAULT_RX_CTL)) < 0) > + goto out; > + > + rx_ctl = asix_read_rx_ctl(dev); > + dbg("RX_CTL is 0x%04x after all initializations", rx_ctl); > + > + rx_ctl = asix_read_medium_status(dev); > + dbg("Medium Status is 0x%04x after all initializations", rx_ctl); > + > + /* Asix framing packs multiple eth frames into a 2K usb bulk transfer */ > + if (dev->driver_info->flags & FLAG_FRAMING_AX) { > + /* hard_mtu is still the default - the device does not support > + jumbo eth frames */ > + dev->rx_urb_size = 2048; > + } > + return 0; > + > +out: > + return ret; > +} > + > static struct ethtool_ops ax88178_ethtool_ops = { > .get_drvinfo = asix_get_drvinfo, > .get_link = asix_get_link, > @@ -1339,6 +1465,17 @@ static const struct driver_info ax88772_ > .tx_fixup = asix_tx_fixup, > }; > > +static const struct driver_info ax88772a_info = { > + .description = "ASIX AX88772A USB 2.0 Ethernet", > + .bind = ax88772a_bind, > + .status = asix_status, > + .link_reset = ax88772_link_reset, > + .reset = ax88772_link_reset, > + .flags = FLAG_ETHER | FLAG_FRAMING_AX, > + .rx_fixup = asix_rx_fixup, > + .tx_fixup = asix_tx_fixup, > +}; > + > static const struct driver_info ax88178_info = { > .description = "ASIX AX88178 USB 2.0 Ethernet", > .bind = ax88178_bind, > @@ -1451,6 +1588,10 @@ static const struct usb_device_id produc > // Cables-to-Go USB Ethernet Adapter > USB_DEVICE(0x0b95, 0x772a), > .driver_info = (unsigned long) &ax88772_info, > +}, { > + // ASIX AX88772A > + USB_DEVICE(0x0b95, 0x772a), > + .driver_info = (unsigned long) &ax88772a_info, This is matching the same id as the entry above, so how will it ever be used? Do the bind operations distinguish them somehow? Please also run checkpatch.pl and fix the errors it finds. Ben. > }, > { }, // END > };
From: David Hollis <dhollis@davehollis.com> Date: Wed, 14 Jan 2009 11:12:00 -0500 > > The attached patch adds support for ASIX AX88772A devices, which is used > in some netbooks, notebooks and embedded devices. It's largely > compatible with the AX88772 family. > > This re-post does not include changes to the ax88772_link_reset() > function that removed the use of the generic MII calls in favor of more > explicit register reads. Those changes were not required for AX88772A > support. > > Thanks to Louis <louis@asix.com.tw> for the initial patch. > > Signed-off-by: David Hollis <dhollis@davehollis.com> Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Ben Hutchings <bhutchings@solarflare.com> Date: Wed, 14 Jan 2009 16:53:56 +0000 > On Wed, 2009-01-14 at 11:12 -0500, David Hollis wrote: > > The attached patch adds support for ASIX AX88772A devices, which is used > > in some netbooks, notebooks and embedded devices. It's largely > > compatible with the AX88772 family. > > > > This re-post does not include changes to the ax88772_link_reset() > > function that removed the use of the generic MII calls in favor of more > > explicit register reads. Those changes were not required for AX88772A > > support. > > > > Thanks to Louis <louis@asix.com.tw> for the initial patch. > > > > Signed-off-by: David Hollis <dhollis@davehollis.com> > > This patch, including the context, has had tabs converted to spaces. That's why he used an attachment. I applied David's patch as-is, we can clean this code up later. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, 2009-01-14 at 16:53 +0000, Ben Hutchings wrote: > > #define AX88772_IPG0_DEFAULT 0x15 > > -#define AX88772_IPG1_DEFAULT 0x0c > > -#define AX88772_IPG2_DEFAULT 0x12 > > +#define AX88772_IPG1_DEFAULT 0x16 > > +#define AX88772_IPG2_DEFAULT 0x1A > > Is this change correct for the plain AX88772 as well? > I had to ask that as I well. I have tested on a standard AX88772 and they do work. I definitely didn't want those devices breaking since I use them quite regularly as well. Apparently these values help in certain half-duplex situations. > > /* GPIO 0 .. 2 toggles */ > > #define AX_GPIO_GPO0EN 0x01 /* GPIO0 Output enable */ > > #define AX_GPIO_GPO_0 0x02 /* GPIO0 Output value */ > > @@ -891,7 +898,11 @@ static int ax88772_link_reset(struct usb > > if (ecmd.duplex != DUPLEX_FULL) > > mode &= ~AX_MEDIUM_FD; > > > > - devdbg(dev, "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", ecmd.speed, ecmd.duplex, mode); > > + devdbg(dev, > > + "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", > > + ecmd.speed, > > + ecmd.duplex, > > + mode); > > This is just a formatting change so it belongs in a separate patch. And > if you're going to change it, the format string should be indented > consistently with the other arguments. > Good catch. I missed that line. > > asix_write_medium_mode(dev, mode); > > > > @@ -1018,6 +1029,121 @@ out: > > return ret; > > } > > > > +static int ax88772a_bind(struct usbnet *dev, struct usb_interface *intf) > > +{ > > + int ret; > > + u16 rx_ctl; > > + struct asix_data *data = (struct asix_data *)&dev->data; > > + u8 buf[ETH_ALEN]; > > + u32 phyid; > > + > > + data->eeprom_len = AX88772_EEPROM_LEN; > > + > > + usbnet_get_endpoints(dev,intf); > > + > > + /* Power up the embedded PHY */ > > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > > + goto out; > > Please separate the assignment and the test. I know the existing code > combines them and you don't need to fix that, but new code should not do > this. > I can work on doing that. I'd really prefer that the whole file be one way or the other, just so it's all consistent. > > + /* Select the embedded PHY as the active PHY */ > > + if ((ret = asix_write_cmd(dev, AX_CMD_SW_PHY_SELECT, > > + AX_PHYSELECT_SSEN | AX_PHYSELECT_PSEL, > > + 0, 0, NULL)) < 0) { > > + dbg("Select PHY #1 failed: %d", ret); > > + goto out; > > + } > > + > > + /* Reload the EEPROM and configure the GPIO pins */ > > + if ((ret = asix_write_gpio(dev, > > + AX_GPIO_RSE | AX_GPIO_GPO_2 | AX_GPIO_GPO2EN, 5)) < 0) > > + goto out; > > "goto out;" should be indented. > Another good catch. Thanks. > > + /* Set embedded PHY in power down state */ > > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL | AX_SWRESET_IPPD)) < 0) > > + goto out; > > + > > + msleep(10); > > + > > + /* Set embedded PHY in power up state */ > > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > > + goto out; > > + > > + msleep(60); > > + > > + /* Set embedded PHY in reset state */ > > + if ((ret = asix_sw_reset(dev, AX_SWRESET_CLEAR)) < 0) > > + goto out; > > + > > + /* Set embedded PHY in operating state */ > > + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) > > + goto out; > > + > > + /* Stop RX operation */ > > + if ((ret = asix_write_rx_ctl(dev, 0)) < 0) > > + goto out; > > + > > + /* Get the MAC address */ > > + if ((ret = asix_read_cmd(dev, AX_CMD_READ_NODE_ID, > > + 0, 0, ETH_ALEN, buf)) < 0) { > > + dbg("Failed to read MAC address: %d", ret); > > + goto out; > > + } > > + memcpy(dev->net->dev_addr, buf, ETH_ALEN); > > + > > + /* Initialize MII structure */ > > + dev->mii.dev = dev->net; > > + dev->mii.mdio_read = asix_mdio_read; > > + dev->mii.mdio_write = asix_mdio_write; > > + dev->mii.phy_id_mask = 0x1f; > > + dev->mii.reg_num_mask = 0x1f; > > + dev->net->do_ioctl = asix_ioctl; > > + dev->mii.phy_id = asix_get_phy_addr(dev); > > + > > + phyid = asix_get_phyid(dev); > > + dbg("PHYID=0x%08x", phyid); > > + > > + dev->net->set_multicast_list = asix_set_multicast; > > + dev->net->ethtool_ops = &ax88772_ethtool_ops; > > + > > + asix_mdio_write(dev->net, dev->mii.phy_id, MII_BMCR, BMCR_RESET); > > + asix_mdio_write(dev->net, dev->mii.phy_id, MII_ADVERTISE, > > + ADVERTISE_ALL | ADVERTISE_CSMA); > > + mii_nway_restart(&dev->mii); > > + > > + if ((ret = asix_write_medium_mode(dev, AX88772_MEDIUM_DEFAULT)) < 0) > > + goto out; > > + > > + if ((ret = asix_write_cmd(dev, AX_CMD_WRITE_IPG0, > > + AX88772_IPG0_DEFAULT | AX88772_IPG1_DEFAULT, > > + AX88772_IPG2_DEFAULT, 0, NULL)) < 0) { > > + dbg("Write IPG,IPG1,IPG2 failed: %d", ret); > > + goto out; > > + } > > + > > + msleep(1); > > + > > + /* Set RX_CTL to default values with 2k buffer, and enable cactus */ > > + if ((ret = asix_write_rx_ctl(dev, AX_DEFAULT_RX_CTL)) < 0) > > + goto out; > > + > > + rx_ctl = asix_read_rx_ctl(dev); > > + dbg("RX_CTL is 0x%04x after all initializations", rx_ctl); > > + > > + rx_ctl = asix_read_medium_status(dev); > > + dbg("Medium Status is 0x%04x after all initializations", rx_ctl); > > + > > + /* Asix framing packs multiple eth frames into a 2K usb bulk transfer */ > > + if (dev->driver_info->flags & FLAG_FRAMING_AX) { > > + /* hard_mtu is still the default - the device does not support > > + jumbo eth frames */ > > + dev->rx_urb_size = 2048; > > + } > > + return 0; > > + > > +out: > > + return ret; > > +} > > + > > static struct ethtool_ops ax88178_ethtool_ops = { > > .get_drvinfo = asix_get_drvinfo, > > .get_link = asix_get_link, > > @@ -1339,6 +1465,17 @@ static const struct driver_info ax88772_ > > .tx_fixup = asix_tx_fixup, > > }; > > > > +static const struct driver_info ax88772a_info = { > > + .description = "ASIX AX88772A USB 2.0 Ethernet", > > + .bind = ax88772a_bind, > > + .status = asix_status, > > + .link_reset = ax88772_link_reset, > > + .reset = ax88772_link_reset, > > + .flags = FLAG_ETHER | FLAG_FRAMING_AX, > > + .rx_fixup = asix_rx_fixup, > > + .tx_fixup = asix_tx_fixup, > > +}; > > + > > static const struct driver_info ax88178_info = { > > .description = "ASIX AX88178 USB 2.0 Ethernet", > > .bind = ax88178_bind, > > @@ -1451,6 +1588,10 @@ static const struct usb_device_id produc > > // Cables-to-Go USB Ethernet Adapter > > USB_DEVICE(0x0b95, 0x772a), > > .driver_info = (unsigned long) &ax88772_info, > > +}, { > > + // ASIX AX88772A > > + USB_DEVICE(0x0b95, 0x772a), > > + .driver_info = (unsigned long) &ax88772a_info, > > This is matching the same id as the entry above, so how will it ever be > used? Do the bind operations distinguish them somehow? > Ugh! I didn't notice that either. Unfortunately, I don't have this new device to be able to test with so I have to rely on the vendors contribution. I would presume that the Cables-to-Go device needs to be using ax88772a_info and in the current state, it isn't going to. I'll check with the contributor for an assist on that. > Please also run checkpatch.pl and fix the errors it finds. > Will do, thanks. > Ben. > > > }, > > { }, // END > > }; > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: David Hollis <dhollis@davehollis.com> Date: Thu, 15 Jan 2009 08:09:34 -0500 > On Wed, 2009-01-14 at 16:53 +0000, Ben Hutchings wrote: > > > > @@ -1451,6 +1588,10 @@ static const struct usb_device_id produc > > > // Cables-to-Go USB Ethernet Adapter > > > USB_DEVICE(0x0b95, 0x772a), > > > .driver_info = (unsigned long) &ax88772_info, > > > +}, { > > > + // ASIX AX88772A > > > + USB_DEVICE(0x0b95, 0x772a), > > > + .driver_info = (unsigned long) &ax88772a_info, > > > > This is matching the same id as the entry above, so how will it ever be > > used? Do the bind operations distinguish them somehow? > > > > Ugh! I didn't notice that either. Unfortunately, I don't have this new > device to be able to test with so I have to rely on the vendors > contribution. I would presume that the Cables-to-Go device needs to be > using ax88772a_info and in the current state, it isn't going to. I'll > check with the contributor for an assist on that. Grrr, I just noticed this and I'm ripping this patch out of my tree. Sigh. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- a/drivers/net/usb/asix.c 2009-01-02 01:06:26.000000000 -0500 +++ b/drivers/net/usb/asix.c 2009-01-14 10:35:11.000000000 -0500 @@ -35,7 +35,7 @@ #include <linux/crc32.h> #include <linux/usb/usbnet.h> -#define DRIVER_VERSION "14-Jun-2006" +#define DRIVER_VERSION "8-Jan-2009" static const char driver_name [] = "asix"; /* ASIX AX8817X based USB 2.0 Ethernet Devices */ @@ -93,8 +93,8 @@ static const char driver_name [] = "asix #define AX_SWRESET_IPPD 0x40 #define AX88772_IPG0_DEFAULT 0x15 -#define AX88772_IPG1_DEFAULT 0x0c -#define AX88772_IPG2_DEFAULT 0x12 +#define AX88772_IPG1_DEFAULT 0x16 +#define AX88772_IPG2_DEFAULT 0x1A /* AX88772 & AX88178 Medium Mode Register */ #define AX_MEDIUM_PF 0x0080 @@ -136,6 +136,13 @@ static const char driver_name [] = "asix #define AX_DEFAULT_RX_CTL \ (AX_RX_CTL_SO | AX_RX_CTL_AB ) +#define AX_PHYSELECT_PSEL 0x01 +#define AX_PHYSELECT_ASEL 0x02 +#define AX_PHYSELECT_SSMII 0x04 +#define AX_PHYSELECT_SSRMII 0x08 +#define AX_PHYSELECT_SSRRMII 0x0C +#define AX_PHYSELECT_SSEN 0x10 + /* GPIO 0 .. 2 toggles */ #define AX_GPIO_GPO0EN 0x01 /* GPIO0 Output enable */ #define AX_GPIO_GPO_0 0x02 /* GPIO0 Output value */ @@ -891,7 +898,11 @@ static int ax88772_link_reset(struct usb if (ecmd.duplex != DUPLEX_FULL) mode &= ~AX_MEDIUM_FD; - devdbg(dev, "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", ecmd.speed, ecmd.duplex, mode); + devdbg(dev, + "ax88772_link_reset() speed: %d duplex: %d setting mode to 0x%04x", + ecmd.speed, + ecmd.duplex, + mode); asix_write_medium_mode(dev, mode); @@ -1018,6 +1029,121 @@ out: return ret; } +static int ax88772a_bind(struct usbnet *dev, struct usb_interface *intf) +{ + int ret; + u16 rx_ctl; + struct asix_data *data = (struct asix_data *)&dev->data; + u8 buf[ETH_ALEN]; + u32 phyid; + + data->eeprom_len = AX88772_EEPROM_LEN; + + usbnet_get_endpoints(dev,intf); + + /* Power up the embedded PHY */ + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) + goto out; + + /* Select the embedded PHY as the active PHY */ + if ((ret = asix_write_cmd(dev, AX_CMD_SW_PHY_SELECT, + AX_PHYSELECT_SSEN | AX_PHYSELECT_PSEL, + 0, 0, NULL)) < 0) { + dbg("Select PHY #1 failed: %d", ret); + goto out; + } + + /* Reload the EEPROM and configure the GPIO pins */ + if ((ret = asix_write_gpio(dev, + AX_GPIO_RSE | AX_GPIO_GPO_2 | AX_GPIO_GPO2EN, 5)) < 0) + goto out; + + /* Set embedded PHY in power down state */ + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL | AX_SWRESET_IPPD)) < 0) + goto out; + + msleep(10); + + /* Set embedded PHY in power up state */ + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) + goto out; + + msleep(60); + + /* Set embedded PHY in reset state */ + if ((ret = asix_sw_reset(dev, AX_SWRESET_CLEAR)) < 0) + goto out; + + /* Set embedded PHY in operating state */ + if ((ret = asix_sw_reset(dev, AX_SWRESET_IPRL)) < 0) + goto out; + + /* Stop RX operation */ + if ((ret = asix_write_rx_ctl(dev, 0)) < 0) + goto out; + + /* Get the MAC address */ + if ((ret = asix_read_cmd(dev, AX_CMD_READ_NODE_ID, + 0, 0, ETH_ALEN, buf)) < 0) { + dbg("Failed to read MAC address: %d", ret); + goto out; + } + memcpy(dev->net->dev_addr, buf, ETH_ALEN); + + /* Initialize MII structure */ + dev->mii.dev = dev->net; + dev->mii.mdio_read = asix_mdio_read; + dev->mii.mdio_write = asix_mdio_write; + dev->mii.phy_id_mask = 0x1f; + dev->mii.reg_num_mask = 0x1f; + dev->net->do_ioctl = asix_ioctl; + dev->mii.phy_id = asix_get_phy_addr(dev); + + phyid = asix_get_phyid(dev); + dbg("PHYID=0x%08x", phyid); + + dev->net->set_multicast_list = asix_set_multicast; + dev->net->ethtool_ops = &ax88772_ethtool_ops; + + asix_mdio_write(dev->net, dev->mii.phy_id, MII_BMCR, BMCR_RESET); + asix_mdio_write(dev->net, dev->mii.phy_id, MII_ADVERTISE, + ADVERTISE_ALL | ADVERTISE_CSMA); + mii_nway_restart(&dev->mii); + + if ((ret = asix_write_medium_mode(dev, AX88772_MEDIUM_DEFAULT)) < 0) + goto out; + + if ((ret = asix_write_cmd(dev, AX_CMD_WRITE_IPG0, + AX88772_IPG0_DEFAULT | AX88772_IPG1_DEFAULT, + AX88772_IPG2_DEFAULT, 0, NULL)) < 0) { + dbg("Write IPG,IPG1,IPG2 failed: %d", ret); + goto out; + } + + msleep(1); + + /* Set RX_CTL to default values with 2k buffer, and enable cactus */ + if ((ret = asix_write_rx_ctl(dev, AX_DEFAULT_RX_CTL)) < 0) + goto out; + + rx_ctl = asix_read_rx_ctl(dev); + dbg("RX_CTL is 0x%04x after all initializations", rx_ctl); + + rx_ctl = asix_read_medium_status(dev); + dbg("Medium Status is 0x%04x after all initializations", rx_ctl); + + /* Asix framing packs multiple eth frames into a 2K usb bulk transfer */ + if (dev->driver_info->flags & FLAG_FRAMING_AX) { + /* hard_mtu is still the default - the device does not support + jumbo eth frames */ + dev->rx_urb_size = 2048; + } + return 0; + +out: + return ret; +} + static struct ethtool_ops ax88178_ethtool_ops = { .get_drvinfo = asix_get_drvinfo, .get_link = asix_get_link, @@ -1339,6 +1465,17 @@ static const struct driver_info ax88772_ .tx_fixup = asix_tx_fixup, }; +static const struct driver_info ax88772a_info = { + .description = "ASIX AX88772A USB 2.0 Ethernet", + .bind = ax88772a_bind, + .status = asix_status, + .link_reset = ax88772_link_reset, + .reset = ax88772_link_reset, + .flags = FLAG_ETHER | FLAG_FRAMING_AX, + .rx_fixup = asix_rx_fixup, + .tx_fixup = asix_tx_fixup, +}; + static const struct driver_info ax88178_info = { .description = "ASIX AX88178 USB 2.0 Ethernet", .bind = ax88178_bind, @@ -1451,6 +1588,10 @@ static const struct usb_device_id produc // Cables-to-Go USB Ethernet Adapter USB_DEVICE(0x0b95, 0x772a), .driver_info = (unsigned long) &ax88772_info, +}, { + // ASIX AX88772A + USB_DEVICE(0x0b95, 0x772a), + .driver_info = (unsigned long) &ax88772a_info, }, { }, // END };
The attached patch adds support for ASIX AX88772A devices, which is used in some netbooks, notebooks and embedded devices. It's largely compatible with the AX88772 family. This re-post does not include changes to the ax88772_link_reset() function that removed the use of the generic MII calls in favor of more explicit register reads. Those changes were not required for AX88772A support. Thanks to Louis <louis@asix.com.tw> for the initial patch. Signed-off-by: David Hollis <dhollis@davehollis.com>