[PATCH 1/2] mvebu: backport linkstation-poweroff upstream driver
Adrian Schmutzler
mail at adrianschmutzler.de
Thu Aug 27 17:54:42 EDT 2020
Hi,
> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org]
> On Behalf Of Daniel González Cabanelas
> Sent: Donnerstag, 27. August 2020 22:21
> To: openwrt-devel at lists.openwrt.org
> Subject: [PATCH 1/2] mvebu: backport linkstation-poweroff upstream driver
>
> Backport the Linkstation poweroff driver from the kernel upstream (commit
> a7f79f99541ef)
>
> This driver is required by the Buffalo LinkStation LS421DE for a correct power
> off operation. It also allows to use the WoL feature.
>
> Signed-off-by: Daniel González Cabanelas <dgcbueu at gmail.com>
> ---
> package/kernel/linux/modules/other.mk | 21 +++
> .../030-linkstation-poweroff.patch | 177 ++++++++++++++++++
> 2 files changed, 198 insertions(+)
> create mode 100644 target/linux/mvebu/patches-5.4/030-linkstation-
> poweroff.patch
>
> diff --git a/package/kernel/linux/modules/other.mk
> b/package/kernel/linux/modules/other.mk
> index 6eb9404837..683d539be4 100644
> --- a/package/kernel/linux/modules/other.mk
> +++ b/package/kernel/linux/modules/other.mk
> @@ -329,6 +329,27 @@ endef
>
> $(eval $(call KernelPackage,gpio-amd-fch))
>
> +define KernelPackage/linkstation-poweroff
> + SUBMENU:=$(OTHER_MENU)
> + DEPENDS:=@TARGET_mvebu
> + TITLE:=Buffalo LinkStation power off driver
> + KCONFIG:= \
> + CONFIG_POWER_RESET=y \
> + CONFIG_POWER_RESET_LINKSTATION \
> + CONFIG_POWER_RESET_QNAP=n
Since this is specific to mvebu and even refers to symbols only defined in the target, I'd have expected it in target/linux/mvebu/modules.mk?
Best
Adrian
> +
> + FILES:=$(LINUX_DIR)/drivers/power/reset/linkstation-poweroff.ko
> + AUTOLOAD:=$(call AutoLoad,31,linkstation-poweroff,1)
> +endef
> +
> +define KernelPackage/linkstation-poweroff/description
> + This driver supports turning off some Buffalo LinkStations by
> + setting an output pin at the ethernet PHY to the correct state.
> + It also makes the device compatible with the WoL function.
> + Say Y here if you have a Buffalo LinkStation LS421D/E.
> +endef
> +
> +$(eval $(call KernelPackage,linkstation-poweroff))
>
> define KernelPackage/ppdev
> SUBMENU:=$(OTHER_MENU)
> diff --git a/target/linux/mvebu/patches-5.4/030-linkstation-poweroff.patch
> b/target/linux/mvebu/patches-5.4/030-linkstation-poweroff.patch
> new file mode 100644
> index 0000000000..983b1de918
> --- /dev/null
> +++ b/target/linux/mvebu/patches-5.4/030-linkstation-poweroff.patch
> @@ -0,0 +1,177 @@
> +--- a/drivers/power/reset/Kconfig
> ++++ b/drivers/power/reset/Kconfig
> +@@ -99,6 +99,17 @@
> + help
> + Reboot support for Hisilicon boards.
> +
> ++config POWER_RESET_LINKSTATION
> ++ tristate "Buffalo LinkStation power-off driver"
> ++ depends on ARCH_MVEBU || COMPILE_TEST
> ++ depends on OF_MDIO && PHYLIB
> ++ help
> ++ This driver supports turning off some Buffalo LinkStations by
> ++ setting an output pin at the ethernet PHY to the correct state.
> ++ It also makes the device compatible with the WoL function.
> ++
> ++ Say Y here if you have a Buffalo LinkStation LS421D/E.
> ++
> + config POWER_RESET_MSM
> + bool "Qualcomm MSM power-off driver"
> + depends on ARCH_QCOM
> +--- a/drivers/power/reset/Makefile
> ++++ b/drivers/power/reset/Makefile
> +@@ -10,6 +10,7 @@
> + obj-$(CONFIG_POWER_RESET_GPIO) += gpio-poweroff.o
> + obj-$(CONFIG_POWER_RESET_GPIO_RESTART) += gpio-restart.o
> + obj-$(CONFIG_POWER_RESET_HISI) += hisi-reboot.o
> ++obj-${CONFIG_POWER_RESET_LINKSTATION} += linkstation-poweroff.o
> + obj-$(CONFIG_POWER_RESET_MSM) += msm-poweroff.o
> + obj-$(CONFIG_POWER_RESET_QCOM_PON) += qcom-pon.o
> + obj-$(CONFIG_POWER_RESET_OCELOT_RESET) += ocelot-reset.o
> +--- a/dev/null
> ++++ b/drivers/power/reset/linkstation-poweroff.c
> +@@ -0,0 +1,144 @@
> ++// SPDX-License-Identifier: GPL-2.0
> ++/*
> ++ * LinkStation power off restart driver
> ++ * Copyright (C) 2020 Daniel González Cabanelas <dgcbueu at gmail.com>
> ++*/
> ++
> ++#include <linux/module.h>
> ++#include <linux/notifier.h>
> ++#include <linux/of.h>
> ++#include <linux/of_mdio.h>
> ++#include <linux/of_platform.h>
> ++#include <linux/reboot.h>
> ++#include <linux/phy.h>
> ++
> ++/* Defines from the eth phy Marvell driver */
> ++#define MII_MARVELL_COPPER_PAGE 0
> ++#define MII_MARVELL_LED_PAGE 3
> ++#define MII_MARVELL_WOL_PAGE 17
> ++#define MII_MARVELL_PHY_PAGE 22
> ++
> ++#define MII_PHY_LED_CTRL 16
> ++#define MII_88E1318S_PHY_LED_TCR 18
> ++#define MII_88E1318S_PHY_WOL_CTRL 16
> ++#define MII_M1011_IEVENT 19
> ++
> ++#define MII_88E1318S_PHY_LED_TCR_INTn_ENABLE BIT(7)
> ++#define MII_88E1318S_PHY_LED_TCR_FORCE_INT BIT(15)
> ++#define MII_88E1318S_PHY_WOL_CTRL_CLEAR_WOL_STATUS BIT(12)
> ++#define LED2_FORCE_ON (0x8 << 8)
> ++#define LEDMASK
> GENMASK(11,8)
> ++
> ++static struct phy_device *phydev;
> ++
> ++static void mvphy_reg_intn(u16 data)
> ++{
> ++ int rc = 0, saved_page;
> ++
> ++ saved_page = phy_select_page(phydev, MII_MARVELL_LED_PAGE);
> ++ if (saved_page < 0)
> ++ goto err;
> ++
> ++ /* Force manual LED2 control to let INTn work */
> ++ __phy_modify(phydev, MII_PHY_LED_CTRL, LEDMASK,
> LED2_FORCE_ON);
> ++
> ++ /* Set the LED[2]/INTn pin to the required state */
> ++ __phy_modify(phydev, MII_88E1318S_PHY_LED_TCR,
> ++ MII_88E1318S_PHY_LED_TCR_FORCE_INT,
> ++ MII_88E1318S_PHY_LED_TCR_INTn_ENABLE | data);
> ++
> ++ if (!data) {
> ++ /* Clear interrupts to ensure INTn won't be holded in high
> state */
> ++ __phy_write(phydev, MII_MARVELL_PHY_PAGE,
> MII_MARVELL_COPPER_PAGE);
> ++ __phy_read(phydev, MII_M1011_IEVENT);
> ++
> ++ /* If WOL was enabled and a magic packet was received
> before powering
> ++ * off, we won't be able to wake up by sending another magic
> packet.
> ++ * Clear WOL status.
> ++ */
> ++ __phy_write(phydev, MII_MARVELL_PHY_PAGE,
> MII_MARVELL_WOL_PAGE);
> ++ __phy_set_bits(phydev, MII_88E1318S_PHY_WOL_CTRL,
> ++
> MII_88E1318S_PHY_WOL_CTRL_CLEAR_WOL_STATUS);
> ++ }
> ++err:
> ++ rc = phy_restore_page(phydev, saved_page, rc);
> ++ if (rc < 0)
> ++ dev_err(&phydev->mdio.dev, "Write register failed, %d\n",
> rc);
> ++
> ++ if (!data) {
> ++ /* Slow down the PHY to save energy */
> ++ rc= phy_speed_down(phydev, false);
> ++ if (rc < 0)
> ++ dev_err(&phydev->mdio.dev, "PHY speed down
> failed, %d\n", rc);
> ++ }
> ++}
> ++
> ++static int linkstation_reboot_notifier(struct notifier_block *nb,
> ++ unsigned long action, void *unused) {
> ++ if (action == SYS_RESTART)
> ++ mvphy_reg_intn(MII_88E1318S_PHY_LED_TCR_FORCE_INT);
> ++
> ++ return NOTIFY_DONE;
> ++}
> ++
> ++static struct notifier_block linkstation_reboot_nb = {
> ++ .notifier_call = linkstation_reboot_notifier, };
> ++
> ++static void linkstation_poweroff(void) {
> ++ unregister_reboot_notifier(&linkstation_reboot_nb);
> ++ mvphy_reg_intn(0);
> ++
> ++ kernel_restart("Power off");
> ++}
> ++
> ++static const struct of_device_id ls_poweroff_of_match[] = {
> ++ { .compatible = "buffalo,ls421d" },
> ++ { .compatible = "buffalo,ls421de" },
> ++ { },
> ++};
> ++
> ++static int __init linkstation_poweroff_init(void) {
> ++ struct mii_bus *bus;
> ++ struct device_node *dn;
> ++
> ++ dn = of_find_matching_node(NULL, ls_poweroff_of_match);
> ++ if (!dn)
> ++ return -ENODEV;
> ++ of_node_put(dn);
> ++
> ++ dn = of_find_node_by_name(NULL, "mdio");
> ++ if (!dn)
> ++ return -ENODEV;
> ++
> ++ bus = of_mdio_find_bus(dn);
> ++ of_node_put(dn);
> ++ if (!bus)
> ++ return -EPROBE_DEFER;
> ++
> ++ phydev = phy_find_first(bus);
> ++ if (!phydev)
> ++ return -EPROBE_DEFER;
> ++
> ++ register_reboot_notifier(&linkstation_reboot_nb);
> ++ pm_power_off = linkstation_poweroff;
> ++
> ++ pr_info("LinkStation power off driver registered\n");
> ++ return 0;
> ++}
> ++
> ++static void __exit linkstation_poweroff_exit(void) {
> ++ pm_power_off = NULL;
> ++ unregister_reboot_notifier(&linkstation_reboot_nb);
> ++}
> ++
> ++module_init(linkstation_poweroff_init);
> ++module_exit(linkstation_poweroff_exit);
> ++
> ++MODULE_AUTHOR("Daniel González Cabanelas <dgcbueu at gmail.com>");
> ++MODULE_DESCRIPTION("LinkStation power off driver");
> ++MODULE_LICENSE("GPL v2");
> --
> 2.28.0
>
>
>
>
>
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel at lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.openwrt.org/pipermail/openwrt-devel/attachments/20200827/f9c80a93/attachment-0001.sig>
More information about the openwrt-devel
mailing list