Re: [PATCH 2/3] i2c: xiic: Improve struct memory alignment

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

 




On 27. 08. 20 16:48, ext-jaakko.laine@xxxxxxxxxxx wrote:
> From: Jaakko Laine <ext-jaakko.laine@xxxxxxxxxxx>
> 
> xiic_i2c struct alignment causes the struct to
> take more space in memory than strictly required.
> Move state -member to end of struct to get less
> padding.
> 
> Signed-off-by: Jaakko Laine <ext-jaakko.laine@xxxxxxxxxxx>
> Suggested-by: Michal Simek <michal.simek@xxxxxxxxxx>
> ---
>  drivers/i2c/busses/i2c-xiic.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c
> index 10380531d45c..1453d82bb664 100644
> --- a/drivers/i2c/busses/i2c-xiic.c
> +++ b/drivers/i2c/busses/i2c-xiic.c
> @@ -54,11 +54,11 @@ enum xiic_endian {
>   * @lock: Mutual exclusion
>   * @tx_pos: Current pos in TX message
>   * @nmsgs: Number of messages in tx_msg
> - * @state: See STATE_
>   * @rx_msg: Current RX message
>   * @rx_pos: Position within current RX message
>   * @endianness: big/little-endian byte order
>   * @clk: Pointer to AXI4-lite input clock
> + * @state: See STATE_
>   */
>  struct xiic_i2c {
>  	struct device *dev;
> @@ -69,11 +69,11 @@ struct xiic_i2c {
>  	struct mutex lock;
>  	unsigned int tx_pos;
>  	unsigned int nmsgs;
> -	enum xilinx_i2c_state state;
>  	struct i2c_msg *rx_msg;
>  	int rx_pos;
>  	enum xiic_endian endianness;
>  	struct clk *clk;
> +	enum xilinx_i2c_state state;
>  };
>  
>  
> 

Acked-by: Michal Simek <michal.simek@xxxxxxxxxx>

Thanks,
Michal



[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux