Hi!
> >diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c
> >index 4f30f06a6f78..61f2d46e52ba 100644
> >--- a/drivers/acpi/acpica/tbxfload.c
> >+++ b/drivers/acpi/acpica/tbxfload.c
> >@@ -297,6 +297,17 @@ acpi_status acpi_load_table(struct acpi_table_header *table)
> > status = acpi_tb_install_and_load_table(ACPI_PTR_TO_PHYSADDR(table),
> > ACPI_TABLE_ORIGIN_EXTERNAL_VIRTUAL,
> > FALSE, &table_index);
> >+
> >+ if (!ACPI_FAILURE(status)) {
> Checkpatch should complain about putting brackets around single statement
> if's.
With multiline statement like that and including comment -- I'd say
keep {'s. It is cleaner this way.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Attachment:
signature.asc
Description: Digital signature
