From patchwork Fri Apr 17 15:55:45 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 11495493 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id C5A2415AB for ; Fri, 17 Apr 2020 15:57:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B822D2223C for ; Fri, 17 Apr 2020 15:57:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729625AbgDQP5A (ORCPT ); Fri, 17 Apr 2020 11:57:00 -0400 Received: from mout.kundenserver.de ([212.227.17.10]:50399 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729282AbgDQP5A (ORCPT ); Fri, 17 Apr 2020 11:57:00 -0400 Received: from threadripper.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue109 [212.227.15.145]) with ESMTPA (Nemesis) id 1MdeSt-1iprPw3KU5-00ZfOr; Fri, 17 Apr 2020 17:56:21 +0200 From: Arnd Bergmann To: dri-devel@lists.freedesktop.org, Jani Nikula , linux-fbdev@vger.kernel.org Cc: Arnd Bergmann , Nicolas Pitre , Andrzej Hajda , Jason Gunthorpe , Saeed Mahameed , masahiroy@kernel.org, Laurent.pinchart@ideasonboard.com, linux-renesas-soc@vger.kernel.org, kieran.bingham+renesas@ideasonboard.com, airlied@linux.ie, daniel@zonque.org, haojian.zhuang@gmail.com, robert.jarzmik@free.fr, daniel@ffwll.ch, marex@denx.de, stefan@agner.ch, linux-graphics-maintainer@vmware.com, thellstrom@vmware.com, jfrederich@gmail.com, dsd@laptop.org, geert@linux-m68k.org Subject: [PATCH 0/8] drm, fbdev: rework dependencies Date: Fri, 17 Apr 2020 17:55:45 +0200 Message-Id: <20200417155553.675905-1-arnd@arndb.de> X-Mailer: git-send-email 2.26.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:ihjqslpA75qTMF6wDF4+VNNTC1NuRX5DyjEZChnbH2+LduferQW jC0LdL6btHPVgdMOdMSVPpZ+znLPsqbExhraTQcyb9rSJjMpz9m6IC8Rv/hCKvIPAJcBGd0 hoMUlKQPUxa5CWHHDUiR9HooUGxQKazIExDHqcmkZ2XigqkWeJHResaEL4oAKQcpaZIWAwD ruyD6lxdbXrDz80gK0WHg== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:3tM+z2ZlTSg=:Y+9HE2EHWXdISGlEwzfCoA znjKjlR25yaG0teoXA4D4CeSSX+90bjonxwvfbjYZQQcAEg91D0TCOo68kMmgHgdjxSeZ/cQE h7ea4mzL0RV4U9lI8gHxzNG7B0A+rf1wcqF+02xr5G+kX4C2wqgW9HW3lxAW7aktqf4q5efZA JXLi+n2fEBOILATNtcdS5xjaMq/0B3qRcSs/zMQKeCDxG+MvNjdR3AMR8grIoOesVD0m0wUvD yeODH5YXBaQtCzFrIkoYRb1enxS0bPVu6rTX8POeJ4YVCxP1ABi+zbsuT4BXoeW7Iap00s+hz 7xlD1BV37+gDlcxZ4GZUzs5tkkaPQIkMCGCvUJOJ3rRut1JwC7X/lr3drfx9hw5cjIaiE1LMC JgK5KNaFDR7oPdLrOS/C1OEbrOiqB8gKsjT5ZPt+0NtBNIJtfgyv9fvvcX75UR1+nfhojAaLF xC49KgXOQS2jnvO1w1VJdB5lRJmfPWJQDsRXP5sZTVRzYeLLqC0EHgTreGAoUOcQ+oxV5QDCL T7OI5dkyb/qbyPE8C09N72z19941QqlXgY5M0x2NkO9rzCPWhDkHqXL9ivrf2NiNfu/G4uOal BEFmbx0RKEk2vk9zgrHJasgEMe36VSFu6DggJ6Gk7Swe0drJO7GkfcCaQ5/jzUA+hgDkAMlWa z5WnF+9VTwvi6zaTpktEnJEvJPz1hiLDMUl/Xt+t2c8zGxhmhMNv3lkHXnPQIcSk+YBRNVSPs 5L8k0Ju92kqHNPL3aAP7VTgciERckhKIKlBGQpFCZz2DlY828VoHZGEjrZjRuxqZFZfd04ylz wzH+eGBO8rrvCSJAO8TPvbKeMFvlMJr5uWCJBB8rNqGirwuBew= Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org I tried to fix up some dependencies after the sii8620 "imply EXTCON" statementn broke, trying a few things but in the backing out a change that would completely reverse the LEDS_CLASS selects into a 'depends on'. However, what I got now are multiple changes that remove gratious "selects" that lead to circular dependencies for sii8620 and others: - Anything doing "select FB" is now gone, or becomes "depends on FB", - DDC support depends on I2C instead of selecting it - backlight class device support is never selected by framebuffer drivers but has proper dependencies I have done thousands of randconfig build tests on this, but no runtime tests. Some of the 'depends on FOO || !FOO' statements could be simplified into a new 'uses FOO' syntax based on a patch from Saeed Mahameed, but I would for the moment treat that as a cleanup that can be done later. If we can agree on these changes, maybe someone can merge them through the drm-misc tree. Please review Arnd Arnd Bergmann (8): fbdev: w100fb: clean up mach-pxa compile-time dependency fbdev/ARM: pxa: avoid selecting CONFIG_FB fbdev: rework FB_DDC dependencies drm/rcar: stop using 'imply' for dependencies drm/vmwgfx: make framebuffer support optional drm: decouple from CONFIG_FB fbdev: rework backlight dependencies drm/bridge/sii8620: fix extcon dependency arch/arm/configs/pxa_defconfig | 3 ++ arch/arm/mach-pxa/Kconfig | 7 --- arch/arm/mach-pxa/eseries.c | 14 +---- arch/arm/mach-pxa/saar.c | 2 +- arch/arm/mach-pxa/tavorevb.c | 2 +- drivers/auxdisplay/Kconfig | 1 + drivers/gpu/drm/Kconfig | 5 +- drivers/gpu/drm/bridge/Kconfig | 2 +- drivers/gpu/drm/mxsfb/Kconfig | 1 - drivers/gpu/drm/rcar-du/Kconfig | 23 +++++--- drivers/gpu/drm/vmwgfx/Kconfig | 17 +++--- drivers/gpu/drm/vmwgfx/Makefile | 4 +- drivers/gpu/drm/vmwgfx/vmwgfx_drv.c | 35 +++++++----- drivers/gpu/drm/zte/Kconfig | 1 - drivers/macintosh/Kconfig | 1 + drivers/staging/fbtft/Kconfig | 1 + drivers/staging/olpc_dcon/Kconfig | 2 +- drivers/video/fbdev/Kconfig | 31 ++++++++--- drivers/video/fbdev/w100fb.c | 84 +++++------------------------ include/video/w100fb.h | 6 +-- 20 files changed, 101 insertions(+), 141 deletions(-)