Acked-by: Chengwen Feng <[email protected]>

On 1/27/2026 7:49 PM, Marat Khalili wrote:
> Macros RTE_SHIFT_VAL32 and RTE_SHIFT_VAL64 were implemented by applying
> UINT32_C or UINT64_C correspondingly to its first argument. As a
> consequence first argument had to be a constant. Replace UINT32_C and
> UINT64_C with casts to uint32_t and uint64_t to allow these arguments be
> variable. For constants the result should be the same.
> 
> (Yes, technically UINT64_C promotes to uint_least64_t, not uint64_t, but
> I think most users of RTE_SHIFT_VAL64 expect the result to be uint64_t.)
> 
> Fixes: 1d8f2285ed3f ("eal: introduce more macros for bit definition")
> Cc: [email protected]
> 
> Signed-off-by: Marat Khalili <[email protected]>
> Reviewed-by: Morten Brørup <[email protected]>
> Acked-by: Konstantin Ananyev <[email protected]>
> Acked-by: Stephen Hemminger <[email protected]>

...

Reply via email to