[PATCH 3/4] media: dvb-usb-v2: fix spelling mistake: "completition" -> "completion"

From: Colin Ian King Trivial fix to spelling mistake in dev_dbg and dev_err messages Signed-off-by: Colin Ian King --- drivers/media/usb/dvb-usb-v2/usb_urb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/dvb-usb-v2/usb_urb.c

[PATCH] [media] cx18: remove redundant zero check on retval

From: Colin Ian King The check for a zero retval is redundant as all paths that lead to this point have set retval to an error return value that is non-zero. Remove the redundant check. Detected by CoverityScan, CID#102589 ("Logically dead code") Signed-off-by: Colin Ian King ---

[PATCH] media: bt8xx: bttv: fix spelling mistake: "culpit" -> "culprit"

From: Colin Ian King Trivial fix to spelling mistake in pr_notice message text Signed-off-by: Colin Ian King --- drivers/media/pci/bt8xx/bttv-driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/bt8xx/bttv-driver.c

[PATCH] media: mtk-vpu: fix spelling mistake: "Prosessor" -> "Processor"

From: Colin Ian King Trivial fix to spelling mistake in module description text. Signed-off-by: Colin Ian King --- drivers/media/platform/mtk-vpu/mtk_vpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] [media] hdpvr: fix spelling mistake: "Hauppage" -> "Hauppauge"

From: Colin Ian King Trivial fix to spelling mistake in name field Signed-off-by: Colin Ian King --- drivers/media/usb/hdpvr/hdpvr-i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/hdpvr/hdpvr-i2c.c

[PATCH] media: dvb_frontends: fix spelling mistake: "unexpcted" -> "unexpected"

From: Colin Ian King Trivial fix to spelling mistake in dprintk message text Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/l64781.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH] [media] cx231xx: Fix spelling mistake: "senario" -> "scenario"

From: Colin Ian King Trivial fix to spelling mistake in dev_err message. Signed-off-by: Colin Ian King --- drivers/media/usb/cx231xx/cx231xx-pcb-cfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH][media-next] media: ddbridge: avoid out-of-bounds write on array demod_in_use

From: Colin Ian King In function stop there is a check to see if state->demod is a stopped value of 0xff, however, later on, array demod_in_use is indexed with this value causing an out-of-bounds write error. Avoid this by only writing to array demod_in_use if

[PATCH][media-next][V2] media: davinci_vpfe: fix memory leaks of params

From: Colin Ian King There are memory leaks of params; when copy_to_user fails and also the exit via the label 'error'. Also, there is a bogos memory allocation check on pointer 'to' when memory allocation fails on params. Fix this by kfree'ing params in error exit

[PATCH][media-next] media: davinci_vpfe: fix memory leaks of params

From: Colin Ian King There are memory leaks of params; when copy_to_user fails and also the exit via the label 'error'. Fix this by kfree'ing params in error exit path and jumping to this on the copy_to_user failure path. Detected by CoverityScan, CID#1467966

[PATCH] media: atomisp: fix spelling mistake: "diregard" -> "disregard"

From: Colin Ian King Trivial fix to spelling mistake in ia_css_print message text Signed-off-by: Colin Ian King --- .../css2400/css_2401_csi2p_system/host/csi_rx_private.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] [media] media/usbvision: fix spelling mistake: "compresion" -> "compression"

From: Colin Ian King Trivial fix to spelling mistake in proc text string Signed-off-by: Colin Ian King --- drivers/media/usb/usbvision/usbvision-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] media: smiapp: fix timeout checking in smiapp_read_nvm

From: Colin Ian King The current code decrements the timeout counter i and the end of each loop i is incremented, so the check for timeout will always be false and hence the timeout mechanism is just a dead code path. Potentially, if the RD_READY bit is not set, we

[PATCH][next] media: ispstat: don't dereference user_cfg before a null check

From: Colin Ian King The pointer user_cfg (a copy of new_conf) is dereference before new_conf is null checked, hence we may have a null pointer dereference on user_cfg when assigning buf_size from user_cfg->buf_size. Ensure this does not occur by moving the assignment

[PATCH] media: cec: set ev rather than v with CEC_PIN_EVENT_FL_DROPPED bit

