From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [PATCH 2/7] acpi, apei, ghes: Introduce more generic mechanism to init/deinit GHES error notifications. Date: Wed, 21 May 2014 20:11:45 +0200 Message-ID: <20140521181145.GO21205@pd.tnic> References: <1397056476-9183-1-git-send-email-tomasz.nowicki@linaro.org> <1397056476-9183-3-git-send-email-tomasz.nowicki@linaro.org> <20140513181353.GC8760@pd.tnic> <5374CFB8.3000107@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <5374CFB8.3000107@linaro.org> Sender: linux-kernel-owner@vger.kernel.org To: Tomasz Nowicki Cc: rjw@rjwysocki.net, lenb@kernel.org, tony.luck@intel.com, bp@suse.de, m.chehab@samsung.com, linux-edac@vger.kernel.org, x86@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linaro-acpi@lists.linaro.org List-Id: linux-acpi@vger.kernel.org On Thu, May 15, 2014 at 04:31:20PM +0200, Tomasz Nowicki wrote: > Well, we need to keep these strings somewhere :) IMO, this is simpler > solution. What would be advantage of additional map function? Nevermind, I got confused here. > I am validating generic->notify.type at the begin of function and > return error in case of unknown value. I think there is no need to > check that again. Yes, you do :) -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. --