Hi Yong, On Mon, Oct 29, 2018 at 03:22:59PM -0700, Yong Zhi wrote: > This add all the structs of IPU3 firmware ABI. > > Signed-off-by: Yong Zhi <yong.zhi@xxxxxxxxx> > Signed-off-by: Rajmohan Mani <rajmohan.mani@xxxxxxxxx> ... > +struct imgu_abi_shd_intra_frame_operations_data { > + struct imgu_abi_acc_operation > + operation_list[IMGU_ABI_SHD_MAX_OPERATIONS] __attribute__((aligned(32))); > + struct imgu_abi_acc_process_lines_cmd_data > + process_lines_data[IMGU_ABI_SHD_MAX_PROCESS_LINES] __attribute__((aligned(32))); > + struct imgu_abi_shd_transfer_luts_set_data > + transfer_data[IMGU_ABI_SHD_MAX_TRANSFERS] __attribute__((aligned(32))); Could you replace this wth __aligned(32), please? The same for the rest of the header. -- Regards, Sakari Ailus sakari.ailus@xxxxxxxxxxxxxxx