[PATCH net-next 15/22] net: hamradio: fix return type of ndo_start_xmit function

2018-09-20 Thread YueHaibing
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t', which is a typedef for an enum type, so make sure the implementation in this driver has returns 'netdev_tx_t' value, and change the function return type to netdev_tx_t. Found by coccinelle. Signed-off-by: YueHaibing

[PATCH -next] x86/hyper-v: Remove unused including

2018-09-23 Thread YueHaibing
Remove including that don't need it. Signed-off-by: YueHaibing --- arch/x86/hyperv/hv_apic.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/hyperv/hv_apic.c b/arch/x86/hyperv/hv_apic.c index 2c43e30..8eb6fbee 100644 --- a/arch/x86/hyperv/hv_apic.c +++ b/arch/x86/hyperv/hv_apic.c

[PATCH -next] ring-buffer: Fix ring buffer size in rb_write_something

2018-09-23 Thread YueHaibing
'cnt' should be used to calculate ring buffer size rather than data->cnt Signed-off-by: YueHaibing --- kernel/trace/ring_buffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c index 65bd461..53538c3 100

[PATCH -next] VMCI: remove set but not used variable 'cid'

2018-09-27 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: drivers/misc/vmw_vmci/vmci_host.c: In function 'vmci_host_do_alloc_queuepair': drivers/misc/vmw_vmci/vmci_host.c:450:6: warning: variable 'cid' set but not used [-Wunused-but-set-variable] u32 cid; ^ Signed-off-by: YueHaibing --- drivers

[PATCH -next] mm/hmm: remove set but not used variable 'devmem'

2018-11-30 Thread YueHaibing
reate() with devm_memremap_pages()") Signed-off-by: YueHaibing --- mm/hmm.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/mm/hmm.c b/mm/hmm.c index 50fbaf8..361f370 100644 --- a/mm/hmm.c +++ b/mm/hmm.c @@ -992,9 +992,6 @@ static void hmm_devmem_ref_exit(void *data) static void hmm_devmem_ref_kill(struct

[PATCH -next] cpuset: Remove set but not used variable 'cs'

2018-11-30 Thread YueHaibing
multi-destination migration from subtree_control enabling") Signed-off-by: YueHaibing --- kernel/cgroup/cpuset.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/kernel/cgroup/cpuset.c b/kernel/cgroup/cpuset.c index e9f6906..48cfe55 100644 --- a/kernel/cgroup/cpuset.c +++ b/kernel/cgrou

[PATCH -next] ocfs2: Use kmemdup rather than duplicating its implementation

2018-11-28 Thread YueHaibing
Use kmemdup rather than duplicating its implementation Signed-off-by: YueHaibing --- fs/ocfs2/localalloc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/ocfs2/localalloc.c b/fs/ocfs2/localalloc.c index 308f05b..3572204 100644 --- a/fs/ocfs2/localalloc.c +++ b/fs

[PATCH -next] coresight: stm: remove set but not used variable 'res_size'

2018-12-01 Thread YueHaibing
237483aa5cf4 ("coresight: stm: adding driver for CoreSight STM component") Signed-off-by: YueHaibing --- drivers/hwtracing/coresight/coresight-stm.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/hwtracing/coresight/coresight-stm.c b/drivers/hwtracing

Re: [PATCH -next] PNP: remove duplicated include from quirks.c

2018-11-30 Thread YueHaibing
Pls ignore this, the maintainer' email is wrong. On 2018/11/30 19:11, YueHaibing wrote: > Remove duplicated include. > > Signed-off-by: YueHaibing > --- > drivers/pnp/quirks.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/pnp/quirks.c b/drivers/pnp/

[PATCH] iwlwifi: mvm: remove duplicated include from ops.c

2018-11-30 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/net/wireless/intel/iwlwifi/mvm/ops.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/ops.c b/drivers/net/wireless/intel/iwlwifi/mvm/ops.c index a31dc99..b85c877 100644 --- a/drivers/net

[PATCH -next] bus: fsl-mc: remove duplicated include files

2018-11-30 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/bus/fsl-mc/dpbp.c | 1 - drivers/bus/fsl-mc/dpcon.c | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/bus/fsl-mc/dpbp.c b/drivers/bus/fsl-mc/dpbp.c index 17e3c5d..9003cd3 100644 --- a/drivers/bus/fsl-mc/dpbp.c +++ b

