As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Christoph Hellwig
Cc: Marek Szyprowski
Cc: Robin Murphy
Reviewed-by: Sergey Senozhatsky
Signed-off-
ewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
sound/soc/samsung/s3c-i2s-v2.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/soc/samsung/s3c-i2s-v2.c b/sound/soc/samsung/s3c-i2s-v2.c
index 7e196b599be1..593be1b668d6 100644
--- a/sound/soc/samsung/s3c-i2s-v
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Karsten Keil
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
drivers/isdn/hardware/mISDN/a
Signed-off-by: Kefeng Wang
---
drivers/platform/x86/eeepc-laptop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/eeepc-laptop.c
b/drivers/platform/x86/eeepc-laptop.c
index f3f74a9c109e..776868d5e458 100644
--- a/drivers/platform/x86/eeepc-laptop.c
+++ b/d
With all pr_warning are removed, saftely drop pr_warning definition.
Cc: Petr Mladek
Cc: Sergey Senozhatsky
Cc: Steven Rostedt
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
include/linux/printk.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/include
Signed-off-by: Kefeng Wang
---
arch/arm64/kernel/hw_breakpoint.c | 8
arch/arm64/kernel/smp.c | 11 +--
2 files changed, 9 insertions(+), 10 deletions(-)
diff --git a/arch/arm64/kernel/hw_breakpoint.c
b/arch/arm64/kernel/hw_breakpoint.c
index 38ee1514cd9c..0b727edf4104 10
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Herbert Xu
Cc: "David S. Miller"
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Steven Rostedt
Cc: Ingo Molnar
Acked-by: Steven Rostedt (VMware)
Reviewed-by: Sergey Senozhatsky
Signed-off-
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Rob Herring
Cc: Frank Rowand
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
driver
For now, all pr_warning are removed, delete pr_warning check in
checkpatch.
Cc: Andy Whitcroft
Cc: Joe Perches
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
scripts/checkpatch.pl | 9 -
1 file changed, 9 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
tools/lib/api/debug-internal.h | 4 ++--
tools/lib/api/debug.c | 4 ++--
tools/lib/api/fs/fs.c | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/tools/lib/api/debug-internal.h b/tools/lib/api/debug
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Benjamin Herrenschmidt
Cc: linuxppc-...@lists.ozlabs.org
Reviewed-by: Sergey Senozhatsky
Signed-off-
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Andrew Morton
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
lib/cpu_rmap.c | 2 +-
1
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: "Rafael J. Wysocki"
Cc: Len Brown
Cc: James Morse
Reviewed-by: Sergey Senozhatsky
Signed-off-by:
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Tony Luck
Cc: Fenghua Yu
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
arch/ia64/kernel/se
: Yonghong Song
Cc: b...@vger.kernel.org
Acked-by: Andrii Nakryiko
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
tools/lib/bpf/btf.c | 56 +--
tools/lib/bpf/btf_dump.c| 18 +-
tools/lib/bpf/libbpf.c | 679
tools/lib/bpf
sky
Signed-off-by: Kefeng Wang
---
drivers/block/drbd/drbd_nl.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/block/drbd/drbd_nl.c b/drivers/block/drbd/drbd_nl.c
index 5d52a2d32155..de2f94d0103a 100644
--- a/drivers/block/drbd/drbd_nl.c
+++ b/dri
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Stephen Rothwell
Cc: Yoshinori Sato
Cc: Rich Felker
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: "David S. Miller"
Cc: Andrew Morton
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wan
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Daniel Lezcano
Acked-by: Daniel Lezcano
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
driv
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Robert Richter
Acked-by: Robert Richter
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
d
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: "James E.J. Bottomley"
Cc: "Martin K. Petersen"
Reviewed-by: Sergey Senozhatsky
Signe
Signed-off-by: Kefeng Wang
---
arch/riscv/kernel/module.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/riscv/kernel/module.c b/arch/riscv/kernel/module.c
index 70bb94ae61c5..b7401858d872 100644
--- a/arch/riscv/kernel/module.c
+++ b/arch/riscv/kernel/module.c
@@ -315,
As said in commit f2c2cbcc35d4 ("powerpc: Use pr_warn instead of
pr_warning"), removing pr_warning so all logging messages use a
consistent _warn style. Let's do it.
Cc: Yoshinori Sato
Cc: Rich Felker
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
drivers/sh/
: Yonghong Song
Cc: b...@vger.kernel.org
Acked-by: Andrii Nakryiko
Reviewed-by: Sergey Senozhatsky
Signed-off-by: Kefeng Wang
---
Based on git://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git
tools/lib/bpf/btf.c | 56 +--
tools/lib/bpf/btf_dump.c| 18 +-
tools/lib
Using dev_get_drvdata directly.
Cc: Andy Gross
Cc: David Brown
Cc: Lee Jones
Cc: linux-arm-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
v2:
-use dev_get_drvdata() instead of to_ssbi()
drivers/mfd/ssbi.c | 6 ++
drivers/mfd/t7l66xb.c | 12
drivers/mfd/tc6387xb.c
if iio_dummy_evgen_create() fails, iio_evgen should be NULL, when call
iio_evgen_release() to cleanup, it throws some warning and could cause
double free.
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
drivers/iio/dummy/iio_dummy_evgen.c | 4
1 file changed, 4 insertions(+)
diff
Using dev_get_drvdata directly.
Cc: David Brown
Cc: Lee Jones
Cc: linux-arm-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
v3:
- fix build issue('dev' undeclared) in tc6393xb_nand_enable()
v2:
-use dev_get_drvdata() instead of to_ssbi()
drivers/mfd/ssbi.c | 6 ++
d
Robot
Signed-off-by: Kefeng Wang
---
drivers/input/misc/wistron_btns.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/input/misc/wistron_btns.c
b/drivers/input/misc/wistron_btns.c
index 43e67f546366..a82ec3d102b4 100644
--- a/drivers/input/misc/wistron_btns.c
+++ b
If ioremap fails, NULL pointer dereference will happen and
leading to a kernel panic when access the virtual address
in check_signature().
Fix it by check the return value of ioremap.
Cc: Jean Delvare
Cc: Wolfram Sang
Cc: linux-...@vger.kernel.org
Reported-by: Hulk Robot
Signed-off-by: Kefeng
If ioremap fails, NULL pointer dereference will happen and
leading to a kernel panic when access the virtual address
in check_signature().
Fix it by check the return value of ioremap.
Cc: Dmitry Torokhov
Cc: linux-in...@vger.kernel.org
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
On 2019/5/10 16:09, Jean Delvare wrote:
> Hi Kefeng,
>
> On Fri, 10 May 2019 11:03:20 +0800, Kefeng Wang wrote:
>> If ioremap fails, NULL pointer dereference will happen and
>> leading to a kernel panic when access the virtual address
>> in check_signature().
>&
On 2019/5/10 20:18, Jean Delvare wrote:
> On Fri, 10 May 2019 17:35:46 +0800, Kefeng Wang wrote:
>> On 2019/5/10 16:09, Jean Delvare wrote:
>>> We don't need this anyway. The comment says it can't fail, so why
>>> bother checking for a condition which will
do_syscall_64+0x72/0x2a0 arch/x86/entry/common.c:298
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Don't call kfree(src) if device_add() fails to fix it.
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
drivers/hwtracing/stm/core.c | 17 -
1 file changed, 8 insertions(+), 9 dele
:3909
do_syscall_64+0x72/0x2a0 arch/x86/entry/common.c:298
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Only free stm once if device_add() fails to fix it.
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
drivers/hwtracing/stm/core.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion
sorry, send wrong version, please ignore.
On 2019/5/20 17:13, Kefeng Wang wrote:
> If device_add() in stm_source_register_device() fails,
> stm_source_device_release() is called to free src, free
> src again on err path will trigger double free issue.
>
> BUG: KASAN: double-free
sorry, send wrong version, please ignore.
On 2019/5/20 17:13, Kefeng Wang wrote:
> If device_add() in stm_register_device() fails, stm_device_release()
> is called to free stm, free stm again on err_device path will trigger
> following warning,
>
> Trying to vfree() none
kernel/module.c:3909
do_syscall_64+0x72/0x2a0 arch/x86/entry/common.c:298
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Change error handling path and only free stm once if device_add() fails
to fix it.
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
drivers/hwtracing/stm/core.c | 8
/driver.c:193
driver_unregister+0x60/0x70 drivers/base/driver.c:193
Fix it by adding platform_registered variable, only unregister platform
driver when it is already successfully registered.
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
drivers/char/ipmi/ipmi_ssif.c | 5 -
1 file
3e67f106f619 ("inet: frags: break the 2GB limit for frags storage"),
changes ipfrag_high/low_thread 'type' from int to long, using CTL_ULONG
instead of CTL_INT to keep consistent.
Cc: Andrew Morton
Cc: "David S. Miller"
Cc: Eric Dumazet
Signed-off-by: Kefeng Wan
:298
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Add 'bool initialized' into struct jffs2_compressor, return error
if initialized is not set in jffs2_unregister_compressor().
Reported-by: Hulk Robot
Signed-off-by: Kefeng Wang
---
fs/jffs2/compr.c | 7 +++
fs/jffs2/compr.h | 1
On 2019/5/24 23:00, Eric Dumazet wrote:
> On Fri, May 24, 2019 at 7:30 AM Kefeng Wang
> wrote:
>> 3e67f106f619 ("inet: frags: break the 2GB limit for frags storage"),
>> changes ipfrag_high/low_thread 'type' from int to long, using CTL_ULONG
>> ins
Use of_clk_get_parent_count() instead of open coding.
Signed-off-by: Kefeng Wang
---
drivers/usb/dwc3/dwc3-qcom.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/usb/dwc3/dwc3-qcom.c b/drivers/usb/dwc3/dwc3-qcom.c
index 184df4daa590..821f5179d7b6 100644
--- a
Use of_clk_get_parent_count() instead of open coding.
Signed-off-by: Kefeng Wang
---
drivers/pwm/pwm-rockchip.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pwm/pwm-rockchip.c b/drivers/pwm/pwm-rockchip.c
index 4d99d468df09..a0fbb9a139bd 100644
--- a/drivers
Use of_clk_get_parent_count() instead of open coding.
Signed-off-by: Kefeng Wang
---
drivers/base/power/clock_ops.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/base/power/clock_ops.c b/drivers/base/power/clock_ops.c
index 59d19dd64928..9c7e83267eac 100644
--- a
Use of_clk_get_parent_count() instead of open coding.
Reviewed-by: Geert Uytterhoeven
Signed-off-by: Kefeng Wang
---
drivers/base/power/clock_ops.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base/power/clock_ops.c b/drivers/base/power/clock_ops.c
index
On 2019/5/27 19:08, Geert Uytterhoeven wrote:
> Hi Kefeng,
>
> On Sat, May 25, 2019 at 1:54 PM Kefeng Wang
> wrote:
>> Use of_clk_get_parent_count() instead of open coding.
>>
>> Signed-off-by: Kefeng Wang
> Thanks for your patch!
>
>> --- a/drive
0x9f/0x450
entry_SYSCALL_64_after_hwframe+0x49/0xbe
Fix it by checking the return of proc_create_single() before
calling remove_proc_entry().
Cc: Hans Verkuil
Cc: Mauro Carvalho Chehab
Cc: Mauro Carvalho Chehab
Signed-off-by: Kefeng Wang
---
drivers/media/pci/saa7164/saa7164-c
Use of_clk_get_parent_count() instead of open coding.
Cc: Thierry Reding
Cc: Heiko Stuebner
Signed-off-by: Kefeng Wang
---
v2:
- add include
drivers/pwm/pwm-rockchip.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/pwm/pwm-rockchip.c b/drivers/pwm/pwm
When !CONFIG_SPARSE_IRQ, request_mutex of struct irq_desc won't
be initialiezed, it leads to system hung, fix it.
Fixes: 9114014cf4e6 ("genirq: Add mutex to irq desc to serialize
request/free_irq()")
Signed-off-by: Kefeng Wang
---
kernel/irq/irqdesc.c | 1 +
1 file chang
nce vim2m_device_release has done.
Cc: Hans Verkuil
Cc: Laurent Pinchart
Cc: Mauro Carvalho Chehab
Reported-by: Hulk Robot
Fixes: ea6c7e34f3b2 ("media: vim2m: replace devm_kzalloc by kzalloc")
Signed-off-by: Kefeng Wang
---
drivers/media/platform/vim2m.c | 6 +++---
1 file changed,
erial: uartlite: Move uart register to probe")
Signed-off-by: Kefeng Wang
---
drivers/tty/serial/uartlite.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/tty/serial/uartlite.c b/drivers/tty/serial/uartlite.c
index b8b912b5a8b9..06e79c11141d 100644
--- a/drivers/t
Using dev_get_drvdata directly.
Cc: Ludovic Desroches
Cc: Jonathan Cameron
Cc: linux-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/iio/adc/at91-sama5d2_adc.c | 12
drivers/iio/adc/at91_adc.c | 4 ++--
2 files changed, 6 insertions(+), 10 deletions(-)
diff
Using dev_get_drvdata directly.
Cc: Alessandro Zummo
Cc: Alexandre Belloni
Cc: linux-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/rtc/rtc-hid-sensor-time.c | 3 +--
drivers/rtc/rtc-pxa.c | 3 +--
drivers/rtc/rtc-rk808.c | 6 ++
drivers/rtc/rtc-tx4939
Using dev_get_drvdata directly.
Cc: Zhang Rui
Cc: Eduardo Valentin
Cc: Daniel Lezcano
Cc: linux...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
.../intel/int340x_thermal/processor_thermal_device.c | 8 +---
drivers/thermal/st/stm_thermal.c | 6 ++
2
Using dev_get_drvdata directly.
Cc: Darren Hart
Cc: Andy Shevchenko
Cc: platform-driver-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/platform/x86/asus-wmi.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform
Using dev_get_drvdata directly.
Cc: Andy Gross
Cc: David Brown
Cc: Lee Jones
Cc: linux-arm-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/mfd/ssbi.c | 2 +-
drivers/mfd/t7l66xb.c | 12
drivers/mfd/tc6387xb.c | 12
drivers/mfd/tc6393xb.c | 21
Using dev_get_drvdata directly.
Cc: Dmitry Torokhov
Cc: linux-in...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/input/keyboard/ep93xx_keypad.c | 10 --
drivers/input/keyboard/gpio_keys.c | 6 ++
drivers/input/keyboard/imx_keypad.c | 10 --
drivers
Using dev_get_drvdata directly.
Cc: Bin Liu
Cc: Greg Kroah-Hartman
Cc: linux-...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/usb/musb/musb_dsps.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb/musb_dsps.c
Many drivers call to_platform_device() to get platform_device, then
get drvdata by platform_get_drvdata(), but in platfrom_get_drvdata(),
the platform_device is converted back to struct device again.
Using dev_get_drvdata() directly, only build test on arm64/x86_64.
Kefeng Wang (25):
ACPI
Using dev_get_drvdata directly.
Cc: Anup Patel
Cc: Jassi Brar
Signed-off-by: Kefeng Wang
---
drivers/mailbox/bcm-flexrm-mailbox.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/mailbox/bcm-flexrm-mailbox.c
b/drivers/mailbox/bcm-flexrm-mailbox.c
index
Using dev_get_drvdata directly.
Cc: Vinod Koul
Cc: dmaeng...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/dma/bcm-sba-raid.c | 3 +--
drivers/dma/nbpfaxi.c | 4 ++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba
Using dev_get_drvdata directly.
Cc: Dmitry Torokhov
Cc: linux-in...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/input/misc/max77693-haptic.c | 6 ++
drivers/input/misc/max8925_onkey.c| 10 --
drivers/input/misc/max8997_haptic.c | 3 +--
drivers/input/misc/msm
Using dev_get_drvdata directly.
Cc: Jean Delvare
Cc: Guenter Roeck
Cc: linux-hw...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/hwmon/s3c-hwmon.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/hwmon/s3c-hwmon.c b/drivers/hwmon/s3c-hwmon.c
index
Using dev_get_drvdata directly.
Cc: Ulf Hansson
Cc: Linus Walleij
Cc: linux-...@vger.kernel.org
Cc: linux-o...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/mmc/host/omap_hsmmc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/mmc/host/omap_hsmmc.c b
Using dev_get_drvdata directly.
Cc: "Rafael J. Wysocki"
Cc: Srinivas Pandruvada
Cc: linux-a...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/acpi/dptf/dptf_power.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/acpi/dptf/dptf_power.c b/drivers
Using dev_get_drvdata directly.
Cc: Jamie Iles
Cc: Herbert Xu
Cc: "David S. Miller"
Cc: linux-cry...@vger.kernel.org
Cc: linux-arm-ker...@lists.infradead.org
Signed-off-by: Kefeng Wang
---
drivers/crypto/picoxcell_crypto.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
Using dev_get_drvdata directly.
Cc: Dmitry Torokhov
Cc: linux-in...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/input/mouse/navpoint.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/input/mouse/navpoint.c b/drivers/input/mouse/navpoint.c
index
Using dev_get_drvdata directly.
Cc: Dmitry Torokhov
Cc: Shawn Guo
Cc: Sascha Hauer
Cc: linux-in...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/input/touchscreen/imx6ul_tsc.c | 6 ++
drivers/input/touchscreen/s3c2410_ts.c | 3 +--
2 files changed, 3 insertions(+), 6 deletions
Using dev_get_drvdata directly.
Cc: "David S. Miller"
Cc: net...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/net/ethernet/calxeda/xgmac.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/calxeda/xgmac.c
b/drivers/net/ethern
Using dev_get_drvdata directly.
Cc: Vinod Koul
Cc: dmaeng...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/dma/bcm-sba-raid.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/dma/bcm-sba-raid.c b/drivers/dma/bcm-sba-raid.c
index 72878ac5c78d..fa81d0177765
Using dev_get_drvdata directly.
Cc: Vinod Koul
Cc: dmaeng...@vger.kernel.org
Signed-off-by: Kefeng Wang
---
drivers/dma/nbpfaxi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/nbpfaxi.c b/drivers/dma/nbpfaxi.c
index a67b292190f4..594409a6e975 100644
--- a
On 2019/4/29 11:47, Zhang Rui wrote:
> On 二, 2019-04-23 at 15:50 +0800, Kefeng Wang wrote:
>> Using dev_get_drvdata directly.
>>
>> Cc: Zhang Rui
>> Cc: Eduardo Valentin
>> Cc: Daniel Lezcano
>> Cc: linux...@vger.kernel.org
>> Signed-off-by: Kefe
and with memory).
After 1) and 2), we could remove unnecessary check whether the 'new'
AND node_states[N_MEMORY] is empty.
Cc: Andrea Arcangeli
Cc: Andrew Morton
Cc: Dan Williams
Cc: Michal Hocko
Cc: Oscar Salvador
Cc: Vlastimil Babka
Cc: linux...@kvack.org
Signed-off-by: K
Kindly ping...
On 2019/7/25 19:30, Xiongfeng Wang wrote:
> When I ran the syzkaller testsuite, it prints out the following calltrace.
>
> [ 138.350029] rcu: INFO: rcu_sched self-detected stall on CPU
> [ 138.354703] rcu: 2-: (104998 ticks this GP)
> idle=ece/1/0x4002 softirq=
According to 78ddc5347341 ("thp: rename split_huge_page_pmd() to
split_huge_pmd()"), update releated comment.
Signed-off-by: Kefeng Wang
---
arch/powerpc/mm/book3s64/hash_utils.c | 2 +-
include/asm-generic/pgtable.h | 5 ++---
2 files changed, 3 insertions(+), 4 deletion
On 2019/8/6 16:36, Vlastimil Babka wrote:
> On 8/6/19 4:36 AM, Kefeng Wang wrote:
[...]
>>
>> [QUESTION]
>>
>> SYSCALL_DEFINE4(migrate_pages, pid_t, pid, unsigned long, maxnode,
>> const unsigned long __user *, old_nodes,
>>
probe of HISI0152:00 failed with error -12
Instead of changing the print to "%px", and leaking kernel addresses,
just remove the print completely.
Signed-off-by: Kefeng Wang
---
drivers/irqchip/irq-mbigen.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/
On 2019/6/18 15:48, Marc Zyngier wrote:
> Hi Kefeng,
>
> On Tue, 18 Jun 2019 04:22:02 +0100,
> Kefeng Wang wrote:
>>
>> After commit ad67b74d2469d9b8 ("printk: hash addresses printed with %p"),
>> it will print "ptrval" instead of a
On 2019/6/18 16:42, Marc Zyngier wrote:
> On 18/06/2019 09:35, Kefeng Wang wrote:
>>
>>
>> On 2019/6/18 15:48, Marc Zyngier wrote:
>>> Hi Kefeng,
>>>
>>> On Tue, 18 Jun 2019 04:22:02 +0100,
>>> Kefeng Wang wrote:
>>>>
>&g
probe of HISI0152:00 failed with error -12
dev_xxx() helper contains the device info, HISI0152:00, which stands for
mbigen ACPI HID and its UID, we can identify the failing probed mbigen,
so just remove the printing "mgn_chip->base", and also add missing "\n".
Signed-of
pcpu_setup_first_chunk() will panic or BUG_ON if the are some
error and doesn't return any error, hence it can be defined to
return void.
Signed-off-by: Kefeng Wang
---
arch/ia64/mm/contig.c| 5 +
arch/ia64/mm/discontig.c | 5 +
include/linux/percpu.h | 2 +-
mm/per
On 2019/7/4 12:20, Dennis Zhou wrote:
> On Wed, Jul 03, 2019 at 04:25:52PM +0800, Kefeng Wang wrote:
>> pcpu_setup_first_chunk() will panic or BUG_ON if the are some
>> error and doesn't return any error, hence it can be defined to
>> return void.
>&g
ioctl, r[0], 0x40086806, 0x40);
Fix it by using div64_ul().
Signed-off-by: Kefeng Wang
Signed-off-by: Zhang HongJun
---
drivers/char/hpet.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c
index 5c39f20378b8..dd52b63b62db 10
On 2019/7/11 20:32, Arnd Bergmann wrote:
> On Thu, Jul 11, 2019 at 1:20 PM Kefeng Wang
> wrote:
>> The base value in do_div() called by hpet_time_div() is truncated from
>> unsigned long to uint32_t, resulting in a divide-by-zero exception.
>
> Good catch!
>
&
ioctl, r[0], 0x40086806, 0x40);
Fix it by using div64_ul().
Signed-off-by: Kefeng Wang
Signed-off-by: Zhang HongJun
---
drivers/char/hpet.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/char/hpet.c b/drivers/char/hpet.c
index 5c39f20378b8..9ac6671bb514 10
../drivers/char/hpet.c:159:17: warning: variable ‘m’ set but not used
[-Wunused-but-set-variable]
unsigned long m, t, mc, base, k;
^
Signed-off-by: Kefeng Wang
---
drivers/char/hpet.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/char/hpet.c b
On 2019/3/8 22:45, Bart Van Assche wrote:
> On 3/7/19 11:37 PM, Kefeng Wang wrote:
>> syzkaller report an issue "KASAN: use-after-free Read in alloc_workqueue",
>>
>> alloc_workqueue
>> - kzalloc wq
>> - wq_init_lockdep(wq);
>> -
ev() fail in
hci_uart_set_proto(), see stack above, fix this by setting
HCI_UART_PROTO_READY bit only when hci_uart_register_dev()
return success.
Reported-by: syzbot+899a33dc0fa0dbaf0...@syzkaller.appspotmail.com
Signed-off-by: Kefeng Wang
---
drivers/bluetooth/hci_ldisc.c | 3 +--
1 file changed, 1 ins
elete the entry from hlist.
Reported-by: syzbot+17335689e239ce135...@syzkaller.appspotmail.com
Fixes: 669de8bda87b ("kernel/workqueue: Use dynamic lockdep keys for
workqueues")
Signed-off-by: Kefeng Wang
---
kernel/workqueue.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/kernel/wo
Signed-off-by: Kefeng Wang
---
note:
- Build error occurred when I backported kcov to an old kernel version
with an old gcc
kernel/kcov.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/kernel/kcov.c b/kernel/kcov.c
index 30e6d05..3cbb0c8 100644
--- a/kernel/kcov.c
+++ b/kernel/kcov.c
It is fragile that some definitions acquired via transitive
dependencies, as shown in below:
atomic_*()
ENOMEM/EN* ()
EXPORT_SYMBOL ()
device_initcall ()
preempt_* ()
Include them to prevent possible issues.
Suggested-by: Mark Rutland
Signed-off-by: Kefeng Wang
read-side critical section.
Signed-off-by: Kefeng Wang
---
net/ipv6/addrconf.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
index c1e124b..f60e88e 100644
--- a/net/ipv6/addrconf.c
+++ b/net/ipv6/addrconf.c
@@ -5540,8 +5540,7 @@ s
cott Branden
> Reviewed-by: Ray Jui
> Signed-off-by: Jason Uy
Reviewed-by: Kefeng Wang
> ---
> drivers/tty/serial/8250/8250_dw.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/tty/serial/8250/8250_dw.c
> b/drivers/tty/serial/82
Using better of_property_read_u32() than generic of_get_property().
Cc: Bartlomiej Zolnierkiewicz
Cc: Hans de Goede
Cc: Tejun Heo
Signed-off-by: Kefeng Wang
---
drivers/ata/pata_of_platform.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/ata
On 2016/10/17 20:07, Pan Wen wrote:
> Add dts files for Hi3516CV300 demo board.
>
> Signed-off-by: Pan Wen
> ---
> arch/arm/boot/dts/Makefile | 1 +
[...]
> diff --git a/arch/arm/boot/dts/hi3516cv300.dtsi
> b/arch/arm/boot/dts/hi3516cv300.dtsi
> new file mode 100644
> index 000
of_clk_init() caused changes
>> in the initialization order breaking booting.
>>
>> Fixes: 69d99e6c0d621f ("xtensa: Remove unnecessary of_platform_populate with
>> default match table")
>> Cc: Kefeng Wang
>> Cc: Guenter Roeck
>> Cc: Max Filip
of_iommu_init()
called before of_platform_populate.
Acked-by: Rich Felker
Tested-by: Marek Szyprowski
Cc: Arnd Bergmann
Cc: Catalin Marinas
Cc: Marek Szyprowski
Cc: Rich Felker
Cc: Rob Herring
Cc: Robin Murphy
Cc: Will Deacon
Signed-off-by: Kefeng Wang
---
Change since v1:
- Rebase to
te, and we
>>> get a straightforward "Unable to hande kernel paging request..." dump.
>>>
>>> The more typical use-case of mmap_kmem() already validates the address
>>> with pfn_valid() as one might expect, so let's make that consistent
>>&
(), and use __func__ to
print function's name of of_platform_bus_create().
Cc: Rob Herring
Cc: Frank Rowand
Cc: Grant Likely
Signed-off-by: Kefeng Wang
---
drivers/of/platform.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/drivers/of/platform.c b/dr
301 - 400 of 576 matches
Mail list logo