| 2786faa33bfc8d61b4fa45dd2e31664de796c837 |
|
01-Jul-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Sanitize USB read/write functions
The original Realtek provided functions suffered badly from clutter to
accommodate broken operating systems. Lets try this lean and clean
version instead.
v2: Do not use the stack for data passed to usb_control_msg(). This
requires reintroducing the mutex used in the old function. In
addition, get rid of the no longer used 'usb_vendor_req_buf'.
Note that rtl8723au_writeN() remains unlocked, so it can be used
for bulk block transfers without having to retake the mutex for
every write().
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| e976cf6f135713cb6f4c904bf24fadfbbf325160 |
|
01-Jul-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: pnp_bstop_trx is never set
In addition remove unused pnp_current_pwr_state
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 63a7cabbd57522fdbf063ccb0d0309fd8b3b7ada |
|
26-Jun-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Wifi_Error_Status isn't used for anything
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 1c69be9facbbb7d6aaa87831444544736c884653 |
|
26-Jun-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: usb_ops_linux.c: Add blank line between declaration and usage
This fixes a minor sparse warning.
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| f9de4a30a5e647bff71509d6f5d419f6967ba291 |
|
28-May-2014 |
Sachin Kamat <sachin.kamat@linaro.org> |
staging: rtl8723au: Remove redundant casting in usb_ops_linux.c
Casting value returned by k[cmz]alloc is useless.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| f8628a47ba502ec2ec795dc0134f052f45ce2cd4 |
|
24-May-2014 |
Joe Perches <joe@perches.com> |
staging: Convert __FUNCTION__ to __func__
Use the normal mechanism for emitting a function name.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 48039e70b647424ee9cf529af7846effb270f417 |
|
21-May-2014 |
Joe Perches <joe@perches.com> |
staging: realtek: Convert /n to \n
Use proper line terminations.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 68552a904c86e3e49d53305933371a1225d8aaf2 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Name port vs register access functions consistently
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| edbfd672cacb0d417096a1ed75e0070f1672c861 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Call usb_write*() functions directly
This allows us to finally remove the ugly HAL interface for accessing
registers, and remove rtw_io.c
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 050abc45833c6a66501cb3f938ef6e93ac18da8d |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Call usb_read*() functions directly
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 1c1bc5f1dd86237c65f571ca72c75b285b0dceeb |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Call usb_read_interrupt() directly instead of via HAL layer
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 633475cf5e6bfca434c7feb06653b0a187274c1d |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Call usb_{read,write}_port() functions directly
Get rid of more HAL layer obfuscations
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 97ccc3a6bead88e637c0150b19aaea27387ba6f7 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Clean up usb_{read,write}_port() usage
Get rid of silly HAL convolution for calling usb_{read,write}_port()
functions.
Benefit from fact we just have one input address for read_port, so no
point storing this and carrying it around.
Pick pipe in the local function instead of based on dst/src address.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 2296fd2d928c2cb9d601cdeef5e41eea4c120b88 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Remove nasty wrappers used to hide wrong function types
Do not use #defines to hack around pre-history functions not matching
current APIs
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 4631e5055eda1e52fdfe952b9e56f9f5743f3910 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Don't wrap calls to usb_control_msg()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 7438ccb93689845d31f5ced3ff6b22eada280d94 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: hal/usb_ops_linux.c: Various cleanups
Reduce excessive parenthesis usage, remove unnecessary null
initialization of variables, and some bad formatting.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 24ed8e4575cdd9bf59750755fadbef67af427c88 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: rtl8723au_xmit_tasklet(): Remove unnecessary variable init
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 43eeb8ee53a22b7cc6a4139ce7173c6fc0794d73 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Remove unused usb_read_mem/usb_write_mem stubs
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| ad899b12192ef2fa0cf81a88a320d5a0de4d5151 |
|
16-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Clean up error codes in IO functions
This also fixes up another messy case assuming _FAIL == false and
true == _SUCCESS
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 099a845251999648b2b24435f48b7486751d1b1b |
|
09-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Eliminate no-op function rtl8723au_init_recvbuf()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| c5779a0d1feff5d4e16f3bd4b1b17a41aac51d8a |
|
09-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: rtw_free_recvframe23a() is always passed precvpriv->free_recv_queue
No point in passing in the pointer to free_recv_queue, when we always
pass in the same value.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| d217e5dd224829a7d2a190c8119f16385f7d275a |
|
09-May-2014 |
Larry Finger <Larry.Finger@lwfinger.net> |
staging: rtl8723au: Move data byte-swap closer to register access
This makes sparse happy
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 980cf72afb0f6eac84d3ea63978db0993b76a3ec |
|
09-May-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Use workqueue to handle interrupt complete processing
Split the old work handler into a workqueue for processing
usb_read_interrupt_complete() events that require more than ust
clearing the event, and leave the old handler to just handle event
clearing.
This means we can get rid of the hacks with magic pointers to
determine what actions needs to be taken in the work handler, and
as an extra bonus obsoletes the bizarre cbuf code.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 28c71e23d2b43a2e687edd9342dca24a21b777e6 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Eliminate struct intf_hdl
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 64cb819b06b79bf9b47d9bd6adc5d074b9336d00 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Remove unused rtw_adapter * from struct intf_hdl
struct intf_hdl is now purely a wrapper around struct _io_ops, so we
can get rid of it in the next patch.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 45c64aa030f5b2674f4ab25e336d22a7029863b1 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Pass a struct rtw_adapter * to the USB functions
This lowers the dependency on struct intf_hdl so we can start getting
rid of if.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| ad0fb71b20524cf48db036d6909fef9e1456011b |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Remove unused pintf_dev entry from struct intf_hdl
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 520106f61e164f07f88844ab20a5c709b9716dd4 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Get rid of struct io_priv
No point wrapping struct intf_hdl into struct io_priv just for the
sake of it.
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 1dbe119b82b7902d0626347a455319f9004166e4 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Remove unused struct rtw_adapter * from struct io_priv
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| 5827c6555ce5edfba003deedc00cc30efca600f7 |
|
26-Apr-2014 |
Jes Sorensen <Jes.Sorensen@redhat.com> |
staging: rtl8723au: Fold rtw_init_io_priv23a() into rtl8723au_set_intf_ops()
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
| f7c92d2cc2beb3367f244480300eaecdd9502932 |
|
29-Mar-2014 |
Larry Finger <Larry.Finger@lwfinger.net> |
staging: r8723au: Add source files for new driver - part 2
The Realtek USB device RTL8723AU is found in Lenovo Yoga 13 tablets.
A driver for it has been available in a GitHub repo for several months.
This commit contains the second part of the source files. The source
is arbitrarily split to avoid E-mail files that are too large.
Jes Sorensen at RedHat has made many improvements to the vendor code,
and he has been doing the testing. I do not have access to this device.
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|