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]
Date:	Sun, 22 May 2016 22:30:48 +0200
From:	Mathias Kresin <m@...sin.me>
To:	Hauke Mehrtens <hauke@...ke-m.de>, f.fainelli@...il.com
Cc:	alexander.stein@...tec-electronic.com, netdev@...r.kernel.org,
	andrew@...n.ch, john@...ozen.org, openwrt@...sin.me,
	hauke.mehrtens@...el.com
Subject: Re: [RFC] NET: PHY: adds driver for Lantiq PHY11G

Hi Hauke,

find my comments in-line.

Am 22.05.2016 um 21:33 schrieb Hauke Mehrtens:
> Supports the Lantiq / Intel CHD 11G and 22E PHYs.
> These PHYs are also named PEF 7061, PEF 7071, PEF 7072
>
> Signed-off-by: John Crispin <john@...ozen.org>
> Signed-off-by: Hauke Mehrtens <hauke@...ke-m.de>
> ---
>
> This is based on a driver from OpenWrt / LEDE. This is send as a RFC
> because the merge window is open now and it adds a new driver. This
> patch was cleaned up on request of Alexander.
>
>
>   .../devicetree/bindings/phy/phy-lanitq.txt         | 216 +++++++++++++++++

Looks like a typo in the filename. lantiq != lanitq

>   drivers/net/phy/Kconfig                            |   6 +
>   drivers/net/phy/Makefile                           |   1 +
>   drivers/net/phy/lantiq.c                           | 269 +++++++++++++++++++++
>   4 files changed, 492 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/phy/phy-lanitq.txt
>   create mode 100644 drivers/net/phy/lantiq.c
>
> diff --git a/Documentation/devicetree/bindings/phy/phy-lanitq.txt b/Documentation/devicetree/bindings/phy/phy-lanitq.txt
> new file mode 100644
> index 0000000..d9746e8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/phy-lanitq.txt
> @@ -0,0 +1,216 @@
> +Lanitq PHY binding

Same typo as mentioned above.

