]> git.baikalelectronics.ru Git - kernel.git/commit
drm/sti: Depend on OF rather than selecting it
authorOliver O'Halloran <oohall@gmail.com>
Tue, 3 Apr 2018 05:34:01 +0000 (15:34 +1000)
committerPhilippe Cornu <philippe.cornu@st.com>
Thu, 5 Apr 2018 12:04:17 +0000 (14:04 +0200)
commit4e411aa90c7ce97619ee8dad627fc7240e939974
tree81c402ecc6a012dd3d39a9f53a2dc46de232cfa2
parent9e1b6873cbea4fc1d83e50ccbf5cffa0839f6c41
drm/sti: Depend on OF rather than selecting it

Commit 6ce085b9fd06 ("drm: sti: remove useless fields from vtg
structure") reworked some code inside of this driver and made it select
CONFIG_OF. This results in the entire OF layer being enabled when
building an allmodconfig on ia64. OF on ia64 is completely unsupported
so this isn't a great state of affairs.

The 0day robot noticed a link-time failure on ia64 caused by
using of_node_to_nid() in an otherwise unrelated driver. The
generic fallback for of_node_to_nid() only exists when:

defined(CONFIG_OF) && defined(CONFIG_NUMA) == false

Since CONFIG_NUMA is usually selected for IA64 we get the link failure.
Fix this by making the driver depend on OF rather than selecting it,
odds are that was the original intent.

Link: https://lists.01.org/pipermail/kbuild-all/2018-March/045172.html
Fixes: 6ce085b9fd06 ("drm: sti: remove useless fields from vtg structure")
Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Cc: Vincent Abriou <vincent.abriou@st.com>
Cc: David Airlie <airlied@linux.ie>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-ia64@vger.kernel.org
Cc: stable@vger.kernel.org
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Signed-off-by: Philippe Cornu <philippe.cornu@st.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180403053401.30045-1-oohall@gmail.com
drivers/gpu/drm/sti/Kconfig