| Message ID | 20211111091123.50853-1-krzysztof.kozlowski@canonical.com |
|---|---|
| State | New |
| Headers | show |
| Series | ARM: s3c: include header for prototype of s3c2410_modify_misccr | expand |
On 11/11/2021 10:46, Arnd Bergmann wrote: > On Thu, Nov 11, 2021 at 10:11 AM Krzysztof Kozlowski > <krzysztof.kozlowski@canonical.com> wrote: >> >> Include the header with prototype of s3c2410_modify_misccr to fix W=1 >> warning: >> >> arch/arm/mach-s3c/gpio-samsung.c:1309:14: warning: >> no previous prototype for ‘s3c2410_modify_misccr’ [-Wmissing-prototypes] >> >> Reported-by: kernel test robot <lkp@intel.com> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> > > Thank you for fixing it, > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > If you like, I can put it into the fixes branch of the soc tree directly, > otherwise I expect you'll send it in a pull request later. It's not a serious bug and was there since long time, so maybe let's send it via regular pull request. Best regards, Krzysztof
On Thu, Nov 11, 2021 at 10:56 AM Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> wrote: > > On 11/11/2021 10:46, Arnd Bergmann wrote: > > On Thu, Nov 11, 2021 at 10:11 AM Krzysztof Kozlowski > > <krzysztof.kozlowski@canonical.com> wrote: > >> > >> Include the header with prototype of s3c2410_modify_misccr to fix W=1 > >> warning: > >> > >> arch/arm/mach-s3c/gpio-samsung.c:1309:14: warning: > >> no previous prototype for ‘s3c2410_modify_misccr’ [-Wmissing-prototypes] > >> > >> Reported-by: kernel test robot <lkp@intel.com> > >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> > > > > Thank you for fixing it, > > > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > > > If you like, I can put it into the fixes branch of the soc tree directly, > > otherwise I expect you'll send it in a pull request later. > > It's not a serious bug and was there since long time, so maybe let's > send it via regular pull request. Ok, sounds good. I was worried for a bit that this would break builds with CONFIG_WERROR, but of course that is already broken in many ways with W=1. Arnd
diff --git a/arch/arm/mach-s3c/gpio-samsung.c b/arch/arm/mach-s3c/gpio-samsung.c index 76ef415789f2..fda2c01f5a08 100644 --- a/arch/arm/mach-s3c/gpio-samsung.c +++ b/arch/arm/mach-s3c/gpio-samsung.c @@ -35,6 +35,7 @@ #include "gpio-core.h" #include "gpio-cfg.h" #include "gpio-cfg-helpers.h" +#include "hardware-s3c24xx.h" #include "pm.h" int samsung_gpio_setpull_updown(struct samsung_gpio_chip *chip,
Include the header with prototype of s3c2410_modify_misccr to fix W=1 warning: arch/arm/mach-s3c/gpio-samsung.c:1309:14: warning: no previous prototype for ‘s3c2410_modify_misccr’ [-Wmissing-prototypes] Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> --- arch/arm/mach-s3c/gpio-samsung.c | 1 + 1 file changed, 1 insertion(+)