- 25 Nov, 2013 40 commits
-
-
Adam Lee authored
The return variable of request_code_segment() is used uninitialized if the argument boot_case is false, this patch fixes it. FYI, in ft1000_usb.h: STATUS_SUCCESS 0, and STATUS_FAILURE 0x1001. Signed-off-by:
Adam Lee <adam8157@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Malcolm Priestley authored
Attach ieee80211_hdr to vnt_beacon_buffer and remove pointer arithmetic. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
Merge structures typedef struct tagSTxShortBufHead and the members of struct vnt_tx_datahead_ab to form single structure vnt_tx_short_buf_head. Remove the duplicate members in struct vnt_beacon_buffer already in typedef struct tagSTxShortBufHead. This removes the need for any pointer arithmetic. Signed-off-by:
-
Malcolm Priestley authored
Remove > 0 code. Signed-off-by:
-
Peng Tao authored
This was mistakenly removed by commit 7fb3d1c5. Now register_sysctl_table() complains about "No proc_handler". Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
Peng Tao <bergwolf@gmail.com> Signed-off-by:
-
Peng Tao authored
The extactly same definition exitst in lustre_lib.h. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Patch generated by coccinelle: @generic_type@ declarer name CFS_MODULE_PARM; declarer name module_param; declarer name MODULE_PARM_DESC; expression E1, E2, E3, E4; type t; @@ -CFS_MODULE_PARM(E1, E2, t, E3, E4); +module_param(E1, t, E3); +MODULE_PARM_DESC(E1, E4); @charp@ expression E1, E2, E3, E4, E5; @@ -CFS_MODULE_PARM(E1, E2, E3, E4, E5); +module_param(E1, E3, E4); +MODULE_PARM_DESC(E1, E5); With manual modification to remove CFS_MODULE_PARM definition and fix up checkpatch.pl warnnings. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
It was never enabled. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
They are in fact definition of how client/server talks to each other about file type. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Use vfs_write instead and call kmap/kunmap to ensure that we can access high memory zone. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
no users. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
we can just call generic vfs_fsync(). Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Alexey Khoroshilov authored
Error handling code in gdm_usb_probe() deallocates all resources, but calls usb_get_dev(usbdev) and returns error code after that. The patch fixes it and, by the way, several other issues: - no need to use GFP_ATOMIC in probe(); - return -ENODEV instead of -1; - kmalloc+memset -> kzalloc Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by:
Alexey Khoroshilov <khoroshilov@ispras.ru> Signed-off-by:
-
Dan Carpenter authored
Instead of writing code like: if (dwc2_qh_is_non_per(qh)) { ... } else { ... } return; write it like: if (dwc2_qh_is_non_per(qh)) { ... return; } ... Signed-off-by:Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Paul Zimmerman <paulz@synopsys.com> Signed-off-by:
-
Dan Carpenter authored
I've made the success and error paths clearer and pulled some code in one indent level. Signed-off-by:
-
Paul Zimmerman authored
Remove #ifdef DEBUG from a couple of places where it is not needed Signed-off-by:
-
Paul Zimmerman authored
Rename dwc2_check_core_status() to dwc2_is_controller_alive(), and make it a boolean function. Also change the message when the controller is dead to say "dead" instead of "disconnected". Signed-off-by:
-
Paul Zimmerman authored
Remove useless cast in dwc2_get_otg_version() Signed-off-by:
-
Paul Zimmerman authored
NO_FS_PHY_HW_CHECKS is never defined, so remove the conditional code that checks for it being set Signed-off-by:
-
Paul Zimmerman authored
We were not checking the return value from any of these functions, so make them void functions Signed-off-by:
-
Paul Zimmerman authored
DWC2_PARAM_TEST is not a very good name for this macro, so rename it to DWC2_OUT_OF_BOUNDS Signed-off-by:
-
Paul Zimmerman authored
dwc2_process_non_isoc_desc() can potentially free the qtd, so null out the qtd pointer if the call fails so we don't try to access it later Signed-off-by:
-
Paul Zimmerman authored
Fix some functions called by dwc2_hcd_qtd_add() to return either a proper error code or 0, instead of somewhat random values. Then change the caller of dwc2_hcd_qtd_add() to just check the return value for 0. Signed-off-by:
-
Julien DELACOU authored
If the GRSTCTL_CSFTRST self-clearing bit never comes back to 0 for any reason, the controller is under reset state and cannot be used. It's preferable to abort initialization in such case. Signed-off-by:
Julien Delacou <julien.delacou@st.com> Acked-by:
-
Julien DELACOU authored
Pending interrupts clearing is done in dwc2_enable_common_interrupts so we don't need to do it twice. Signed-off-by:
-
Alexandre Demers authored
Remove useless netdev_info slowly flooding dmesg. Fixes bug https://bugzilla.kernel.org/show_bug.cgi?id=64231Signed-off-by:
Alexandre Demers <alexandre.f.demers@gmail.com> Acked-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
H Hartley Sweeten authored
The DPRINTK messages in this driver are just added noise. Remove them. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
H Hartley Sweeten authored
Use dev_dbg() instead of the DPRINTK macro to output the comedi debugging information. The dev_dbg() will prefix the messages appropriately so remove the "comedi%d: amplc_pci230: " portion and use __func__ to show the actual function name for debugging. Signed-off-by:
-
H Hartley Sweeten authored
Use dev_dbg() instead of the DPRINTK macro to output the comedi debugging information. The dev_dbg() will prefix the messages appropriately so remove the "comedi%d: " DRIVER_NAME portion and use __func__ to show the actual function name for debugging. Signed-off-by:
-
H Hartley Sweeten authored
Remove the DPRINTK messages that are just function trace noise. Use dev_dbg() instead of the DPRINTK macro to output the comedi debugging information. Signed-off-by:
-
H Hartley Sweeten authored
Use dev_dbg() instead of the DPRINTK macro to output the comedi debugging information. Signed-off-by:
-
H Hartley Sweeten authored
Remove this module parameter and use the CONFIG_COMEDI_DEBUG option to enable normal kernel debugging with -DDEBUG flag. Remove the #undef DEBUG from all the comedi source files so they will honour the -DDEBUG flag. Signed-off-by:
-
H Hartley Sweeten authored
This define unables some debug code that prints the status flags during the interrupt handler. These messages are just added noise and it's probably not a good idea to spew them during the interrupt anyway. Remove the DEBUG define as well as the debug code. Signed-off-by:
-
H Hartley Sweeten authored
This define is only used to enable some debug messages during the board attach. These are just added noise, remove them as well as the DEBUG define. Signed-off-by:
-
H Hartley Sweeten authored
These defines are not used in the driver. Remove them. Signed-off-by:
-
H Hartley Sweeten authored
The comedi core defines a DPRINTK macro in comedidev.h. Use that macro instead of defining a private version in this driver. Also, remove the DEBUG define since it was only used to enable the DPRINTK macro. Signed-off-by:
-
H Hartley Sweeten authored
This define is not used by the driver. Just remove it. Signed-off-by:
-
