From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A327BC2BA80 for ; Thu, 9 Apr 2020 00:46:27 +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 7034220757 for ; Thu, 9 Apr 2020 00:46:27 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="uWTJR6Wf" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7034220757 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 F093E6EB20; Thu, 9 Apr 2020 00:46:26 +0000 (UTC) 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 2AA306EB20 for ; Thu, 9 Apr 2020 00:46:26 +0000 (UTC) Received: from pendragon.bb.dnainternet.fi (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 44AB02DC; Thu, 9 Apr 2020 02:46:24 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1586393184; bh=XrImY9BvJeKbhh8QnJm/IVyeaDkiA3BMUm0hl4NRemY=; h=From:To:Cc:Subject:Date:From; b=uWTJR6WfIbrGc6PfSOY/8QElBelrCrO2EYtcfL/UQe6sA/Z9mAcZq+38t6Am94rtP xJj0A4Q/HhvjAC9ESjB5d+crmS5gEVkx7DRsaFeUOM9CagW96+DdomWfocD6N1mRsJ fFuqpl+1vTt5QdWHsCMsBJpGU0YByMJ2YivIWmAg= From: Laurent Pinchart To: dri-devel@lists.freedesktop.org Subject: [PATCH 0/4] drm: bridge: adv7511: Enable usage with DRM bridge connector helper Date: Thu, 9 Apr 2020 03:46:06 +0300 Message-Id: <20200409004610.12346-1-laurent.pinchart+renesas@ideasonboard.com> X-Mailer: git-send-email 2.24.1 MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Rob Clark , Jernej Skrabec , Jonas Karlman , Neil Armstrong , linux-renesas-soc@vger.kernel.org, Andrzej Hajda , Sean Paul Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hello, This patch series enables usage of the adv7511 driver with the DRM bridge connector helper (drm_bridge_connector.c). Patch 1/4 and 2/4 start by splitting EDID read and connector creation to separate functions to ease review of 3/4 and 4/4. Patch 3/4 performs the bulk of the work by implementing the DRM bridge connector-related operations, and patch 4/4 then makes connector creation optional. I've had trouble wrapping my head around the HPD handling code in the av7511 driver (this is why I've CC'ed Rob and Sean who last touched this). The split of the code between the .detect() operation and the HPD IRQ seems a bit weird to me, and I haven't dared touching it as it also appears fragile. In particular, I'm not sure why we need to retrieve modes in the .detect() operation. git blame didn't help, as the code has been there since the beginning. I'd like to remove that completely, but to avoid breakages, patch 3/4 only does so when the adv7511 is used without creating a DRM connector. Usage of both adv7511->status and adv7511->connector.status in the two operations also seems awkward, and I would like to drop usage of the latter in the new code path, but I also haven't dared refactoring that yet. Feedback would be welcome. Laurent Pinchart (4): drm: bridge: adv7511: Split EDID read to a separate function drm: bridge: adv7511: Split connector creation to a separate function drm: bridge: adv7511: Implement bridge connector operations drm: bridge: adv7511: Make connector creation optional drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 137 +++++++++++++------ 1 file changed, 98 insertions(+), 39 deletions(-) -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel