Commit Graph

231 Commits

Author SHA1 Message Date
Phillip Potter
1f0cf855dc remove function _rtw_vmalloc
Remove the function _rtw_vmalloc from os_dep/osdep_service.c, converting
its only user (also in os_dep/osdep_service.c) to use plain vmalloc.
This function is just an inline wrapper around vmalloc which returns a u8
pointer, which isn't needed. Also remove the declaration from
include/osdep_service.h.

It is considered generally bad practice to declare functions as inline in
the majority of cases, as not only can this qualifier be ignored by the
compiler but the compiler generally makes good decisions about inlining
anyway.

Link: https://lore.kernel.org/r/20210818234853.208448-4-phil@philpotter.co.uk
2021-10-11 15:50:48 +02:00
Phillip Potter
6e2f371df5 remove rtw_vmalloc preprocessor definition
Remove rtw_vmalloc preprocessor definition from include/osdep_service.h
as it now has no callers.

Link: https://lore.kernel.org/r/20210818234853.208448-3-phil@philpotter.co.uk
2021-10-11 15:46:55 +02:00
Phillip Potter
397bbaeacf convert only rtw_vmalloc call to vmalloc
Convert the only call to rtw_vmalloc in os_dep/ioctl_linux.c to the
kernel's existing vmalloc function, as rtw_malloc is just a preprocessor
definition for _rtw_vmalloc. The _rtw_vmalloc function is defined inline
and returns a u8, wrapping standard vmalloc. This behaviour is not necessary.

Link: https://lore.kernel.org/r/20210818234853.208448-2-phil@philpotter.co.uk
2021-10-11 15:45:31 +02:00
Greg Kroah-Hartman
76cefd5ae2 remove inline markings from functions in rtw_br_ext.c
The compiler is free to ignore, or follow, "inline" markings so they
really have no use in .c files, so just remove them.

This allows functions to properly show up as being unused when all
callers to them are removed, otherwise gcc does not warn you about this.

Link: https://lore.kernel.org/r/20210816073450.668993-1-gregkh@linuxfoundation.org
2021-10-11 15:15:20 +02:00
Michael Straube
c1f70ba056 use common ieee80211 constants
Many defined constants in wifi.h are unused and/or available from
<linux/ieee80211.h>, some with slightly different names. Remove
the constants from wifi.h and use the common ones. Rename where
necessary.

Link: https://lore.kernel.org/r/20210814165518.8672-1-straube.linux@gmail.com
2021-10-11 15:03:09 +02:00
Fabio M. De Francesco
22a000acb9 Remove defined but not used variables
Remove defined but not used const variables. Issues detected by GCC
running with -Wunused-const-variable option enabled.

Link: https://lore.kernel.org/r/20210814160804.11634-1-fmdefrancesco@gmail.com
2021-10-11 14:25:08 +02:00
Nathan Chancellor
c597fa83da Reorganize error handling in rtw_drv_init()
Looking at the error function as a whole, the error handling is odd
compared to the rest of the kernel, which prefers to set error codes on
goto paths, rather than a global "status" variable which determines the
error code at the end of the function and function calls in the case of
error.

Rearrange the error handling of this function to bring it more inline
with how the kernel does it in most cases, which helps readability.

Link: https://lore.kernel.org/r/20210812204027.338872-4-nathan@kernel.org
Link: https://lore.kernel.org/r/20210813201418.4018631-1-nathan@kernel.org
2021-10-11 14:22:28 +02:00
Nathan Chancellor
8d93cdeefa Remove unused static inline functions in rtw_recv.h
The functions are not used or are called
from functions that are not used so just remove them to remove the
warnings.

Link: https://lore.kernel.org/r/20210812204027.338872-2-nathan@kernel.org
2021-10-11 14:06:16 +02:00
Michael Straube
c8fd0937e4 replace custom hwaddr_aton_i() with mac_pton()
Replace custom hwaddr_aton_i() with mac_pton() in core/ioctl_linux.c
and remove the now unused function hwaddr_aton_i().

Link: https://lore.kernel.org/r/20210812132057.22579-1-straube.linux@gmail.com
2021-10-11 13:55:23 +02:00
Martin Kaiser
0faae6386e use IW_HANDLER to declare wext handlers
Use the IW_HANDLER macro to declare the handler functions for
wext ioctls. We don't have to skip unused ioctl numbers manually.

