* [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static @ 2016-06-08 17:41 Ben Dooks 2016-06-13 1:18 ` Jason Cooper 0 siblings, 1 reply; 5+ messages in thread From: Ben Dooks @ 2016-06-08 17:41 UTC (permalink / raw) To: linux-arm-kernel The bcm2836_smp_boot_secondary() is not declared or used elsewhere so make it static to fix the following warning: drivers/irqchip/irq-bcm2836.c:227:12: warning: symbol 'bcm2836_smp_boot_secondary' was not declared. Should it be static? Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Jason Cooper <jason@lakedaemon.net> Cc: Marc Zyngier <marc.zyngier@arm.com> Cc: linux-arm-kernel at lists.infradead.org --- drivers/irqchip/irq-bcm2836.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/irq-bcm2836.c b/drivers/irqchip/irq-bcm2836.c index 72ff1d5..6c72e60 100644 --- a/drivers/irqchip/irq-bcm2836.c +++ b/drivers/irqchip/irq-bcm2836.c @@ -224,8 +224,8 @@ static struct notifier_block bcm2836_arm_irqchip_cpu_notifier = { }; #ifdef CONFIG_ARM -int __init bcm2836_smp_boot_secondary(unsigned int cpu, - struct task_struct *idle) +static int __init bcm2836_smp_boot_secondary(unsigned int cpu, + struct task_struct *idle) { unsigned long secondary_startup_phys = (unsigned long)virt_to_phys((void *)secondary_startup); -- 2.8.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static 2016-06-08 17:41 [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static Ben Dooks @ 2016-06-13 1:18 ` Jason Cooper 2016-06-16 12:48 ` Ben Dooks 0 siblings, 1 reply; 5+ messages in thread From: Jason Cooper @ 2016-06-13 1:18 UTC (permalink / raw) To: linux-arm-kernel Hey Ben, On Wed, Jun 08, 2016 at 06:41:37PM +0100, Ben Dooks wrote: > The bcm2836_smp_boot_secondary() is not declared or used elsewhere > so make it static to fix the following warning: > > drivers/irqchip/irq-bcm2836.c:227:12: warning: symbol 'bcm2836_smp_boot_secondary' was not declared. Should it be static? > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > --- > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: Jason Cooper <jason@lakedaemon.net> > Cc: Marc Zyngier <marc.zyngier@arm.com> > Cc: linux-arm-kernel at lists.infradead.org > --- > drivers/irqchip/irq-bcm2836.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) This, and the rest of the irqchip/ ones, applied to irqchip/misc. thx, Jason. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static 2016-06-13 1:18 ` Jason Cooper @ 2016-06-16 12:48 ` Ben Dooks 2016-06-16 13:05 ` Jason Cooper 0 siblings, 1 reply; 5+ messages in thread From: Ben Dooks @ 2016-06-16 12:48 UTC (permalink / raw) To: linux-arm-kernel On 13/06/16 02:18, Jason Cooper wrote: > Hey Ben, > > On Wed, Jun 08, 2016 at 06:41:37PM +0100, Ben Dooks wrote: >> The bcm2836_smp_boot_secondary() is not declared or used elsewhere >> so make it static to fix the following warning: >> >> drivers/irqchip/irq-bcm2836.c:227:12: warning: symbol 'bcm2836_smp_boot_secondary' was not declared. Should it be static? >> >> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> >> --- >> Cc: Thomas Gleixner <tglx@linutronix.de> >> Cc: Jason Cooper <jason@lakedaemon.net> >> Cc: Marc Zyngier <marc.zyngier@arm.com> >> Cc: linux-arm-kernel at lists.infradead.org >> --- >> drivers/irqchip/irq-bcm2836.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > This, and the rest of the irqchip/ ones, applied to irqchip/misc. Thank you, so hopefully it'll be included in -next soon? -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static 2016-06-16 12:48 ` Ben Dooks @ 2016-06-16 13:05 ` Jason Cooper 2016-06-16 13:11 ` Ben Dooks 0 siblings, 1 reply; 5+ messages in thread From: Jason Cooper @ 2016-06-16 13:05 UTC (permalink / raw) To: linux-arm-kernel On Thu, Jun 16, 2016 at 01:48:45PM +0100, Ben Dooks wrote: > On 13/06/16 02:18, Jason Cooper wrote: > > Hey Ben, > > > > On Wed, Jun 08, 2016 at 06:41:37PM +0100, Ben Dooks wrote: > >> The bcm2836_smp_boot_secondary() is not declared or used elsewhere > >> so make it static to fix the following warning: > >> > >> drivers/irqchip/irq-bcm2836.c:227:12: warning: symbol 'bcm2836_smp_boot_secondary' was not declared. Should it be static? > >> > >> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > >> --- > >> Cc: Thomas Gleixner <tglx@linutronix.de> > >> Cc: Jason Cooper <jason@lakedaemon.net> > >> Cc: Marc Zyngier <marc.zyngier@arm.com> > >> Cc: linux-arm-kernel at lists.infradead.org > >> --- > >> drivers/irqchip/irq-bcm2836.c | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > > > > This, and the rest of the irqchip/ ones, applied to irqchip/misc. > > Thank you, so hopefully it'll be included in -next soon? hmmm, I just checked myself. It appears that the series in irqchip/misc has been in at least two runs of linux-next. Do you see something different? Here's the commit hashes: $ git ldo --graph v4.7-rc1^..HEAD * ebf63bb87f67 (HEAD -> irqchip/for-next, infradead/irqchip/for-next) Merge branch 'irqchip/misc' into irqchip/for-next |\ | * df042a5f4ce4 (infradead/irqchip/misc, irqchip/misc) irqchip/vic: Make vic_syscore_ops and vic_of_init static | * f3142635de32 irqchip/omap-intc: Fix missing <linux/irqchip/irq-omap-intc.h> include | * 7c034f169973 irqchip/gic-v2m: Fix missing include of <linux/irqchip/arm-gic.h> | * 2ae9add9d82d irqchip/brcmstb-l2: Make of probe function static | * dde7e6d1abdc irqchip/bcm7120-l2: Make probe functions static | * 1eeb67362d93 irqchip/bcm2836: Make bcm2836_smp_boot_secondary static | * 6c880902419e irqchip/armada-370-xp: Make syscore_ops static | * f6fbaaa4c0fc irqchip/tegra: Fix sparse warnings on __iomem | * 4a3691ccb3ce irqchip/sirfsoc: Fix sparse warnings on __iomem * | d7e3528eed85 (infradead/irqchip/bcm, irqchip/bcm) irqchip: bcm2835: Avoid arch/arm-specific handle_IRQ |/ * 1a695a905c18 (tag: v4.7-rc1, linux-phy/next, infradead/irqchip/urgent, infradead/irqchip/core, arm-soc/master, irqchip/urgent, irqchip/core) Linux 4.7-rc1 hth, Jason. ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static 2016-06-16 13:05 ` Jason Cooper @ 2016-06-16 13:11 ` Ben Dooks 0 siblings, 0 replies; 5+ messages in thread From: Ben Dooks @ 2016-06-16 13:11 UTC (permalink / raw) To: linux-arm-kernel On 16/06/16 14:05, Jason Cooper wrote: > On Thu, Jun 16, 2016 at 01:48:45PM +0100, Ben Dooks wrote: >> On 13/06/16 02:18, Jason Cooper wrote: >>> Hey Ben, >>> >>> On Wed, Jun 08, 2016 at 06:41:37PM +0100, Ben Dooks wrote: >>>> The bcm2836_smp_boot_secondary() is not declared or used elsewhere >>>> so make it static to fix the following warning: >>>> >>>> drivers/irqchip/irq-bcm2836.c:227:12: warning: symbol 'bcm2836_smp_boot_secondary' was not declared. Should it be static? >>>> >>>> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> >>>> --- >>>> Cc: Thomas Gleixner <tglx@linutronix.de> >>>> Cc: Jason Cooper <jason@lakedaemon.net> >>>> Cc: Marc Zyngier <marc.zyngier@arm.com> >>>> Cc: linux-arm-kernel at lists.infradead.org >>>> --- >>>> drivers/irqchip/irq-bcm2836.c | 4 ++-- >>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>> >>> This, and the rest of the irqchip/ ones, applied to irqchip/misc. >> >> Thank you, so hopefully it'll be included in -next soon? > > hmmm, I just checked myself. It appears that the series in irqchip/misc > has been in at least two runs of linux-next. Do you see something > different? yes, I did git remote-update in a different git repo tree and then looked in a gitg on an different repo :( -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-16 13:11 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-06-08 17:41 [PATCH] irqchip/bcm2836: make bcm2836_smp_boot_secondary static Ben Dooks 2016-06-13 1:18 ` Jason Cooper 2016-06-16 12:48 ` Ben Dooks 2016-06-16 13:05 ` Jason Cooper 2016-06-16 13:11 ` Ben Dooks
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).