> +============================================
> +
> +This devicetree binding controls the lantiq ethernet phys led functionality.
> +
> +Example:
> +	mdio@0 {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +		compatible = "lantiq,xrx200-mdio";
> +			phy5: ethernet-phy@5 {
> +			reg = <0x1>;
> +			compatible = "lantiq,phy11g", "ethernet-phy-ieee802.3-c22";
> +		};
> +		phy11: ethernet-phy@11 {
> +			reg = <0x11>;
> +			compatible = "lantiq,phy22f", "ethernet-phy-ieee802.3-c22";
> +			lantiq,led2h = <0x00>;
> +			lantiq,led2l = <0x03>;
> +		};
> +		phy12: ethernet-phy@12 {
> +			reg = <0x12>;
> +			compatible = "lantiq,phy22f", "ethernet-phy-ieee802.3-c22";
> +			lantiq,led1h = <0x00>;
> +			lantiq,led1l = <0x03>;
> +		};
> +		phy13: ethernet-phy@13 {
> +			reg = <0x13>;
> +			compatible = "lantiq,phy22f", "ethernet-phy-ieee802.3-c22";
> +			lantiq,led2h = <0x00>;
> +			lantiq,led2l = <0x03>;
> +		};
> +		phy14: ethernet-phy@14 {
> +			reg = <0x14>;
> +			compatible = "lantiq,phy22f", "ethernet-phy-ieee802.3-c22";
> +			lantiq,led1h = <0x00>;
> +			lantiq,led1l = <0x03>;
> +		};
> +	};
> +
> +Register Description
> +============================================
> +
> +LEDCH:
> +
> +Name	Hardware Reset Value
> +LEDCH	0x00C5
> +
> +| 15 |    |    |    |    |    |    |  8 |
> +=========================================
> +|		RES			|
> +=========================================
> +
> +|  7 |    |    |    |    |    |    |  0 |
> +=========================================
> +|   FBF   |   SBF   |RES |     NACS     |
> +=========================================
> +
> +Field	Bits	Type	Description
> +FBC	7:6	RW	Fast Blink Frequency
> +			---
> +			0x0 (00b) F02HZ 2 Hz blinking frequency
> +			0x1 (01b) F04HZ 4 Hz blinking frequency
> +			0x2 (10b) F08HZ 8 Hz blinking frequency
> +			0x3 (11b) F16HZ 16 Hz blinking frequency
> +
> +SBF	5:4	RW	Slow Blink Frequency
> +			---
> +			0x0 (00b) F02HZ 2 Hz blinking frequency
> +			0x1 (01b) F04HZ 4 Hz blinking frequency
> +			0x2 (10b) F08HZ 8 Hz blinking frequency
> +			0x3 (11b) F16HZ 16 Hz blinking frequency
> +
> +NACS	2:0	RW	Inverse of Scan Function
> +			---
> +			0x0 (000b) NONE No Function
> +			0x1 (001b) LINK Complex function enabled when link is up
> +			0x2 (010b) PDOWN Complex function enabled when device is powered-down
> +			0x3 (011b) EEE Complex function enabled when device is in EEE mode
> +			0x4 (100b) ANEG Complex function enabled when auto-negotiation is running
> +			0x5 (101b) ABIST Complex function enabled when analog self-test is running
> +			0x6 (110b) CDIAG Complex function enabled when cable diagnostics are running
> +			0x7 (111b) TEST Complex function enabled when test mode is running
> +
> +LEDCL:
> +
> +Name	Hardware Reset Value
> +LEDCL	0x0067
> +
> +| 15 |    |    |    |    |    |    |  8 |
> +=========================================
> +|		RES			|
> +=========================================
> +
> +|  7 |    |    |    |    |    |    |  0 |
> +=========================================
> +|RES |     SCAN     |RES |    CBLINK    |
> +=========================================
> +
> +Field	Bits	Type	Description
> +SCAN	6:4	RW	Complex Scan Configuration
> +			---
> +			000 B NONE No Function
> +			001 B LINK Complex function enabled when link is up
> +			010 B PDOWN Complex function enabled when device is powered-down
> +			011 B EEE Complex function enabled when device is in EEE mode
> +			100 B ANEG Complex function enabled when auto-negotiation is running
> +			101 B ABIST Complex function enabled when analog self-test is running
> +			110 B CDIAG Complex function enabled when cable diagnostics are running
> +			111 B TEST Complex function enabled when test mode is running
> +
> +CBLINK	2:0	RW	Complex Blinking Configuration
> +			---
> +			000 B NONE No Function
> +			001 B LINK Complex function enabled when link is up
> +			010 B PDOWN Complex function enabled when device is powered-down
> +			011 B EEE Complex function enabled when device is in EEE mode
> +			100 B ANEG Complex function enabled when auto-negotiation is running
> +			101 B ABIST Complex function enabled when analog self-test is running
> +			110 B CDIAG Complex function enabled when cable diagnostics are running
> +			111 B TEST Complex function enabled when test mode is running
> +
> +LEDxH:
> +
> +Name	Hardware Reset Value
> +LED0H	0x0070
> +LED1H	0x0020
> +LED2H	0x0040
> +LED3H	0x0040
> +
> +| 15 |    |    |    |    |    |    |  8 |
> +=========================================
> +|		RES			|
> +=========================================
> +
> +|  7 |    |    |    |    |    |    |  0 |
> +=========================================
> +|        CON        |       BLINKF      |
> +=========================================
> +
> +Field	Bits	Type	Description
> +CON	7:4	RW	Constant On Configuration
> +			---
> +			0x0 (0000b) NONE LED does not light up constantly
> +			0x1 (0001b) LINK10 LED is on when link is 10 Mbit/s
> +			0x2 (0010b) LINK100 LED is on when link is 100 Mbit/s
> +			0x3 (0011b) LINK10X LED is on when link is 10/100 Mbit/s
> +			0x4 (0100b) LINK1000 LED is on when link is 1000 Mbit/s
> +			0x5 (0101b) LINK10_0 LED is on when link is 10/1000 Mbit/s
> +			0x6 (0110b) LINK100X LED is on when link is 100/1000 Mbit/s
> +			0x7 (0111b) LINK10XX LED is on when link is 10/100/1000 Mbit/s
> +			0x8 (1000b) PDOWN LED is on when device is powered-down
> +			0x9 (1001b) EEE LED is on when device is in EEE mode
> +			0xA (1010b) ANEG LED is on when auto-negotiation is running
> +			0xB (1011b) ABIST LED is on when analog self-test is running
> +			0xC (1100b) CDIAG LED is on when cable diagnostics are running
> +
> +BLINKF	3:0	RW	Fast Blinking Configuration
> +			---
> +			0x0 (0000b) NONE No Blinking
> +			0x1 (0001b) LINK10 Blink when link is 10 Mbit/s
> +			0x2 (0010b) LINK100 Blink when link is 100 Mbit/s
> +			0x3 (0011b) LINK10X Blink when link is 10/100 Mbit/s
> +			0x4 (0100b) LINK1000 Blink when link is 1000 Mbit/s
> +			0x5 (0101b) LINK10_0 Blink when link is 10/1000 Mbit/s
> +			0x6 (0110b) LINK100X Blink when link is 100/1000 Mbit/s
> +			0x7 (0111b) LINK10XX Blink when link is 10/100/1000 Mbit/s
> +			0x8 (1000b) PDOWN Blink when device is powered-down
> +			0x9 (1001b) EEE Blink when device is in EEE mode
> +			0xA (1010b) ANEG Blink when auto-negotiation is running
> +			0xB (1011b) ABIST Blink when analog self-test is running
> +			0xC (1100b) CDIAG Blink when cable diagnostics are running
> +
> +LEDxL:
> +
> +Name	Hardware Reset Value
> +LED0L	0x0003
> +LED1L	0x0000
> +LED2L	0x0000
> +LED3L	0x0020
> +
> +| 15 |    |    |    |    |    |    |  8 |
> +=========================================
> +|		RES			|
> +=========================================
> +
> +|  7 |    |    |    |    |    |    |  0 |
> +=========================================
> +|      BLINKS       |       PULSE       |
> +=========================================
> +
> +Field	Bits	Type	Description
> +BLINKS	7:4	RW	Slow Blinkin Configuration
> +			---
> +			0x0 (0000b) NONE No Blinking
> +			0x1 (0001b) LINK10 Blink when link is 10 Mbit/s
> +			0x2 (0010b) LINK100 Blink when link is 100 Mbit/s
> +			0x3 (0011b) LINK10X Blink when link is 10/100 Mbit/s
> +			0x4 (0100b) LINK1000 Blink when link is 1000 Mbit/s
> +			0x5 (0101b) LINK10_0 Blink when link is 10/1000 Mbit/s
> +			0x6 (0110b) LINK100X Blink when link is 100/1000 Mbit/s
> +			0x7 (0111b) LINK10XX Blink when link is 10/100/1000 Mbit/s
> +			0x8 (1000b) PDOWN Blink when device is powered-down
> +			0x9 (1001b) EEE Blink when device is in EEE mode
> +			0xA (1010b) ANEG Blink when auto-negotiation is running
> +			0xB (1011b) ABIST Blink when analog self-test is running
> +			0xC (1100b) CDIAG Blink when cable diagnostics are runningning
> +
> +PULSE	3:0	RW	Pulsing Configuration
> +			The pulse field is a mask field by which certain events can be combined
> +			---
> +			0x0 (0000b) NONE No pulsing
> +			0x1 (0001b) TXACT Transmit activity
> +			0x2 (0010b) RXACT Receive activity
> +			0x4 (0100b) COL Collision
> +			0x8 (1000b) RES Reserved
> diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig
> index f0a7702..f3685ef 100644
> --- a/drivers/net/phy/Kconfig
> +++ b/drivers/net/phy/Kconfig
> @@ -257,6 +257,12 @@ config MDIO_BCM_IPROC
>   	  This module provides a driver for the MDIO busses found in the
>   	  Broadcom iProc SoC's.
>
> +config LANTIQ_PHY
> +	tristate "Driver for Lantiq PHYs"
> +	---help---
> +	  Supports the Lantiq / Intel CHD 11G and 22E PHYs.
> +	  These PHYs are also named PEF 7061, PEF 7071 and PEF 7072
> +
>   endif # PHYLIB
>
>   config MICREL_KS8995MA
> diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile
> index 680e88f9..042b0ef6 100644
> --- a/drivers/net/phy/Makefile
> +++ b/drivers/net/phy/Makefile
> @@ -42,3 +42,4 @@ obj-$(CONFIG_MDIO_MOXART)	+= mdio-moxart.o
>   obj-$(CONFIG_MDIO_BCM_UNIMAC)	+= mdio-bcm-unimac.o
>   obj-$(CONFIG_MICROCHIP_PHY)	+= microchip.o
>   obj-$(CONFIG_MDIO_BCM_IPROC)	+= mdio-bcm-iproc.o
> +obj-$(CONFIG_LANTIQ_PHY)	+= lantiq.o
> diff --git a/drivers/net/phy/lantiq.c b/drivers/net/phy/lantiq.c
> new file mode 100644
> index 0000000..65a8086
> --- /dev/null
> +++ b/drivers/net/phy/lantiq.c
> @@ -0,0 +1,269 @@
> +/*
> + * Copyright (C) 2012 Daniel Schwierzeck <daniel.schwierzeck@...glemail.com>
> + * Copyright (C) 2016 Hauke Mehrtens <hauke@...ke-m.de>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <linux/mdio.h>
> +#include <linux/module.h>
> +#include <linux/phy.h>
> +#include <linux/of.h>
> +
> +#define MII_VR9_11G_IMASK	0x19	/* interrupt mask */
> +#define MII_VR9_11G_ISTAT	0x1a	/* interrupt status */
> +
> +#define INT_VR9_11G_WOL		BIT(15)	/* Wake-On-LAN */
> +#define INT_VR9_11G_ANE		BIT(11)	/* Auto-Neg error */
> +#define INT_VR9_11G_ANC		BIT(10)	/* Auto-Neg complete */
> +#define INT_VR9_11G_ADSC	BIT(5)	/* Link auto-downspeed detect */
> +#define INT_VR9_11G_DXMC	BIT(2)	/* Duplex mode change */
> +#define INT_VR9_11G_LSPC	BIT(1)	/* Link speed change */
> +#define INT_VR9_11G_LSTC	BIT(0)	/* Link state change */
> +#define INT_VR9_11G_MASK	(INT_VR9_11G_LSTC | INT_VR9_11G_ADSC)
> +
> +#define ADVERTISED_MPD		BIT(10)	/* Multi-port device */
> +
> +#define PHY_ID_PHY11G_1_3	0x030260D1
> +#define PHY_ID_PHY22F_1_3	0x030260E1
> +#define PHY_ID_PHY11G_1_4	0xD565A400
> +#define PHY_ID_PHY22F_1_4	0xD565A410
> +#define PHY_ID_PHY11G_1_5	0xD565A401
> +#define PHY_ID_PHY22F_1_5	0xD565A411
> +
> +static void lantiq_gphy_of_reg_init(struct phy_device *phydev)
> +{
> +	struct device_node *node = phydev->mdio.dev.of_node;
> +	u32 tmp;
> +
> +	if (!IS_ENABLED(CONFIG_OF_MDIO))
> +		return;
> +
> +	/* store the led values if one was passed by the device tree */
> +	if (!of_property_read_u32(node, "lantiq,ledch", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e0, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,ledcl", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e1, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led0h", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e2, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led0l", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e3, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led1h", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e4, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led1l", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e5, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led2h", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e6, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led2l", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e7, MDIO_MMD_VEND2, tmp);
> +
> +	/* The LED3 is only available in PEF 7072 package. */
> +	if (!of_property_read_u32(node, "lantiq,led3h", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e8, MDIO_MMD_VEND2, tmp);
> +
> +	if (!of_property_read_u32(node, "lantiq,led3l", &tmp))
> +		phy_write_mmd_indirect(phydev, 0x1e9, MDIO_MMD_VEND2, tmp);
> +}
> +
> +static int lantiq_gphy_config_init(struct phy_device *phydev)
> +{
> +	int err;
> +
> +	/* Mask all interrupts */
> +	err = phy_write(phydev, MII_VR9_11G_IMASK, 0);
> +	if (err)
> +		return err;
> +
> +	/* Clear all pending interrupts */
> +	phy_read(phydev, MII_VR9_11G_ISTAT);
> +
> +	phy_write_mmd_indirect(phydev, 0x1e0, MDIO_MMD_VEND2, 0xc5);
> +	phy_write_mmd_indirect(phydev, 0x1e1, MDIO_MMD_VEND2, 0x67);

Any specific reason why the complex functions are enabled by default?

> +	phy_write_mmd_indirect(phydev, 0x1e2, MDIO_MMD_VEND2, 0x42);
> +	phy_write_mmd_indirect(phydev, 0x1e3, MDIO_MMD_VEND2, 0x10);
> +	phy_write_mmd_indirect(phydev, 0x1e4, MDIO_MMD_VEND2, 0x70);
> +	phy_write_mmd_indirect(phydev, 0x1e5, MDIO_MMD_VEND2, 0x03);
> +	phy_write_mmd_indirect(phydev, 0x1e6, MDIO_MMD_VEND2, 0x20);
> +	phy_write_mmd_indirect(phydev, 0x1e7, MDIO_MMD_VEND2, 0x00);
> +	phy_write_mmd_indirect(phydev, 0x1e8, MDIO_MMD_VEND2, 0x40);
> +	phy_write_mmd_indirect(phydev, 0x1e9, MDIO_MMD_VEND2, 0x20);

I would suggest to use the same blink/permanent on configuration for all 
led pins (as it is in LEDE/OpenWrt):

     Constant On: 10/100/1000MBit
     Blink Fast: None
     Blink Slow: None
     Pulse: TX/RX

I'm aware of only one CPE that uses more than one led for status 
indication. All other have a single led attached to any of the pins.

This way it's only required to change the default configuration via the 
device tree bindings for the minority of the devices.


> +
> +	lantiq_gphy_of_reg_init(phydev);
> +
> +	return 0;
> +}
> +
> +static int lantiq_gphy14_config_aneg(struct phy_device *phydev)
> +{
> +	int reg, err;
> +
> +	/* Advertise as multi-port device, see IEEE802.3-2002 40.5.1.1 */
> +	/* This is a workaround for an errata in rev < 1.5 devices */
> +	reg = phy_read(phydev, MII_CTRL1000);
> +	reg |= ADVERTISED_MPD;
> +	err = phy_write(phydev, MII_CTRL1000, reg);
> +	if (err)
> +		return err;
> +
> +	return genphy_config_aneg(phydev);
> +}
> +
> +static int lantiq_gphy_ack_interrupt(struct phy_device *phydev)
> +{
> +	int reg;
> +
> +	/**
> +	 * Possible IRQ numbers:
> +	 * - IM3_IRL18 for GPHY0
> +	 * - IM3_IRL17 for GPHY1
> +	 *
> +	 * Due to a silicon bug IRQ lines are not really independent from
> +	 * each other. Sometimes the two lines are driven at the same time
> +	 * if only one GPHY core raises the interrupt.
> +	 */
> +	reg = phy_read(phydev, MII_VR9_11G_ISTAT);
> +
> +	return (reg < 0) ? reg : 0;
> +}
> +
> +static int lantiq_gphy_did_interrupt(struct phy_device *phydev)
> +{
> +	int reg;
> +
> +	reg = phy_read(phydev, MII_VR9_11G_ISTAT);
> +
> +	return reg > 0;
> +}
> +
> +static int lantiq_gphy_config_intr(struct phy_device *phydev)
> +{
> +	int err;
> +
> +	if (phydev->interrupts == PHY_INTERRUPT_ENABLED)
> +		err = phy_write(phydev, MII_VR9_11G_IMASK, INT_VR9_11G_MASK);
> +	else
> +		err = phy_write(phydev, MII_VR9_11G_IMASK, 0);
> +
> +	return err;
> +}
> +
> +static struct phy_driver lantiq_gphy[] = {
> +	{
> +		.phy_id		= PHY_ID_PHY11G_1_3,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY11G (PEF 7071/PEF 7072) v1.3",
> +		.features	= (PHY_GBIT_FEATURES | SUPPORTED_Pause),
> +		 /* there is an errata regarding irqs in this rev */
> +		.flags		= 0,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= lantiq_gphy14_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	}, {
> +		.phy_id		= PHY_ID_PHY22F_1_3,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY22F (PEF 7061) v1.3",
> +		.features	= (PHY_BASIC_FEATURES | SUPPORTED_Pause),
> +		 /* there is an errata regarding irqs in this rev */
> +		.flags		= 0,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= lantiq_gphy14_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	}, {
> +		.phy_id		= PHY_ID_PHY11G_1_4,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY11G (PEF 7071/PEF 7072) v1.4",
> +		.features	= (PHY_GBIT_FEATURES | SUPPORTED_Pause),
> +		.flags		= PHY_HAS_INTERRUPT,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= lantiq_gphy14_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	}, {
> +		.phy_id		= PHY_ID_PHY22F_1_4,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY22F (PEF 7061) v1.4",
> +		.features	= (PHY_BASIC_FEATURES | SUPPORTED_Pause),
> +		.flags		= PHY_HAS_INTERRUPT,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= lantiq_gphy14_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	}, {
> +		.phy_id		= PHY_ID_PHY11G_1_5,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY11G (PEF 7071/PEF 7072) v1.5 / v1.6",
> +		.features	= (PHY_GBIT_FEATURES | SUPPORTED_Pause),
> +		.flags		= PHY_HAS_INTERRUPT,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= genphy_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	}, {
> +		.phy_id		= PHY_ID_PHY22F_1_5,
> +		.phy_id_mask	= 0xffffffff,
> +		.name		= "Lantiq XWAY PHY22F (PEF 7061) v1.5 / v1.6",
> +		.features	= (PHY_BASIC_FEATURES | SUPPORTED_Pause),
> +		.flags		= PHY_HAS_INTERRUPT,
> +		.config_init	= lantiq_gphy_config_init,
> +		.config_aneg	= genphy_config_aneg,
> +		.read_status	= genphy_read_status,
> +		.ack_interrupt	= lantiq_gphy_ack_interrupt,
> +		.did_interrupt	= lantiq_gphy_did_interrupt,
> +		.config_intr	= lantiq_gphy_config_intr,
> +		.suspend	= genphy_suspend,
> +		.resume		= genphy_resume,
> +	},
> +};
> +module_phy_driver(lantiq_gphy);
> +
> +static struct mdio_device_id __maybe_unused lantiq_gphy_tbl[] = {
> +	{ PHY_ID_PHY11G_1_3, 0xffffffff },
> +	{ PHY_ID_PHY22F_1_3, 0xffffffff },
> +	{ PHY_ID_PHY11G_1_4, 0xffffffff },
> +	{ PHY_ID_PHY22F_1_4, 0xffffffff },
> +	{ PHY_ID_PHY11G_1_5, 0xffffffff },
> +	{ PHY_ID_PHY22F_1_5, 0xffffffff },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(mdio, lantiq_gphy_tbl);
> +
> +MODULE_DESCRIPTION("Lantiq PHY driver");
> +MODULE_AUTHOR("Daniel Schwierzeck <daniel.schwierzeck@...glemail.com>");
> +MODULE_LICENSE("GPL");
>



Download attachment "smime.p7s" of type "application/pkcs7-signature" (4342 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