[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Wed, 4 Apr 2018 10:00:39 +0200
From: Mike Looijmans <mike.looijmans@...ic.nl>
To: Jarkko Nikula <jarkko.nikula@...ux.intel.com>
CC: <linux-kernel@...r.kernel.org>, <netdev@...r.kernel.org>,
Andrew Lunn <andrew@...n.ch>,
Florian Fainelli <f.fainelli@...il.com>,
"David S. Miller" <davem@...emloft.net>
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: mike.looijmans@...icproducts.com
Website: www.topicproducts.com
Please consider the environment before printing this e-mail
Powered by blists - more mailing lists