On 14/05/2020 14.37, Janosch Frank wrote:
> Let's make it a bit more clear that we check the full 64 bits to fit
> into the 32 we return.
>
> Signed-off-by: Janosch Frank <fran...@linux.ibm.com>
> Suggested-by: David Hildenbrand <da...@redhat.com>
> Reviewed-by: David Hildenbrand <da...@redhat.com>
> ---
> pc-bios/s390-ccw/helper.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/pc-bios/s390-ccw/helper.h b/pc-bios/s390-ccw/helper.h
> index 78d5bc7442..8a50ec91bb 100644
> --- a/pc-bios/s390-ccw/helper.h
> +++ b/pc-bios/s390-ccw/helper.h
> @@ -18,7 +18,7 @@
> /* Avoids compiler warnings when casting a pointer to a u32 */
> static inline uint32_t ptr2u32(void *ptr)
> {
> - IPL_assert((uint64_t)ptr <= 0xffffffff, "ptr2u32: ptr too large");
> + IPL_assert((uint64_t)ptr <= 0xffffffffull, "ptr2u32: ptr too large");
> return (uint32_t)(uint64_t)ptr;
> }
>
>
Reviewed-by: Thomas Huth <th...@redhat.com>
