From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: ghes_edac: enable HIP08 platform edac driver From: Borislav Petkov Message-Id: <20180518111132.GC17285@pd.tnic> Date: Fri, 18 May 2018 13:11:32 +0200 To: James Morse Cc: Zhengqiang , Tyler Baicar , mchehab@kernel.org, toshi.kani@hpe.com, linux-edac@vger.kernel.org, linuxarm@huawei.com, "linux-arm-kernel@lists.infradead.org" List-ID: T24gVGh1LCBNYXkgMTcsIDIwMTggYXQgMDc6MDI6MThQTSArMDEwMCwgSmFtZXMgTW9yc2Ugd3Jv dGU6Cj4gdjQuMTctcmM1IGhhcyAncmV0dXJuIDAnIGhlcmUuIFdvdWxkbid0IHRoaXMgY2hhbmdl IG1lYW5zIG5vIGdoZXMgY2FuIGJlCj4gcmVnaXN0ZXJlZCB1bmxlc3MgZ2hlc19lZGFjIGlzIGFs c28gc3VwcG9ydGVkIGJ5IHRoZSBwbGF0Zm9ybT8KPiBTaG91bGRuJ3QgdGhpcyBiZSAnMCcgZm9y IGEgc2lsZW50IGZhaWx1cmU/CgpodHRwczovL2dpdC5rZXJuZWwub3JnL3B1Yi9zY20vbGludXgv a2VybmVsL2dpdC9icC9icC5naXQvY29tbWl0Lz9oPWZvci1uZXh0JmlkPWNjN2YzZjEzMjY1ODI4 OWI2NjYxYWI4Mjk0YWIwOGE5ZDMyZWEwMjYKCj4gVGVzdGVkIG9uIFNlYXR0bGUgYW5kIHNvbWUg Y3Jhbmt5IGhvbWVicmV3LW5vLURNSSBmaXJtd2FyZToKPiBUZXN0ZWQtYnk6IEphbWVzIE1vcnNl IDxqYW1lcy5tb3JzZUBhcm0uY29tPgo+IAo+IFdpdGggdGhlIEVOT0RFVi8wIHRoaW5nIGFib3Zl Ogo+IFJldmlld2VkLWJ5OiBKYW1lcyBNb3JzZSA8amFtZXMubW9yc2VAYXJtLmNvbT4KClRoYW5r cywgYWRkaW5nLgo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: bp@alien8.de (Borislav Petkov) Date: Fri, 18 May 2018 13:11:32 +0200 Subject: [PATCH] ghes_edac: enable HIP08 platform edac driver In-Reply-To: <8602b133-e0fa-57e2-5159-9d34a1ded85f@arm.com> References: <1526039543-180996-1-git-send-email-zhengqiang10@huawei.com> <20180511121901.GA12705@pd.tnic> <5AF90C70.408@huawei.com> <20180514094709.GC23049@pd.tnic> <20180514164720.GH23049@pd.tnic> <20180516182958.GB17092@pd.tnic> <8602b133-e0fa-57e2-5159-9d34a1ded85f@arm.com> Message-ID: <20180518111132.GC17285@pd.tnic> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, May 17, 2018 at 07:02:18PM +0100, James Morse wrote: > v4.17-rc5 has 'return 0' here. Wouldn't this change means no ghes can be > registered unless ghes_edac is also supported by the platform? > Shouldn't this be '0' for a silent failure? https://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git/commit/?h=for-next&id=cc7f3f132658289b6661ab8294ab08a9d32ea026 > Tested on Seattle and some cranky homebrew-no-DMI firmware: > Tested-by: James Morse > > With the ENODEV/0 thing above: > Reviewed-by: James Morse Thanks, adding. -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.