linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
@ 2016-04-05  1:34 kbuild test robot
  2016-04-05  3:10 ` Viresh Kumar
  0 siblings, 1 reply; 4+ messages in thread
From: kbuild test robot @ 2016-04-05  1:34 UTC (permalink / raw)
  To: Viresh Kumar; +Cc: kbuild-all, linux-kernel, Rafael J. Wysocki, Stephen Boyd

[-- Attachment #1: Type: text/plain, Size: 1640 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   9735a22799b9214d17d3c231fe377fc852f042e9
commit: f59d3ee8480d30f41914cb4bed5086237e8507b0 PM / OPP: Move cpu specific code to opp/cpu.c
date:   7 months ago
config: i386-randconfig-c0-04050854 (attached as .config)
reproduce:
        git checkout f59d3ee8480d30f41914cb4bed5086237e8507b0
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

>> drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
    int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
        ^
   In file included from drivers/base/power/opp/opp.h:20:0,
                    from drivers/base/power/opp/cpu.c:21:
   include/linux/pm_opp.h:165:19: note: previous definition of 'dev_pm_opp_set_sharing_cpus' was here
    static inline int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
                      ^

vim +/dev_pm_opp_set_sharing_cpus +120 drivers/base/power/opp/cpu.c

   114		*table = NULL;
   115	}
   116	EXPORT_SYMBOL_GPL(dev_pm_opp_free_cpufreq_table);
   117	#endif	/* CONFIG_CPU_FREQ */
   118	
   119	/* Required only for V1 bindings, as v2 can manage it from DT itself */
 > 120	int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
   121	{
   122		struct device_list_opp *list_dev;
   123		struct device_opp *dev_opp;

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 24991 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
  2016-04-05  1:34 drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus' kbuild test robot
@ 2016-04-05  3:10 ` Viresh Kumar
  2016-04-05  7:18   ` Li, Philip
  0 siblings, 1 reply; 4+ messages in thread
From: Viresh Kumar @ 2016-04-05  3:10 UTC (permalink / raw)
  To: kbuild test robot
  Cc: kbuild-all, linux-kernel, Rafael J. Wysocki, Stephen Boyd

On 05-04-16, 09:34, kbuild test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   9735a22799b9214d17d3c231fe377fc852f042e9
> commit: f59d3ee8480d30f41914cb4bed5086237e8507b0 PM / OPP: Move cpu specific code to opp/cpu.c
> date:   7 months ago
> config: i386-randconfig-c0-04050854 (attached as .config)
> reproduce:
>         git checkout f59d3ee8480d30f41914cb4bed5086237e8507b0
>         # save the attached .config to linux build tree
>         make ARCH=i386 

Something isn't right in your setup. This is a 7 months old patch you have
reported now and I can see no problem with the patch as well.

Over that, the attached .config doesn't have CONFIG_PM_OPP enabled, so below
can't really compile.

> All errors (new ones prefixed by >>):
> 
> >> drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
>     int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
>         ^
>    In file included from drivers/base/power/opp/opp.h:20:0,
>                     from drivers/base/power/opp/cpu.c:21:
>    include/linux/pm_opp.h:165:19: note: previous definition of 'dev_pm_opp_set_sharing_cpus' was here
>     static inline int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
>                       ^
> 
> vim +/dev_pm_opp_set_sharing_cpus +120 drivers/base/power/opp/cpu.c
> 
>    114		*table = NULL;
>    115	}
>    116	EXPORT_SYMBOL_GPL(dev_pm_opp_free_cpufreq_table);
>    117	#endif	/* CONFIG_CPU_FREQ */
>    118	
>    119	/* Required only for V1 bindings, as v2 can manage it from DT itself */
>  > 120	int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
>    121	{
>    122		struct device_list_opp *list_dev;
>    123		struct device_opp *dev_opp;
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
viresh

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
  2016-04-05  3:10 ` Viresh Kumar
@ 2016-04-05  7:18   ` Li, Philip
  2016-04-05  7:20     ` Viresh Kumar
  0 siblings, 1 reply; 4+ messages in thread
From: Li, Philip @ 2016-04-05  7:18 UTC (permalink / raw)
  To: Viresh Kumar, Wu, Fengguang
  Cc: kbuild-all, linux-kernel, Rafael J. Wysocki, Stephen Boyd



