1105592cb8
The first member of struct tcan4x5x_map_buf is the struct tcan4x5x_buf_cmd, which has a size of 4 bytes. It's followed by an array of u8. The compiler places the array directly after the struct tcan4x5x_buf_cmd. This patch removes the not needed attribute __packed from the struct tcan4x5x_map_buf. Suggested-by: Jakub Kicinski <kuba@kernel.org> Link: https://lore.kernel.org/r/20210113203955.912916-1-mkl@pengutronix.de Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
58 lines
1.1 KiB
C
58 lines
1.1 KiB
C
/* SPDX-License-Identifier: GPL-2.0
|
|
*
|
|
* tcan4x5x - Texas Instruments TCAN4x5x Family CAN controller driver
|
|
*
|
|
* Copyright (c) 2020 Pengutronix,
|
|
* Marc Kleine-Budde <kernel@pengutronix.de>
|
|
*/
|
|
|
|
#ifndef _TCAN4X5X_H
|
|
#define _TCAN4X5X_H
|
|
|
|
#include <linux/gpio/consumer.h>
|
|
#include <linux/regmap.h>
|
|
#include <linux/regmap.h>
|
|
#include <linux/regulator/consumer.h>
|
|
#include <linux/spi/spi.h>
|
|
|
|
#include "m_can.h"
|
|
|
|
#define TCAN4X5X_SANITIZE_SPI 1
|
|
|
|
struct __packed tcan4x5x_buf_cmd {
|
|
u8 cmd;
|
|
__be16 addr;
|
|
u8 len;
|
|
};
|
|
|
|
struct tcan4x5x_map_buf {
|
|
struct tcan4x5x_buf_cmd cmd;
|
|
u8 data[256 * sizeof(u32)];
|
|
} ____cacheline_aligned;
|
|
|
|
struct tcan4x5x_priv {
|
|
struct m_can_classdev cdev;
|
|
|
|
struct regmap *regmap;
|
|
struct spi_device *spi;
|
|
|
|
struct gpio_desc *reset_gpio;
|
|
struct gpio_desc *device_wake_gpio;
|
|
struct gpio_desc *device_state_gpio;
|
|
struct regulator *power;
|
|
|
|
struct tcan4x5x_map_buf map_buf_rx;
|
|
struct tcan4x5x_map_buf map_buf_tx;
|
|
};
|
|
|
|
static inline void
|
|
tcan4x5x_spi_cmd_set_len(struct tcan4x5x_buf_cmd *cmd, u8 len)
|
|
{
|
|
/* number of u32 */
|
|
cmd->len = len >> 2;
|
|
}
|
|
|
|
int tcan4x5x_regmap_init(struct tcan4x5x_priv *priv);
|
|
|
|
#endif
|