Re: linux-next: build warning after merge of the pm tree
From: Rafael J. Wysocki
Date: Thu Jul 11 2019 - 08:26:23 EST
On Thursday, July 11, 2019 7:46:29 AM CEST wen.yang99@xxxxxxxxxx wrote:
> > Hi all,
> >
> > After merging the pm tree, today's linux-next build (powerpc
> > ppc64_defconfig) produced this warning:
> >
> > drivers/cpufreq/pasemi-cpufreq.c: In function 'pas_cpufreq_cpu_init':
> > drivers/cpufreq/pasemi-cpufreq.c:199:1: warning: label 'out_unmap_sdcpwr' defined but not used [-Wunused-label]
> > out_unmap_sdcpwr:
> > ^~~~~~~~~~~~~~~~
> >
> > Introduced by commit
> >
> > f43e075f7252 ("cpufreq/pasemi: fix an use-after-free in pas_cpufreq_cpu_init()")
>
> Thank you very much.
>
> We need to delete the useless code here:
> diff --git a/drivers/cpufreq/pasemi-cpufreq.c b/drivers/cpufreq/pasemi-cpufreq.c
> index 1f0beb7..624c34a 100644
> --- a/drivers/cpufreq/pasemi-cpufreq.c
> +++ b/drivers/cpufreq/pasemi-cpufreq.c
> @@ -195,9 +195,6 @@ static int pas_cpufreq_cpu_init(struct cpufreq_policy *policy)
>
> return cpufreq_generic_init(policy, pas_freqs, get_gizmo_latency());
>
> -out_unmap_sdcpwr:
> - iounmap(sdcpwr_mapbase);
> -
> out_unmap_sdcasr:
> iounmap(sdcasr_mapbase);
> out:
I'm dropping commit f43e075f7252 from my linux-next branch, so please send a new version
of that patch with the issue fixed.
If you consider alternatives to the patch, please describe them in the message section between
the patch and the changelog.