[PATCH -next] PNP: remove duplicated include from quirks.c

2018-11-30 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/pnp/quirks.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c index 803666a..280943d 100644 --- a/drivers/pnp/quirks.c +++ b/drivers/pnp/quirks.c @@ -226,8 +226,6 @@ static void

Re: [PATCH] iwlwifi: mvm: remove duplicated include from ops.c

2018-11-30 Thread YueHaibing
Sorry, pls ignore this On 2018/11/30 19:13, YueHaibing wrote: > Remove duplicated include. > > Signed-off-by: YueHaibing > --- > drivers/net/wireless/intel/iwlwifi/mvm/ops.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/net/wireless/intel/iwlwifi/mv

[PATCH -next] PNP: remove duplicated include from quirks.c

2018-11-30 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/pnp/quirks.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/pnp/quirks.c b/drivers/pnp/quirks.c index 803666a..280943d 100644 --- a/drivers/pnp/quirks.c +++ b/drivers/pnp/quirks.c @@ -226,8 +226,6 @@ static void

[PATCH -next] remoteproc: remove duplicated include from qcom_sysmon.c

2018-11-30 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- drivers/remoteproc/qcom_sysmon.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/remoteproc/qcom_sysmon.c b/drivers/remoteproc/qcom_sysmon.c index e976a60..603b813 100644 --- a/drivers/remoteproc/qcom_sysmon.c +++ b/drivers

[PATCH -next] platform/x86: intel-ips: remove unnecessary checks in ips_debugfs_init

2018-11-26 Thread YueHaibing
/x86/intel_ips.c:1328 ips_debugfs_init() warn: passing zero to 'PTR_ERR' Signed-off-by: YueHaibing --- drivers/platform/x86/intel_ips.c | 21 +++-- 1 file changed, 3 insertions(+), 18 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c

[PATCH -next] platform/x86: intel-ips: fix 'passing zero to ERR_PTR()' warning

2018-11-26 Thread YueHaibing
is enabled, which should not passing to ERR_PTR. Fixes: aa7ffc01d254 ("x86 platform driver: intelligent power sharing driver") Signed-off-by: YueHaibing --- drivers/platform/x86/intel_ips.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/platform/x86/int

Re: [PATCH -next] platform/x86: intel-ips: fix 'passing zero to ERR_PTR()' warning

2018-11-26 Thread YueHaibing
On 2018/11/26 21:06, YueHaibing wrote: > Fix a static code checker warning: > drivers/platform/x86/intel_ips.c:1314 > ips_debugfs_init() warn: passing zero to 'PTR_ERR' > drivers/platform/x86/intel_ips.c:1328 > ips_debugfs_init() warn: passing zero to 'PTR_ERR' >

[PATCH v2 -next] platform/x86: intel-ips: fix 'passing zero to PTR_ERR()' warning

2018-11-26 Thread YueHaibing
is enabled, which should not passing to PTR_ERR. Fixes: aa7ffc01d254 ("x86 platform driver: intelligent power sharing driver") Signed-off-by: YueHaibing --- v2:Fix typo in patch log and title --- drivers/platform/x86/intel_ips.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletion

[PATCH -next] exportfs: fix 'passing zero to ERR_PTR()' warning

2018-11-18 Thread YueHaibing
Fix a static code checker warning: fs/exportfs/expfs.c:171 reconnect_one() warn: passing zero to 'ERR_PTR' The error path for lookup_one_len_unlocked failure should set err to PTR_ERR. Fixes: bbf7a8a3562f ("exportfs: move most of reconnect_path to helper function") Signed-off-by:

[PATCH net-next] e100: Fix passing zero to 'PTR_ERR' warning in e100_load_ucode_wait

2018-11-19 Thread YueHaibing
Fix a static code checker warning: drivers/net/ethernet/intel/e100.c:1349 e100_load_ucode_wait() warn: passing zero to 'PTR_ERR' Signed-off-by: YueHaibing --- drivers/net/ethernet/intel/e100.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel

[PATCH -next] Smack: remove set but not used variable 'root_inode'

2018-09-13 Thread YueHaibing
Fixes gcc '-Wunused-but-set-variable' warning: security/smack/smackfs.c: In function 'smk_fill_super': security/smack/smackfs.c:2856:16: warning: variable 'root_inode' set but not used [-Wunused-but-set-variable] Signed-off-by: YueHaibing --- security/smack/smackfs.c | 3 --- 1 file changed