From: Colin Ian King Setting v with the CEC_PIN_EVENT_FL_DROPPED is incorrect, instead ev should be set with this bit. Fix this. Detected by CoverityScan, CID#1467974 ("Extra high-order bits") Fixes: 6ec1cbf6b125 ("media: cec: improve CEC pin event handling")

[PATCH] [media] include/media: fix missing | operator when setting cfg

From: Colin Ian King The value from a readl is being masked with ITE_REG_CIOCAN_MASK however this is not being used and cfg is being re-assigned. I believe the assignment operator should actually be instead the |= operator. Detected by CoverityScan, CID#1467987

[PATCH] staging: media: davinci_vpfe: fix spelling of resizer_configure_in_continious_mode

From: Colin Ian King Trivial fix: rename function resizer_configure_in_continious_mode to resizer_configure_in_continuous_mode to fix spelling mistake. Signed-off-by: Colin Ian King --- drivers/staging/media/davinci_vpfe/dm365_resizer.c | 4

[PATCH] media: cx25821: prevent out-of-bounds read on array card

From: Colin Ian King Currently an out of range dev->nr is detected by just reporting the issue and later on an out-of-bounds read on array card occurs because of this. Fix this by checking the upper range of dev->nr with the size of array card (removes the hard coded

[PATCH] media: staging: atomisp: remove redundant assignments to various variables

From: Colin Ian King There are various assignments that are being made to variables that are not read and the variables are being updated later on, hence the redundant assignments can be removed. Cleans up clang warnings:

[PATCH] [media] exynos4-is: make array 'cmd' static, shrinks object size

From: Colin Ian King Don't populate the const read-only array 'cmd' on the stack but instead make it static. Makes the object code smaller by 38 bytes: Before: textdata bss dec hex filename 4950 868 0581816ba fimc-is-regs.o

[PATCH][next] media: lirc: don't kfree the uninitialized pointer txbuf

From: Colin Ian King The current error exit path if ir_raw_encode_scancode fails is via the label out_kfree which kfree's an uninitialized pointer txbuf. Fix this by exiting via a new exit path that does not kfree txbuf. Also exit via this new exit path for a failed

[PATCH] [media] dvb_frontend: remove redundant status self assignment

From: Colin Ian King The assignment status to itself is redundant and can be removed. Detected with Coccinelle. Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/drxd_hard.c| 3 --- drivers/media/dvb-frontends/tda18271c2dd.c

[PATCH] [media] dvb-frontends/stv0367: remove redundant self assignment of temporary

From: Colin Ian King The self assignment of temporary is redundant and can be removed. Detected using coccinelle. Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/stv0367.c | 2 -- 1 file changed, 2 deletions(-) diff --git

[PATCH] [media] stb0899: remove redundant self assignment of k_indirect

From: Colin Ian King The self assignment of k_indirect is redundant and can be removed. Detected using coccinelle. Signed-off-by: Colin Ian King --- drivers/media/dvb-frontends/stb0899_algo.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH] media: v4l: async: fix return of unitialized variable ret

From: Colin Ian King A shadow declaration of variable ret is being assigned a return error status and this value is being lost when the error exit goto's jump out of the local scope. This leads to an uninitalized error return value in the outer scope being returned. Fix

[PATCH] media: imx274: fix missing return assignment from call to imx274_mode_regs

From: Colin Ian King The variable ret is being checked for failure however it is not being set from the return status from the call to imx274_mode_regs. Currently ret is alwayus zero and the check is redundant. Fix this by assigning it. Detected by CoverityScan,

[PATCH] [media] au0828: make const array addr_list static

From: Colin Ian King Don't populate array addr_list on the stack but instead make it static. Makes the object code smaller by over 360 bytes: Before: textdata bss dec hex filename 80361488 192971625f4 au0828-input.o After: text

[PATCH] [media] cx88: make const arrays default_addr_list and pvr2000_addr_list static

From: Colin Ian King Don't populate arrays default_addr_list and pvr2000_addr_list on the stack but instead make them static. Makes the object code smaller by over 340 bytes: Before: textdata bss dec hex filename 125202800 64 15384

[PATCH] media: drxd: make const array fastIncrDecLUT static

From: Colin Ian King Don't populate array fastIncrDecLUT on the stack but instead make it static. Makes the object code smaller by over 360 bytes: textdata bss dec hex filename 32680 944 64 336888398 drxd_hard.o textdata

[PATCH 3/7] em28xx: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/em28xx/em28xx-dvb.c | 4 ++-- drivers/media/usb/em28xx/em28xx-video.c | 4 ++-- 2 files changed, 4

[PATCH 4/7] msi2500: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/msi2500/msi2500.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/7] au0828: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/au0828/au0828-video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 2/7] cx231xx: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/cx231xx/cx231xx-dvb.c | 4 ++-- drivers/media/usb/cx231xx/cx231xx-vbi.c | 4 ++--

[PATCH 5/7] pwc: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text and break line to clean up checkpatch warning Signed-off-by: Colin Ian King --- drivers/media/usb/pwc/pwc-if.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

[PATCH 6/7] stk1160: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/stk1160/stk1160-video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 7/7] tm6000: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/tm6000/tm6000-dvb.c | 4 ++-- drivers/media/usb/tm6000/tm6000-video.c | 4 ++-- 2 files changed, 4

[PATCH] [media] drivers/media/pci/zoran: remove redundant assignment to pointer h

From: Colin Ian King The pointer h is already initialized to codeclist_top so the second identical assignment is redundant and can be removed. Cleans up clang warning: drivers/media/pci/zoran/videocodec.c:322:21: warning: Value stored to 'h' during its initialization

[PATCH] [media] pt3: remove redundant assignment to mask

From: Colin Ian King Variable mask is being set to 0x80 and then set to this value again in the following for-loop. Remove the extraneous first setting of mask. Cleans up clang warning: drivers/media/pci/pt3/pt3_i2c.c:88:2: warning: Value stored to 'mask' is never read

[PATCH][V2] bdisp: remove redundant assignment to pix

From: Colin Ian King Pointer pix is being initialized to a value and a little later being assigned the same value again. Remove the initial assignment to avoid a duplicate assignment. Cleans up the clang warning: drivers/media/platform/sti/bdisp/bdisp-v4l2.c:726:26:

[PATCH] [media] bdisp: remove redundant assignment to pix

From: Colin Ian King Pointer pix is being initialized to a value and a little later being assigned the same value again. Remove the redundant second duplicate assignment. Cleans up the clang warning: drivers/media/platform/sti/bdisp/bdisp-v4l2.c:726:26: warning: Value

[PATCH] [media] gspca: remove redundant assignment to variable j

From: Colin Ian King Variable j is being set to zero before a loop and also immediately inside the loop and is not used after the loop. Hence the first assignment is redundant and can be removed. Cleans up clang warning: drivers/media/usb/gspca/gspca.c:1078:2: warning:

[PATCH] [media] mxl111sf: remove redundant assignment to index

From: Colin Ian King Variable index is set to zero and then set to zero again a few lines later in a for loop initialization. Remove the redundant setting of index to zero. Cleans up the clang warning: drivers/media/usb/dvb-usb-v2/mxl111sf-i2c.c:519:3: warning: Value

[PATCH] [media] radio-raremono: remove redundant initialization of freq

From: Colin Ian King Variable freq is initialized to f->frequency however this value is never read and freq is later updated; hence the initialization is redundant and can be removed. Cleans up clang warning: drivers/media/radio/radio-raremono.c:257:6: warning: Value

[PATCH] media: dvb_ca_en50221: sanity check slot number from userspace

From: Colin Ian King Currently a user can pass in an unsanitized slot number which will lead to and out of range index into ca->slot_info. Fix this by checking that the slot number is no more than the allowed maximum number of slots. Seems that this bug has been in the

[PATCH] [media] cxusb: pass buf as a const u8 * pointer and make buf static const

From: Colin Ian King Don't populate the read-only u8 array buf on the stack at run time but instead make it static const; makes object code smaller saving over 480 bytes: Before: textdata bss dec hex filename 33030 65936 192 99158 18356

[PATCH] dma-buf: remove redundant initialization of sg_table

From: Colin Ian King sg_table is being initialized and is never read before it is updated again later on, hence making the initialization redundant. Remove the initialization. Detected by clang scan-build: "warning: Value stored to 'sg_table' during its initialization

[PATCH] [media] gspca: make arrays static, reduces object code size

From: Colin Ian King Don't populate const arrays on the stack, instead make them static. Makes the object code smaller by over 5200 bytes: Before: textdata bss dec hex filename 582598880 128 67267 106c3 ov519.o After: textdata

[PATCH] [media] ov2640: make array reset_seq static, reduces object code size

From: Colin Ian King Don't populate the array reset_seq on the stack, instead make it static. Makes the object code smaller by over 50 bytes: Before: textdata bss dec hex filename 117376000 64 178014589 drivers/media/i2c/ov2640.o

[PATCH] [media] tuners: mxl5005s: make arrays static const, reduces object code size

From: Colin Ian King Don't populate the arrays RegAddr on the stack, instead make them static const. Makes the object code smaller by over 980 bytes: Before: textdata bss dec hex filename 64923 304 0 65227fecb

[PATCH] media: cx23885: make const array buf static, reduces object code size

From: Colin Ian King Don't populate the array buf on the stack, instead make it static. Makes the object code smaller by over 240 bytes: Before: textdata bss dec hex filename 21689 22992 416 45097b029 cx23885-cards.o After: text

[PATCH] [media] ov9640: make const arrays res_x/y static const, reduces object code size

From: Colin Ian King Don't populate the arrays res_x and resy_y on the stack, instead make them static const. Makes the object code smaller by over 160 bytes: Before: textdata bss dec hex filename 105092800 64 13373343d ov9640.o

[PATCH] [media] cobalt: remove redundant zero check on retval

From: Colin Ian King The error handling paths all end up with retval being non-zero, so the check for retval being zero is always false and hence is redundant. Remove it. Detected by CoverityScan CID#1309479 ("Logically dead code") Signed-off-by: Colin Ian King

[PATCH] [media] imon: make two const arrays static, reduces object code size

From: Colin Ian King Don't populate the const arrays vfd_packet6 and fp_packet on the stack, instead make them static. Makes the object code smaller by over 600 bytes: Before: textdata bss dec hex filename 43794 179201024 62738f512

[PATCH] [media] cx25840: make array stds static const, reduces object code size

From: Colin Ian King Don't populate the array syds on the stack, instead make it static const. Makes the object code smaller by over 280 bytes: Before: textdata bss dec hex filename 81451 12784 704 94939 172db cx25840-core.o text

[PATCH] [media] rtl28xxu: make array rc_nec_tab static const

From: Colin Ian King Don't populate the array rc_nec_tab on the stack, instead make it static const. Makes the object code smaller by over 620 bytes: Before: textdata bss dec hex filename 49511 17040 64 66615 10437 rtl28xxu.o After:

[PATCH][media-next] media: qcom: camss: Make function vfe_set_selection static

From: Colin Ian King The function vfe_set_selection is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: warning: symbol 'vfe_set_selection' was not declared. Should it be static? Signed-off-by: Colin Ian King

[PATCH] [media] em28xx: calculate left volume level correctly

From: Colin Ian King The calculation of the left volume looks suspect, the value of 0x1f - ((val << 8) & 0x1f) is always 0x1f. The debug prior to the assignemnt of value[1] prints the left volume setting using the calculation 0x1f - (val >> 8) & 0x1f which looks correct

[PATCH] [media] docs-next: update the fe_status documentation for FE_NONE

From: Colin Ian King Recently added FE_NONE to the enum fe_status, so update the documentation accordingly. Signed-off-by: Colin Ian King --- Documentation/media/uapi/dvb/fe-read-status.rst | 20 ++-- 1 file changed, 14

[PATCH] dvb_frontend: initialize variable s with FE_NONE instead of 0

From: Colin Ian King In a previous commit, we added FE_NONE as an unknown fe_status. Initialize variable s to FE_NONE instead of the more opaque value 0. Signed-off-by: Colin Ian King --- drivers/media/dvb-core/dvb_frontend.c | 2 +- 1 file

[PATCH][media-next] media: dvb-frontends/stv0910: make various local variables static

