Re: [PATCH 1/1] include/acpi/processor.h: replace __attribute__((packed)) by __packed

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

 



On Sunday, June 01, 2014 07:16:17 PM Fabian Frederick wrote:
> This patch fixes checkpatch warnings:
> 
> "WARNING: __packed is preferred over __attribute__((packed))"
> 
> Cc: Robert Moore <robert.moore@xxxxxxxxx>
> Cc: "Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx>
> Signed-off-by: Fabian Frederick <fabf@xxxxxxxxx>

Applied, thanks!

> ---
>  include/acpi/processor.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/include/acpi/processor.h b/include/acpi/processor.h
> index 6eb1d3c..9b9b6f2 100644
> --- a/include/acpi/processor.h
> +++ b/include/acpi/processor.h
> @@ -53,7 +53,7 @@ struct acpi_power_register {
>  	u8 bit_offset;
>  	u8 access_size;
>  	u64 address;
> -} __attribute__ ((packed));
> +} __packed;
>  
>  struct acpi_processor_cx {
>  	u8 valid;
> @@ -83,7 +83,7 @@ struct acpi_psd_package {
>  	u64 domain;
>  	u64 coord_type;
>  	u64 num_processors;
> -} __attribute__ ((packed));
> +} __packed;
>  
>  struct acpi_pct_register {
>  	u8 descriptor;
> @@ -93,7 +93,7 @@ struct acpi_pct_register {
>  	u8 bit_offset;
>  	u8 reserved;
>  	u64 address;
> -} __attribute__ ((packed));
> +} __packed;
>  
>  struct acpi_processor_px {
>  	u64 core_frequency;	/* megahertz */
> @@ -124,7 +124,7 @@ struct acpi_tsd_package {
>  	u64 domain;
>  	u64 coord_type;
>  	u64 num_processors;
> -} __attribute__ ((packed));
> +} __packed;
>  
>  struct acpi_ptc_register {
>  	u8 descriptor;
> @@ -134,7 +134,7 @@ struct acpi_ptc_register {
>  	u8 bit_offset;
>  	u8 reserved;
>  	u64 address;
> -} __attribute__ ((packed));
> +} __packed;
>  
>  struct acpi_processor_tx_tss {
>  	u64 freqpercentage;	/* */
> 

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux