Merge git://git.skbuff.net/gitroot/yoshfuji/linux-2.6-git-rfc3542
[sfrench/cifs-2.6.git] / drivers / acpi / resources / rscalc.c
index 98176f2fcb5d878442808affed0d4486e8e97fe7..378f58390fc14935fff3f9fa4e519e6d2293e143 100644 (file)
  * POSSIBILITY OF SUCH DAMAGES.
  */
 
-
 #include <acpi/acpi.h>
 #include <acpi/acresrc.h>
 #include <acpi/amlcode.h>
 #include <acpi/acnamesp.h>
 
 #define _COMPONENT          ACPI_RESOURCES
-        ACPI_MODULE_NAME    ("rscalc")
-
+ACPI_MODULE_NAME("rscalc")
 
 /*******************************************************************************
  *
  *              the resource data.
  *
  ******************************************************************************/
-
 acpi_status
-acpi_rs_get_byte_stream_length (
-       struct acpi_resource            *linked_list,
-       acpi_size                       *size_needed)
+acpi_rs_get_byte_stream_length(struct acpi_resource *linked_list,
+                              acpi_size * size_needed)
 {
-       acpi_size                       byte_stream_size_needed = 0;
-       acpi_size                       segment_size;
-       u8                              done = FALSE;
-
-
-       ACPI_FUNCTION_TRACE ("rs_get_byte_stream_length");
+       acpi_size byte_stream_size_needed = 0;
+       acpi_size segment_size;
+       u8 done = FALSE;
 
+       ACPI_FUNCTION_TRACE("rs_get_byte_stream_length");
 
        while (!done) {
                /* Init the variable that will hold the size to add to the total. */
@@ -145,11 +139,11 @@ acpi_rs_get_byte_stream_length (
                         */
                        if (linked_list->data.vendor_specific.length > 7) {
                                segment_size = 3;
-                       }
-                       else {
+                       } else {
                                segment_size = 1;
                        }
-                       segment_size += linked_list->data.vendor_specific.length;
+                       segment_size +=
+                           linked_list->data.vendor_specific.length;
                        break;
 
                case ACPI_RSTYPE_END_TAG:
@@ -194,9 +188,11 @@ acpi_rs_get_byte_stream_length (
                         */
                        segment_size = 16;
 
-                       if (linked_list->data.address16.resource_source.string_ptr) {
+                       if (linked_list->data.address16.resource_source.
+                           string_ptr) {
                                segment_size +=
-                                       linked_list->data.address16.resource_source.string_length;
+                                   linked_list->data.address16.resource_source.
+                                   string_length;
                                segment_size++;
                        }
                        break;
@@ -211,9 +207,11 @@ acpi_rs_get_byte_stream_length (
                         */
                        segment_size = 26;
 
-                       if (linked_list->data.address32.resource_source.string_ptr) {
+                       if (linked_list->data.address32.resource_source.
+                           string_ptr) {
                                segment_size +=
-                                       linked_list->data.address32.resource_source.string_length;
+                                   linked_list->data.address32.resource_source.
+                                   string_length;
                                segment_size++;
                        }
                        break;
@@ -227,9 +225,11 @@ acpi_rs_get_byte_stream_length (
                         */
                        segment_size = 46;
 
-                       if (linked_list->data.address64.resource_source.string_ptr) {
+                       if (linked_list->data.address64.resource_source.
+                           string_ptr) {
                                segment_size +=
-                                       linked_list->data.address64.resource_source.string_length;
+                                   linked_list->data.address64.resource_source.
+                                   string_length;
                                segment_size++;
                        }
                        break;
@@ -244,11 +244,14 @@ acpi_rs_get_byte_stream_length (
                         * Resource Source + 1 for the null.
                         */
                        segment_size = 9 + (((acpi_size)
-                               linked_list->data.extended_irq.number_of_interrupts - 1) * 4);
+                                            linked_list->data.extended_irq.
+                                            number_of_interrupts - 1) * 4);
 
-                       if (linked_list->data.extended_irq.resource_source.string_ptr) {
+                       if (linked_list->data.extended_irq.resource_source.
+                           string_ptr) {
                                segment_size +=
-                                       linked_list->data.extended_irq.resource_source.string_length;
+                                   linked_list->data.extended_irq.
+                                   resource_source.string_length;
                                segment_size++;
                        }
                        break;
@@ -257,9 +260,9 @@ acpi_rs_get_byte_stream_length (
 
                        /* If we get here, everything is out of sync, exit with error */
 
-                       return_ACPI_STATUS (AE_AML_INVALID_RESOURCE_TYPE);
+                       return_ACPI_STATUS(AE_AML_INVALID_RESOURCE_TYPE);
 
-               } /* switch (linked_list->Id) */
+               }               /* switch (linked_list->Id) */
 
                /* Update the total */
 
@@ -267,17 +270,16 @@ acpi_rs_get_byte_stream_length (
 
                /* Point to the next object */
 
-               linked_list = ACPI_PTR_ADD (struct acpi_resource,
-                                 linked_list, linked_list->length);
+               linked_list = ACPI_PTR_ADD(struct acpi_resource,
+                                          linked_list, linked_list->length);
        }
 
        /* This is the data the caller needs */
 
        *size_needed = byte_stream_size_needed;
-       return_ACPI_STATUS (AE_OK);
+       return_ACPI_STATUS(AE_OK);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_rs_get_list_length
@@ -297,32 +299,28 @@ acpi_rs_get_byte_stream_length (
  ******************************************************************************/
 
 acpi_status
-acpi_rs_get_list_length (
-       u8                              *byte_stream_buffer,
-       u32                             byte_stream_buffer_length,
-       acpi_size                       *size_needed)
+acpi_rs_get_list_length(u8 * byte_stream_buffer,
+                       u32 byte_stream_buffer_length, acpi_size * size_needed)
 {
-       u32                             buffer_size = 0;
-       u32                             bytes_parsed = 0;
-       u8                              number_of_interrupts = 0;
-       u8                              number_of_channels = 0;
-       u8                              resource_type;
-       u32                             structure_size;
-       u32                             bytes_consumed;
-       u8                              *buffer;
-       u8                              temp8;
-       u16                             temp16;
-       u8                              index;
-       u8                              additional_bytes;
-
-
-       ACPI_FUNCTION_TRACE ("rs_get_list_length");
-
+       u32 buffer_size = 0;
+       u32 bytes_parsed = 0;
+       u8 number_of_interrupts = 0;
+       u8 number_of_channels = 0;
+       u8 resource_type;
+       u32 structure_size;
+       u32 bytes_consumed;
+       u8 *buffer;
+       u8 temp8;
+       u16 temp16;
+       u8 index;
+       u8 additional_bytes;
+
+       ACPI_FUNCTION_TRACE("rs_get_list_length");
 
        while (bytes_parsed < byte_stream_buffer_length) {
                /* The next byte in the stream is the resource type */
 
-               resource_type = acpi_rs_get_resource_type (*byte_stream_buffer);
+               resource_type = acpi_rs_get_resource_type(*byte_stream_buffer);
 
                switch (resource_type) {
                case ACPI_RDESC_TYPE_MEMORY_24:
@@ -331,10 +329,10 @@ acpi_rs_get_list_length (
                         */
                        bytes_consumed = 12;
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_mem24);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_mem24);
                        break;
 
-
                case ACPI_RDESC_TYPE_LARGE_VENDOR:
                        /*
                         * Vendor Defined Resource
@@ -342,38 +340,39 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
                        ++buffer;
 
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
                        bytes_consumed = temp16 + 3;
 
                        /* Ensure a 32-bit boundary for the structure */
 
-                       temp16 = (u16) ACPI_ROUND_UP_to_32_bITS (temp16);
+                       temp16 = (u16) ACPI_ROUND_UP_to_32_bITS(temp16);
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_vendor) +
-                                          (temp16 * sizeof (u8));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_vendor) +
+                           (temp16 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_MEMORY_32:
                        /*
                         * 32-Bit Memory Range Resource
                         */
                        bytes_consumed = 20;
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_mem32);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_mem32);
                        break;
 
-
                case ACPI_RDESC_TYPE_FIXED_MEMORY_32:
                        /*
                         * 32-Bit Fixed Memory Resource
                         */
                        bytes_consumed = 12;
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_fixed_mem32);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct
+                                                acpi_resource_fixed_mem32);
                        break;
 
-
                case ACPI_RDESC_TYPE_EXTENDED_ADDRESS_SPACE:
                        /*
                         * 64-Bit Address Resource
@@ -381,13 +380,14 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
 
                        ++buffer;
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        bytes_consumed = temp16 + 3;
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_address64);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct
+                                                acpi_resource_address64);
                        break;
 
-
                case ACPI_RDESC_TYPE_QWORD_ADDRESS_SPACE:
                        /*
                         * 64-Bit Address Resource
@@ -395,7 +395,7 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
 
                        ++buffer;
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        bytes_consumed = temp16 + 3;
 
@@ -409,20 +409,19 @@ acpi_rs_get_list_length (
                         */
                        if (43 < temp16) {
                                temp8 = (u8) (temp16 - 44);
-                       }
-                       else {
+                       } else {
                                temp8 = 0;
                        }
 
                        /* Ensure a 64-bit boundary for the structure */
 
-                       temp8 = (u8) ACPI_ROUND_UP_to_64_bITS (temp8);
+                       temp8 = (u8) ACPI_ROUND_UP_to_64_bITS(temp8);
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_address64) +
-                                          (temp8 * sizeof (u8));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_address64)
+                           + (temp8 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_DWORD_ADDRESS_SPACE:
                        /*
                         * 32-Bit Address Resource
@@ -430,7 +429,7 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
 
                        ++buffer;
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        bytes_consumed = temp16 + 3;
 
@@ -444,20 +443,19 @@ acpi_rs_get_list_length (
                         */
                        if (23 < temp16) {
                                temp8 = (u8) (temp16 - 24);
-                       }
-                       else {
+                       } else {
                                temp8 = 0;
                        }
 
                        /* Ensure a 32-bit boundary for the structure */
 
-                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS (temp8);
+                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS(temp8);
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_address32) +
-                                          (temp8 * sizeof (u8));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_address32)
+                           + (temp8 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_WORD_ADDRESS_SPACE:
                        /*
                         * 16-Bit Address Resource
@@ -465,7 +463,7 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
 
                        ++buffer;
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        bytes_consumed = temp16 + 3;
 
@@ -479,20 +477,19 @@ acpi_rs_get_list_length (
                         */
                        if (13 < temp16) {
                                temp8 = (u8) (temp16 - 14);
-                       }
-                       else {
+                       } else {
                                temp8 = 0;
                        }
 
                        /* Ensure a 32-bit boundary for the structure */
 
-                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS (temp8);
+                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS(temp8);
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_address16) +
-                                          (temp8 * sizeof (u8));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_address16)
+                           + (temp8 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_EXTENDED_XRUPT:
                        /*
                         * Extended IRQ
@@ -500,7 +497,7 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
 
                        ++buffer;
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        bytes_consumed = temp16 + 3;
 
@@ -527,21 +524,20 @@ acpi_rs_get_list_length (
                         */
                        if (9 + additional_bytes < temp16) {
                                temp8 = (u8) (temp16 - (9 + additional_bytes));
-                       }
-                       else {
+                       } else {
                                temp8 = 0;
                        }
 
                        /* Ensure a 32-bit boundary for the structure */
 
-                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS (temp8);
+                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS(temp8);
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_ext_irq) +
-                                          (additional_bytes * sizeof (u8)) +
-                                          (temp8 * sizeof (u8));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_ext_irq) +
+                           (additional_bytes * sizeof(u8)) +
+                           (temp8 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_IRQ_FORMAT:
                        /*
                         * IRQ Resource.
@@ -550,10 +546,9 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
                        temp8 = *buffer;
 
-                       if(temp8 & 0x01) {
+                       if (temp8 & 0x01) {
                                bytes_consumed = 4;
-                       }
-                       else {
+                       } else {
                                bytes_consumed = 3;
                        }
 
@@ -563,7 +558,7 @@ acpi_rs_get_list_length (
 
                        /* Look at the number of bits set */
 
-                       ACPI_MOVE_16_TO_16 (&temp16, buffer);
+                       ACPI_MOVE_16_TO_16(&temp16, buffer);
 
                        for (index = 0; index < 16; index++) {
                                if (temp16 & 0x1) {
@@ -573,11 +568,11 @@ acpi_rs_get_list_length (
                                temp16 >>= 1;
                        }
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_io) +
-                                          (number_of_interrupts * sizeof (u32));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_io) +
+                           (number_of_interrupts * sizeof(u32));
                        break;
 
-
                case ACPI_RDESC_TYPE_DMA_FORMAT:
                        /*
                         * DMA Resource
@@ -593,19 +588,19 @@ acpi_rs_get_list_length (
 
                        temp8 = *buffer;
 
-                       for(index = 0; index < 8; index++) {
-                               if(temp8 & 0x1) {
+                       for (index = 0; index < 8; index++) {
+                               if (temp8 & 0x1) {
                                        ++number_of_channels;
                                }
 
                                temp8 >>= 1;
                        }
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_dma) +
-                                          (number_of_channels * sizeof (u32));
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_dma) +
+                           (number_of_channels * sizeof(u32));
                        break;
 
-
                case ACPI_RDESC_TYPE_START_DEPENDENT:
                        /*
                         * Start Dependent Functions Resource
@@ -614,17 +609,17 @@ acpi_rs_get_list_length (
                        buffer = byte_stream_buffer;
                        temp8 = *buffer;
 
-                       if(temp8 & 0x01) {
+                       if (temp8 & 0x01) {
                                bytes_consumed = 2;
-                       }
-                       else {
+                       } else {
                                bytes_consumed = 1;
                        }
 
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_start_dpf);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct
+                                                acpi_resource_start_dpf);
                        break;
 
-
                case ACPI_RDESC_TYPE_END_DEPENDENT:
                        /*
                         * End Dependent Functions Resource
@@ -633,25 +628,24 @@ acpi_rs_get_list_length (
                        structure_size = ACPI_RESOURCE_LENGTH;
                        break;
 
-
                case ACPI_RDESC_TYPE_IO_PORT:
                        /*
                         * IO Port Resource
                         */
                        bytes_consumed = 8;
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_io);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_io);
                        break;
 
