From patchwork Fri Feb 12 12:04:40 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 8290421 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 0465FBEEE5 for ; Fri, 12 Feb 2016 12:09:58 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 29933203EB for ; Fri, 12 Feb 2016 12:09:57 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 14111203E9 for ; Fri, 12 Feb 2016 12:09:56 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1aUCWd-0005At-Qs; Fri, 12 Feb 2016 12:08:39 +0000 Received: from mout.kundenserver.de ([212.227.126.130]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1aUCWX-00052b-HK for linux-arm-kernel@lists.infradead.org; Fri, 12 Feb 2016 12:08:36 +0000 Received: from wuerfel.lan. ([78.42.132.4]) by mrelayeu.kundenserver.de (mreue003) with ESMTPA (Nemesis) id 0MIRzP-1aRAbW2ll1-0049jg; Fri, 12 Feb 2016 13:05:29 +0100 From: Arnd Bergmann To: Andrew Jones , Konrad Rzeszutek Wilk Subject: [PATCH] xen kconfig: clarify INPUT_XEN_KBDDEV_FRONTEND select Date: Fri, 12 Feb 2016 13:04:40 +0100 Message-Id: <1455278707-2008263-1-git-send-email-arnd@arndb.de> X-Mailer: git-send-email 2.7.0 X-Provags-ID: V03:K0:GS9pxNJWKUNuy/wX2avT8l5Qjxbaz72im+RaI1dK+fScq8J/0y8 RqK+jW6H6bPTudFCDtByCQSGqE1xx23e5+Mjwc+kLPG68ODWHu43JwhH8EFs9fxbhe9XBpY xktLUQT+afptbpa7AYA4waX2pSVpeoZQErgMDaMUUSGApPvwJyulc+4sitXrCRt5zwXauuu fh7Xf4u99vKDMetGCNacQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:e8ERRFrAeKk=:fT5KTQqeCbx8D3DIV13oR7 1OlRoQxi8zul0UkuHHzH3VbRF1UJARYL9XhnvQie1qm0kfuZJ6+UnqFFC1/DvswR8FvoPUyDO Fj8C+zOqD2RF7XiAhHlckaPh1sIFfDsgnLG62UxvaBNmFoCOuCIxIU7R5J8KVcssWdqXc9vEG 364JFRJA6r27zt0WbCK/LGvv5Lq8ePzmzCThCETEmXdsKQaa8LOj14FbyqJLmTiBLjctYQoZf 48eLwAM9C0Sso0c2+R/QJluJJj5H1AAn9LnLiHGTjC1COPVUHL731C6ZWfhLJToWs1p2ucIxI 88KpXEDZ+OnU6Qw3ivpGfGgt57wGY40jKW/maG8/xi7mKRuN5h1BI/SbBa2NMY2q/un4bOpYo od5g0CGu0Jx89zULGXO87c4eV39YGf24EHddkZKyGizhHu3uUxWkEyZwBexs0oLSMJsKOyJ9v DL4Oy3oLcvq+C51OxXl2/WcTlSlsFYpNsWLZAPUqkhc7bggBill3Vli9vGo/oXWWXN4yBeGo7 BtRD+aPmqXgt+CsOCwnqvyJ5+R0K4eEYSMhDgQ+xZ2pO+YJYV+7+Tp45/u/ZK79iDEgjZKhme 6+QY9xtRG3pAzo9VX+iNprv/EJDlgr8sgm7lR54mrXQTtiHRsl5x0P9Zer5YpnrNyLPns+auS QtBQ7sCvaFlIkfgG9UwAOeAkQrN9OC469gp0fjzqWCSBOX2oJPZuI1o47FlltBSCDYgA= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20160212_040834_094296_BC5DBE12 X-CRM114-Status: GOOD ( 13.08 ) X-Spam-Score: -1.9 (-) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-fbdev@vger.kernel.org, Arnd Bergmann , linux-kernel@vger.kernel.org, Tomi Valkeinen , Jean-Christophe Plagniol-Villard , linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-4.3 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP The Xen framebuffer driver selects the xen keyboard driver, so the latter will be built-in if XEN_FBDEV_FRONTEND=y. However, when CONFIG_INPUT is a loadable module, this configuration cannot work. On mainline kernels, the symbol will be enabled but not used, while in combination with a patch I have to detect such useless configurations, we get the expected link failure: drivers/input/built-in.o: In function `xenkbd_remove': xen-kbdfront.c:(.text+0x2f0): undefined reference to `input_unregister_device' xen-kbdfront.c:(.text+0x30e): undefined reference to `input_unregister_device' This changes the dependencies of XEN_FBDEV_FRONTEND so it cannot be built-in if CONFIG_INPUT=m && CONFIG_INPUT_MISC=y, as that would result in the broken select. As usual, we would be much better off without the 'select', but removing it now would likely break existing user configurations that depend on it, so this adds another hack on top to get it working. Signed-off-by: Arnd Bergmann Fixes: 36c1132e34bd ("xen kconfig: fix select INPUT_XEN_KBDDEV_FRONTEND") Reviewed-by: Andrew Jones --- drivers/video/fbdev/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/video/fbdev/Kconfig b/drivers/video/fbdev/Kconfig index 8ea45a5cd806..fd3d6fd290a9 100644 --- a/drivers/video/fbdev/Kconfig +++ b/drivers/video/fbdev/Kconfig @@ -2241,6 +2241,7 @@ config FB_VIRTUAL config XEN_FBDEV_FRONTEND tristate "Xen virtual frame buffer support" depends on FB && XEN + depends on INPUT || !INPUT_MISC select FB_SYS_FILLRECT select FB_SYS_COPYAREA select FB_SYS_IMAGEBLIT