From: "Hawa, Hanna" <hhhawa@amazon.com>
To: Rob Herring <robh+dt@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
James Morse <james.morse@arm.com>,
Frank Rowand <frowand.list@gmail.com>,
David Miller <davem@davemloft.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Linus Walleij" <linus.walleij@linaro.org>,
<daniel@iogearbox.net>,
"Paul E. McKenney" <paulmck@linux.ibm.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
linux-edac <linux-edac@vger.kernel.org>,
<devicetree@vger.kernel.org>,
"Woodhouse, David" <dwmw@amazon.co.uk>, <benh@amazon.com>,
"Krupnik, Ronen" <ronenk@amazon.com>,
Talel Shenhar <talel@amazon.com>,
Jonathan Chocron <jonnyc@amazon.com>,
"Hanoch, Uri" <hanochu@amazon.com>
Subject: Re: [PATCH v6 3/3] edac: Add support for Amazon's Annapurna Labs L2 EDAC
Date: Thu, 10 Oct 2019 17:03:57 +0300 [thread overview]
Message-ID: <872efced-ec4d-5f24-fca5-c501ed96e570@amazon.com> (raw)
In-Reply-To: <CAL_JsqLZOHx=3d9jPy+7y0a92wA-VKEDQ4PVNvo6L8fRe7xJCQ@mail.gmail.com>
Hi,
On 10/10/2019 2:19 AM, Rob Herring wrote:
> On Mon, Oct 7, 2019 at 10:18 AM Hanna Hawa <hhhawa@amazon.com> wrote:
>>
>> Adds support for Amazon's Annapurna Labs L2 EDAC driver to detect and
>> report L2 errors.
>>
>> Signed-off-by: Hanna Hawa <hhhawa@amazon.com>
>> ---
>> MAINTAINERS | 5 +
>> drivers/edac/Kconfig | 8 ++
>> drivers/edac/Makefile | 1 +
>> drivers/edac/al_l2_edac.c | 251 ++++++++++++++++++++++++++++++++++++++
>> 4 files changed, 265 insertions(+)
>> create mode 100644 drivers/edac/al_l2_edac.c
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7887a62dc843..0eabcfcf91a9 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -748,6 +748,11 @@ M: Hanna Hawa <hhhawa@amazon.com>
>> S: Maintained
>> F: drivers/edac/al_l1_edac.c
>>
>> +AMAZON ANNAPURNA LABS L2 EDAC
>> +M: Hanna Hawa <hhhawa@amazon.com>
>> +S: Maintained
>> +F: drivers/edac/al_l2_edac.c
>> +
>> AMAZON ANNAPURNA LABS THERMAL MMIO DRIVER
>> M: Talel Shenhar <talel@amazon.com>
>> S: Maintained
>> diff --git a/drivers/edac/Kconfig b/drivers/edac/Kconfig
>> index e8161d7c7469..cb394aff1cab 100644
>> --- a/drivers/edac/Kconfig
>> +++ b/drivers/edac/Kconfig
>> @@ -82,6 +82,14 @@ config EDAC_AL_L1
>> for Amazon's Annapurna Labs SoCs.
>> This driver detects errors of L1 caches.
>>
>> +config EDAC_AL_L2
>> + tristate "Amazon's Annapurna Labs L2 EDAC"
>
> I still think this should be an "A57 L2 ECC" driver, but if no one
> cares I'll shut up and the 2nd person can rename everything.
>
>> + depends on ARCH_ALPINE
>
> || COMPILE_TEST
Will be add in next patchset.
>
> Maybe it needs an ARM64 dependency too in this case?
Yes, it need ARM64 dependency, I'll add.
Thanks,
Hanna
>
>> + help
>> + Support for L2 error detection and correction
>> + for Amazon's Annapurna Labs SoCs.
>> + This driver detects errors of L2 caches.
>> +
>
>> +
>> + ret = platform_driver_register(&al_l2_edac_driver);
>> + if (ret) {
>> + pr_err("Failed to register %s (%d)\n", DRV_NAME, ret);
>> + return ret;
>> + }
>> +
>> + edac_l2_device = platform_device_register_simple(DRV_NAME, -1, NULL, 0);
>> + if (IS_ERR(edac_l2_device)) {
>> + pr_err("Failed to register EDAC AL L2 platform device\n");
>> + return PTR_ERR(edac_l2_device);
>> + }
>> +
>> + return 0;
>> +}
>> +
>> +static void __exit al_l2_exit(void)
>> +{
>> + platform_device_unregister(edac_l2_device);
>> + platform_driver_unregister(&al_l2_edac_driver);
>> +}
>> +
>> +late_initcall(al_l2_init);
>> +module_exit(al_l2_exit);
>> +
>> +MODULE_LICENSE("GPL v2");
>> +MODULE_AUTHOR("Hanna Hawa <hhhawa@amazon.com>");
>> +MODULE_DESCRIPTION("Amazon's Annapurna Lab's L2 EDAC Driver");
>> --
>> 2.17.1
>>
prev parent reply other threads:[~2019-10-10 14:04 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-07 15:17 [PATCH v6 0/3] Add support for Amazon's Annapurna Labs EDAC for L1/L2 Hanna Hawa
2019-10-07 15:17 ` [PATCH v6 1/3] edac: Add support for Amazon's Annapurna Labs L1 EDAC Hanna Hawa
2019-10-07 15:17 ` [PATCH v6 2/3] of: EXPORT_SYMBOL_GPL of_find_next_cache_node Hanna Hawa
2019-10-07 15:17 ` [PATCH v6 3/3] edac: Add support for Amazon's Annapurna Labs L2 EDAC Hanna Hawa
2019-10-09 23:09 ` Rob Herring
2019-10-10 14:28 ` Hawa, Hanna
2019-10-09 23:19 ` Rob Herring
2019-10-10 14:03 ` Hawa, Hanna [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=872efced-ec4d-5f24-fca5-c501ed96e570@amazon.com \
--to=hhhawa@amazon.com \
--cc=benh@amazon.com \
--cc=bp@alien8.de \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=dwmw@amazon.co.uk \
--cc=frowand.list@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=hanochu@amazon.com \
--cc=james.morse@arm.com \
--cc=jonnyc@amazon.com \
--cc=linus.walleij@linaro.org \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mchehab@kernel.org \
--cc=paulmck@linux.ibm.com \
--cc=robh+dt@kernel.org \
--cc=ronenk@amazon.com \
--cc=talel@amazon.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).