summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorTang Chen <tangchen@cn.fujitsu.com>2013-08-14 11:37:08 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-08-14 23:24:01 +0200
commit7702ae0dd9b40930931914866999a2ac9734d3eb (patch)
tree2e9fc055c4b4ca5aa54408ebe1d69411d837c00d /lib
parentearlycpio.c: Fix the confusing comment of find_cpio_data(). (diff)
downloadlinux-7702ae0dd9b40930931914866999a2ac9734d3eb.tar.xz
linux-7702ae0dd9b40930931914866999a2ac9734d3eb.zip
ACPI / osl: Kill macro INVALID_TABLE().
The macro INVALID_TABLE() is defined like this: #define INVALID_TABLE(x, path, name) \ { pr_err("ACPI OVERRIDE: " x " [%s%s]\n", path, name); continue; } And it is used like this: for (...) { ... if (...) INVALID_TABLE() ... } The "continue" in the macro makes the code hard to understand. And also, this macro is only used several times in a single file. As suggested by Joe Perches <joe@perches.com>, we can remote it and use pr_err directly. So after this patch, this macro is removed, and pr_err() is used like this: for (...) { ... if (...) { pr_err("ACPI OVERRIDE: ......"); continue; } ... } Signed-off-by: Tang Chen <tangchen@cn.fujitsu.com> Suggested-by: Joe Perches <joe@perches.com> Acked-by: Tejun Heo <tj@kernel.org> Acked-by: Toshi Kani <toshi.kani@hp.com> Reviewed-by: Zhang Yanfei <zhangyanfei@cn.fujitsu.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions