On 16/03/2023 13:36, Oded Gabbay wrote:
> Don't use padX for actual reservedX fields.
>
> Signed-off-by: Oded Gabbay <[email protected]>
> ---
> include/uapi/drm/habanalabs_accel.h | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/include/uapi/drm/habanalabs_accel.h
> b/include/uapi/drm/habanalabs_accel.h
> index 7ca0ef802fd1..7d457eb4da74 100644
> --- a/include/uapi/drm/habanalabs_accel.h
> +++ b/include/uapi/drm/habanalabs_accel.h
> @@ -915,17 +915,18 @@ struct hl_info_hw_ip_info {
> __u64 dram_page_size;
> __u32 edma_enabled_mask;
> __u16 number_of_user_interrupts;
> - __u16 pad2;
> - __u64 reserved4;
> + __u8 reserved1;
> + __u8 reserved2;
> + __u64 reserved3;
> __u64 device_mem_alloc_default_page_size;
> + __u64 reserved4;
> __u64 reserved5;
> - __u64 reserved6;
> - __u32 reserved7;
> - __u8 reserved8;
> + __u32 reserved6;
> + __u8 reserved7;
> __u8 revision_id;
> __u16 tpc_interrupt_id;
> + __u32 reserved8;
> __u32 reserved9;
> - __u8 pad3[4];
> __u64 engine_core_interrupt_reg_addr;
> };
>
Reviewed-by: Ofir Bitton <[email protected]>