- 15 Aug, 2016 40 commits
-
-
Christian Gromm authored
The function aim_register_videodev() uses the variables 'ret' and 'retval' to represent the same value. This patch removes 'retval' and replaces it with 'ret'. Further, it replaces the constant return value '-ENODEV' with the result returned by function video_register_device() in the event something went wrong. Signed-off-by:
Andrey Shvetsov <andrey.shvetsov@k2l.de> Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Christian Gromm authored
This patch replaces pr_info, pr_err, etc. function calls by the v4l2_... ones to get the device dependent logs. Signed-off-by:
-
Christian Gromm authored
The current V4L2 AIM implementation assigns to all video devices and to all V4L devices the same names. This patch makes use of hardware dependent names for the video devices and allows the user to choose the names for the V4L devices. Signed-off-by:
-
Christian Gromm authored
As the function get_aim_dev() does not delete elements of the list, the use of macro list_for_each_entry_safe is not necessary. This patch replaces the macro list_for_each_entry_safe with the macro list_for_each_entry. Signed-off-by:
-
Christian Gromm authored
The functions get_aim_dev() and aim_rx_data() are using interrupt unsafe spinlocks even though they may be called from an interrupt context. This patch fixes the described problem. Signed-off-by:
-
Colin Ian King authored
synth_add allows one to add MAXSYNTHS synths to the synths array; however it always NULLifies the next synth in the array which means that on the MAXSYNTHS synth we get an out-of-bounds write of the NULL to the synths array. Make the synths array MAXSYNTHS + 1 elements in size to allow for the final NULL sentinal to avoid the out-of-bounds write. Issue found wit CoverityScan, CID#744671 Signed-off-by:
Colin Ian King <colin.king@canonical.com> Reviewed-by:
Samuel Thibault <samuel.thibault@ens-lyon.org> Signed-off-by:
-
Laurence Rochfort authored
Fix the checkpatch.pl warning "No space is necessary after a cast". Signed-off-by:
Laurence Rochfort <laurence.rochfort@gmail.com> Signed-off-by:
-
Daeseok Youn authored
The 'ch_intr_rx' variable was used only for increasing. So the 'ch_intr_rx' variable is not useful for this driver. Signed-off-by:
Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by:
-
Daeseok Youn authored
The 'ch_intr_tx' variable was used only for increasing. So the 'ch_intr_tx' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
The 'ch_intr_modem' variable was used only for increasing. So the 'ch_intr_modem' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
The 'intr_rx' variable was used only for increasing. So the 'intr_rx' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
The 'intr_tx' variable was used only for increasing. So the 'intr_tx' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
The 'intr_modem' variable was used only for increasing. So the 'intr_modem' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
the "brd" was already checked for NULL before calling dgnc_do_remap(). the dgnc_do_remap() function was called only from the dgnc_found_board() and the DGNC_BOARD_MAGIC value was assigned to "brd->magic" in dgcn_found_board(). So it doesn't need to check about magic value. Signed-off-by:
-
Daeseok Youn authored
The 'intr_count' variable was used only for increasing. So the 'intr_count' variable is not useful for this driver. Signed-off-by:
-
Daeseok Youn authored
the "brd" value cannot be NULL in dgnc_finalize_board_init(). Because "brd" as a parameter of this function was already checked for NULL. the dgnc_finalize_board_init() as a static function was called only from dgnc_found_board() function and brd->magic value was assigned once in dgnc_found_board(). So it doesn't need to check for DGNC_BOARD_MAGIC value. Signed-off-by:
-
Daeseok Youn authored
Re-arrange the functions for removing forward declarations in dgnc_cls.c file. Signed-off-by:
-
Daeseok Youn authored
The dgnc_set_modem_info() used only channel_t variable. Any other variables were used only for checking NULL. So fist parameter changed from "tty_struct" to "channel_t" and useless NULL checks and variables are removed. Signed-off-by:
-
Daeseok Youn authored
The unit struct(un_t) was not used in dgnc_tty_hangup(). Signed-off-by:
-
Daeseok Youn authored
The local variable "bd" was not used in dgnc_carrier() function. Signed-off-by:
-
Daeseok Youn authored
The "result" variable in dgnc_get_mstat() was initialized with "-EIO". But if the "ch" is not null, "result" will be set to zero and if the "ch" is null, dgnc_get_mstat() will return "-ENXIO" as an error. So "-EIO" error value was useless in dgnc_get_mstat(). Signed-off-by:
-
Daeseok Youn authored
The dgnc_block_til_ready() is only used in dgnc_tty_open(). The unit data(struct un_t) was stored into tty->driver_data in dgnc_tty_open(). And also tty and un were tested about NULL so these variables doesn't need to check for NULL in dgnc_block_til_ready(). Signed-off-by:
-
Joshua Houghton authored
Add the __exit macro to the dgnc_cleanup_module(void) function in dgnc_driver.c Signed-off-by:
Joshua Houghton <josh@awful.name> Signed-off-by:
-
Al Viro authored
Signed-off-by:
Al Viro <viro@zeniv.linux.org.uk> Signed-off-by:
-
Al Viro authored
... by using ITER_BVEC recvmsg Signed-off-by:
-
Al Viro authored
Signed-off-by:
-
wang di authored
Increase the post-work for the statahead thread in the readdir case since it can become very busy. Signed-off-by:
wang di <di.wang@intel.com> Reviewed-on: http://review.whamcloud.com/10761 Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4906Reviewed-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
-
wang di authored
Some cases we want to preserve the ctx->pos value or use a different value altogther. So allow the passing in of a position offset to ll_dir_read. Signed-off-by:
-
wang di authored
We have if (1) conditionals which is pointless so remove it and the next code block is never called so remove that as well. Signed-off-by:
Jinshan Xiong <jinshan.xiong@intel.com> Reviewed-by:
-
wang di authored
Make some of the error reporting more clear for the statahead thread startup and is_first_dirent() function. Signed-off-by:
-
wang di authored
The variables api32 and hash64 was renamed to is_api32 and is_hash64. Signed-off-by:
-
wang di authored
Change the done flag from integer to bool. Signed-off-by:
-
wang di authored
Invert the done flag test to reduce the code indentation. If done is true release the page and break out of the while loop. Signed-off-by:
-
wang di authored
The variable next needs only to be set when done is false. Signed-off-by:
-
wang di authored
Instead of making a large chunk of code conditional based on if a page is valid we reverse the page validity test and exit the loop if the page is invalid instead. This allows a section of code to reduce its indentation one level. Signed-off-by:
-
wang di authored
Remove debug message and struct ll_inode_info. Signed-off-by:
-
wang di authored
Add struct md_op_data as a parameter to ll_dir_read. Signed-off-by:
-
wang di authored
We report the inode size but never print in the debug message that the number reported is the size. Signed-off-by:
-
wang di authored
The readdir operations lock was incomplete. This patch fills in the missing pieces. Signed-off-by:
-
wang di authored
Newer lustre version on metadata servers support different version of lmv magic. This add the new data structures to handle these new lmv magic versions. Signed-off-by:
-
