Re: [PATCH 1/1] staging: wilc1000: replace addresses variables type from u32 to size_t for x64 systems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2015-08-17 13:12 GMT-04:00  <Valdis.Kletnieks@xxxxxx>:
> If you're going to do this, do it *right* and chase down where all these
> variables are defined and nail them down to a size_t *there* so you can get rid
> of most of these casts - plus potentially find *other* places they're playing
> fast and loose with the types.

You're right. I tried first to use the struct that is used to declare
pstrWFIDrv, but
as it needed a lot of code refactoring I switched to find an easier
path, but that's
not as elegant and clean.

> Umm.. this isn't for a brain-dead card that has to have its addresses
> down in the the first 4G of memory, is it?  Or was the driver author
> stuck on a 32-bit kernel?

Actually, that's what I wondered also! With further researchs, I just ended
up finding those patches in the linux-wireless mailing list:
http://www.spinics.net/lists/linux-wireless/msg138655.html
http://www.spinics.net/lists/linux-wireless/msg138656.html

These patches are not integrated with Greg's staging tree yet it seems. That's
probably why I missed them at first... They however seem more clean than mine.

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux