| 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
| ||
|
Message-ID: <5626D70C.90403@ti.com>
Date: Tue, 20 Oct 2015 19:06:36 -0500
From: Dan Murphy <dmurphy@...com>
To: "Andrew F. Davis" <afd@...com>,
Florian Fainelli <f.fainelli@...il.com>,
"David S. Miller" <davem@...emloft.net>
CC: <netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] net: phy: dp83848: Add TI DP83848 Ethernet PHY
On 10/20/2015 04:28 PM, Andrew F. Davis wrote:
> Add support for the TI DP83848 Ethernet PHY device.
>
> The DP83848 is a highly reliable, feature rich, IEEE 802.3 compliant
> single port 10/100 Mb/s Ethernet Physical Layer Transceiver supporting
> the MII and RMII interfaces.
>
> Signed-off-by: Andrew F. Davis <afd@...com>
> Signed-off-by: Dan Murphy <dmurphy@...com>
> ---
> drivers/net/phy/Kconfig | 5 +++
> drivers/net/phy/Makefile | 1 +
> drivers/net/phy/dp83848.c | 99 +++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 105 insertions(+)
> create mode 100644 drivers/net/phy/dp83848.c
>
> diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig
> index c5ad98a..12fa29a 100644
> --- a/drivers/net/phy/Kconfig
> +++ b/drivers/net/phy/Kconfig
> @@ -122,6 +122,11 @@ config MICREL_PHY
> ---help---
> Supports the KSZ9021, VSC8201, KS8001 PHYs.
>
> +config DP83848_PHY
> + tristate "Driver for Texas Instruments DP83848 PHY"
> + ---help---
> + Supports the DP83848 PHY.
> +
> config DP83867_PHY
> tristate "Drivers for Texas Instruments DP83867 Gigabit PHY"
> ---help---
> diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile
> index 87f079c..b748224 100644
> --- a/drivers/net/phy/Makefile
> +++ b/drivers/net/phy/Makefile
> @@ -24,6 +24,7 @@ obj-$(CONFIG_MDIO_BITBANG) += mdio-bitbang.o
> obj-$(CONFIG_MDIO_GPIO) += mdio-gpio.o
> obj-$(CONFIG_NATIONAL_PHY) += national.o
> obj-$(CONFIG_DP83640_PHY) += dp83640.o
> +obj-$(CONFIG_DP83848_PHY) += dp83848.o
> obj-$(CONFIG_DP83867_PHY) += dp83867.o
> obj-$(CONFIG_STE10XP) += ste10Xp.o
> obj-$(CONFIG_MICREL_PHY) += micrel.o
> diff --git a/drivers/net/phy/dp83848.c b/drivers/net/phy/dp83848.c
> new file mode 100644
> index 0000000..5ce9bef
> --- /dev/null
> +++ b/drivers/net/phy/dp83848.c
> @@ -0,0 +1,99 @@
> +/*
> + * Driver for the Texas Instruments DP83848 PHY
> + *
> + * Copyright (C) 2015 Texas Instruments Inc.
> + *
> + * 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.
> + *
> + * 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/module.h>
> +#include <linux/phy.h>
> +
> +#define DP83848_PHY_ID 0x20005c90
> +
> +/* Registers */
> +#define DP83848_MICR 0x11
> +#define DP83848_MISR 0x12
> +
> +/* MICR Register Fields */
> +#define DP83848_MICR_INT_OE BIT(0) /* Interrupt Output Enable */
> +#define DP83848_MICR_INTEN BIT(1) /* Interrupt Enable */
> +
> +/* MISR Register Fields */
> +#define DP83848_MISR_RHF_INT_EN BIT(0) /* Receive Error Counter */
> +#define DP83848_MISR_FHF_INT_EN BIT(1) /* False Carrier Counter */
> +#define DP83848_MISR_ANC_INT_EN BIT(2) /* Auto-negotiation complete */
> +#define DP83848_MISR_DUP_INT_EN BIT(3) /* Duplex Status */
> +#define DP83848_MISR_SPD_INT_EN BIT(4) /* Speed status */
> +#define DP83848_MISR_LINK_INT_EN BIT(5) /* Link status */
> +#define DP83848_MISR_ED_INT_EN BIT(6) /* Energy detect */
> +#define DP83848_MISR_LQM_INT_EN BIT(7) /* Link Quality Monitor */
> +
> +static int dp83848_ack_interrupt(struct phy_device *phydev)
> +{
> + int err = phy_read(phydev, DP83848_MISR);
> +
> + return err < 0 ? err : 0;
> +}
> +
> +static int dp83848_config_intr(struct phy_device *phydev)
> +{
> + int err;
> +
> + if (phydev->interrupts == PHY_INTERRUPT_ENABLED) {
> + err = phy_write(phydev, DP83848_MICR,
> + DP83848_MICR_INT_OE |
> + DP83848_MICR_INTEN);
> + if (err < 0)
> + return err;
> +
> + return phy_write(phydev, DP83848_MISR,
> + DP83848_MISR_ANC_INT_EN |
> + DP83848_MISR_DUP_INT_EN |
> + DP83848_MISR_SPD_INT_EN |
> + DP83848_MISR_LINK_INT_EN);
> + }
> +
> + return phy_write(phydev, DP83848_MICR, 0x0);
> +}
> +
> +static struct mdio_device_id __maybe_unused dp83848_tbl[] = {
> + { DP83848_PHY_ID, 0xfffffff0 },
> + { }
> +};
> +MODULE_DEVICE_TABLE(mdio, dp83848_tbl);
> +
> +static struct phy_driver dp83848_driver[] = {
> + {
> + .phy_id = DP83848_PHY_ID,
> + .phy_id_mask = 0xfffffff0,
> + .name = "TI DP83848",
> + .features = PHY_BASIC_FEATURES,
> + .flags = PHY_HAS_INTERRUPT,
> +
> + .soft_reset = genphy_soft_reset,
> + .config_init = genphy_config_init,
> + .suspend = genphy_suspend,
> + .resume = genphy_resume,
> + .config_aneg = genphy_config_aneg,
> + .read_status = genphy_read_status,
> +
> + /* IRQ related */
> + .ack_interrupt = dp83848_ack_interrupt,
> + .config_intr = dp83848_config_intr,
> +
> + .driver = { .owner = THIS_MODULE, },
> + },
> +};
> +module_phy_driver(dp83848_driver);
> +
> +MODULE_DESCRIPTION("Texas Instruments DP83848 PHY driver");
> +MODULE_AUTHOR("Andrew F. Davis <afd@...com");
> +MODULE_LICENSE("GPL");
Acked-by: Dan Murphy <dmurphy@...com>
I know I have my SoB but figured I would give an Acked-by too
--
------------------
Dan Murphy
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists