Re: git: 74a6f2a76b32 - main - iwx: Shorten log code field

From: Colin Percival <cperciva_at_tarsnap.com>
Date: Thu, 03 Apr 2025 17:38:46 UTC
On 4/3/25 00:07, Tom Jones wrote:
> commit 74a6f2a76b32e01d05c4ad71897bd3a6831e703e
>      
>      This fixes the build on i386.

You sure about that?

/usr/src/sys/dev/iwx/if_iwx.c:5762:22: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5762 |         if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[0].tb_len)) >> 32)
       |                             ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:5762:80: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5762 |         if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[0].tb_len)) >> 32)
       | 
                 ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:5769:22: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5769 |         if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[1].tb_len)) >> 32)
       |                             ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:5769:80: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5769 |         if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[1].tb_len)) >> 32)
       | 
                 ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:5778:23: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5778 |                 if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[i + 2].tb_len)) >> 32)
       |                                     ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:5778:85: error: shift count >= width of type 
[-Werror,-Wshift-count-overflow]
  5778 |                 if (data->cmd_paddr >> 32 != (data->cmd_paddr + 
le32toh(desc->tbs[i + 2].tb_len)) >> 32)
       | 
                             ^  ~~
/usr/src/sys/dev/iwx/if_iwx.c:10971:47: error: format specifies type 'unsigned 
long' but the argument has type 'uint64_t' (aka 'unsigned long long') 
[-Werror,-Wformat]
  10971 |         DPRINTF(("%s: k->wk_keytsc=%lu\n", __func__, k->wk_keytsc));
        |                                    ~~~               ^~~~~~~~~~~~
        |                                    %llu

-- 
Colin Percival
FreeBSD Release Engineering Lead & EC2 platform maintainer
Founder, Tarsnap | www.tarsnap.com | Online backups for the truly paranoid