[PATCH 0/2] code optimizations for btext.c

2021-04-07 Thread Yu Kuai
Yu Kuai (2): powerpc: remove set but not used variable 'force_printk_to_btext' powerpc: make 'boot_text_mapped' static arch/powerpc/kernel/btext.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) -- 2.25.4

[PATCH 1/2] powerpc: remove set but not used variable 'force_printk_to_btext'

2021-04-07 Thread Yu Kuai
Fixes gcc '-Wunused-but-set-variable' warning: arch/powerpc/kernel/btext.c:49:12: error: 'force_printk_to_btext' defined but not used. It is never used, and so can be removed. Signed-off-by: Yu Kuai --- arch/powerpc/kernel/btext.c | 1 - 1 file changed, 1 deletion(-

[PATCH 2/2] powerpc: make 'boot_text_mapped' static

2021-04-07 Thread Yu Kuai
The sparse tool complains as follow: arch/powerpc/kernel/btext.c:48:5: warning: symbol 'boot_text_mapped' was not declared. Should it be static? This symbol is not used outside of btext.c, so this commit make it static. Signed-off-by: Yu Kuai --- arch/powerpc/kernel/btext.c | 2

[PATCH] macintosh/windfarm: Make symbol 'pm121_sys_state' static

2021-04-07 Thread Yu Kuai
The sparse tool complains as follows: drivers/macintosh/windfarm_pm121.c:436:24: warning: symbol 'pm121_sys_state' was not declared. Should it be static? This symbol is not used outside of windfarm_pm121.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-b

[PATCH] powerpc: Make some symbols static

2021-04-07 Thread Yu Kuai
are not used outside of btext.c, so this commit marks them static. Reported-by: Hulk Robot Signed-off-by: Yu Kuai --- arch/powerpc/kernel/btext.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/btext.c b/arch/powerpc/kernel/btext.c index 803c2a45b22a.

[PATCH] powerpc/smp: Make some symbols static

2021-04-07 Thread Yu Kuai
it be static? arch/powerpc/kernel/smp.c:132:1: warning: symbol '__pcpu_scope_thread_group_l2_cache_map' was not declared. Should it be static? These symbols are not used outside of smp.c, so this commit marks them static. Reported-by: Hulk Robot Signed-off-by: Yu Kuai --- arch/powe

[PATCH] tty: hvc: make symbol 'hvc_udbg_dev' static

2021-04-07 Thread Yu Kuai
The sparse tool complains as follows: drivers/tty/hvc/hvc_udbg.c:20:19: warning: symbol 'hvc_udbg_dev' was not declared. Should it be static? This symbol is not used outside of hvc_udbg.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Yu Kuai --- drive

[PATCH] macintosh/via-pmu: Make some symbols static

2021-04-07 Thread Yu Kuai
used outside of via-pmu.c, so this commit marks them static. Reported-by: Hulk Robot Signed-off-by: Yu Kuai --- drivers/macintosh/via-pmu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/macintosh/via-pmu.c b/drivers/macintosh/via-pmu.c index 73e6ae88fafd..47876

[PATCH] windfarm: make symbol 'wf_thread' static

2021-04-07 Thread Yu Kuai
The sparse tool complains as follows: drivers/macintosh/windfarm_core.c:59:20: warning: symbol 'wf_thread' was not declared. Should it be static? This symbol is not used outside of windfarm_core.c, so this commit marks it static. Reported-by: Hulk Robot Signed-off-by: Yu Kuai --

[PATCH V2] ASoC: fsl: imx-es8328: add missing put_device() call in imx_es8328_probe()

2020-08-25 Thread Yu Kuai
if of_find_device_by_node() succeed, imx_es8328_probe() doesn't have a corresponding put_device(). Thus add a jump target to fix the exception handling for this function implementation. Fixes: 7e7292dba215 ("ASoC: fsl: add imx-es8328 machine driver") Signed-off-by: Yu Kuai ---

[PATCH 2/2] ASoC: fsl: imx-es8328: add missing put_device() call in imx_es8328_probe()

2020-08-25 Thread Yu Kuai
if of_find_device_by_node() succeed, imx_es8328_probe() doesn't have a corresponding put_device(). Thus add a jump target to fix the exception handling for this function implementation. Fixes: 7e7292dba215 ("ASoC: fsl: add imx-es8328 machine driver") Signed-off-by: Yu Kuai --- so

[PATCH 0/2] do exception handling appropriately in imx_es8328_probe()

2020-08-25 Thread Yu Kuai
Yu Kuai (2): ASoC: fsl: imx-es8328: add missing kfree() call in imx_es8328_probe() ASoC: fsl: imx-es8328: add missing put_device() call in imx_es8328_probe() sound/soc/fsl/imx-es8328.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) -- 2.25.4

[PATCH 1/2] ASoC: fsl: imx-es8328: add missing kfree() call in imx_es8328_probe()

2020-08-25 Thread Yu Kuai
If memory allocation for 'data' or 'comp' succeed, imx_es8328_probe() doesn't have corresponding kfree() in exception handling. Thus add kfree() for this function implementation. Fixes: 7e7292dba215 ("ASoC: fsl: add imx-es8328 machine driver") Signed-off-by: Yu