From: Tom Rix <t...@redhat.com>

A break is not needed if it is preceded by a return

Signed-off-by: Tom Rix <t...@redhat.com>
---
 drivers/acpi/utils.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
index 838b719ec7ce..d5411a166685 100644
--- a/drivers/acpi/utils.c
+++ b/drivers/acpi/utils.c
@@ -104,7 +104,6 @@ acpi_extract_package(union acpi_object *package,
                                              " [%c]\n",
                                              i, format_string[i]);
                                return AE_BAD_DATA;
-                               break;
                        }
                        break;
 
@@ -129,7 +128,6 @@ acpi_extract_package(union acpi_object *package,
                                              " expecting [%c]\n",
                                              i, format_string[i]);
                                return AE_BAD_DATA;
-                               break;
                        }
                        break;
                case ACPI_TYPE_LOCAL_REFERENCE:
@@ -144,7 +142,6 @@ acpi_extract_package(union acpi_object *package,
                                              " expecting [%c]\n",
                                              i, format_string[i]);
                                return AE_BAD_DATA;
-                               break;
                        }
                        break;
 
@@ -155,7 +152,6 @@ acpi_extract_package(union acpi_object *package,
                                          i));
                        /* TBD: handle nested packages... */
                        return AE_SUPPORT;
-                       break;
                }
        }
 
-- 
2.18.1

Reply via email to