[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAOiHx=mgNB5V178vNQM0oePd9uhDsz7bnqKRCcbfc22WxZm1jQ@mail.gmail.com>
Date: Tue, 23 Jan 2024 15:41:47 +0100
From: Jonas Gorski <jonas.gorski@...il.com>
To: Florian Fainelli <f.fainelli@...il.com>
Cc: linux-mips@...r.kernel.org,
Thomas Bogendoerfer <tsbogend@...ha.franken.de>, open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] MIPS: BCM63XX: Fix missing prototypes
Hi,
On Tue, 23 Jan 2024 at 05:08, Florian Fainelli <f.fainelli@...il.com> wrote:
>
> We have a number of missing prototypes warnings for the BCM63XX machine,
> fix those by providing adequate function prototypes.
None of these prototypes should be needed, most functions are never
called outside of their units and are just missing a static
annotation, and one already has a prototype in a different header.
>
> Signed-off-by: Florian Fainelli <f.fainelli@...il.com>
> ---
> arch/mips/include/asm/mach-bcm63xx/bcm63xx_cpu.h | 8 ++++++++
> arch/mips/include/asm/mach-bcm63xx/bcm63xx_timer.h | 1 +
> arch/mips/include/asm/mach-bcm63xx/board_bcm963xx.h | 2 ++
> arch/mips/include/asm/mach-bcm63xx/irq.h | 3 +++
> 4 files changed, 14 insertions(+)
>
> diff --git a/arch/mips/include/asm/mach-bcm63xx/bcm63xx_cpu.h b/arch/mips/include/asm/mach-bcm63xx/bcm63xx_cpu.h
> index 1cad18e6681d..b9ab5ee1c7b2 100644
> --- a/arch/mips/include/asm/mach-bcm63xx/bcm63xx_cpu.h
> +++ b/arch/mips/include/asm/mach-bcm63xx/bcm63xx_cpu.h
> @@ -1065,4 +1065,12 @@ void bcm63xx_machine_halt(void);
>
> void bcm63xx_machine_reboot(void);
>
> +int bcm63xx_register_devices(void);
defined and used only in arch/mips/bcm63xx/setup.c
> +
> +int bcm63xx_rng_register(void);
defined and used only in arch/mips/bcm63xx/dev-rng.c
> +
> +int bcm63xx_uart_register(unsigned int id);
This one already has a prototype in bcm63xx_dev_uart.h,
arch/mips/bcm63xx/dev-uart.c just needs to include that header file.
> +
> +int bcm63xx_wdt_register(void);
defined and used only in arch/mips/bcm63xx/dev-wdt.c
> +
> #endif /* !BCM63XX_CPU_H_ */
> diff --git a/arch/mips/include/asm/mach-bcm63xx/bcm63xx_timer.h b/arch/mips/include/asm/mach-bcm63xx/bcm63xx_timer.h
> index bcbece793fd8..f920c7274f5e 100644
> --- a/arch/mips/include/asm/mach-bcm63xx/bcm63xx_timer.h
> +++ b/arch/mips/include/asm/mach-bcm63xx/bcm63xx_timer.h
> @@ -8,5 +8,6 @@ int bcm63xx_timer_set(int id, int monotonic, unsigned int countdown_us);
> int bcm63xx_timer_enable(int id);
> int bcm63xx_timer_disable(int id);
> unsigned int bcm63xx_timer_countdown(unsigned int countdown_us);
> +int bcm63xx_timer_init(void);
defined and used only in arch/mips/bcm63xx/timer.c
>
> #endif /* !BCM63XX_TIMER_H_ */
> diff --git a/arch/mips/include/asm/mach-bcm63xx/board_bcm963xx.h b/arch/mips/include/asm/mach-bcm63xx/board_bcm963xx.h
> index 830f53f28e3f..7b905c8a424d 100644
> --- a/arch/mips/include/asm/mach-bcm63xx/board_bcm963xx.h
> +++ b/arch/mips/include/asm/mach-bcm63xx/board_bcm963xx.h
> @@ -51,4 +51,6 @@ struct board_info {
> unsigned long ephy_reset_gpio_flags;
> };
>
> +int bcm63xx_get_fallback_sprom(struct ssb_bus *bus, struct ssb_sprom *out);
defined and used only in arch/mips/bcm63xx/boards/board_bcm963xx.c
> +
> #endif /* ! BOARD_BCM963XX_H_ */
> diff --git a/arch/mips/include/asm/mach-bcm63xx/irq.h b/arch/mips/include/asm/mach-bcm63xx/irq.h
> index b016f0615d5f..cbb92a609835 100644
> --- a/arch/mips/include/asm/mach-bcm63xx/irq.h
> +++ b/arch/mips/include/asm/mach-bcm63xx/irq.h
> @@ -5,4 +5,7 @@
> #define NR_IRQS 128
> #define MIPS_CPU_IRQ_BASE 0
>
> +void __dispatch_internal_32(int cpu);
> +void __dispatch_internal_64(int cpu);
dispatch __dispatch_internal_32()/__dispatch_internal_64() are defined
and used only in arch/mips/bcm63xx/irq.c
Best Regards,
Jonas
Powered by blists - more mailing lists