[PATCH -next] RDMA: Remove duplicated include from ib_addr.h

2018-09-13 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- include/rdma/ib_addr.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/rdma/ib_addr.h b/include/rdma/ib_addr.h index 77c7908..8964e84 100644 --- a/include/rdma/ib_addr.h +++ b/include/rdma/ib_addr.h @@ -46,7 +46,6 @@ #include

[PATCH -next] iio: potentiometer: mcp4531: merge calls to of_match_device and of_device_get_match_data

2018-09-15 Thread YueHaibing
Drop call to of_match_device, which is subsumed by the subsequent call to of_device_get_match_data. The code becomes simpler, and a temporary variable can be dropped. Found by coccinelle. Signed-off-by: YueHaibing --- drivers/iio/potentiometer/mcp4531.c | 7 ++- 1 file changed, 2

[PATCH -next] iio: potentiometer: mcp4018: merge calls to of_match_device and of_device_get_match_data

2018-09-15 Thread YueHaibing
Drop call to of_match_device, which is subsumed by the subsequent call to of_device_get_match_data. The code becomes simpler, and a temporary variable can be dropped. Found by coccinelle. Signed-off-by: YueHaibing --- drivers/iio/potentiometer/mcp4018.c | 7 ++- 1 file changed, 2

Re: [PATCH v2] ARM: dma-mapping: always clear allocated buffers

2018-09-15 Thread YueHaibing
Ping. On 2018/7/25 15:07, YueHaibing wrote: > Sean Wang reported dma_zalloc_coherent doesn't work as expect on his > armv7,the allocated mem is not zeroed.The reason is __alloc_from_pool > doesn't honor __GFP_ZERO. > > As commit 6829e274a623 ("arm64: dma-mapping: always cle

[PATCH -next] iio: potentiometer: max5481: merge calls to of_match_device and of_device_get_match_data

2018-09-15 Thread YueHaibing
Drop call to of_match_device, which is subsumed by the subsequent call to of_device_get_match_data. The code becomes simpler, and a temporary variable can be dropped. Found by coccinelle. Signed-off-by: YueHaibing --- drivers/iio/potentiometer/max5481.c | 7 ++- 1 file changed, 2

[PATCH] xfs: remove duplicated include from alloc.c

2018-09-17 Thread YueHaibing
Remove duplicated include xfs_alloc.h Signed-off-by: YueHaibing --- fs/xfs/scrub/alloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/xfs/scrub/alloc.c b/fs/xfs/scrub/alloc.c index 036b5c7..376bcb5 100644 --- a/fs/xfs/scrub/alloc.c +++ b/fs/xfs/scrub/alloc.c @@ -17,7 +17,6 @@ #include

[PATCH -next] fs/iomap.c: remove duplicated include

2018-09-17 Thread YueHaibing
Remove duplicated include linux/swap.h Signed-off-by: YueHaibing --- fs/iomap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/iomap.c b/fs/iomap.c index 9fb78e5..a5613e5 100644 --- a/fs/iomap.c +++ b/fs/iomap.c @@ -30,7 +30,6 @@ #include #include #include -#include #include

[PATCH -next] ASoC: qdsp6: q6asm-dai: remove duplicated include from q6asm-dai.c

2018-09-13 Thread YueHaibing
Remove duplicated include. Signed-off-by: YueHaibing --- sound/soc/qcom/qdsp6/q6asm-dai.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/soc/qcom/qdsp6/q6asm-dai.c b/sound/soc/qcom/qdsp6/q6asm-dai.c index c75fab3..c3806d7 100644 --- a/sound/soc/qcom/qdsp6/q6asm-dai.c +++ b/sound/soc

[PATCH net-next] net: toshiba: fix return type of ndo_start_xmit function

2018-09-19 Thread YueHaibing
The method ndo_start_xmit() is defined as returning an 'netdev_tx_t', which is a typedef for an enum type, so make sure the implementation in this driver has returns 'netdev_tx_t' value, and change the function return type to netdev_tx_t. Found by coccinelle. Signed-off-by: YueHaibing

Re: [PATCH 6/6] net: sun: fix return type of ndo_start_xmit function

2018-09-19 Thread YueHaibing
Sorry for wrongly title, will repost. On 2018/9/19 19:07, YueHaibing wrote: > The method ndo_start_xmit() is defined as returning an 'netdev_tx_t', > which is a typedef for an enum type, but the implementation in this > driver returns an 'int'. > > Found by coccinelle.

[PATCH -next] ipc/shm: Use ERR_CAST to return error code

2018-09-19 Thread YueHaibing
Fix gcc warning: ipc/shm.c: In function 'shm_lock' ipc/shm.c:210:2: note: randstruct: casting between randomized structure pointer types (ssa): 'struct shmid_kernel' and 'struct kern_ipc_perm' Signed-off-by: YueHaibing --- ipc/shm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH net-next] net/smc: cast sizeof to int for comparison

2018-09-16 Thread YueHaibing
On 2018/9/15 19:35, Andreas Schwab wrote: > On Sep 15 2018, YueHaibing wrote: > >> Comparing an int to a size, which is unsigned, causes the int to become >> unsigned, giving the wrong result. kernel_sendmsg can return a negative >> error code. >> >> Signed-of

[PATCH -next] mm: swap: remove duplicated include from swap.c

2018-09-17 Thread YueHaibing
Remove duplicated include linux/memremap.h Signed-off-by: YueHaibing --- mm/swap.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/swap.c b/mm/swap.c index 26fc9b5..87a54c8 100644 --- a/mm/swap.c +++ b/mm/swap.c @@ -29,7 +29,6 @@ #include #include #include -#include #include

[tip:perf/core] perf bpf: Fix NULL return handling in bpf__prepare_load()

2018-05-16 Thread tip-bot for YueHaibing
Commit-ID: 7a36a287de9fbb1ba906e70573d3f2315f7fd609 Gitweb: https://git.kernel.org/tip/7a36a287de9fbb1ba906e70573d3f2315f7fd609 Author: YueHaibing <yuehaib...@huawei.com> AuthorDate: Fri, 11 May 2018 19:21:42 +0800 Committer: Arnaldo Carvalho de Melo <a...@redhat.com> CommitD

[tip:perf/urgent] perf bpf: Fix NULL return handling in bpf__prepare_load()

2018-05-31 Thread tip-bot for YueHaibing
Commit-ID: ab4e32ff5aa797eaea551dbb67946e2fcb56cc7e Gitweb: https://git.kernel.org/tip/ab4e32ff5aa797eaea551dbb67946e2fcb56cc7e Author: YueHaibing AuthorDate: Fri, 11 May 2018 19:21:42 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 30 May 2018 15:35:31 -0300 perf bpf: Fix

[tip:x86/cleanups] x86/platform/pcspeaker: Use PTR_ERR_OR_ZERO() to fix ptr_ret.cocci warning

2018-07-24 Thread tip-bot for YueHaibing
Commit-ID: 2397134ce2d73dcf1d4846579f594c9f3880b2ec Gitweb: https://git.kernel.org/tip/2397134ce2d73dcf1d4846579f594c9f3880b2ec Author: YueHaibing AuthorDate: Fri, 20 Jul 2018 15:32:13 +0800 Committer: Ingo Molnar CommitDate: Tue, 24 Jul 2018 09:46:42 +0200 x86/platform/pcspeaker: Use

[tip:perf/core] x86/cpu: Drop pointless static qualifier in punit_dev_state_show()

2018-10-12 Thread tip-bot for YueHaibing
Commit-ID: f703fd374ef8fb06e46713b326d255e20d6278ad Gitweb: https://git.kernel.org/tip/f703fd374ef8fb06e46713b326d255e20d6278ad Author: YueHaibing AuthorDate: Fri, 12 Oct 2018 10:40:22 + Committer: Ingo Molnar CommitDate: Fri, 12 Oct 2018 12:48:44 +0200 x86/cpu: Drop pointless

[tip:x86/hyperv] x86/hyperv: Remove unused include

2018-09-27 Thread tip-bot for YueHaibing
Commit-ID: 5140a6f471137205687428b0b8f12f7187bffd18 Gitweb: https://git.kernel.org/tip/5140a6f471137205687428b0b8f12f7187bffd18 Author: YueHaibing AuthorDate: Sun, 23 Sep 2018 08:20:22 + Committer: Thomas Gleixner CommitDate: Thu, 27 Sep 2018 21:21:00 +0200 x86/hyperv: Remove

<    1   2