diff mbox series

[U-Boot,2/4] dm: sysreset: x86: Add a sysreset driver

Message ID 1530611322-20965-2-git-send-email-bmeng.cn@gmail.com
State Accepted
Commit fabb2b4c7f28e672080a97980c4bb1a45375cbb3
Delegated to: Bin Meng
Headers show
Series [U-Boot,1/4] x86: quark: acpi: Add full reset bit to the reset register value in FADT | expand

Commit Message

Bin Meng July 3, 2018, 9:48 a.m. UTC
This adds a generic reset driver for x86 processor.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---

 drivers/sysreset/Kconfig        |  6 +++++
 drivers/sysreset/Makefile       |  1 +
 drivers/sysreset/sysreset_x86.c | 49 +++++++++++++++++++++++++++++++++++++++++
 3 files changed, 56 insertions(+)
 create mode 100644 drivers/sysreset/sysreset_x86.c

Comments

Simon Glass July 9, 2018, 2:39 a.m. UTC | #1
On 3 July 2018 at 02:48, Bin Meng <bmeng.cn@gmail.com> wrote:
> This adds a generic reset driver for x86 processor.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
>  drivers/sysreset/Kconfig        |  6 +++++
>  drivers/sysreset/Makefile       |  1 +
>  drivers/sysreset/sysreset_x86.c | 49 +++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 56 insertions(+)
>  create mode 100644 drivers/sysreset/sysreset_x86.c
>

Reviewed-by: Simon Glass <sjg@chromium.org>
Bin Meng July 19, 2018, 5:33 a.m. UTC | #2
On Mon, Jul 9, 2018 at 10:39 AM, Simon Glass <sjg@chromium.org> wrote:
> On 3 July 2018 at 02:48, Bin Meng <bmeng.cn@gmail.com> wrote:
>> This adds a generic reset driver for x86 processor.
>>
>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>> ---
>>
>>  drivers/sysreset/Kconfig        |  6 +++++
>>  drivers/sysreset/Makefile       |  1 +
>>  drivers/sysreset/sysreset_x86.c | 49 +++++++++++++++++++++++++++++++++++++++++
>>  3 files changed, 56 insertions(+)
>>  create mode 100644 drivers/sysreset/sysreset_x86.c
>>
>
> Reviewed-by: Simon Glass <sjg@chromium.org>

applied to u-boot-x86, thanks!
diff mbox series

Patch

diff --git a/drivers/sysreset/Kconfig b/drivers/sysreset/Kconfig
index a6d48e8..2afeadc 100644
--- a/drivers/sysreset/Kconfig
+++ b/drivers/sysreset/Kconfig
@@ -37,4 +37,10 @@  config SYSRESET_WATCHDOG
 	help
 	  Reboot support for generic watchdog reset.
 
+config SYSRESET_X86
+	bool "Enable support for x86 processor reboot driver"
+	depends on X86
+	help
+	  Reboot support for generic x86 processor reset.
+
 endmenu
diff --git a/drivers/sysreset/Makefile b/drivers/sysreset/Makefile
index 0da58a1..0eb0dc7 100644
--- a/drivers/sysreset/Makefile
+++ b/drivers/sysreset/Makefile
@@ -6,6 +6,7 @@  obj-$(CONFIG_SYSRESET) += sysreset-uclass.o
 obj-$(CONFIG_SYSRESET_PSCI) += sysreset_psci.o
 obj-$(CONFIG_SYSRESET_SYSCON) += sysreset_syscon.o
 obj-$(CONFIG_SYSRESET_WATCHDOG) += sysreset_watchdog.o
+obj-$(CONFIG_SYSRESET_X86) += sysreset_x86.o
 obj-$(CONFIG_ARCH_ROCKCHIP) += sysreset_rockchip.o
 obj-$(CONFIG_SANDBOX) += sysreset_sandbox.o
 obj-$(CONFIG_ARCH_STI) += sysreset_sti.o
diff --git a/drivers/sysreset/sysreset_x86.c b/drivers/sysreset/sysreset_x86.c
new file mode 100644
index 0000000..5943a63
--- /dev/null
+++ b/drivers/sysreset/sysreset_x86.c
@@ -0,0 +1,49 @@ 
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2018, Bin Meng <bmeng.cn@gmail.com>
+ *
+ * Generic reset driver for x86 processor
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <sysreset.h>
+#include <asm/io.h>
+#include <asm/processor.h>
+
+static int x86_sysreset_request(struct udevice *dev, enum sysreset_t type)
+{
+	int value;
+
+	switch (type) {
+	case SYSRESET_WARM:
+		value = SYS_RST | RST_CPU;
+		break;
+	case SYSRESET_COLD:
+		value = SYS_RST | RST_CPU | FULL_RST;
+		break;
+	default:
+		return -ENOSYS;
+	}
+
+	outb(value, IO_PORT_RESET);
+
+	return -EINPROGRESS;
+}
+
+static const struct udevice_id x86_sysreset_ids[] = {
+	{ .compatible = "x86,reset" },
+	{ }
+};
+
+static struct sysreset_ops x86_sysreset_ops = {
+	.request = x86_sysreset_request,
+};
+
+U_BOOT_DRIVER(x86_sysreset) = {
+	.name = "x86-sysreset",
+	.id = UCLASS_SYSRESET,
+	.of_match = x86_sysreset_ids,
+	.ops = &x86_sysreset_ops,
+	.flags = DM_FLAG_PRE_RELOC,
+};