lkml.org 
[lkml]   [2017]   [Jan]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/1] EDAC: kill ->show_interleave_mode()
On Sun, Jan 22, 2017 at 06:28:06PM +0100, Nicolas Iooss wrote:
> Function sbridge_register_mci() sets pvt->info.show_interleave_mode to
> knl_show_interleave_mode() on Knight's Landing and
> show_interleave_mode() anywhere else. These functions are only called in
> a debug statement and knl_show_interleave_mode() implementation causes a
> compiler warning (the compiler fails to understand that the default
> branch of the switch condition can never be reached because reg is a
> 2-bit value).
>
> Merge show_interleave_mode() and knl_show_interleave_mode() in a single
> implementation and use it without an indirect function pointer.
>
> Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>
> ---
> drivers/edac/sb_edac.c | 44 ++++++++++----------------------------------
> 1 file changed, 10 insertions(+), 34 deletions(-)

Applied, thanks.

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.

\
 
 \ /
  Last update: 2017-01-23 13:05    [W:0.033 / U:0.192 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site