Re: [PATCH 1/1] intel-ipu3: Align struct ipu3_uapi_awb_fr_config_s to 32 bytes

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

 



On Mon, Dec 02, 2019 at 10:38:20AM +0200, Sakari Ailus wrote:
> A struct that needs to be aligned to 32 bytes has a size of 28. Increase
> the size to 32.
> 
> This makes elements of arrays of this struct aligned to 32 as well, and
> other structs where members are aligned to 32 mixing
> ipu3_uapi_awb_fr_config_s as well as other types.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> Tested-by: Bingbu Cao <bingbu.cao@xxxxxxxxx>

The tags should have included:

Fixes: commit dca5ef2aa1e6 ("media: staging/intel-ipu3: remove the unnecessary compiler flags")

> ---
>  drivers/staging/media/ipu3/include/intel-ipu3.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/media/ipu3/include/intel-ipu3.h b/drivers/staging/media/ipu3/include/intel-ipu3.h
> index 08eaa0bad0de..1c9c3ba4d518 100644
> --- a/drivers/staging/media/ipu3/include/intel-ipu3.h
> +++ b/drivers/staging/media/ipu3/include/intel-ipu3.h
> @@ -449,7 +449,7 @@ struct ipu3_uapi_awb_fr_config_s {
>  	__u16 reserved1;
>  	__u32 bayer_sign;
>  	__u8 bayer_nf;
> -	__u8 reserved2[3];
> +	__u8 reserved2[7];
>  } __attribute__((aligned(32))) __packed;
>  
>  /**
> -- 
> 2.20.1
> 

-- 
Sakari Ailus



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux