[PATCH] staging: exfat: remove fs_func struct.

2020-01-16 Thread T.Kohada
Remove 'fs_func struct' and change indirect calls to direct calls. The following issues are described in exfat's TODO. > Create helper function for exfat_set_entry_time () and > exfat_set_entry_type () because it's sort of ugly to be calling the same > functionn directly and other code calling th

[PATCH v2] Staging: kpc2000: Remove warning: "dubious: x | !y" detected by sparse

2020-01-16 Thread Felipe Cardoso Resende
Sparse complains about "dubious: x | !y". This patch adds some macros to make it clear if a flag will be enabled or not so Sparse stops complaining about dubious code construct. Signed-off-by: Felipe Cardoso Resende --- Changes in v2: - Add "Staging: kpc2000" to Subject - Fix commit message to

Re: [PATCH v2 2/4] media: cedrus: hevc: Add support for scaling matrix

2020-01-16 Thread Jernej Škrabec
Dne sreda, 08. januar 2020 ob 15:46:50 CET je Hans Verkuil napisal(a): > On 1/7/20 6:10 PM, Jernej Škrabec wrote: > > Hi! > > > > Dne torek, 07. januar 2020 ob 16:01:16 CET je Hans Verkuil napisal(a): > >> On 12/13/19 5:04 PM, Jernej Skrabec wrote: > >>> HEVC frames may use scaling list feature. A

[staging:staging-linus] BUILD SUCCESS 9fea3a40f6b07de977a2783270c8c3bc82544d45

2020-01-16 Thread kbuild test robot
fuloong2e_defconfig mips malta_kvm_defconfig x86_64 randconfig-b001-20200116 x86_64 randconfig-b002-20200116 x86_64 randconfig-b003-20200116 i386 randconfig-b001-20200116 i386 randconfig-b002-20200116 i386

[staging:staging-testing] BUILD SUCCESS 793769120b109e3cec536bb8f5b6f9ef959c315f

2020-01-16 Thread kbuild test robot
malta_kvm_defconfig x86_64 randconfig-b001-20200116 x86_64 randconfig-b002-20200116 x86_64 randconfig-b003-20200116 i386 randconfig-b001-20200116 i386 randconfig-b002-20200116 i386 randconfig

[PATCH AUTOSEL 4.4 005/174] staging: comedi: ni_mio_common: protect register write overflow

2020-01-16 Thread Sasha Levin
From: "Spencer E. Olson" [ Upstream commit 1cbca5852d6c16e85a21487a15d211195aacd4a1 ] Fixes two problems introduced as early as commit 03aef4b6dc12 ("Staging: comedi: add ni_mio_common code"): (1) Ensures that the last four bits of NISTC_RTSI_TRIGB_OUT_REG register is not unduly overwritten

[PATCH AUTOSEL 4.9 193/251] staging: greybus: light: fix a couple double frees

2020-01-16 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 329101244f214952606359d254ae883b7109e1a5 ] The problem is in gb_lights_request_handler(). If we get a request to change the config then we release the light with gb_lights_light_release() and re-allocated it. However, if the allocation fails part way throu

[PATCH AUTOSEL 4.9 056/251] staging: most: cdev: add missing check for cdev_add failure

2020-01-16 Thread Sasha Levin
From: Colin Ian King [ Upstream commit 5ae890780e1b4d08f2c0c5d4ea96fc3928fc0ee9 ] Currently the call to cdev_add is missing a check for failure. Fix this by checking for failure and exiting via a new error path that ensures the allocated comp_channel struct is kfree'd. Detected by CoverityScan,

[PATCH AUTOSEL 4.9 007/251] staging: comedi: ni_mio_common: protect register write overflow

2020-01-16 Thread Sasha Levin
From: "Spencer E. Olson" [ Upstream commit 1cbca5852d6c16e85a21487a15d211195aacd4a1 ] Fixes two problems introduced as early as commit 03aef4b6dc12 ("Staging: comedi: add ni_mio_common code"): (1) Ensures that the last four bits of NISTC_RTSI_TRIGB_OUT_REG register is not unduly overwritten

[PATCH AUTOSEL 4.14 283/371] staging: greybus: light: fix a couple double frees

2020-01-16 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 329101244f214952606359d254ae883b7109e1a5 ] The problem is in gb_lights_request_handler(). If we get a request to change the config then we release the light with gb_lights_light_release() and re-allocated it. However, if the allocation fails part way throu

[PATCH AUTOSEL 4.14 115/371] staging: rtlwifi: Use proper enum for return in halmac_parse_psd_data_88xx

2020-01-16 Thread Sasha Levin
From: Nathan Chancellor [ Upstream commit e8edc32d70a4e09160835792eb5d1af71a0eec14 ] Clang warns: drivers/staging/rtlwifi/halmac/halmac_88xx/halmac_func_88xx.c:2472:11: warning: implicit conversion from enumeration type 'enum halmac_cmd_process_status' to different enumeration type 'enum halmac

[PATCH AUTOSEL 4.14 104/371] staging: r8822be: check kzalloc return or bail

2020-01-16 Thread Sasha Levin
From: Nicholas Mc Guire [ Upstream commit e4b08e16b7d9d030b6475ef48f94d734a39f3c81 ] The kzalloc() in halmac_parse_psd_data_88xx() can fail and return NULL so check the psd_set->data after allocation and if allocation failed return HALMAC_CMD_PROCESS_ERROR. Signed-off-by: Nicholas Mc Guire Fix

[PATCH AUTOSEL 4.14 081/371] staging: most: cdev: add missing check for cdev_add failure

2020-01-16 Thread Sasha Levin
From: Colin Ian King [ Upstream commit 5ae890780e1b4d08f2c0c5d4ea96fc3928fc0ee9 ] Currently the call to cdev_add is missing a check for failure. Fix this by checking for failure and exiting via a new error path that ensures the allocated comp_channel struct is kfree'd. Detected by CoverityScan,

[PATCH AUTOSEL 4.14 011/371] staging: comedi: ni_mio_common: protect register write overflow

2020-01-16 Thread Sasha Levin
From: "Spencer E. Olson" [ Upstream commit 1cbca5852d6c16e85a21487a15d211195aacd4a1 ] Fixes two problems introduced as early as commit 03aef4b6dc12 ("Staging: comedi: add ni_mio_common code"): (1) Ensures that the last four bits of NISTC_RTSI_TRIGB_OUT_REG register is not unduly overwritten

[PATCH AUTOSEL 4.14 022/371] staging: bcm2835-camera: Abort probe if there is no camera

2020-01-16 Thread Sasha Levin
From: Stefan Wahren [ Upstream commit 7566f39dfdc11f8a97d5810c6e6295a88f97ef91 ] Abort the probing of the camera driver in case there isn't a camera actually connected to the Raspberry Pi. This solution also avoids a NULL ptr dereference of mmal instance on driver unload. Fixes: 7b3ad5abf027 ("

[PATCH AUTOSEL 4.19 518/671] staging: greybus: light: fix a couple double frees

2020-01-16 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 329101244f214952606359d254ae883b7109e1a5 ] The problem is in gb_lights_request_handler(). If we get a request to change the config then we release the light with gb_lights_light_release() and re-allocated it. However, if the allocation fails part way throu

[PATCH AUTOSEL 4.19 387/671] media: Staging: media: Release the correct resource in an error handling path

2020-01-16 Thread Sasha Levin
From: Christophe JAILLET [ Upstream commit 3b6471c7becd06325eb5e701cc2602b2edbbc7b6 ] 'res' is reassigned several times in the function and if we 'goto error_unmap', its value is not the returned value of 'request_mem_region()' anymore. Introduce a new 'struct resource *' variable (i.e. res2) t

[PATCH AUTOSEL 4.19 296/671] staging: android: vsoc: fix copy_from_user overrun

2020-01-16 Thread Sasha Levin
From: Vincent Stehlé [ Upstream commit 060ea4271a82270be6d44e8e9aefe8f155fb5626 ] The `np->permission' structure is smaller than the `np' structure but sizeof(*np) worth of data is copied in there. Fix the size passed to copy_from_user() to avoid overrun. Fixes: 3d2ec9dcd553 ("staging: Android:

[PATCH AUTOSEL 4.19 193/671] staging: r8822be: check kzalloc return or bail

2020-01-16 Thread Sasha Levin
From: Nicholas Mc Guire [ Upstream commit e4b08e16b7d9d030b6475ef48f94d734a39f3c81 ] The kzalloc() in halmac_parse_psd_data_88xx() can fail and return NULL so check the psd_set->data after allocation and if allocation failed return HALMAC_CMD_PROCESS_ERROR. Signed-off-by: Nicholas Mc Guire Fix

[PATCH AUTOSEL 4.19 209/671] staging: rtlwifi: Use proper enum for return in halmac_parse_psd_data_88xx

2020-01-16 Thread Sasha Levin
From: Nathan Chancellor [ Upstream commit e8edc32d70a4e09160835792eb5d1af71a0eec14 ] Clang warns: drivers/staging/rtlwifi/halmac/halmac_88xx/halmac_func_88xx.c:2472:11: warning: implicit conversion from enumeration type 'enum halmac_cmd_process_status' to different enumeration type 'enum halmac

[PATCH AUTOSEL 4.19 154/671] staging: most: cdev: add missing check for cdev_add failure

2020-01-16 Thread Sasha Levin
From: Colin Ian King [ Upstream commit 5ae890780e1b4d08f2c0c5d4ea96fc3928fc0ee9 ] Currently the call to cdev_add is missing a check for failure. Fix this by checking for failure and exiting via a new error path that ensures the allocated comp_channel struct is kfree'd. Detected by CoverityScan,

[PATCH AUTOSEL 4.19 065/671] staging: bcm2835-camera: fix module autoloading

2020-01-16 Thread Sasha Levin
From: Stefan Wahren [ Upstream commit 3a2c20024a2b47adbf514e7f3ab79342739c3926 ] In order to make the module bcm2835-camera load automatically, we need to add a module alias. Fixes: 4bebb0312ea9 ("staging/bcm2835-camera: Set ourselves up as a platform driver.") Signed-off-by: Stefan Wahren Re

[PATCH AUTOSEL 4.19 064/671] staging: bcm2835-camera: Abort probe if there is no camera

2020-01-16 Thread Sasha Levin
From: Stefan Wahren [ Upstream commit 7566f39dfdc11f8a97d5810c6e6295a88f97ef91 ] Abort the probing of the camera driver in case there isn't a camera actually connected to the Raspberry Pi. This solution also avoids a NULL ptr dereference of mmal instance on driver unload. Fixes: 7b3ad5abf027 ("

[PATCH AUTOSEL 4.19 031/671] staging: comedi: ni_mio_common: protect register write overflow

2020-01-16 Thread Sasha Levin
From: "Spencer E. Olson" [ Upstream commit 1cbca5852d6c16e85a21487a15d211195aacd4a1 ] Fixes two problems introduced as early as commit 03aef4b6dc12 ("Staging: comedi: add ni_mio_common code"): (1) Ensures that the last four bits of NISTC_RTSI_TRIGB_OUT_REG register is not unduly overwritten

[PATCH AUTOSEL 5.4 125/205] media: hantro: Set H264 FIELDPIC_FLAG_E flag correctly

2020-01-16 Thread Sasha Levin
From: Jonas Karlman [ Upstream commit a2cbf80a842add9663522bf898cf13cb2ac4e423 ] The FIELDPIC_FLAG_E bit should be set when field_pic_flag exists in stream, it is currently set based on field_pic_flag of current frame. The PIC_FIELDMODE_E bit is correctly set based on the field_pic_flag. Fix th

[PATCH AUTOSEL 5.4 123/205] media: cedrus: Use correct H264 8x8 scaling list

2020-01-16 Thread Sasha Levin
From: Jonas Karlman [ Upstream commit a6b8feae7c88343212686120740cf7551dd16e08 ] Documentation now defines the expected order of scaling lists, change to use correct indices. Fixes: 6eb9b758e307 ("media: cedrus: Add H264 decoding support") Signed-off-by: Jonas Karlman Reviewed-by: Philipp Zabe

[driver-core:debugfs_remove_return_value] BUILD SUCCESS 25ea9282415f494178fe1b0130e3bd5621278554

2020-01-16 Thread kbuild test robot
shmobile_defconfig arm sunxi_defconfig arm64allmodconfig arm64 allnoconfig arm64allyesconfig arm64 defconfig alpharandconfig-a001-20200116 m68k

[driver-core:driver-core-testing] BUILD SUCCESS a37f4958f7b63d2b3cd17a76151fdfc29ce1da5f

2020-01-16 Thread kbuild test robot
allnoconfig arm64allyesconfig arm64 defconfig alpharandconfig-a001-20200116 m68k randconfig-a001-20200116 mips randconfig-a001-20200116 nds32randconfig-a001-20200116 parisc

[driver-core:debugfs_cleanup] BUILD SUCCESS e72e05ef0e56cf2fe9c81c8b0bc679fd16b80688

2020-01-16 Thread kbuild test robot
defconfig arc randconfig-a001-20200116 arm randconfig-a001-20200116 arm64randconfig-a001-20200116 ia64 randconfig-a001-20200116 powerpc randconfig-a001-20200116 sparcrandconfig-a001-20200116 csky

Re: [PATCH][next] staging: wilc1000: remove redundant assignment to variable result

2020-01-16 Thread Claudiu.Beznea
On 14.01.2020 19:27, Colin King wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > From: Colin Ian King > > The variable result is being initialized with a value that > is never read and is being re-assigned later on. The assignment > is