lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6f31208e-d7da-44ad-92f2-54bbc22b390f@microchip.com>
Date: Wed, 31 Jul 2024 08:03:36 +0000
From: <Parthiban.Veerasooran@...rochip.com>
To: <linux@...ler.io>
CC: <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>,
	<pabeni@...hat.com>, <horms@...nel.org>, <saeedm@...dia.com>,
	<anthony.l.nguyen@...el.com>, <netdev@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <andrew@...n.ch>, <corbet@....net>,
	<linux-doc@...r.kernel.org>, <robh+dt@...nel.org>,
	<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
	<devicetree@...r.kernel.org>, <Horatiu.Vultur@...rochip.com>,
	<ruanjinjie@...wei.com>, <Steen.Hegelund@...rochip.com>,
	<vladimir.oltean@....com>, <masahiroy@...nel.org>, <alexanderduyck@...com>,
	<krzk+dt@...nel.org>, <robh@...nel.org>, <rdunlap@...radead.org>,
	<hkallweit1@...il.com>, <linux@...linux.org.uk>,
	<UNGLinuxDriver@...rochip.com>, <Thorsten.Kummermehr@...rochip.com>,
	<Pier.Beruto@...emi.com>, <Selvamani.Rajagopal@...emi.com>,
	<Nicolas.Ferre@...rochip.com>, <benjamin.bigler@...nformulastudent.ch>
Subject: Re: [PATCH net-next v5 13/14] microchip: lan865x: add driver support
 for Microchip's LAN865X MAC-PHY

Hi Stefan,

