Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread kbuild test robot
Hi Philipp, [auto build test ERROR on pza/reset/next] [also build test ERROR on v4.8-rc3 next-20160824] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Alban
On Wed, 24 Aug 2016 15:28:53 +0200 Philipp Zabel wrote: > Visible only if COMPILE_TEST is enabled, this allows to include the > driver in build tests. > > Cc: Alban Bedel > Signed-off-by: Philipp Zabel Acked-by: Aban Bedel Alban pgpJ53OvyG2pP.pgp Description: OpenPGP digital signature

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Arnd Bergmann
On Thursday, August 25, 2016 6:31:48 PM CEST kbuild test robot wrote: > >drivers/reset/reset-ath79.c: In function 'ath79_reset_update': > >> drivers/reset/reset-ath79.c:38:2: error: implicit declaration of function > >> 'readl' [-Werror=implicit-function-declaration] > >> drivers/reset/reset-

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Philipp Zabel
Am Donnerstag, den 25.08.2016, 12:50 +0200 schrieb Arnd Bergmann: > On Thursday, August 25, 2016 6:31:48 PM CEST kbuild test robot wrote: > > > >drivers/reset/reset-ath79.c: In function 'ath79_reset_update': > > >> drivers/reset/reset-ath79.c:38:2: error: implicit declaration of > > >> functi

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Philipp Zabel
Am Donnerstag, den 25.08.2016, 16:27 +0900 schrieb Masahiro Yamada: > 2016-08-25 16:22 GMT+09:00 Philipp Zabel : > > And the hi6220 > > driver, which doesn't have a user yet. > > It does. > > See arch/arm64/configs/defconfig My mistake, I missed that hi6220_reset_init is a postcore_initcall, so

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Masahiro Yamada
2016-08-25 16:22 GMT+09:00 Philipp Zabel : > And the hi6220 > driver, which doesn't have a user yet. It does. See arch/arm64/configs/defconfig -- Best Regards Masahiro Yamada

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-25 Thread Philipp Zabel
Am Mittwoch, den 24.08.2016, 22:06 +0200 schrieb Arnd Bergmann: > On Thursday, August 25, 2016 3:18:55 AM CEST Masahiro Yamada wrote: > > Hi Arnd, > > > > > > 2016-08-25 0:51 GMT+09:00 Arnd Bergmann : > > > On Wednesday, August 24, 2016 3:28:53 PM CEST Philipp Zabel wrote: > > >> if RESET_CONTRO

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Masahiro Yamada
2016-08-25 5:06 GMT+09:00 Arnd Bergmann : > On Thursday, August 25, 2016 3:18:55 AM CEST Masahiro Yamada wrote: >> Hi Arnd, >> >> >> 2016-08-25 0:51 GMT+09:00 Arnd Bergmann : >> > On Wednesday, August 24, 2016 3:28:53 PM CEST Philipp Zabel wrote: >> >> if RESET_CONTROLLER >> >> >> >> +config RESET

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Arnd Bergmann
On Thursday, August 25, 2016 3:18:55 AM CEST Masahiro Yamada wrote: > Hi Arnd, > > > 2016-08-25 0:51 GMT+09:00 Arnd Bergmann : > > On Wednesday, August 24, 2016 3:28:53 PM CEST Philipp Zabel wrote: > >> if RESET_CONTROLLER > >> > >> +config RESET_ATH79 > >> + bool "AR71xx Reset Driver" if

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Masahiro Yamada
Hi Arnd, 2016-08-25 0:51 GMT+09:00 Arnd Bergmann : > On Wednesday, August 24, 2016 3:28:53 PM CEST Philipp Zabel wrote: >> if RESET_CONTROLLER >> >> +config RESET_ATH79 >> + bool "AR71xx Reset Driver" if COMPILE_TEST >> + default ATH79 >> + help >> + This enables the AT

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Masahiro Yamada
2016-08-24 22:28 GMT+09:00 Philipp Zabel : > Visible only if COMPILE_TEST is enabled, this allows to include the > driver in build tests. > > Cc: Alban Bedel > Signed-off-by: Philipp Zabel Reviewed-by: Masahiro Yamada -- Best Regards Masahiro Yamada

Re: [PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Arnd Bergmann
On Wednesday, August 24, 2016 3:28:53 PM CEST Philipp Zabel wrote: > if RESET_CONTROLLER > > +config RESET_ATH79 > + bool "AR71xx Reset Driver" if COMPILE_TEST > + default ATH79 > + help > + This enables the ATH79 reset controller driver that supports the > + AR

[PATCH 01/10] reset: ath79: add driver Kconfig option

2016-08-24 Thread Philipp Zabel
Visible only if COMPILE_TEST is enabled, this allows to include the driver in build tests. Cc: Alban Bedel Signed-off-by: Philipp Zabel --- drivers/reset/Kconfig | 7 +++ drivers/reset/Makefile | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/reset/Kconfig b/dr