[PATCH v1 0/3 RESEND] NI Ettus Research USRP E3x0 soft-poweroff.

[PATCH v1 0/3 RESEND] NI Ettus Research USRP E3x0 soft-poweroff.

[Moritz Fischer]

This patchset adds support for the NI Ettus Research USRP E3x0 SDR's poweroff
functionality.

v1:
 - Using syscon regmap instead of directly exposing registers.
 - Separate commit for dt bindings.
 - Fixed typo in Kconfig

Moritz Fischer (3):
  power: reset: Add support for NI Ettus Research USRP E3x0
    soft-poweroff.
  dt: power: Add documentation for NI USRP E3x0 soft-poweroff.
  MAINTAINERS: add info for e3x0-poweroff driver.

 .../bindings/power/reset/e3x0-poweroff.txt         | 16 +++++
 MAINTAINERS                                        |  8 +++
 drivers/power/reset/Kconfig                        | 10 +++
 drivers/power/reset/Makefile                       |  1 +
 drivers/power/reset/e3x0-poweroff.c                | 79 ++++++++++++++++++++++
 5 files changed, 114 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/power/reset/e3x0-poweroff.txt
 create mode 100644 drivers/power/reset/e3x0-poweroff.c

-- 
1.9.3

[PATCH v1 1/3 RESEND] power: reset: Add support for NI Ettus Research USRP E3x0 soft-poweroff.

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 drivers/power/reset/Kconfig         | 10 +++++
 drivers/power/reset/Makefile        |  1 +
 drivers/power/reset/e3x0-poweroff.c | 79 +++++++++++++++++++++++++++++++++++++
 3 files changed, 90 insertions(+)

diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig
index 27f6646..7e81c93 100644
--- a/drivers/power/reset/Kconfig
+++ b/drivers/power/reset/Kconfig
@@ -48,6 +48,16 @@ config POWER_RESET_BRCMSTB
 	  Say Y here if you have a Broadcom STB board and you wish
 	  to have restart support.
 
+config POWER_RESET_E3X0
+	tristate "NI Ettus Research USRP E3x0 soft poweroff driver."
+	default n
+	help
+	  Say Y here to enable support for the NI Ettus Research
+	  USRP E3x0 soft poweroff.
+
+	  To compile this driver as a module, chose M here: the
+	  module will be called e3x0-poweroff.
+
 config POWER_RESET_GPIO
 	bool "GPIO power-off driver"
 	depends on OF_GPIO
diff --git a/drivers/power/reset/Makefile b/drivers/power/reset/Makefile
index 11de15b..eef50e9a 100644
--- a/drivers/power/reset/Makefile
+++ b/drivers/power/reset/Makefile
@@ -3,6 +3,7 @@ obj-$(CONFIG_POWER_RESET_AT91_POWEROFF) += at91-poweroff.o
 obj-$(CONFIG_POWER_RESET_AT91_RESET) += at91-reset.o
 obj-$(CONFIG_POWER_RESET_AXXIA) += axxia-reset.o
 obj-$(CONFIG_POWER_RESET_BRCMSTB) += brcmstb-reboot.o
+obj-$(CONFIG_POWER_RESET_E3X0) += e3x0-poweroff.o
 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
