Re: xFrom e2ebdef7c3a5e7ba998ff20bf62806c388efeaac Mon Sep 17 00:00:00 2001

From: Ilkka Koskinen <ilkka@os.amperecomputing.com>
To: Ilkka Koskinen <ilkka@os.amperecomputing.com>
Cc: lorenzo.pieralisi@arm.com, guohanjun@huawei.com,
	sudeep.holla@arm.com,  rafael@kernel.org, linux@armlinux.org.uk,
	lenb@kernel.org,  robert.moore@intel.com, james.morse@arm.com,
	linux-acpi@vger.kernel.org,
	 linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,  patches@amperecomputing.com,
	scott@os.amperecomputing.com,  darren@os.amperecomputing.com
Subject: Re: xFrom e2ebdef7c3a5e7ba998ff20bf62806c388efeaac Mon Sep 17 00:00:00 2001
Date: Thu, 3 Mar 2022 21:32:26 -0800 (PST)	[thread overview]
Message-ID: <alpine.DEB.2.22.394.2203032130410.2294@ubuntu200401> (raw)
In-Reply-To: <20220304052241.151946-1-ilkka@os.amperecomputing.com>

Sorry about the mess. I'll resubmit the patch set.

On Fri, 4 Mar 2022, Ilkka Koskinen wrote:

> Hi Lorenzo,
>
> Would you prefer this version, which doesn't have platform device/driver
> any more?
>
> --Ilkka
>
>
> ----
>
> Arm Generic Diagnostic Dump and Reset device enables a maintainer to
> request OS to perform a diagnostic dump and reset a system via SDEI
> event or an interrupt. This patchset adds support for the SDEI path.
>
> I do have a patch to enable the interrupt path as well but I'm holding
> it back since AGDI table is missing interrupt configuration fields
> (trigger type etc.).
>
> The recently published specification is available at
> https://developer.arm.com/documentation/den0093/latest
>
> The patchset was tested on Ampere Altra/Mt. Jade.
>
> The patchset applies on top of
>  git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm bleeding-edge (9db71e8e3027)
>
> I also tested it works on top of Shuai Xue's new patches in bleeding-edge branch (c6f4ba2d2b9a)
>
> From v1:
>     * Moved pdata to the stack and dropped unnecessary kzalloc() in agdi_init()
>     * Changed the ACPICA patch upstreaming order comment in the paragraph above
>
> From v2:
>     * The first patch was split. The most of it was merged to ACPICA project
>       at first and later ported to linux-acpi
>       (fd919e37cb15914c6fe13e13d530a4f732407c6d). The rest are in the first
>       patch.
>
> From v3:
>     Fixed:
> 	* Moved header files in alphabetical order and removed unnecessary ones
>
> From v4:
> 	* Platform device/driver stuff removed
> 	* acpi_agdi_init() call moved from device_initcall to acpi_init()
> 	* Slightly modified Kconfig text to keep checkpatch happy
>
> Ilkka Koskinen (2):
>  ACPI: tables: Add AGDI to the list of known table signatures
>  ACPI: AGDI: Add support for Arm Generic Diagnostic Dump and Reset
>    device
>
> drivers/acpi/arm64/Kconfig  | 10 +++++++
> drivers/acpi/arm64/Makefile |  1 +
> drivers/acpi/arm64/agdi.c   | 52 +++++++++++++++++++++++++++++++++++++
> drivers/acpi/bus.c          |  2 ++
> drivers/acpi/tables.c       |  2 +-
> include/linux/acpi_agdi.h   | 13 ++++++++++
> 6 files changed, 79 insertions(+), 1 deletion(-)
> create mode 100644 drivers/acpi/arm64/agdi.c
> create mode 100644 include/linux/acpi_agdi.h
>
> -- 
> 2.17.1
>
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel