Linux-Crypto Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] hwrng: ba431 - Add dependency on HAS_IOMEM
       [not found] <202006292036.INytijnT%lkp@intel.com>
@ 2020-06-30  4:20 ` Herbert Xu
  2020-06-30  6:58   ` Olivier Sobrie
  0 siblings, 1 reply; 2+ messages in thread
From: Herbert Xu @ 2020-06-30  4:20 UTC (permalink / raw)
  To: kernel test robot
  Cc: Olivier Sobrie, kbuild-all, Waleed Ziad,
	Linux Crypto Mailing List, Arnd Bergmann

On Mon, Jun 29, 2020 at 08:46:38PM +0800, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> head:   c28e58ee9dadc99f79cf16ca805221feddd432ad
> commit: 0289e9be5dc26d84dda6fc5492f08ca1ff599744 [1846/4145] hwrng: ba431 - add support for BA431 hwrng
> config: s390-zfcpdump_defconfig (attached as .config)
> compiler: s390-linux-gcc (GCC) 9.3.0
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 0289e9be5dc26d84dda6fc5492f08ca1ff599744
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>    s390-linux-ld: drivers/char/hw_random/ba431-rng.o: in function `ba431_trng_probe':
> >> ba431-rng.c:(.text+0xd4): undefined reference to `devm_ioremap_resource'

This patch should fix the problem:

---8<---
The ba431 driver depends on HAS_IOMEM and this was missing from
the Kconfig file.

Reported-by: kernel test robot <lkp@intel.com>
Fixes: 0289e9be5dc2 ("hwrng: ba431 - add support for BA431 hwrng")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig
index 7b876dfdbaaf..edbaf6154764 100644
--- a/drivers/char/hw_random/Kconfig
+++ b/drivers/char/hw_random/Kconfig
@@ -76,6 +76,7 @@ config HW_RANDOM_ATMEL
 
 config HW_RANDOM_BA431
 	tristate "Silex Insight BA431 Random Number Generator support"
+	depends on HAS_IOMEM
 	default HW_RANDOM
 	help
 	  This driver provides kernel-side support for the Random Number
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

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

* Re: [PATCH] hwrng: ba431 - Add dependency on HAS_IOMEM
  2020-06-30  4:20 ` [PATCH] hwrng: ba431 - Add dependency on HAS_IOMEM Herbert Xu
@ 2020-06-30  6:58   ` Olivier Sobrie
  0 siblings, 0 replies; 2+ messages in thread
From: Olivier Sobrie @ 2020-06-30  6:58 UTC (permalink / raw)
  To: Herbert Xu
  Cc: kernel test robot, kbuild-all, Waleed Ziad,
	Linux Crypto Mailing List, Arnd Bergmann

On Tue, Jun 30, 2020 at 02:20:37PM +1000, Herbert Xu wrote:
> On Mon, Jun 29, 2020 at 08:46:38PM +0800, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head:   c28e58ee9dadc99f79cf16ca805221feddd432ad
> > commit: 0289e9be5dc26d84dda6fc5492f08ca1ff599744 [1846/4145] hwrng: ba431 - add support for BA431 hwrng
> > config: s390-zfcpdump_defconfig (attached as .config)
> > compiler: s390-linux-gcc (GCC) 9.3.0
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         git checkout 0289e9be5dc26d84dda6fc5492f08ca1ff599744
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=s390 
> > 
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> > 
> > All errors (new ones prefixed by >>):
> > 
> >    s390-linux-ld: drivers/char/hw_random/ba431-rng.o: in function `ba431_trng_probe':
> > >> ba431-rng.c:(.text+0xd4): undefined reference to `devm_ioremap_resource'
> 
> This patch should fix the problem:
> 
> ---8<---
> The ba431 driver depends on HAS_IOMEM and this was missing from
> the Kconfig file.

Indeed, it was missing. Sorry for that and thanks for the fix!

> 
> Reported-by: kernel test robot <lkp@intel.com>
> Fixes: 0289e9be5dc2 ("hwrng: ba431 - add support for BA431 hwrng")
> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
> 
> diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig
> index 7b876dfdbaaf..edbaf6154764 100644
> --- a/drivers/char/hw_random/Kconfig
> +++ b/drivers/char/hw_random/Kconfig
> @@ -76,6 +76,7 @@ config HW_RANDOM_ATMEL
>  
>  config HW_RANDOM_BA431
>  	tristate "Silex Insight BA431 Random Number Generator support"
> +	depends on HAS_IOMEM
>  	default HW_RANDOM
>  	help
>  	  This driver provides kernel-side support for the Random Number
> -- 
> Email: Herbert Xu <herbert@gondor.apana.org.au>
> Home Page: http://gondor.apana.org.au/~herbert/
> PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

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

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <202006292036.INytijnT%lkp@intel.com>
2020-06-30  4:20 ` [PATCH] hwrng: ba431 - Add dependency on HAS_IOMEM Herbert Xu
2020-06-30  6:58   ` Olivier Sobrie

Linux-Crypto Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-crypto/0 linux-crypto/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-crypto linux-crypto/ https://lore.kernel.org/linux-crypto \
		linux-crypto@vger.kernel.org
	public-inbox-index linux-crypto

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-crypto


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git