Gustavo,

>>>           struct {
>>> -            struct ssp_command_iu task;
>>>               u32 prot[PROT_BUF_SIZE];
>>> +            struct ssp_command_iu task;
>>>           };
>
> Actually, I have a question here:
>
> is u32 prot[PROT_BUF_SIZE]; intended to overlap flex array
> task.add_cdb[] at some point?
>
> if not, this change is just fine. Otherwise, I'd need to update this
> patch to account for the overlap.

I am not familiar with the hisi hardware interface so I don't know
whether the overlap is intentional.

-- 
Martin K. Petersen

Reply via email to