diff --git a/drivers/power/reset/e3x0-poweroff.c b/drivers/power/reset/e3x0-poweroff.c
new file mode 100644
index 0000000..767fa19
--- /dev/null
+++ b/drivers/power/reset/e3x0-poweroff.c
@@ -0,0 +1,79 @@
+/*
+ * Copyright (c) 2015, National Instruments Corp. All rights reserved.
+ *
+ * Driver for NI Ettus Research USRP E3x0 soft power off.
+ *
+ * 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; 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/device.h>
+#include <linux/io.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/platform_device.h>
+#include <linux/regmap.h>
+#include <linux/mfd/syscon.h>
+
+static const u32 SC_POWEROFF = 0x0;
+static const u32 SC_POWEROFF_MAGIC = 0x7a;
+
+static struct regmap *syscon;
+
+static void e3x0_pm_power_off(void)
+{
+	regmap_write(syscon, SC_POWEROFF, SC_POWEROFF_MAGIC);
+}
+
+static int e3x0_poweroff_probe(struct platform_device *pdev)
+{
+	struct device *dev = &pdev->dev;
+
+	syscon = syscon_regmap_lookup_by_phandle(dev->of_node, "syscon");
+	if (IS_ERR(syscon)) {
+		dev_err(dev, "Failed to get syscon");
+		return PTR_ERR(syscon);
+	}
+
+	if (!pm_power_off)
+		pm_power_off = e3x0_pm_power_off;
+
+	return 0;
+}
+
+static int e3x0_poweroff_remove(struct platform_device *pdev)
+{
+	if (pm_power_off == e3x0_pm_power_off)
+		pm_power_off = NULL;
+	return 0;
+}
+
+#ifdef CONFIG_OF
+static const struct of_device_id e3x0_poweroff_match[] = {
+	{ .compatible = "ettus,e3x0-poweroff", },
+	{ },
+};
+MODULE_DEVICE_TABLE(of, e3x0_poweroff_match);
+#endif
+
+static struct platform_driver e3x0_poweroff_driver = {
+	.driver	=	{
+		.name	=	"e3x0-poweroff",
+		.of_match_table	=	of_match_ptr(e3x0_poweroff_match),
+	},
+	.probe	=	e3x0_poweroff_probe,
+	.remove	=	e3x0_poweroff_remove,
+};
+module_platform_driver(e3x0_poweroff_driver);
+
+MODULE_LICENSE("GPL v2");
+MODULE_AUTHOR("Moritz Fischer <moritz.fischer@ettus.com>");
+MODULE_DESCRIPTION("NI Ettus Research USRP E3x0 soft-poweroff driver");
+MODULE_ALIAS("platform:e3x0-poweroff");
-- 
1.9.3

[PATCH v1 2/3 RESEND] dt: power: Add documentation for NI USRP E3x0 soft-poweroff.

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 .../devicetree/bindings/power/reset/e3x0-poweroff.txt    | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/Documentation/devicetree/bindings/power/reset/e3x0-poweroff.txt b/Documentation/devicetree/bindings/power/reset/e3x0-poweroff.txt
new file mode 100644
index 0000000..47896e3
--- /dev/null
+++ b/Documentation/devicetree/bindings/power/reset/e3x0-poweroff.txt
@@ -0,0 +1,16 @@
+National Instruments Ettus Research USRP E3x0 poweroff driver
+
+This module is part of the NI Ettus Research USRP E3x0 SDR.
+
+Required properties:
+- compatible: should be one of the following
+  - "ettus,e3x0-poweroff": For devices such as the NI Ettus Research USRP E3x0
+- syscon: this is the phandle to the register map node
+
+Note: Addresses may vary depending on the FPGA configuration
+
+Example:
+	poweroff0: poweroff@40300000 {
+		compatible = "ettus,e3x0-poweroff";
+		syscon = <&regmapnode>;
+	};
-- 
1.9.3

[PATCH v1 3/3 RESEND] MAINTAINERS: add info for e3x0-poweroff driver.

[Moritz Fischer]

Signed-off-by: Moritz Fischer <moritz.fischer@ettus.com>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 85024e2..19c0ae1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3522,6 +3522,14 @@ S:	Supported
 F:	drivers/input/misc/e3x0-button.c
 F:	Documentation/devicetree/bindings/input/e3x0-button.txt
 
+E3X0 POWEROFF DRIVER
+M:	Moritz Fischer <moritz.fischer@ettus.com>
+L:	usrp-users@lists.ettus.com
+W:	http://www.ettus.com
+S:	Supported
+F:	drivers/power/reset/e3x0-poweroff.c
+F:	Documentation/devicetree/bindings/power/reset/e3x0-poweroff.txt
+
 E4000 MEDIA DRIVER
 M:	Antti Palosaari <crope@iki.fi>
 L:	linux-media@vger.kernel.org
-- 
1.9.3

Re: [PATCH v1 0/3 RESEND] NI Ettus Research USRP E3x0 soft-poweroff.

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 284 bytes --]

Hi,

Please CC linux-pm.

On Wed, Mar 04, 2015 at 02:25:46PM -0800, Moritz Fischer wrote:
> This patchset adds support for the NI Ettus Research USRP E3x0 SDR's poweroff
> functionality.

I would prefer to have a generic driver like syscon-reboot does for
system reset.

-- Sebastian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH v1 0/3 RESEND] NI Ettus Research USRP E3x0 soft-poweroff.

[Moritz Fischer]

Hi,

On Sat, Mar 7, 2015 at 11:53 AM, Sebastian Reichel <sre@kernel.org> wrote:

> Please CC linux-pm.

Will do.

> I would prefer to have a generic driver like syscon-reboot does for
> system reset.

Will look into it,

-- Moritz