The same modification was applied to the deprecated rtl8188eu driver in
commit af249fce390f ("staging: rtl8188eu: use IW_HANDLER to declare wext
handlers").

Link: https://lore.kernel.org/r/20210807153636.11712-11-martin@kaiser.cx
2021-10-11 13:37:30 +02:00
Michael Straube
03af80bee9 replace custom macros with is_broadcast_ether_addr
Replace usage of custom macros with is_broadcast_ether_addr. All buffers
are properly aligned. Remove the now unsued macros MacAddr_isBcst and
IS_MAC_ADDRESS_BROADCAST.

Link: https://lore.kernel.org/r/20210805205010.31192-1-straube.linux@gmail.com
2021-10-11 13:29:13 +02:00
Martin Kaiser
2965c273c9 remove the RT_TRACE macro
We've deleted all RT_TRACE calls. The macro itself can now be removed
as well.

Link: https://lore.kernel.org/r/20210807153636.11712-13-martin@kaiser.cx
2021-10-11 13:18:29 +02:00
Larry Finger
9ad10523e3 Remove wrapper rtw_sleep_schedulable()
This wrapper is never used.

Link: https://lore.kernel.org/r/20210805192644.15978-6-Larry.Finger@lwfinger.net
2021-10-11 13:03:59 +02:00
Larry Finger
a5adfa895b Remove wrapper rtw_udelay_os()
This wrapper is a simple call to udelay(). Remove it.

Link: https://lore.kernel.org/r/20210805192644.15978-4-Larry.Finger@lwfinger.net
2021-10-11 13:00:27 +02:00
Larry Finger
7f8451a86e Remove wrapper rtw_mdelay_os()
This wrapper just calls mdelay(). Remove it.

Link: https://lore.kernel.org/r/20210805192644.15978-3-Larry.Finger@lwfinger.net
2021-10-11 12:57:06 +02:00
Carlos Garcés
5bc12edecb
Merge pull request #245 from CGarces/staging_cherrypick
Adapted commits from staging area
2021-10-11 12:18:18 +02:00
Carlos Garcés
cf246719f1 Remove wrapper for memset()
Link: 545d963a9e (diff-16d649c4743131c1aae5f812af5596dae6423bb63eaaf2990b8c89c9a681a913)
2021-10-11 12:09:34 +02:00
Carlos Garcés
76f68bbfcf Remove hidden warnings from makefile 2021-10-11 12:05:35 +02:00
Larry Finger
fdbea81f08 Remove wrapper routine rtw_msleep_os()
The effect of this macro is to call msleep(). Remove the wrapper.

Link: https://lore.kernel.org/r/20210805192644.15978-2-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
1fcfef2e07 Remove rtw_yield_os()
This wrapper is just a call to yield().

Link: https://lore.kernel.org/r/20210805192644.15978-1-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
f1aa80a838 Remove more empty routines
After removing some empty routines in os_dep/osdep_service.c, new routines
become empty. These include rtw_mfree_stainfo(),
_rtw_free_sta_xmit_priv_lock(), and rtw_mfree_recv_priv_lock.

Link: https://lore.kernel.org/r/20210805183717.23007-5-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
51e1f13a91 Remove all calls to _rtw_spinlock_free()
This routine is empty, thus all calls to it can be removed.

Link: https://lore.kernel.org/r/20210805183717.23007-4-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
34ec238387 Remove wrapper around vfree
This call is so simple that there is no need, or use, for a wrapper.

Link: https://lore.kernel.org/r/20210805183717.23007-3-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
4d3d23dd24 Remove wrapper around do_div
Wrapper routine rtw_modular64() contains only a call to do_div() and
is used once in the code. Remove the wrapper.

Link: https://lore.kernel.org/r/20210803135223.12543-9-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Larry Finger
d7c3737cd5 Remove wrappers for atomic operations
These wrappers were useful when this driver had the hooks for Windows,
but are no longer needed.

Link: https://lore.kernel.org/r/20210802192721.23110-4-Larry.Finger@lwfinger.net
2021-10-11 10:44:02 +02:00
Phillip Potter
e8b235e056 correct set/defined but unused warnings from debug cleanup
This patch fixes an existing warning
related to an array within include/rtw_security.h not being used.

Link: https://lore.kernel.org/r/20210731133804.196631-1-phil@philpotter.co.uk
2021-10-11 10:44:01 +02:00
Greg Kroah-Hartman
6b5e7665c9 remove rtw_usb_control_msg() macro
It is a wrapper around usb_control_msg() that does nothing, so remove
the macro and just call the correct USB function instead in the one
place it is used.

Link: https://lore.kernel.org/r/20210730092417.1014392-12-gregkh@linuxfoundation.org
2021-10-11 10:44:01 +02:00
Greg Kroah-Hartman
904e95f373 remove rtw_usb_bulk_msg() macro
No one is using this wrapper macro, so just remove it as it is
pointless.

Link: https://lore.kernel.org/r/20210730092417.1014392-11-gregkh@linuxfoundation.org
2021-10-11 10:44:01 +02:00
Greg Kroah-Hartman
ca6e4a43f7 remove include/nic_spec.h
No one was including this file, so just remove it.

Link: https://lore.kernel.org/r/20210730092417.1014392-10-gregkh@linuxfoundation.org
2021-10-11 10:44:01 +02:00
Greg Kroah-Hartman
361918ceb1 remove unused enum _NIC_VERSION
None of these enumerated values were used anywhere in the driver, so
just remove them all.

Link: https://lore.kernel.org/r/20210730092417.1014392-9-gregkh@linuxfoundation.org
2021-10-11 10:44:01 +02:00
Greg Kroah-Hartman
ac1790da7c remove rtw_mfree2d() function
It is just a wrapper around kfree(), so remove it and just call kfree()
instead.

Adaptation of: https://lore.kernel.org/r/20210730092417.1014392-7-gregkh@linuxfoundation.org
2021-10-11 10:44:01 +02:00
Larry Finger
1401b54fd0 Remove empty header file
File include/drv_types_linux.h is empty. Remove it.

Adaptation of: https://lore.kernel.org/r/20210729170930.23171-2-Larry.Finger@lwfinger.net
2021-10-10 23:13:31 +02:00
Carlos Garcés
bc765550ed
Merge pull request #246 from Mange/fix_ci
Fix libc6 dependency
2021-10-10 23:09:32 +02:00
Carlos Garcés
fa64647e38
Fix libc6 dependency 2021-10-10 23:02:43 +02:00
Carlos Garcés
4c9751be79
Merge pull request #244 from CGarces/fix_5.15
Fix buils with kernel 5.15
2021-10-09 13:46:06 +02:00
Carlos Garcés
59fd03940c Fix for kernel 5.15
Fixed compilation warnings with GCC 11
Fix libc6 dependency 2.34
2021-10-09 13:33:01 +02:00
Carlos Garcés
5464196753
Merge pull request #236 from CGarces/fix_5.12
Fixed 5.12 support.
2021-05-12 09:43:22 +02:00
Carlos Garcés
4013a01fe6
Fix libc6 2.33 dependency 2021-05-11 23:17:21 +02:00
Carlos Garcés
09c5222d59 Fixed 5.12 support. 2021-05-10 19:06:13 +02:00
Carlos Garcés
ab35c7e967
Merge pull request #233 from CGarces/fix_5.12
Add kernel 5.12(+) (GRO_DROP removed) support
2021-04-03 16:53:01 +02:00
Carlos Garcés
e103b7c701
Merge pull request #232 from CGarces/centos8_fix
Add RHEL/CENTOS 8 support
2021-04-03 16:52:43 +02:00
Carlos Garcés
6a9f1c3ab1
Merge pull request #227 from Mange/kernel_5.11
Fix wireless regulatory API misuse
2021-04-03 16:52:22 +02:00
clnhub
ac90a311a3 Add kernel 5.12(+) (GRO_DROP removed) support 2021-04-03 14:46:24 +00:00
Carlos
48460503c0 Add RHEL/CENTOS 8 support 2021-04-01 15:37:13 +00:00
Carlos Garces
8fa56156a8 Fix wireless regulatory API misuse
Port of 81f153faac
2021-03-11 23:39:00 +01:00
Carlos Garcés
6a758d54cf
Merge pull request #226 from CGarces/github_actions
Improved github actions
2021-03-06 17:34:09 +01:00
Carlos Garces
d062e34d79 Improved github actions
Added step to detect misuse of USB_VENDOR_ID_REALTEK
Added check for duplicated USB IDs
Check for .deb files before download it
Move git checkout before build step
2021-03-06 17:13:00 +01:00
Carlos Garcés
7e8858e0df
Merge pull request #222 from glebm/realtek-4.4.x-fix-gcc10
Fix build with gcc-10 (-Wrestrict)
Close #221
2021-02-06 17:18:48 +01:00
Gleb Mazovetskiy
d6e1690ae8 Fix build with gcc-10 (-Wrestrict)
Fixes errors such as this:

```
os_dep/linux/ioctl_mp.c:611:4: warning: 'sprintf' argument 3 overlaps destination object 'extra' [-Wrestrict]
error, forbidden warning:ioctl_mp.c:611
  611 |    sprintf(extra, "%s,pathb=%d", extra, txpower_inx);
```

Fixes #221
2021-02-06 17:13:16 +01:00
Carlos Garcés
722ebf1e6d
Merge pull request #223 from CGarces/fix_build
Github actions enhancements
2021-02-06 14:01:36 +01:00