Hi Sakari, > -----Original Message----- > From: Sakari Ailus [mailto:[email protected]] > Sent: Monday, November 05, 2018 12:28 AM > To: Zhi, Yong <[email protected]> > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; Mani, > Rajmohan <[email protected]>; Zheng, Jian Xu > <[email protected]>; Hu, Jerry W <[email protected]>; Toivonen, > Tuukka <[email protected]>; Qiu, Tian Shu > <[email protected]>; Cao, Bingbu <[email protected]> > Subject: Re: [PATCH v7 05/16] intel-ipu3: abi: Add structs > > 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 <[email protected]> > > Signed-off-by: Rajmohan Mani <[email protected]> > > ... > > > +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. >
Using __aligned(32) in the uAPI header resulted in compilation errors in user space / camera HAL code. e.g ../../../../../../../../usr/include/linux/intel-ipu3.h:464:57: error: expected ';' at end of declaration list __u8 bayer_table[IPU3_UAPI_AWB_FR_BAYER_TABLE_MAX_SIZE] __aligned(32); So we ended up using __attribute__((aligned(32))) format in uAPI header and to be consistent, we followed the same format in ABI header as well. Let us know if it's okay to deviate between uAPI and ABI header for this alignment qualifier. > -- > Regards, > > Sakari Ailus > [email protected]