From: Colin Ian King The list stvlist and arrays padc_lookup, s1_sn_lookup and s2_sn_lookup do not need to be in global scope, so make them all static. Cleans up a bunch of smatch warnings: symbol 'padc_lookup' was not declared. Should it be static? symbol

[PATCH][V2] dvb_frontend: ensure that inital front end status initialized

From: Colin Ian King The fe_status variable s is not initialized meaning it can have any random garbage status. This could be problematic if fe->ops.tune is false as s is not updated by the call to fe->ops.tune() and a subsequent check on the change status will using a

[PATCH] [media] dvb_frontend: ensure that front end status is initialized

From: Colin Ian King The fe_status variable s is not initialized meaning it can have any random garbage status. This could be problematic if fe->ops.tune is false as s is not updated by the call to fe->ops.tune() and a subsequent check on the change status will using a

[PATCH][media-next] media: v4l: make local function v4l2_fwnode_endpoint_parse_csi1_bus static

From: Colin Ian King The function v4l2_fwnode_endpoint_parse_csi1_bus does not need to be in global scope, so make it static. Also reformat the function arguments as adding the static keyword made one of the source lines more than 80 chars wide and checkpatch does not

[PATCH] [media] staging: media: davinci_vpfe: fix spelling mistake in variable

From: Colin Ian King Trivial fix to spelling mistake, rename the function name resizer_configure_in_continious_mode to resizer_configure_in_continuous_mode and also remove an extraneous space. Signed-off-by: Colin Ian King ---

[PATCH] [media] dib0090: make const array dib0090_tuning_table_cband_7090e_aci static

From: Colin Ian King Don't populate array dib0090_tuning_table_cband_7090e_aci on the stack but instead make it static. Makes the object code smaller by over 180 bytes: Before: textdata bss dec hex filename 400527320 192 47564b9cc

[PATCH] [media] drxj: make several const arrays static

From: Colin Ian King Don't populate const arrays on the stack but instead make them static. Makes the object code smaller by over 1800 bytes: Before: textdata bss dec hex filename 941009160 0 103260 1935c drxj.o After: textdata

[PATCH] [media] drxd: make const arrays slowIncrDecLUT and fastIncrDecLUT static

From: Colin Ian King Don't populate arrays slowIncrDecLUT and fastIncrDecLUT on the stack but instead make them static. Makes the object code smaller by over 100 bytes: textdata bss dec hex filename 27776 832 64 286727000 drxd_hard.o

[PATCH] staging: fbtft: make const array gamma_par_mask static

From: Colin Ian King Don't populate array gamma_par_mask on the stack but instead make it static. Makes the object code smaller by 148 bytes: Before: textdata bss dec hex filename 29931104 040971001

[PATCH] [media] fc001[23]: make const gain table arrays static

From: Colin Ian King Don't populate the gain tables on the stack but make them static const. Makes the object code smaller: Before: textdata bss dec hex filename 78011408 0920923f9 drivers/media/tuners/fc0012.o 8483 936

[PATCH] [media] solo6x10: make const array saa7128_regs_ntsc static

From: Colin Ian King Don't populate const array saa7128_regs_ntsc on the stack but insteaed make it static. Makes the object code smaller and saves nearly 840 bytes Before: textdata bss dec hex filename 9218 360 09578256a

[PATCH] [media] smiapp: make various const arrays static

From: Colin Ian King Don't populate const arrays on the stack but instead make them static. Makes the object code smaller and saves nearly 550 bytes. Before: textdata bss dec hex filename 3638 752 043901126 smiapp-quirk.o After:

[PATCH] [media] dvb-frontends/cxd2841er: make several arrays static

From: Colin Ian King Don't populate arrays on the stack but make them static. Makes the object code smaller: Before: textdata bss dec hex filename 89299 21704 64 111067 1b1db cxd2841er.o After: textdata bss dec hex

[PATCH] [media] uvcvideo: fix spelling mistake: "entites" -> "entities"

From: Colin Ian King Trivial fix to spelling mistake in uvc_printk message Signed-off-by: Colin Ian King --- drivers/media/usb/uvc/uvc_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] [media] media/i2c/saa717x: fix spelling mistake: "implementd" -> "implemented"

From: Colin Ian King Trivial fix to spelling mistake in v4l2_dbg debug message Signed-off-by: Colin Ian King --- drivers/media/i2c/saa717x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/saa717x.c

[PATCH] [media] media: i2c: m5mols: fix spelling mistake: "Machanics" -> "Mechanics"

From: Colin Ian King Trivial fix to spelling mistake in v4l2_info message Signed-off-by: Colin Ian King --- drivers/media/i2c/m5mols/m5mols_core.c | 2 +- drivers/scsi/qedi/qedi_fw.c| 2 +- 2 files changed, 2 insertions(+), 2

[PATCH][media-next] media: venus: fix loop wrap in cleanup of clks

From: Colin Ian King The current pre-decrement is incorrect and should be replaced with a post-decrement. Consider the case where the very first clk_prepare_enable fails when i is 0; in this case the error clean up will decrement the unsigned int which wraps to the

[PATCH][media] atomisp: make repool_pgnr and punit_ddr_dvfs_enable static

From: Colin Ian King integer repool_pgnr and function punit_ddr_dvfs_enable can be made static as they do not need to be in global scope. Cleans up sparse warnings: "symbol 'repool_pgnr' was not declared. Should it be static?" "symbol 'punit_ddr_dvfs_enable' was not

[PATCH] [media] em28xx: fix spelling mistake: "missdetected" -> "misdetected"

From: Colin Ian King Trivial fix to spelling mistake in dev_err message Signed-off-by: Colin Ian King --- drivers/media/usb/em28xx/em28xx-cards.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH][V2] [media] rainshadow-cec: ensure exit_loop is intialized

From: Colin Ian King exit_loop is not being initialized, so it contains garbage. Ensure it is initialized to false. Detected by CoverityScan, CID#1436409 ("Uninitialized scalar variable") Fixes: ea6a69defd3311 ("[media] rainshadow-cec: avoid -Wmaybe-uninitialized

[PATCH] [media] rainshadow-cec: ensure exit_loop is initialized

From: Colin Ian King exit_loop is not being initialized, so it contains garbage. Ensure it is initialized to false. Detected by CoverityScan, CID#1436409 ("Uninitialzed scalar variable") Fixes: ea6a69defd3311 ("[media] rainshadow-cec: avoid -Wmaybe-uninitialized

[PATCH] [media] s5p-mfc: fix spelling mistake: "destionation" -> "destination"

From: Colin Ian King Trivial fix to spelling mistake in mfc_err error messages Signed-off-by: Colin Ian King --- drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c | 2 +- drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +- 2 files changed,

[PATCH] [media] pvrusb2: remove redundant check on cnt > 8

From: Colin Ian King The 2nd check of cnt > 8 is redundant as cnt is already checked and thresholded to a maximum of 8 a few statements earlier. Remove this redundant 2nd check. Detected by CoverityScan, CID#114281 ("Logically dead code") Signed-off-by: Colin Ian King

[PATCH] [media] cx18: fix spelling mistake: "demodualtor" -> "demodulator"

From: Colin Ian King trivial fix to spelling mistake and add in a white space in a CX18_ERR error message Signed-off-by: Colin Ian King --- drivers/media/pci/cx18/cx18-dvb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] staging: media: atomisp: fix range checking on clk_num

From: Colin Ian King The range checking on clk_num is incorrect; fix these so that invalid clk_num values are detected correctly. Detected by static analysis with by PVS-Studio Signed-off-by: Colin Ian King ---

[PATCH] [media] xc5000: fix spelling mistake: "calibration"

From: Colin Ian King Trivial fix to spelling mistake on calibration, make Self lowercase and re-join multiple lined printk since checkpatch allows this coding style. Signed-off-by: Colin Ian King --- drivers/media/tuners/xc5000.c | 3 +-- 1

[PATCH] staging/atomisp: fix spelling mistake: "falied" -> "failed"

From: Colin Ian King trivial fix to spelling mistake in dev_err error message Signed-off-by: Colin Ian King --- drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm_bo_dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] [media] coda: remove redundant call to v4l2_m2m_get_vq

