Page MenuHomeFreeBSD

iwx: Fix 32-bit compilation
ClosedPublic

Authored by olce on Mar 5 2026, 1:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 1, 7:18 AM
Unknown Object (File)
Thu, Mar 26, 7:56 AM
Unknown Object (File)
Sun, Mar 22, 11:36 PM
Unknown Object (File)
Sat, Mar 21, 6:42 PM
Unknown Object (File)
Sat, Mar 21, 10:05 AM
Unknown Object (File)
Thu, Mar 19, 1:33 AM
Unknown Object (File)
Tue, Mar 17, 4:25 PM
Unknown Object (File)
Fri, Mar 13, 9:20 PM
Subscribers

Details

Summary
  • Avoid shifts wider than integer types, by wrapping the corresponding checks into '#if SIZEOF_SIZE_T > 32' blocks. 'bus_addr_t' has the same width as 'size_t' on all architectures (and this is not going to change for 32-bit architectures).
  • Use appropriate printf(3) format for 'wk_keytsc'.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

olce requested review of this revision.Mar 5 2026, 1:04 PM

(Alternative: Don't compile this on 32-bit architectures at all, if if_iwx in reality does not support those.)

(Alternative: Don't compile this on 32-bit architectures at all, if if_iwx in reality does not support those.)

it doesn't? what about it doesn't support 32 bit archs?

adrian added inline comments.
sys/dev/iwx/if_iwx.c
5821

ugh if this is a hardware limitation then god i wish we could describe it somehow to busdma...

This revision is now accepted and ready to land.Mar 5 2026, 4:07 PM
This revision was automatically updated to reflect the committed changes.
olce marked an inline comment as done.