diff options
author | Tang Chen <[email protected]> | 2013-08-14 17:37:08 +0800 |
---|---|---|
committer | Rafael J. Wysocki <[email protected]> | 2013-08-14 23:24:01 +0200 |
commit | 7702ae0dd9b40930931914866999a2ac9734d3eb (patch) | |
tree | 2e9fc055c4b4ca5aa54408ebe1d69411d837c00d /fs/jbd/commit.c | |
parent | 598bae70c2a8e35c8d39b610cca2b32afcf047af (diff) |
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 <[email protected]>, 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 <[email protected]>
Suggested-by: Joe Perches <[email protected]>
Acked-by: Tejun Heo <[email protected]>
Acked-by: Toshi Kani <[email protected]>
Reviewed-by: Zhang Yanfei <[email protected]>
Signed-off-by: Rafael J. Wysocki <[email protected]>
Diffstat (limited to 'fs/jbd/commit.c')
0 files changed, 0 insertions, 0 deletions