From patchwork Wed Aug 21 18:50:00 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Pinchart X-Patchwork-Id: 11108027 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 5F56B912 for ; Wed, 21 Aug 2019 18:50:17 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 48105214DA for ; Wed, 21 Aug 2019 18:50:17 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 48105214DA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 389DE6E38D; Wed, 21 Aug 2019 18:50:16 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by gabe.freedesktop.org (Postfix) with ESMTPS id A8CBF6E340 for ; Wed, 21 Aug 2019 18:50:15 +0000 (UTC) Received: from pendragon.bb.dnainternet.fi (dfj612yhrgyx302h3jwwy-3.rev.dnainternet.fi [IPv6:2001:14ba:21f5:5b00:ce28:277f:58d7:3ca4]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 0A6C533D; Wed, 21 Aug 2019 20:50:13 +0200 (CEST) From: Laurent Pinchart To: dri-devel@lists.freedesktop.org Subject: [PATCH/RFC 0/5] Decouple EDID retrieval from drm_connector Date: Wed, 21 Aug 2019 21:50:00 +0300 Message-Id: <20190821185005.9789-1-laurent.pinchart@ideasonboard.com> X-Mailer: git-send-email 2.21.0 MIME-Version: 1.0 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1566413414; bh=wlsR7rdAiklreUp0q7OxHNou6vm9GVZ8vCKTA3UcJd4=; h=From:To:Cc:Subject:Date:From; b=ZnUD/MpX+APEb96towEaTbztPgPZ6uA0Y8+iW8e1OLR1ULg9TMzqX/iKuUY4iHR4K c0uZqiaxxpCyM7KisNDz38JnA+b4EFc25hNIJCr6SRomI/xnYyWZOoOOnlNTYD18tk /Ni5SpmWyXRRQ6A/M2uLRnl1f7iHlfcSeEpHlRwg= 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: Tomi Valkeinen Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hello, This small patch series attemps at decoupling EDID retrieval from drm_connector, following a discussion with Daniel Vetter [1]. While working on this I noticed a few issues with EDID retrieval, which I have attempted to fix in patches 1/5 to 4/5. Patch 5/5 then tries to decouple the EDID retrieval from drm_connector, in what I think is a pretty bad way. I would like to discuss this further to see if a) there's an interest, and b) someone has a better idea :-) Regardless of the outcome of the discussion on patch 5/5, the first four patches are candidates for upstreaming as they try to address existing issues (they may also introduce new bugs, but hopefully they will be pointed out during review). The patches are available at git://linuxtv.org/pinchartl/media.git omapdrm/edid [1] https://lists.freedesktop.org/archives/dri-devel/2019-August/231930.html Laurent Pinchart (5): drm/edid: Reorganise the DisplayID parsing code drm/edid: Move functions to avoid forward declaration drm/edid: Move DisplayID tile parsing to drm_connector.c drm/edid: Honour connector->force in drm_do_get_edid() [HACK] drm/edid: Decouple EDID retrieval from connector drivers/gpu/drm/drm_connector.c | 137 +++- drivers/gpu/drm/drm_dp_mst_topology.c | 3 +- drivers/gpu/drm/drm_edid.c | 857 ++++++++++++-------------- include/drm/drm_connector.h | 3 +- include/drm/drm_displayid.h | 2 + include/drm/drm_edid.h | 2 + 6 files changed, 534 insertions(+), 470 deletions(-)