-
                case ACPI_RDESC_TYPE_FIXED_IO_PORT:
                        /*
                         * Fixed IO Port Resource
                         */
                        bytes_consumed = 4;
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_fixed_io);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_fixed_io);
                        break;
 
-
                case ACPI_RDESC_TYPE_SMALL_VENDOR:
                        /*
                         * Vendor Specific Resource
@@ -664,12 +658,12 @@ acpi_rs_get_list_length (
 
                        /* Ensure a 32-bit boundary for the structure */
 
-                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS (temp8);
-                       structure_size = ACPI_SIZEOF_RESOURCE (struct acpi_resource_vendor) +
-                                          (temp8 * sizeof (u8));
+                       temp8 = (u8) ACPI_ROUND_UP_to_32_bITS(temp8);
+                       structure_size =
+                           ACPI_SIZEOF_RESOURCE(struct acpi_resource_vendor) +
+                           (temp8 * sizeof(u8));
                        break;
 
-
                case ACPI_RDESC_TYPE_END_TAG:
                        /*
                         * End Tag
@@ -679,18 +673,17 @@ acpi_rs_get_list_length (
                        byte_stream_buffer_length = bytes_parsed;
                        break;
 
-
                default:
                        /*
                         * If we get here, everything is out of sync,
                         * exit with an error
                         */