On 30/07/24 11:46 am, Stefan Bigler wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Hi Parthiban
> 
> Thanks for v5. I tested and after some fixes it works (I'm still on 6.6.y but it was easy to apply the patches).
> 
> I found the following issues:
> 
> 1) set of Mac-Addr is not working correctly
> be ware to the fact that eth_commit_mac_addr_change() takes the pointer to the sockaddr and not to the sa_data
> 
> drivers/net/ethernet/microchip/lan865x/lan865x.c
> @@ -278,7 +278,7 @@ static int lan865x_set_mac_address(struct net_device *netdev, void *addr)
>          if (ret)
>                  return ret;
> -        eth_commit_mac_addr_change(netdev, address->sa_data);
> +       eth_commit_mac_addr_change(netdev, address);
Ah ok, sorry for the wrong one. Sure will correct it in the next version.
> 
> 2) Missing symbol export
> drivers/net/ethernet/oa_tc6.c
> @@ -1224,6 +1224,8 @@ int oa_tc6_zero_align_receive_frame_enable(struct oa_tc6 *tc6)
>          return oa_tc6_write_register(tc6, OA_TC6_REG_CONFIG0, regval);
>   }
> +EXPORT_SYMBOL_GPL(oa_tc6_zero_align_receive_frame_enable);
Yes, I already changed it in the reposted v5 due to net-next window was 
closed previously.
> 
> 3) My patch for Multicast support was incomplete
> Sorry for this. I did forgott to write that the MAC_NET_CFG_MULTICAST_MODE must be set.
> I addition I also added the correct implemenation for IFF_ALLMULTI
> 
> drivers/net/ethernet/microchip/lan865x/lan865x.c
> @@ -330,6 +330,22 @@ static void lan865x_set_specific_multicast_addr(struct net_device *netdev)
>                  netdev_err(netdev, "Failed to write reg_hashl");
>   }
> 
> +static void lan865x_set_all_multicast_addr(struct net_device *netdev)
> +{
> +       struct lan865x_priv *priv = netdev_priv(netdev);
> +       u32 hash_lo = 0xffffffff;
> +       u32 hash_hi = 0xffffffff;
> +
> +       /* Enabling specific multicast addresses */
> +       if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_H_HASH, hash_hi)) {
> +               netdev_err(netdev, "Failed to write reg_hashh");
> +               return;
> +       }
> +
> +       if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_L_HASH, hash_lo))
> +               netdev_err(netdev, "Failed to write reg_hashl");
> +}
> +
>   static void lan865x_multicast_work_handler(struct work_struct *work)
>   {
>          struct lan865x_priv *priv = container_of(work, struct lan865x_priv,
> @@ -343,14 +359,15 @@ static void lan865x_multicast_work_handler(struct work_struct *work)
>                  regval &= (~MAC_NET_CFG_UNICAST_MODE);
>          } else if (priv->netdev->flags & IFF_ALLMULTI) {
>                  /* Enabling all multicast mode */
> +               lan865x_set_all_multicast_addr(priv->netdev);
>                  regval &= (~MAC_NET_CFG_PROMISCUOUS_MODE);
>                  regval |= MAC_NET_CFG_MULTICAST_MODE;
>                  regval &= (~MAC_NET_CFG_UNICAST_MODE);
>          } else if (!netdev_mc_empty(priv->netdev)) {
>                  lan865x_set_specific_multicast_addr(priv->netdev);
>                  regval &= (~MAC_NET_CFG_PROMISCUOUS_MODE);
> -               regval &= (~MAC_NET_CFG_MULTICAST_MODE);
> -               regval |= MAC_NET_CFG_UNICAST_MODE;
> +               regval |= MAC_NET_CFG_MULTICAST_MODE;
> +               regval &= (~MAC_NET_CFG_UNICAST_MODE);
>          } else {
Sure, I will consider this in the next version. Thanks a lot for your 
support on this.

Best regards,
Parthiban V
> 
> Thanks for your work.
> Best Regards
> Stefan
> 
> 
> Am 2024-07-30T06:09:05.000+02:00 hat Parthiban Veerasooran <Parthiban.Veerasooran@...rochip.com> geschrieben:
> 
>>   The LAN8650/1 is designed to conform to the OPEN Alliance 10BASE-T1x
>> MAC-PHY Serial Interface specification, Version 1.1. The IEEE Clause 4
>> MAC integration provides the low pin count standard SPI interface to any
>> microcontroller therefore providing Ethernet functionality without
>> requiring MAC integration within the microcontroller. The LAN8650/1
>> operates as an SPI client supporting SCLK clock rates up to a maximum of
>> 25 MHz. This SPI interface supports the transfer of both data (Ethernet
>> frames) and control (register access).
>>
>> By default, the chunk data payload is 64 bytes in size. The Ethernet
>> Media Access Controller (MAC) module implements a 10 Mbps half duplex
>> Ethernet MAC, compatible with the IEEE 802.3 standard. 10BASE-T1S
>> physical layer transceiver integrated is into the LAN8650/1. The PHY and
>> MAC are connected via an internal Media Independent Interface (MII).
>>
>> Signed-off-by: Parthiban Veerasooran <Parthiban.Veerasooran@...rochip.com>
>> ---
>>   MAINTAINERS                                   |   6 +
>>   drivers/net/ethernet/microchip/Kconfig        |   1 +
>>   drivers/net/ethernet/microchip/Makefile       |   1 +
>>   .../net/ethernet/microchip/lan865x/Kconfig    |  19 +
>>   .../net/ethernet/microchip/lan865x/Makefile   |   6 +
>>   .../net/ethernet/microchip/lan865x/lan865x.c  | 391 ++++++++++++++++++
>>   6 files changed, 424 insertions(+)
>>   create mode 100644 drivers/net/ethernet/microchip/lan865x/Kconfig
>>   create mode 100644 drivers/net/ethernet/microchip/lan865x/Makefile
>>   create mode 100644 drivers/net/ethernet/microchip/lan865x/lan865x.c
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index ee490b9e363c..907522277010 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -14949,6 +14949,12 @@ L:   netdev@...r.kernel.org
>>   S:   Maintained
>>   F:   drivers/net/ethernet/microchip/lan743x_*
>>
>> +MICROCHIP LAN8650/1 10BASE-T1S MACPHY ETHERNET DRIVER
>> +M:   Parthiban Veerasooran <parthiban.veerasooran@...rochip.com>
>> +L:   netdev@...r.kernel.org
>> +S:   Maintained
>> +F:   drivers/net/ethernet/microchip/lan865x/lan865x.c
>> +
>>   MICROCHIP LAN87xx/LAN937x T1 PHY DRIVER
>>   M:   Arun Ramadoss <arun.ramadoss@...rochip.com>
>>   R:   UNGLinuxDriver@...rochip.com
>> diff --git a/drivers/net/ethernet/microchip/Kconfig b/drivers/net/ethernet/microchip/Kconfig
>> index 43ba71e82260..06ca79669053 100644
>> --- a/drivers/net/ethernet/microchip/Kconfig
>> +++ b/drivers/net/ethernet/microchip/Kconfig
>> @@ -56,6 +56,7 @@ config LAN743X
>>          To compile this driver as a module, choose M here. The module will be
>>          called lan743x.
>>
>> +source "drivers/net/ethernet/microchip/lan865x/Kconfig"
>>   source "drivers/net/ethernet/microchip/lan966x/Kconfig"
>>   source "drivers/net/ethernet/microchip/sparx5/Kconfig"
>>   source "drivers/net/ethernet/microchip/vcap/Kconfig"
>> diff --git a/drivers/net/ethernet/microchip/Makefile b/drivers/net/ethernet/microchip/Makefile
>> index bbd349264e6f..15dfbb321057 100644
>> --- a/drivers/net/ethernet/microchip/Makefile
>> +++ b/drivers/net/ethernet/microchip/Makefile
>> @@ -9,6 +9,7 @@ obj-$(CONFIG_LAN743X) += lan743x.o
>>
>>   lan743x-objs := lan743x_main.o lan743x_ethtool.o lan743x_ptp.o
>>
>> +obj-$(CONFIG_LAN865X) += lan865x/
>>   obj-$(CONFIG_LAN966X_SWITCH) += lan966x/
>>   obj-$(CONFIG_SPARX5_SWITCH) += sparx5/
>>   obj-$(CONFIG_VCAP) += vcap/
>> diff --git a/drivers/net/ethernet/microchip/lan865x/Kconfig b/drivers/net/ethernet/microchip/lan865x/Kconfig
>> new file mode 100644
>> index 000000000000..f3d60d14e202
>> --- /dev/null
>> +++ b/drivers/net/ethernet/microchip/lan865x/Kconfig
>> @@ -0,0 +1,19 @@
>> +# SPDX-License-Identifier: GPL-2.0-only
>> +#
>> +# Microchip LAN865x Driver Support
>> +#
>> +
>> +if NET_VENDOR_MICROCHIP
>> +
>> +config LAN865X
>> +     tristate "LAN865x support"
>> +     depends on SPI
>> +     depends on OA_TC6
>> +     help
>> +       Support for the Microchip LAN8650/1 Rev.B1 MACPHY Ethernet chip. It
>> +       uses OPEN Alliance 10BASE-T1x Serial Interface specification.
>> +
>> +       To compile this driver as a module, choose M here. The module will be
>> +       called lan865x.
>> +
>> +endif # NET_VENDOR_MICROCHIP
>> diff --git a/drivers/net/ethernet/microchip/lan865x/Makefile b/drivers/net/ethernet/microchip/lan865x/Makefile
>> new file mode 100644
>> index 000000000000..9f5dd89c1eb8
>> --- /dev/null
>> +++ b/drivers/net/ethernet/microchip/lan865x/Makefile
>> @@ -0,0 +1,6 @@
>> +# SPDX-License-Identifier: GPL-2.0-only
>> +#
>> +# Makefile for the Microchip LAN865x Driver
>> +#
>> +
>> +obj-$(CONFIG_LAN865X) += lan865x.o
>> diff --git a/drivers/net/ethernet/microchip/lan865x/lan865x.c b/drivers/net/ethernet/microchip/lan865x/lan865x.c
>> new file mode 100644
>> index 000000000000..b25c927659f4
>> --- /dev/null
>> +++ b/drivers/net/ethernet/microchip/lan865x/lan865x.c
>> @@ -0,0 +1,391 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +/*
>> + * Microchip's LAN865x 10BASE-T1S MAC-PHY driver
>> + *
>> + * Author: Parthiban Veerasooran <parthiban.veerasooran@...rochip.com>
>> + */
>> +
>> +#include <linux/module.h>
>> +#include <linux/kernel.h>
>> +#include <linux/phy.h>
>> +#include <linux/oa_tc6.h>
>> +
>> +#define DRV_NAME                     "lan8650"
>> +
>> +/* MAC Network Control Register */
>> +#define LAN865X_REG_MAC_NET_CTL              0x00010000
>> +#define MAC_NET_CTL_TXEN             BIT(3) /* Transmit Enable */
>> +#define MAC_NET_CTL_RXEN             BIT(2) /* Receive Enable */
>> +
>> +/* MAC Network Configuration Reg */
>> +#define LAN865X_REG_MAC_NET_CFG              0x00010001
>> +#define MAC_NET_CFG_PROMISCUOUS_MODE BIT(4)
>> +#define MAC_NET_CFG_MULTICAST_MODE   BIT(6)
>> +#define MAC_NET_CFG_UNICAST_MODE     BIT(7)
>> +
>> +/* MAC Hash Register Bottom */
>> +#define LAN865X_REG_MAC_L_HASH               0x00010020
>> +/* MAC Hash Register Top */
>> +#define LAN865X_REG_MAC_H_HASH               0x00010021
>> +/* MAC Specific Addr 1 Bottom Reg */
>> +#define LAN865X_REG_MAC_L_SADDR1     0x00010022
>> +/* MAC Specific Addr 1 Top Reg */
>> +#define LAN865X_REG_MAC_H_SADDR1     0x00010023
>> +
>> +struct lan865x_priv {
>> +     struct work_struct multicast_work;
>> +     struct net_device *netdev;
>> +     struct spi_device *spi;
>> +     struct oa_tc6 *tc6;
>> +};
>> +
>> +static int lan865x_set_hw_macaddr_low_bytes(struct oa_tc6 *tc6, const u8 *mac)
>> +{
>> +     u32 regval;
>> +
>> +     regval = (mac[3] << 24) | (mac[2] << 16) | (mac[1] << 8) | mac[0];
>> +
>> +     return oa_tc6_write_register(tc6, LAN865X_REG_MAC_L_SADDR1, regval);
>> +}
>> +
>> +static int lan865x_set_hw_macaddr(struct lan865x_priv *priv, const u8 *mac)
>> +{
>> +     int restore_ret;
>> +     u32 regval;
>> +     int ret;
>> +
>> +     /* Configure MAC address low bytes */
>> +     ret = lan865x_set_hw_macaddr_low_bytes(priv->tc6, mac);
>> +     if (ret)
>> +             return ret;
>> +
>> +     /* Prepare and configure MAC address high bytes */
>> +     regval = (mac[5] << 8) | mac[4];
>> +     ret = oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_H_SADDR1,
>> +                                 regval);
>> +     if (!ret)
>> +             return 0;
>> +
>> +     /* Restore the old MAC address low bytes from netdev if the new MAC
>> +      * address high bytes setting failed.
>> +      */
>> +     restore_ret = lan865x_set_hw_macaddr_low_bytes(priv->tc6,
>> +                                                    priv->netdev->dev_addr);
>> +     if (restore_ret)
>> +             return restore_ret;
>> +
>> +     return ret;
>> +}
>> +
>> +static void
>> +lan865x_get_drvinfo(struct net_device *netdev, struct ethtool_drvinfo *info)
>> +{
>> +     strscpy(info->driver, DRV_NAME, sizeof(info->driver));
>> +     strscpy(info->bus_info, dev_name(netdev->dev.parent),
>> +             sizeof(info->bus_info));
>> +}
>> +
>> +static const struct ethtool_ops lan865x_ethtool_ops = {
>> +     .get_drvinfo        = lan865x_get_drvinfo,
>> +     .get_link_ksettings = phy_ethtool_get_link_ksettings,
>> +     .set_link_ksettings = phy_ethtool_set_link_ksettings,
>> +};
>> +
>> +static int lan865x_set_mac_address(struct net_device *netdev, void *addr)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +     struct sockaddr *address = addr;
>> +     int ret;
>> +
>> +     ret = eth_prepare_mac_addr_change(netdev, addr);
>> +     if (ret < 0)
>> +             return ret;
>> +
>> +     if (ether_addr_equal(address->sa_data, netdev->dev_addr))
>> +             return 0;
>> +
>> +     ret = lan865x_set_hw_macaddr(priv, address->sa_data);
>> +     if (ret)
>> +             return ret;
>> +
>> +     eth_commit_mac_addr_change(netdev, address->sa_data);
>> +
>> +     return 0;
>> +}
>> +
>> +static u32 get_address_bit(u8 addr[ETH_ALEN], u32 bit)
>> +{
>> +     return ((addr[bit / 8]) >> (bit % 8)) & 1;
>> +}
>> +
>> +static u32 lan865x_hash(u8 addr[ETH_ALEN])
>> +{
>> +     u32 hash_index = 0;
>> +
>> +     for (int i = 0; i < 6; i++) {
>> +             u32 hash = 0;
>> +
>> +             for (int j = 0; j < 8; j++)
>> +                     hash ^= get_address_bit(addr, (j * 6) + i);
>> +
>> +             hash_index |= (hash << i);
>> +     }
>> +
>> +     return hash_index;
>> +}
>> +
>> +static void lan865x_set_specific_multicast_addr(struct net_device *netdev)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +     struct netdev_hw_addr *ha;
>> +     u32 hash_lo = 0;
>> +     u32 hash_hi = 0;
>> +
>> +     netdev_for_each_mc_addr(ha, netdev) {
>> +             u32 bit_num = lan865x_hash(ha->addr);
>> +
>> +             if (bit_num &gt;= BIT(5))
>> +                     hash_hi |= (1 << (bit_num - BIT(5)));
>> +             else
>> +                     hash_lo |= (1 << bit_num);
>> +     }
>> +
>> +     /* Enabling specific multicast addresses */
>> +     if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_H_HASH, hash_hi)) {
>> +             netdev_err(netdev, "Failed to write reg_hashh");
>> +             return;
>> +     }
>> +
>> +     if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_L_HASH, hash_lo))
>> +             netdev_err(netdev, "Failed to write reg_hashl");
>> +}
>> +
>> +static void lan865x_multicast_work_handler(struct work_struct *work)
>> +{
>> +     struct lan865x_priv *priv = container_of(work, struct lan865x_priv,
>> +                                              multicast_work);
>> +     u32 regval = 0;
>> +
>> +     if (priv->netdev->flags & IFF_PROMISC) {
>> +             /* Enabling promiscuous mode */
>> +             regval |= MAC_NET_CFG_PROMISCUOUS_MODE;
>> +             regval &= (~MAC_NET_CFG_MULTICAST_MODE);
>> +             regval &= (~MAC_NET_CFG_UNICAST_MODE);
>> +     } else if (priv->netdev->flags & IFF_ALLMULTI) {
>> +             /* Enabling all multicast mode */
>> +             regval &= (~MAC_NET_CFG_PROMISCUOUS_MODE);
>> +             regval |= MAC_NET_CFG_MULTICAST_MODE;
>> +             regval &= (~MAC_NET_CFG_UNICAST_MODE);
>> +     } else if (!netdev_mc_empty(priv->netdev)) {
>> +             lan865x_set_specific_multicast_addr(priv->netdev);
>> +             regval &= (~MAC_NET_CFG_PROMISCUOUS_MODE);
>> +             regval &= (~MAC_NET_CFG_MULTICAST_MODE);
>> +             regval |= MAC_NET_CFG_UNICAST_MODE;
>> +     } else {
>> +             /* enabling local mac address only */
>> +             if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_H_HASH,
>> +                                       0)) {
>> +                     netdev_err(priv->netdev, "Failed to write reg_hashh");
>> +                     return;
>> +             }
>> +             if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_L_HASH,
>> +                                       0)) {
>> +                     netdev_err(priv->netdev, "Failed to write reg_hashl");
>> +                     return;
>> +             }
>> +     }
>> +     if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_NET_CFG, regval))
>> +             netdev_err(priv->netdev,
>> +                        "Failed to enable promiscuous/multicast/normal mode");
>> +}
>> +
>> +static void lan865x_set_multicast_list(struct net_device *netdev)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +
>> +     schedule_work(&priv->multicast_work);
>> +}
>> +
>> +static netdev_tx_t lan865x_send_packet(struct sk_buff *skb,
>> +                                    struct net_device *netdev)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +
>> +     return oa_tc6_start_xmit(priv->tc6, skb);
>> +}
>> +
>> +static int lan865x_hw_disable(struct lan865x_priv *priv)
>> +{
>> +     u32 regval;
>> +
>> +     if (oa_tc6_read_register(priv->tc6, LAN865X_REG_MAC_NET_CTL, ®val))
>> +             return -ENODEV;
>> +
>> +     regval &= ~(MAC_NET_CTL_TXEN | MAC_NET_CTL_RXEN);
>> +
>> +     if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_NET_CTL, regval))
>> +             return -ENODEV;
>> +
>> +     return 0;
>> +}
>> +
>> +static int lan865x_net_close(struct net_device *netdev)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +     int ret;
>> +
>> +     netif_stop_queue(netdev);
>> +     phy_stop(netdev->phydev);
>> +     ret = lan865x_hw_disable(priv);
>> +     if (ret) {
>> +             netdev_err(netdev, "Failed to disable the hardware: %d\n", ret);
>> +             return ret;
>> +     }
>> +
>> +     return 0;
>> +}
>> +
>> +static int lan865x_hw_enable(struct lan865x_priv *priv)
>> +{
>> +     u32 regval;
>> +
>> +     if (oa_tc6_read_register(priv->tc6, LAN865X_REG_MAC_NET_CTL, ®val))
>> +             return -ENODEV;
>> +
>> +     regval |= MAC_NET_CTL_TXEN | MAC_NET_CTL_RXEN;
>> +
>> +     if (oa_tc6_write_register(priv->tc6, LAN865X_REG_MAC_NET_CTL, regval))
>> +             return -ENODEV;
>> +
>> +     return 0;
>> +}
>> +
>> +static int lan865x_net_open(struct net_device *netdev)
>> +{
>> +     struct lan865x_priv *priv = netdev_priv(netdev);
>> +     int ret;
>> +
>> +     ret = lan865x_hw_enable(priv);
>> +     if (ret) {
>> +             netdev_err(netdev, "Failed to enable hardware: %d\n", ret);
>> +             return ret;
>> +     }
>> +
>> +     phy_start(netdev->phydev);
>> +
>> +     return 0;
>> +}
>> +
>> +static const struct net_device_ops lan865x_netdev_ops = {
>> +     .ndo_open               = lan865x_net_open,
>> +     .ndo_stop               = lan865x_net_close,
>> +     .ndo_start_xmit         = lan865x_send_packet,
>> +     .ndo_set_rx_mode        = lan865x_set_multicast_list,
>> +     .ndo_set_mac_address    = lan865x_set_mac_address,
>> +};
>> +
>> +static int lan865x_probe(struct spi_device *spi)
>> +{
>> +     struct net_device *netdev;
>> +     struct lan865x_priv *priv;
>> +     int ret;
>> +
>> +     netdev = alloc_etherdev(sizeof(struct lan865x_priv));
>> +     if (!netdev)
>> +             return -ENOMEM;
>> +
>> +     priv = netdev_priv(netdev);
>> +     priv->netdev = netdev;
>> +     priv->spi = spi;
>> +     spi_set_drvdata(spi, priv);
>> +     INIT_WORK(&priv->multicast_work, lan865x_multicast_work_handler);
>> +
>> +     priv->tc6 = oa_tc6_init(spi, netdev);
>> +     if (!priv->tc6) {
>> +             ret = -ENODEV;
>> +             goto free_netdev;
>> +     }
>> +
>> +     /* As per the point s3 in the below errata, SPI receive Ethernet frame
>> +      * transfer may halt when starting the next frame in the same data block
>> +      * (chunk) as the end of a previous frame. The RFA field should be
>> +      * configured to 01b or 10b for proper operation. In these modes, only
>> +      * one receive Ethernet frame will be placed in a single data block.
>> +      * When the RFA field is written to 01b, received frames will be forced
>> +      * to only start in the first word of the data block payload (SWO=0). As
>> +      * recommended, enable zero align receive frame feature for proper
>> +      * operation.
>> +      *
>> +      * https://ww1.microchip.com/downloads/aemDocuments/documents/AIS/ProductDocuments/Errata/LAN8650-1-Errata-80001075.pdf
>> +      */
>> +     ret = oa_tc6_zero_align_receive_frame_enable(priv->tc6);
>> +     if (ret) {
>> +             dev_err(&spi->dev, "Failed to set ZARFE: %d\n", ret);
>> +             goto oa_tc6_exit;
>> +     }
>> +
>> +     /* Get the MAC address from the SPI device tree node */
>> +     if (device_get_ethdev_address(&spi->dev, netdev))
>> +             eth_hw_addr_random(netdev);
>> +
>> +     ret = lan865x_set_hw_macaddr(priv, netdev->dev_addr);
>> +     if (ret) {
>> +             dev_err(&spi->dev, "Failed to configure MAC: %d\n", ret);
>> +             goto oa_tc6_exit;
>> +     }
>> +
>> +     netdev->if_port = IF_PORT_10BASET;
>> +     netdev->irq = spi->irq;
>> +     netdev->netdev_ops = &lan865x_netdev_ops;
>> +     netdev->ethtool_ops = &lan865x_ethtool_ops;
>> +
>> +     ret = register_netdev(netdev);
>> +     if (ret) {
>> +             dev_err(&spi->dev, "Register netdev failed (ret = %d)", ret);
>> +             goto oa_tc6_exit;
>> +     }
>> +
>> +     return 0;
>> +
>> +oa_tc6_exit:
>> +     oa_tc6_exit(priv->tc6);
>> +free_netdev:
>> +     free_netdev(priv->netdev);
>> +     return ret;
>> +}
>> +
>> +static void lan865x_remove(struct spi_device *spi)
>> +{
>> +     struct lan865x_priv *priv = spi_get_drvdata(spi);
>> +
>> +     cancel_work_sync(&priv->multicast_work);
>> +     unregister_netdev(priv->netdev);
>> +     oa_tc6_exit(priv->tc6);
>> +     free_netdev(priv->netdev);
>> +}
>> +
>> +static const struct spi_device_id spidev_spi_ids[] = {
>> +     { .name = "lan8650" },
>> +     {},
>> +};
>> +
>> +static const struct of_device_id lan865x_dt_ids[] = {
>> +     { .compatible = "microchip,lan8650" },
>> +     { /* Sentinel */ }
>> +};
>> +MODULE_DEVICE_TABLE(of, lan865x_dt_ids);
>> +
>> +static struct spi_driver lan865x_driver = {
>> +     .driver = {
>> +             .name = DRV_NAME,
>> +             .of_match_table = lan865x_dt_ids,
>> +      },
>> +     .probe = lan865x_probe,
>> +     .remove = lan865x_remove,
>> +     .id_table = spidev_spi_ids,
>> +};
>> +module_spi_driver(lan865x_driver);
>> +
>> +MODULE_DESCRIPTION(DRV_NAME " 10Base-T1S MACPHY Ethernet Driver");
>> +MODULE_AUTHOR("Parthiban Veerasooran <parthiban.veerasooran@...rochip.com>");
>> +MODULE_LICENSE("GPL");
>> --
>> 2.34.1

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