diff options
| author | Anson Huang <Anson.Huang@nxp.com> | 2020-03-02 10:51:25 +0800 |
|---|---|---|
| committer | Daniel Lezcano <daniel.lezcano@linaro.org> | 2020-03-12 11:56:53 +0100 |
| commit | 1fd213f38b999b95777d4dddc7e232e08f2294c8 (patch) | |
| tree | d6de7816b26d93ce2e6144ed14d21e7efd6bca5b /drivers/thermal/imx_sc_thermal.c | |
| parent | 7375f2ac6b2d5f42ebc42e25f5251aa3284f30ed (diff) | |
thermal: imx_sc_thermal: Align imx sc thermal msg structs to 4
The i.MX SCU API strongly assumes that messages are composed out
of 4-bytes words but some of our message structs have odd sizeofs,
use __packed __aligned(4) to avoid potential oopses.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/1583117485-30922-1-git-send-email-Anson.Huang@nxp.com
Diffstat (limited to 'drivers/thermal/imx_sc_thermal.c')
| -rw-r--r-- | drivers/thermal/imx_sc_thermal.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/thermal/imx_sc_thermal.c b/drivers/thermal/imx_sc_thermal.c index 4955dfd285ce..dbb277af232d 100644 --- a/drivers/thermal/imx_sc_thermal.c +++ b/drivers/thermal/imx_sc_thermal.c @@ -27,12 +27,12 @@ struct imx_sc_sensor { struct req_get_temp { u16 resource_id; u8 type; -} __packed; +} __packed __aligned(4); struct resp_get_temp { u16 celsius; u8 tenths; -} __packed; +} __packed __aligned(4); struct imx_sc_msg_misc_get_temp { struct imx_sc_rpc_msg hdr; @@ -40,7 +40,7 @@ struct imx_sc_msg_misc_get_temp { struct req_get_temp req; struct resp_get_temp resp; } data; -}; +} __packed __aligned(4); static int imx_sc_thermal_get_temp(void *data, int *temp) { |