2018-04-04 07:23:54

by Jarkko Nikula

[permalink] [raw]
Subject: Build failure due commit 9217e566bdee

Hi

Today's head and linux-next doesn't compile due commit 9217e566bdee
("of_net: Implement of_get_nvmem_mac_address helper"):

drivers/of/of_net.o: In function `of_get_nvmem_mac_address':
.../drivers/of/of_net.c:100: undefined reference to `of_nvmem_cell_get'
.../drivers/of/of_net.c:104: undefined reference to `nvmem_cell_read'
.../drivers/of/of_net.c:106: undefined reference to `nvmem_cell_put'
Makefile:1033: recipe for target 'vmlinux' failed
make: *** [vmlinux] Error 1

Builds if I set CONFIG_NVMEM=y from CONFIG_NVMEM=m or revert above commit.

--
Jarkko


2018-04-04 09:16:14

by Mike Looijmans

[permalink] [raw]
Subject: Re: Build failure due commit 9217e566bdee

On 04-04-18 09:22, Jarkko Nikula wrote:
> Hi
>
> Today's head and linux-next doesn't compile due commit 9217e566bdee ("of_net:
> Implement of_get_nvmem_mac_address helper"):
>
> drivers/of/of_net.o: In function `of_get_nvmem_mac_address':
> .../drivers/of/of_net.c:100: undefined reference to `of_nvmem_cell_get'
> .../drivers/of/of_net.c:104: undefined reference to `nvmem_cell_read'
> .../drivers/of/of_net.c:106: undefined reference to `nvmem_cell_put'
> Makefile:1033: recipe for target 'vmlinux' failed
> make: *** [vmlinux] Error 1
>
> Builds if I set CONFIG_NVMEM=y from CONFIG_NVMEM=m or revert above commit.
>


I've been looking into it, but haven't come up with a proper solution.

The issue is that OF_NET now requires NVMEM as either not-present or built-in,
but cannot support it as module.

I don't know if that requirement can be expressed. I think having nvnmem_core
as a module doesn't make much sense anyway, but that's not up to me.

Moving the code from of to nvmem seems weird, but would solve it.



Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: [email protected]
Website: http://www.topicproducts.com

Please consider the environment before printing this e-mail