[PATCH] media/platform/exynos4-is/fimc-is - Unmap region obtained by of_iomap

2016-12-05 Thread Arvind Yadav
Free memory mapping, if fimc_is_probe is not successful. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/exynos4-is/fimc-is.c |8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/exynos4-is/fimc-is.c b/d

[PATCH] Staging: media: Unmap and release region obtained by ioremap_nocache

2017-03-17 Thread Arvind Yadav
Free memory mapping, if vpfe_ipipe_init is not successful. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/staging/media/davinc

[PATCH] [media] radio: constify pnp_device_id

2017-08-15 Thread Arvind Yadav
pnp_device_id are not supposed to change at runtime. All functions working with pnp_device_id provided by work with const pnp_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/radio/radio-cadet.c| 2 +- drivers

[PATCH 0/3] constify media platform_device_id

2017-08-15 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Arvind Yadav (3): [PATCH 1/3] [media] coda: constify platform_device_id [PATCH 2/3] [media] davinci

[PATCH 3/3] [media] omap3isp: constify platform_device_id

2017-08-15 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/omap3isp/isp

[PATCH 1/3] [media] coda: constify platform_device_id

2017-08-15 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/coda/coda-co

[PATCH 2/3] [media] davinci: constify platform_device_id

2017-08-15 Thread Arvind Yadav
platform_device_id are not supposed to change at runtime. All functions working with platform_device_id provided by work with const platform_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/davinci/vpbe

[PATCH 1/3] [media] usb: constify usb_device_id

2017-08-13 Thread Arvind Yadav
'['. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/usb/airspy/airspy.c | 2 +- drivers/media/usb/as102/as102_usb_drv.c | 2 +- drivers/media/usb/b2c2/flexcop-usb.c | 2 +- drivers/media/usb/cpia2/cpia2_usb.c

[PATCH 0/3] constify media usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Arvind Yadav (3): [PATCH 1/3] [media] usb: constify usb_device_id [PATCH 2/3] [media] rc: constify usb_device_id

[PATCH 3/3] [media] radio: constify usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/radio/dsbr100.c

[PATCH 2/3] [media] rc: constify usb_device_id

2017-08-13 Thread Arvind Yadav
usb_device_id are not supposed to change at runtime. All functions working with usb_device_id provided by work with const usb_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/rc/ati_remote.c | 2 +- drivers/me

[PATCH] [media] usb: pulse8-cec: constify serio_device_id

2017-08-17 Thread Arvind Yadav
serio_device_id are not supposed to change at runtime. All functions working with serio_device_id provided by work with const serio_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/usb/pulse8-cec/pulse8-cec.c | 2

[PATCH] [media] usb: rainshadow-cec: constify serio_device_id

2017-08-17 Thread Arvind Yadav
serio_device_id are not supposed to change at runtime. All functions working with serio_device_id provided by work with const serio_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/usb/rainshadow-cec/rainshadow

[PATCH] staging: atomisp: gc2235: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/gc2235.o File size After adding 'const': textdata bss dec hex filename 108181296 4 121182f56 drivers/staging/media/atomisp/i2c/gc2235.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: mt9m114: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/mt9m114.o File size After adding 'const': textdata bss dec hex filename 152442512 8 177644564 drivers/staging/media/atomisp/i2c/mt9m114.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] [media] rc: constify attribute_group structures.

2017-07-07 Thread Arvind Yadav
drivers/media/rc/rc-main.o File size After adding 'const': textdata bss dec hex filename 11797 720 20 1253730f9 drivers/media/rc/rc-main.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/rc/rc-main.c | 6 +++--- 1 file chan

[PATCH] media: vb2 dma-contig: Constify dma_buf_ops structures.

2017-07-01 Thread Arvind Yadav
/v4l2-core/videobuf2-dma-contig.o File size After adding 'const': textdata bss dec hex filename 6155 160 0631518ab drivers/media/v4l2-core/videobuf2-dma-contig.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/v4l2-core/videobu

[PATCH] media: vb2 vmalloc: Constify dma_buf_ops structures.

2017-07-01 Thread Arvind Yadav
/v4l2-core/videobuf2-vmalloc.o File size After adding 'const': textdata bss dec hex filename 3291 80 03371 d2b drivers/media/v4l2-core/videobuf2-vmalloc.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/v4l2-core/videobuf2-vma

[PATCH] media: vb2 dma-sg: Constify dma_buf_ops structures.

2017-07-01 Thread Arvind Yadav
/v4l2-core/videobuf2-dma-sg.o File size After adding 'const': textdata bss dec hex filename 5358 0 4536214f2 drivers/media/v4l2-core/videobuf2-dma-sg.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/v4l2-core/videobuf2-dma-sg

[PATCH] staging: atomisp: lm3554: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/lm3554.o File size After adding 'const': textdata bss dec hex filename 54111856 2472911c7b drivers/staging/media/atomisp/i2c/lm3554.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: ov2680: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/ov2680.o File size After adding 'const': textdata bss dec hex filename 125303056 8 155943cea drivers/staging/media/atomisp/i2c/ov2680.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: ov8858: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/ov8858.o File size After adding 'const': textdata bss dec hex filename 238688384 0 322527dfc drivers/staging/media/atomisp/i2c/ov8858.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: gc0310: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/gc0310.o File size After adding 'const': textdata bss dec hex filename 103611824 0 121852f99 drivers/staging/media/atomisp/i2c/gc0310.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: ov2722: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/ov2722.o File size After adding 'const': textdata bss dec hex filename 148351816 0 16651410b drivers/staging/media/atomisp/i2c/ov2722.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomi

[PATCH] staging: atomisp: ov5693: constify acpi_device_id.

2017-07-06 Thread Arvind Yadav
/staging/media/atomisp/i2c/ov5693/ov5693.o File size After adding 'const': textdata bss dec hex filename 207933200 0 239935db9 drivers/staging/media/atomisp/i2c/ov5693/ov5693.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging

[PATCH] media: exynos4-is: fimc-is-i2c: constify dev_pm_ops structures.

2017-06-29 Thread Arvind Yadav
/platform/exynos4-is/fimc-is-i2c.o File size After adding 'const': textdata bss dec hex filename 1403 176 01579 62b drivers/media/platform/exynos4-is/fimc-is-i2c.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/exynos4-i

[PATCH] [media] imon: constify attribute_group structures.

2017-07-07 Thread Arvind Yadav
drivers/media/rc/imon.o File size After adding 'const': textdata bss dec hex filename 186792160 77 2091651b4 drivers/media/rc/imon.o Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/rc/imon.c | 4 ++-- 1 file changed, 2 insertions

[PATCH 08/18] [media] bt8xx: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/bt8xx/bt878.c | 2 +- 1 file chan

[PATCH 16/18] [media] mantis: hopper_cards: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/mantis/hopper_cards.c | 2 +-

[PATCH 15/18] [media] mantis: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/mantis/mantis_cards.c | 2 +-

[PATCH 13/18] [media] saa7164: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/saa7164/saa7164-core.c | 2 +-

[PATCH 17/18] [media] cx18: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/cx18/cx18-driver.c | 2 +-

[PATCH 04/18] [media] meye: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/meye/meye.c | 2 +- 1 file chan

[PATCH 07/18] [media] zoran: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/zoran/zoran_card.c | 2 +-

[PATCH 12/18] [media] b2c2: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/b2c2/flexcop-pci.c | 2 +-

[PATCH 05/18] [media] pluto2: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/pluto2/pluto2.c | 2 +- 1 file c

[PATCH 06/18] [media] dm1105: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/dm1105/dm1105.c | 2 +- 1 file c

[PATCH 18/18] [media] radio: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/radio/radio-maxiradio.c | 2 +-

[PATCH 14/18] [media] pt1: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/pt1/pt1.c | 2 +- 1 file chan

[PATCH 10/18] [media] ivtv: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ivtv/ivtv-driver.c | 2 +-

[PATCH 11/18] [media] cobalt: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/cobalt/cobalt-driver.c | 2 +-

[PATCH 03/18] [media] cx23885: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/cx23885/cx23885-core.c | 2 +-

[PATCH 01/18] [media] marvell-ccic: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/marvell-ccic/cafe-driver

[PATCH 00/18] constify media pci_device_id

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Arvind Yadav (18): [PATCH 01/18] [media] marvell-ccic: constify pci_device_id. [PATCH 02/18] [media] netup_unidvb

[PATCH 02/18] [media] netup_unidvb: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/netup_unidvb/netup_unidvb_core

[PATCH 09/18] [media] bt8xx: bttv: constify pci_device_id.

2017-08-01 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/bt8xx/bttv-driver.c | 2 +-

[PATCH 9/9] [media] saa7146: hexium_gemini: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/saa7146/hexium_gemini.c | 2

[PATCH 7/9] [media] saa7146: mxb: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/saa7146/mxb.c | 2 +-

[PATCH 6/9] [media] ttpci: av7110: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ttpci/av7110.c | 2 +-

[PATCH 3/9] [media] ttpci: budget-patch: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ttpci/budget-patch.c | 2 +-

[PATCH 8/9] [media] saa7146: hexium_orion: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/saa7146/hexium_orion.c | 2

[PATCH 4/9] [media] ttpci: budget-ci: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ttpci/budget-ci.c | 2 +-

[PATCH 5/9] [media] ttpci: budget-av: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ttpci/budget-av.c | 2 +-

[PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by work with const pci_device_id. So mark the non-const structs as const. So making 'pci_tbl' as const member of 'struct saa7146_extension'. Signed-off-by: Arvind Yadav <arvind.ya

[PATCH 2/9] [media] ttpci: budget: constify pci_device_id.

2017-08-02 Thread Arvind Yadav
pci_device_id are not supposed to change at runtime. All functions working with pci_device_id provided by and work with const pci_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/ttpci/budget.c | 2 +-

[PATCH 0/9] constify media pci_device_id/pci_tbl.

2017-08-02 Thread Arvind Yadav
SAA7146 DVD card base pci device id const. Arvind Yadav (9): [PATCH 1/9] [media] drv-intf: saa7146: constify pci_device_id. [PATCH 2/9] [media] ttpci: budget: constify pci_device_id. [PATCH 3/9] [media] ttpci: budget-patch: constify pci_device_id. [PATCH 4/9] [media] ttpci: budget-ci

[PATCH 2/6] [media] adv7511: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/adv7511.c | 2 +- 1 file chan

[PATCH] [media] tc358743: Handle return value of clk_prepare_enable

2017-05-31 Thread Arvind Yadav
clk_prepare_enable() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/tc358743.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc35

[PATCH] Staging: media: davinci_vpfe Fix resource leaks in error paths.

2017-06-07 Thread Arvind Yadav
Free memory, if ipipe_s_config and ipipe_g_config are not successful. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 1/2] [media] coda: Handle return value of kasprintf

2017-09-20 Thread Arvind Yadav
kasprintf() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/coda/coda-bit.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/platform/coda/coda-bit.c b/drivers/media/platform/coda/coda

[PATCH 0/2] [media] Handle return value of kasprintf

2017-09-20 Thread Arvind Yadav
kasprintf() can fail here and we must check its return value. Arvind Yadav (2): [PATCH 1/2][media] coda: Handle return value of kasprintf [PATCH 2/2][media] cx23885: Handle return value of kasprintf drivers/media/pci/cx23885/cx23885-input.c | 15 +-- drivers/media/platform/coda

[PATCH 2/2] [media] cx23885: Handle return value of kasprintf

2017-09-20 Thread Arvind Yadav
kasprintf() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/cx23885/cx23885-input.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/media/pci/cx23885/cx23885-inpu

[PATCH v2 1/2] [media] coda: Handle return value of kasprintf

2017-09-20 Thread Arvind Yadav
kasprintf() can fail here and we must check its return value. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- changes in v2 : Calling coda_free_framebuffers to release already allocated buffers. drivers/media/platform/coda/coda-bit.c | 4 1 file changed, 4 inse

Re: usb/media/hdpvr: trying to register non-static key in hdpvr_probe

2017-09-22 Thread Arvind Yadav
Hi, I have a doubt. Why we are calling flush_work in hdpvr_probe for every failure. We are flushing work which is not defined yet. Here, hdpvr_register_videodev() is responsible for setup and register a video device. Also defining and initializing a worker. we are calling

[PATCH] [media] hdpvr: Fix an error handling path in hdpvr_probe()

2017-09-22 Thread Arvind Yadav
. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/usb/hdpvr/hdpvr-core.c | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/drivers/media/usb/hdpvr/hdpvr-core.c b/drivers/media/usb/hdpvr/hdpvr-core.c index dbe29c6..1e8cbaf

Re: usb/media/hdpvr: trying to register non-static key in hdpvr_probe

2017-09-22 Thread Arvind Yadav
Hi Andrey, On Friday 22 September 2017 05:16 PM, Andrey Konovalov wrote: On Fri, Sep 22, 2017 at 9:41 AM, Arvind Yadav <arvind.yadav...@gmail.com> wrote: Hi, I have a doubt. Why we are calling flush_work in hdpvr_probe for every failure. We are flushing work which is not defined yet.

[PATCH] media: imon: Fix null-ptr-deref in imon_probe

2017-10-09 Thread Arvind Yadav
It seems that the return value of usb_ifnum_to_if() can be NULL and needs to be checked. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov usb/media/imon: null-ptr-deref in imon_probe drivers/media/rc/imon.c | 5 + 1 file changed, 5 inse

[RFT v2] [media] siano: FIX use-after-free in worker_thread

2017-09-27 Thread Arvind Yadav
Call flush_work() on failure and disconnect. Work initialize and schedule in smsusb_onresponse(). it should be freed in smsusb_stop_streaming(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov "usb/media/smsusb: use-after-free in wo

[PATCH 0/4] staging: rtlwifi: pr_err() strings should end with newlines

2017-10-03 Thread Arvind Yadav
pr_err() messages should end with a new-line to avoid other messages being concatenated. Arvind Yadav (4): [PATCH 1/4] staging: gs_fpgaboot: pr_err() strings should end with newlines [PATCH 2/4] staging: media: davinci_vpfe: pr_err() strings should end with newlines [PATCH 3/4] staging

[PATCH 2/4] staging: media: davinci_vpfe: pr_err() strings should end with newlines

2017-10-03 Thread Arvind Yadav
pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/davinci_vpfe/dm365_ipipe_hw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging

[PATCH 3/4] staging: bcm2835-camera: pr_err() strings should end with newlines

2017-10-03 Thread Arvind Yadav
pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s

[PATCH 1/4] staging: gs_fpgaboot: pr_err() strings should end with newlines

2017-10-03 Thread Arvind Yadav
pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gs_fp

[PATCH 4/4] staging: rtlwifi: pr_err() strings should end with newlines

2017-10-03 Thread Arvind Yadav
pr_err() messages should end with a new-line to avoid other messages being concatenated. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/rtlwifi/halmac/halmac_88xx/halmac_func_88xx.c | 6 +++--- drivers/staging/rtlwifi/rtl8822be/phy.c | 4 +

[PATCH] staging: atomisp: constify v4l2_subdev_sensor_ops

2017-08-26 Thread Arvind Yadav
v4l2_subdev_sensor_ops are not supposed to change at runtime. v4l2_subdev_sensor_ops are working with const 'sensor' field of sturct v4l2_subdev_ops. So mark the non-const v4l2_subdev_sensor_ops structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging

[PATCH 3/4] [media] platform: constify videobuf_queue_ops structures

2017-08-22 Thread Arvind Yadav
videobuf_queue_ops are not supposed to change at runtime. All functions working with videobuf_queue_ops provided by work with const videobuf_queue_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/d

[PATCH 0/4] constify videobuf_queue_ops structures

2017-08-22 Thread Arvind Yadav
videobuf_queue_ops are not supposed to change at runtime. All functions working with videobuf_queue_ops provided by work with const videobuf_queue_ops. So mark the non-const structs as const. Arvind Yadav (4): [PATCH 1/4] [media] saa7146: constify videobuf_queue_ops structures [PATCH 2/4

[PATCH 1/4] [media] saa7146: constify videobuf_queue_ops structures

2017-08-22 Thread Arvind Yadav
videobuf_queue_ops are not supposed to change at runtime. All functions working with videobuf_queue_ops provided by work with const videobuf_queue_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/common/saa7146/saa7146

[PATCH 2/4] [media] pci: constify videobuf_queue_ops structures

2017-08-22 Thread Arvind Yadav
videobuf_queue_ops are not supposed to change at runtime. All functions working with videobuf_queue_ops provided by work with const videobuf_queue_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/bt8xx/bttv-driver

[PATCH 4/4] [media] usb: constify videobuf_queue_ops structures

2017-08-22 Thread Arvind Yadav
videobuf_queue_ops are not supposed to change at runtime. All functions working with videobuf_queue_ops provided by work with const videobuf_queue_ops. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/usb/cx231xx/cx231xx

[PATCH 6/6] [media] ths8200: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/ths8200.c | 2 +- 1 file chan

[PATCH 5/6] [media] tc358743: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/tc358743.c | 2 +- 1 file chan

[PATCH 4/6] [media] saa7127: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/saa7127.c | 2 +- 1 file chan

[PATCH 1/6] [media] ad9389b: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/ad9389b.c | 2 +- 1 file chan

[PATCH 3/6] [media] adv7842: constify i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/i2c/adv7842.c | 2 +- 1 file chan

[PATCH 0/6] constify media i2c_device_id

2017-08-19 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Arvind Yadav (6): [PATCH 1/6] [media] ad9389b: constify i2c_device_id [PATCH 2/6] [media] adv7511: constify

[PATCH] Staging: atomisp: constify driver_attribute

2017-08-31 Thread Arvind Yadav
driver_attribute are not supposed to change at runtime. Functions driver_create_file/driver_remove_file are working with const driver_attribute. So mark the non-const structs as const. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/staging/media/atomisp/pci/at

[RFT] [media] siano: FIX use-after-free in worker_thread

2017-09-27 Thread Arvind Yadav
. It's allocate first in siano_media_device_register() and it should be freed last in smsusb_disconnect(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov "usb/media/smsusb: use-after-free in worker_thread". drivers/media/usb/sia

[RFT] media: dvb_frontend: Fix use-after-free in __dvb_frontend_free

2017-10-24 Thread Arvind Yadav
Here, dvb_free_device will free dvb_device. dvb_frontend_invoke_release is using dvb_device after free. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov (usb/media/dtt200u: use-after-free in __dvb_frontend_free). drivers/media/dv

[PATCH] [media] winbond-cir: Fix pnp_irq's error checking for wbcir_probe

2017-11-15 Thread Arvind Yadav
The pnp_irq() function returns -1 if an error occurs. pnp_irq() error checking for zero is not correct. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-c

[RFT] [media] em28xx: Fix use-after-free in v4l2_fh_init

2017-11-08 Thread Arvind Yadav
Here, em28xx_free_v4l2 is release "v4l2->dev->v4l2" Which is allready release by em28xx_v4l2_init. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- This bug report by Andrey Konovalov "net/media/em28xx: use-after-free in v4l2_fh_init" drivers/med

[PATCH 0/2] Use gpio_is_valid()

2018-04-27 Thread Arvind Yadav
Replace the manual validity checks for the GPIO with the gpio_is_valid(). Arvind Yadav (2): [PATCH 1/2] [media] platform: Use gpio_is_valid() [PATCH 2/2] [media] sta2x11: Use gpio_is_valid() and remove unnecessary check drivers/media/pci/sta2x11/sta2x11_vip.c | 31

[PATCH 2/2] [media] sta2x11: Use gpio_is_valid() and remove unnecessary check

2018-04-27 Thread Arvind Yadav
vip_gpio_reserve() is also checking for valid gpio pin. So removed extra error checking for gpio 'reset_pin'. Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/pci/sta2x11/sta2x11_vip.c | 31 +++ 1 file changed, 15 insertions(+), 16 del

[PATCH 1/2] [media] platform: Use gpio_is_valid()

2018-04-27 Thread Arvind Yadav
Replace the manual validity checks for the GPIO with the gpio_is_valid(). Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- drivers/media/platform/via-camera.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/via-camera.c b/drivers

[RFT] media: hdpvr: Fix Double kfree() error

2018-03-20 Thread Arvind Yadav
4l2_device_release => hdpvr_device_release (CALLBACK) => kfree(dev) error_free_dev: kfree(dev) Signed-off-by: Arvind Yadav <arvind.yadav...@gmail.com> --- reported by: Dan Carpenter<dan.carpen...@oracle.com> drivers/media/usb/hdpvr/hdpvr-core.c | 1 +