Hi, On 23/05/2022 15:50, Lin Liu wrote:
Update to use byteswap to swap bytes be*_to_cpup(p) is short for be*to_cpu(*p), update to use latter one explictly
But why? I really don't have a suggestion on the comment because I disagree (and AFAICT Jan as well) with the approach.
In any case, I think it would be helpful if you participate in the discussion rather than sending a new version quickly. This would make sure you don't spend time on resending with unfinished discussion.
Cheers, -- Julien Grall
