[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3 23/62] acpi/table: Introduce acpi_parse_entries



>>> On 17.11.15 at 10:40, <shannon.zhao@xxxxxxxxxx> wrote:
> From: Parth Dixit <parth.dixit@xxxxxxxxxx>
> 
> Add new function acpi_parse_entries which takes acpi table as argument.
> This will avoid fetching table everytime in acpi_table_parse_entries.
> This function will be used in acpi_gicv2_init within another patch.
> 
> Signed-off-by: Naresh Bhat <Naresh.Bhat@xxxxxxxxxx>
> Signed-off-by: Parth Dixit <parth.dixit@xxxxxxxxxx>
> Signed-off-by: Shannon Zhao <shannon.zhao@xxxxxxxxxx>

Please clarify authorship: From and first S-o-b aren't really in line.

> --- a/xen/drivers/acpi/tables.c
> +++ b/xen/drivers/acpi/tables.c
> @@ -221,29 +221,27 @@ void __init acpi_table_print_madt_entry(struct 
> acpi_subtable_header *header)
>       }
>  }
>  
> -
>  int __init
> -acpi_table_parse_entries(char *id,
> -                          unsigned long table_size,
> -                          int entry_id,
> -                          acpi_table_entry_handler handler,
> -                          unsigned int max_entries)
> +acpi_parse_entries(char *id,
> +                   unsigned long table_size,
> +                   acpi_table_entry_handler handler,
> +                   struct acpi_table_header *table_header,
> +                   int entry_id,
> +                   unsigned int max_entries)
>  {
> -     struct acpi_table_header *table_header = NULL;

Looks like Linux has gained similar functionality. Can't you simply
pull in one or more Linux commits?

> -     if (!handler)
> +     if ( !handler )

But in any event please don't change the file's coding style.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.