From patchwork Thu Aug 16 18:54:14 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sean Paul X-Patchwork-Id: 10568049 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 15C98921 for ; Thu, 16 Aug 2018 18:54:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 0E4042B47A for ; Thu, 16 Aug 2018 18:54:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 027F02B844; Thu, 16 Aug 2018 18:54:19 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 8B60F2B47A for ; Thu, 16 Aug 2018 18:54:19 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A10056E4F7; Thu, 16 Aug 2018 18:54:17 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-yb0-x242.google.com (mail-yb0-x242.google.com [IPv6:2607:f8b0:4002:c09::242]) by gabe.freedesktop.org (Postfix) with ESMTPS id 830EE6E4F5 for ; Thu, 16 Aug 2018 18:54:16 +0000 (UTC) Received: by mail-yb0-x242.google.com with SMTP id d18-v6so1018546ybq.5 for ; Thu, 16 Aug 2018 11:54:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=OtB+k98b5SQGH7KJPw21QceUoqDsD+/5rkZmVWyCZM4=; b=MgAE/DmcI2xTEwAfGf1MrhFcrecGLYQEi7gRHj42HiInCaHXEu7FXllGfhtDqishZp 2Nw3o2WAggVvSy9+PESIDIfRInQ7uQHj5UbcUPyAs3hsbNjKJoFiWxt6MDC0D6cCxIo8 lQzq2tVThrAeIiFVKgwaGoOh6MxGmWQsAIE3cesFXLTGlRisMFbkmB53ql8slC7BusXy XUD3668Pl/9UcDgu0r43d53dFSju2q3ZgiPDLcW0MwR3B6ZoGcNYjelh++6Od1Z63lDL zeNYdQ7xfGx4XYOdViFOHzlnC1b/JMRLfqD1jHi0ll8t4WdsXTQl8UxoAxnLEZZBUJ8Y 9skA== X-Gm-Message-State: AOUpUlHuIvuuJCa1wSUn7KDTtdmuAjrXEXo+MEIbySB5vtSdTkACBTpq luozXWCO6YVvQUoEcY9xdcso8HWJpD4= X-Google-Smtp-Source: AA+uWPzEWvducT+zM66OXu5XuWhKi/DSS+XCNlF0YJPQGbxB8EOzdn8GmYFYVJ65l3jxPfjl0iielg== X-Received: by 2002:a25:73c4:: with SMTP id o187-v6mr10680ybc.69.1534445655665; Thu, 16 Aug 2018 11:54:15 -0700 (PDT) Received: from rosewood.cam.corp.google.com ([2620:0:1013:11:ad55:b1db:adfe:3b9f]) by smtp.gmail.com with ESMTPSA id f3-v6sm26751ywh.47.2018.08.16.11.54.15 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 16 Aug 2018 11:54:15 -0700 (PDT) From: Sean Paul To: dri-devel@lists.freedesktop.org Subject: [PATCH] drm: Rationalize DRM_MIPI_DSI inclusion in Kconfig Date: Thu, 16 Aug 2018 14:54:14 -0400 Message-Id: <20180816185414.104152-1-sean@poorly.run> X-Mailer: git-send-email 2.18.0.865.gffc8e1a3cd6-goog MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jani Nikula , Thierry Reding , Sean Paul Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP From: Sean Paul DRM_MIPI_DSI is included via both "select" and "depends on", this is trouble waiting to happen since this will result in different behavior depending on which is used. This patch resolves the problem by: - Converting all inclusion to "select" since DRM_MIPI_DSI is not a menu item. - Remove "depends on DRM" since "select"ing DRM_MIPI_DSI won't include DRM anyways, so this is misleading. Cc: Jani Nikula Cc: Thierry Reding Suggested-by: Daniel Vetter Signed-off-by: Sean Paul Reviewed-by: Daniel Vetter Acked-by: Jani Nikula --- drivers/gpu/drm/Kconfig | 1 - drivers/gpu/drm/panel/Kconfig | 22 +++++++++++----------- 2 files changed, 11 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig index cb88528e7b10..197691b10c6b 100644 --- a/drivers/gpu/drm/Kconfig +++ b/drivers/gpu/drm/Kconfig @@ -25,7 +25,6 @@ menuconfig DRM config DRM_MIPI_DSI bool - depends on DRM config DRM_DP_AUX_CHARDEV bool "DRM DP AUX Interface" diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig index 6020c30a33b3..a1e00e9fa566 100644 --- a/drivers/gpu/drm/panel/Kconfig +++ b/drivers/gpu/drm/panel/Kconfig @@ -49,7 +49,7 @@ config DRM_PANEL_ILITEK_IL9322 config DRM_PANEL_ILITEK_ILI9881C tristate "Ilitek ILI9881C-based panels" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y if you want to enable support for panels based on the @@ -58,7 +58,7 @@ config DRM_PANEL_ILITEK_ILI9881C config DRM_PANEL_INNOLUX_P079ZCA tristate "Innolux P079ZCA panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Innolux P079ZCA @@ -69,7 +69,7 @@ config DRM_PANEL_INNOLUX_P079ZCA config DRM_PANEL_JDI_LT070ME05000 tristate "JDI LT070ME05000 WUXGA DSI panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for JDI DSI video mode @@ -93,7 +93,7 @@ config DRM_PANEL_LG_LG4573 config DRM_PANEL_ORISETECH_OTM8009A tristate "Orise Technology otm8009a 480x800 dsi 2dl panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Orise Technology @@ -102,7 +102,7 @@ config DRM_PANEL_ORISETECH_OTM8009A config DRM_PANEL_PANASONIC_VVX10F034N00 tristate "Panasonic VVX10F034N00 1920x1200 video mode panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Panasonic VVX10F034N00 @@ -111,7 +111,7 @@ config DRM_PANEL_PANASONIC_VVX10F034N00 config DRM_PANEL_RASPBERRYPI_TOUCHSCREEN tristate "Raspberry Pi 7-inch touchscreen panel" - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI help Say Y here if you want to enable support for the Raspberry Pi 7" Touchscreen. To compile this driver as a module, @@ -120,7 +120,7 @@ config DRM_PANEL_RASPBERRYPI_TOUCHSCREEN config DRM_PANEL_RAYDIUM_RM68200 tristate "Raydium RM68200 720x1280 DSI video mode panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Raydium RM68200 @@ -129,14 +129,14 @@ config DRM_PANEL_RAYDIUM_RM68200 config DRM_PANEL_SAMSUNG_S6E3HA2 tristate "Samsung S6E3HA2 DSI video mode panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE select VIDEOMODE_HELPERS config DRM_PANEL_SAMSUNG_S6E63J0X03 tristate "Samsung S6E63J0X03 DSI command mode panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE select VIDEOMODE_HELPERS @@ -158,7 +158,7 @@ config DRM_PANEL_SEIKO_43WVF1G config DRM_PANEL_SHARP_LQ101R1SX01 tristate "Sharp LQ101R1SX01 panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Sharp LQ101R1SX01 @@ -172,7 +172,7 @@ config DRM_PANEL_SHARP_LQ101R1SX01 config DRM_PANEL_SHARP_LS043T1LE01 tristate "Sharp LS043T1LE01 qHD video mode panel" depends on OF - depends on DRM_MIPI_DSI + select DRM_MIPI_DSI depends on BACKLIGHT_CLASS_DEVICE help Say Y here if you want to enable support for Sharp LS043T1LE01 qHD