This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: dvb: as102-fe: Fix as10x_register_addr packing
Author:  Ricardo Ribalda <riba...@chromium.org>
Date:    Wed Apr 10 12:24:37 2024 +0000

This structure is embedded in multiple other structures that are packed,
which conflicts with it being aligned.

drivers/media/usb/as102/as10x_cmd.h:379:30: warning: field reg_addr within 
'struct as10x_dump_memory::(unnamed at 
drivers/media/usb/as102/as10x_cmd.h:373:2)' is less aligned than 'struct 
as10x_register_addr' and is usually due to 'struct as10x_dump_memory::(unnamed 
at drivers/media/usb/as102/as10x_cmd.h:373:2)' being packed, which can lead to 
unaligned accesses [-Wunaligned-access]

Mark it as being packed.

Marking the inner struct as 'packed' does not change the layout, since the
whole struct is already packed, it just silences the clang warning. See
also this llvm discussion:

https://github.com/llvm/llvm-project/issues/55520

Signed-off-by: Ricardo Ribalda <riba...@chromium.org>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/dvb-frontends/as102_fe_types.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/dvb-frontends/as102_fe_types.h 
b/drivers/media/dvb-frontends/as102_fe_types.h
index 297f9520ebf9..8a4e392c8896 100644
--- a/drivers/media/dvb-frontends/as102_fe_types.h
+++ b/drivers/media/dvb-frontends/as102_fe_types.h
@@ -174,6 +174,6 @@ struct as10x_register_addr {
        uint32_t addr;
        /* register mode access */
        uint8_t mode;
-};
+} __packed;
 
 #endif

Reply via email to