From: Colin Ian King The call to v4ls_m2m_get_vq is only used to get the return value which is not being used, so it appears to be redundant and can be removed. Detected with CoverityScan, CID#1420674 ("Useless call") Signed-off-by: Colin Ian King

[PATCH] [media] usb: au0828: remove redundant code

From: Colin Ian King The check for ret being non-zero is false as ret is always zero, hence we have redundant dead code that can be removed. Detected with CoverityScan, CID#112968 ("Constant' variable guards dead code (DEADCODE)'") Signed-off-by: Colin Ian King

[PATCH] staging: atomisp: fix missing break in switch statement

From: Colin Ian King I believe there is a missing break in the switch statement for case V4L2_CID_FOCUS_STATUS as the current fall-through looks suspect to me. Detected by CoverityScan, CID#1416580 ("Missing break in switch") Signed-off-by: Colin Ian King

[PATCH] staging/atomisp: remove redundant null check on frame

From: Colin Ian King There is no need to perform a null check on frame as there is an earlier null check check and return hence making the null check redundant. Remove it. Detected by CoverityScan, CID#1416563 ("Logically Dead Code") Signed-off-by: Colin Ian King

[PATCH] staging: atomisp: remove redundant check for client being null

From: Colin Ian King The previous statement checks if client is null, so the null check when assigning dev is redundant and can be removed. Detected by CoverityScan, CID#1416555 ("Logically Dead Code") Signed-off-by: Colin Ian King ---

[PATCH] staging: atomisp: clean up return logic, remove redunant code

From: Colin Ian King There is no need to check if ret is non-zero, remove this redundant check and just return the error status from the call to mt9m114_write_reg_array. Detected by CoverityScan, CID#1416577 ("Identical code for different branches") Signed-off-by:

[PATCH] [media] atmel-isc: fix off-by-one comparison and out of bounds read issue

From: Colin Ian King The are only HIST_ENTRIES worth of entries in hist_entry however the for-loop is iterating one too many times leasing to a read access off the end off the array ctrls->hist_entry. Fix this by iterating by the correct number of times. Detected by

[PATCH] [media] Staging: media/lirc: don't call put_ir_rx on rx twice

From: Colin Ian King There is an exit path where rx is kfree'd on put_ir_rx and then a jump to label out_put_xx will again kfree it with another call to put_ir_rx. Fix this by adding a new label that avoids this 2nd call to put_ir_rx for this specific case. Detected

[PATCH] [media] saa7164: "first image" should be "second image" in error message

From: Colin Ian King The error message when the second image is not available is incorrect, replace "first image" with "second image". Fixes CoverityScan CID#1077508 ("Copy-paste error") Signed-off-by: Colin Ian King ---

[PATCH] [media] gspca_stv06xx: remove redundant check for err < 0

From: Colin Ian King The comparison of err < 0 is redundant as err has been previously been assigned to 0 and has not changed. Remove the redundant check. Fixes CoverityScan CID#703363 ("Logically dead code") Signed-off-by: Colin Ian King

[PATCH][V3] [media] gp8psk: fix spelling mistake: "firmare" -> "firmware"

From: Colin Ian King trivial fix to spelling mistake in err message Signed-off-by: Colin Ian King --- drivers/media/usb/dvb-usb/gp8psk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/gp8psk.c

[PATCH][V2] [media] gp8psk: fix spelling mistake: "firmare" -> "firmware"

From: Colin Ian King Trivial fix to spelling mistake in err message. Also change "don't" to "does not". Signed-off-by: Colin Ian King --- drivers/media/usb/dvb-usb/gp8psk.c | 2 +- drivers/net/wireless/realtek/rtlwifi/core.c | 2 +-

[PATCH] [media] gp8psk: fix spelling mistake: "firmare" -> "firmware"

From: Colin Ian King trivial fix to spelling mistake in err message Signed-off-by: Colin Ian King --- drivers/media/usb/dvb-usb/gp8psk.c | 2 +- drivers/net/wireless/realtek/rtlwifi/core.c | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH] [media] b2c2: fix spelling mistake: "Contunuity" -> "Continuity"

From: Colin Ian King trivial fix to spelling mistake in deb_chk message Signed-off-by: Colin Ian King --- drivers/media/pci/b2c2/flexcop-pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

  1   2   >