-                       return_ACPI_STATUS (AE_AML_INVALID_RESOURCE_TYPE);
+                       return_ACPI_STATUS(AE_AML_INVALID_RESOURCE_TYPE);
                }
 
                /* Update the return value and counter */
 
-               buffer_size += (u32) ACPI_ALIGN_RESOURCE_SIZE (structure_size);
+               buffer_size += (u32) ACPI_ALIGN_RESOURCE_SIZE(structure_size);
                bytes_parsed += bytes_consumed;
 
                /* Set the byte stream to point to the next resource */
@@ -701,10 +694,9 @@ acpi_rs_get_list_length (
        /* This is the data the caller needs */
 
        *size_needed = buffer_size;
-       return_ACPI_STATUS (AE_OK);
+       return_ACPI_STATUS(AE_OK);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_rs_get_pci_routing_table_length
@@ -723,22 +715,19 @@ acpi_rs_get_list_length (
  ******************************************************************************/
 
 acpi_status
-acpi_rs_get_pci_routing_table_length (
-       union acpi_operand_object       *package_object,
-       acpi_size                       *buffer_size_needed)
+acpi_rs_get_pci_routing_table_length(union acpi_operand_object *package_object,
+                                    acpi_size * buffer_size_needed)
 {
-       u32                             number_of_elements;
-       acpi_size                       temp_size_needed = 0;
-       union acpi_operand_object       **top_object_list;
-       u32                             index;
-       union acpi_operand_object       *package_element;
-       union acpi_operand_object       **sub_object_list;
-       u8                              name_found;
-       u32                             table_index;
-
-
-       ACPI_FUNCTION_TRACE ("rs_get_pci_routing_table_length");
+       u32 number_of_elements;
+       acpi_size temp_size_needed = 0;
+       union acpi_operand_object **top_object_list;
+       u32 index;
+       union acpi_operand_object *package_element;
+       union acpi_operand_object **sub_object_list;
+       u8 name_found;
+       u32 table_index;
 
+       ACPI_FUNCTION_TRACE("rs_get_pci_routing_table_length");
 
        number_of_elements = package_object->package.count;
 
@@ -769,53 +758,51 @@ acpi_rs_get_pci_routing_table_length (
 
                name_found = FALSE;
 
-               for (table_index = 0; table_index < 4 && !name_found; table_index++) {
+               for (table_index = 0; table_index < 4 && !name_found;
+                    table_index++) {
                        if ((ACPI_TYPE_STRING ==
-                                       ACPI_GET_OBJECT_TYPE (*sub_object_list)) ||
-
-                               ((ACPI_TYPE_LOCAL_REFERENCE ==
-                                       ACPI_GET_OBJECT_TYPE (*sub_object_list)) &&
-
-                                       ((*sub_object_list)->reference.opcode ==
-                                               AML_INT_NAMEPATH_OP))) {
+                            ACPI_GET_OBJECT_TYPE(*sub_object_list))
+                           ||
+                           ((ACPI_TYPE_LOCAL_REFERENCE ==
+                             ACPI_GET_OBJECT_TYPE(*sub_object_list))
+                            && ((*sub_object_list)->reference.opcode ==
+                                AML_INT_NAMEPATH_OP))) {
                                name_found = TRUE;
-                       }
-                       else {
+                       } else {
                                /* Look at the next element */
 
                                sub_object_list++;
                        }
                }
 
-               temp_size_needed += (sizeof (struct acpi_pci_routing_table) - 4);
+               temp_size_needed += (sizeof(struct acpi_pci_routing_table) - 4);
 
                /* Was a String type found? */
 
                if (name_found) {
-                       if (ACPI_GET_OBJECT_TYPE (*sub_object_list) == ACPI_TYPE_STRING) {
+                       if (ACPI_GET_OBJECT_TYPE(*sub_object_list) ==
+                           ACPI_TYPE_STRING) {
                                /*
                                 * The length String.Length field does not include the
                                 * terminating NULL, add 1
                                 */
                                temp_size_needed += ((acpi_size)
-                                       (*sub_object_list)->string.length + 1);
+                                                    (*sub_object_list)->string.
+                                                    length + 1);
+                       } else {
+                               temp_size_needed += acpi_ns_get_pathname_length((*sub_object_list)->reference.node);
                        }
-                       else {
-                               temp_size_needed += acpi_ns_get_pathname_length (
-                                                  (*sub_object_list)->reference.node);
-                       }
-               }
-               else {
+               } else {
                        /*
                         * If no name was found, then this is a NULL, which is
                         * translated as a u32 zero.
                         */
-                       temp_size_needed += sizeof (u32);
+                       temp_size_needed += sizeof(u32);
                }
 
                /* Round up the size since each element must be aligned */
 
-               temp_size_needed = ACPI_ROUND_UP_to_64_bITS (temp_size_needed);
+               temp_size_needed = ACPI_ROUND_UP_to_64_bITS(temp_size_needed);
 
                /* Point to the next union acpi_operand_object */
 
@@ -826,6 +813,7 @@ acpi_rs_get_pci_routing_table_length (
         * Adding an extra element to the end of the list, essentially a
         * NULL terminator
         */
-       *buffer_size_needed = temp_size_needed + sizeof (struct acpi_pci_routing_table);
-       return_ACPI_STATUS (AE_OK);
+       *buffer_size_needed =
+           temp_size_needed + sizeof(struct acpi_pci_routing_table);
+       return_ACPI_STATUS(AE_OK);
 }