x86, acpi: LLVMLinux: Remove nested functions from Thinkpad ACPI
authorBehan Webster <behanw@converseincode.com>
Wed, 13 Feb 2013 03:25:57 +0000 (22:25 -0500)
committerBehan Webster <behanw@converseincode.com>
Wed, 9 Apr 2014 20:44:35 +0000 (13:44 -0700)
commit3ddf5ce966a7b87b0605fe9a364a492bd1915d0f
tree4fc3e3475e6ed9c701f7db457fd27c8b55e03736
parent565cbdc2fec92ef2ae75995e06e69172ed2edecd
x86, acpi: LLVMLinux: Remove nested functions from Thinkpad ACPI

The only real change is passing in event_mask to the formerly nested functions.
Otherwise it's just moving around function and macro code.

This is the only place in the Linux kernel where nested functions are still in
use. Nested functions aren't part of the C standards, and complicate the
generated code. Although the Linux Kernel has never set out to be entirely C
standard compliant, it is increasingly compliant to the standard which is
supported by other compilers such as Clang. The LLVMLinux project is working on
being able to compile the Linux kernel with Clang. The use of nested functions
blocks this effort.

Signed-off-by: Behan Webster <behanw@converseincode.com>
Signed-off-by: Jan-Simon Möller <dl9pf@gmx.de>
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
drivers/platform/x86/thinkpad_acpi.c