> -----Original Message-----
> From: Viresh Kumar [mailto:viresh.kumar@linaro.org]
> Sent: Tuesday, April 5, 2016 11:10 AM
> To: Wu, Fengguang <fengguang.wu@intel.com>
> Cc: kbuild-all@01.org; linux-kernel@vger.kernel.org; Rafael J. Wysocki
> <rjw@rjwysocki.net>; Stephen Boyd <sboyd@codeaurora.org>
> Subject: Re: drivers/base/power/opp/cpu.c:120:5: error: redefinition of
> 'dev_pm_opp_set_sharing_cpus'
> 
> On 05-04-16, 09:34, kbuild test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> master
> > head:   9735a22799b9214d17d3c231fe377fc852f042e9
> > commit: f59d3ee8480d30f41914cb4bed5086237e8507b0 PM / OPP: Move
> cpu specific code to opp/cpu.c
> > date:   7 months ago
> > config: i386-randconfig-c0-04050854 (attached as .config)
> > reproduce:
> >         git checkout f59d3ee8480d30f41914cb4bed5086237e8507b0
> >         # save the attached .config to linux build tree
> >         make ARCH=i386
> 
> Something isn't right in your setup. This is a 7 months old patch you have
> reported now and I can see no problem with the patch as well.
Thanks, Viresh, will check why old commit is reported. On the other hand, I think even one config is not set, it should not introduce build error, which seems a valid failure. 

> 
> Over that, the attached .config doesn't have CONFIG_PM_OPP enabled, so
> below can't really compile.
> 
> > All errors (new ones prefixed by >>):
> >
> > >> drivers/base/power/opp/cpu.c:120:5: error: redefinition of
> 'dev_pm_opp_set_sharing_cpus'
> >     int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev,
> cpumask_var_t cpumask)
> >         ^
> >    In file included from drivers/base/power/opp/opp.h:20:0,
> >                     from drivers/base/power/opp/cpu.c:21:
> >    include/linux/pm_opp.h:165:19: note: previous definition of
> 'dev_pm_opp_set_sharing_cpus' was here
> >     static inline int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev,
> cpumask_var_t cpumask)
> >                       ^
> >
> > vim +/dev_pm_opp_set_sharing_cpus +120 drivers/base/power/opp/cpu.c
> >
> >    114		*table = NULL;
> >    115	}
> >    116	EXPORT_SYMBOL_GPL(dev_pm_opp_free_cpufreq_table);
> >    117	#endif	/* CONFIG_CPU_FREQ */
> >    118
> >    119	/* Required only for V1 bindings, as v2 can manage it from DT
> itself */
> >  > 120	int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev,
> cpumask_var_t cpumask)
> >    121	{
> >    122		struct device_list_opp *list_dev;
> >    123		struct device_opp *dev_opp;
> >
> > ---
> > 0-DAY kernel test infrastructure                Open Source Technology
> Center
> > https://lists.01.org/pipermail/kbuild-all                   Intel
> Corporation
> 
> 
> 
> --
> viresh

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus'
  2016-04-05  7:18   ` Li, Philip
@ 2016-04-05  7:20     ` Viresh Kumar
  0 siblings, 0 replies; 4+ messages in thread
From: Viresh Kumar @ 2016-04-05  7:20 UTC (permalink / raw)
  To: Li, Philip
  Cc: Wu, Fengguang, kbuild-all, linux-kernel, Rafael J. Wysocki, Stephen Boyd

On 05-04-16, 07:18, Li, Philip wrote:
> Thanks, Viresh, will check why old commit is reported. On the other hand, I think even one config is not set, it should not introduce build error, which seems a valid failure. 

What I am saying is that I couldn't reproduce the issue at my end and
the reason for that being the .config file doesn't have necessary
CONFIG_* options to build the code you reported failures for.

-- 
viresh

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-04-05  7:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-05  1:34 drivers/base/power/opp/cpu.c:120:5: error: redefinition of 'dev_pm_opp_set_sharing_cpus' kbuild test robot
2016-04-05  3:10 ` Viresh Kumar
2016-04-05  7:18   ` Li, Philip
2016-04-05  7:20     ` Viresh Kumar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).