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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3FC66C54FB3 for ; Mon, 26 May 2025 21:15:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=AVwR3RqXfhEXlhoGhmwUftz4FLAY7joifHIAsoI1opk=; b=g/BXBNtX7xSQd16qK199PgbQky Ow0S9iaoJyHf6i6OaEaBIuwCZmzThW7gBJ6SoUL/7fEB1zFOcsLBMQ3vAutaZOuWDMo2JZMfs+DrU rUvUjwVoL1vyOHtA/o6nR02wG43lsbMYO3c6bMzi8rpeFadIeoWpk7v4zmtNsl5fGQOQLBPlvzp1G 6QNhk6CzRph7hymFf+qLlO6fP4RNgovldxS8P+jsgSrVKY4sUuDQtcc5CZ9OyjypRbIzxyK7UpePw qi4w8YK9pNqb0FIOtQHJvsCGqiUs3I/VpSjP2HzjjP6MX1MI5WNCu1XaH+AFaXaaxJOO8i68DPCdl t9gfDZaw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uJfAU-00000009WYi-2LS7; Mon, 26 May 2025 21:15:34 +0000 Received: from mo4-p01-ob.smtp.rzone.de ([85.215.255.52]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uJf8M-00000009WUE-3jFT for linux-arm-kernel@lists.infradead.org; Mon, 26 May 2025 21:13:24 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1748293984; cv=none; d=strato.com; s=strato-dkim-0002; b=SBnCxxSXiIiddnKsNUoos8EzazWtIvC4bKbWPUepcnmdhmXdHyJi8hPakBcnU7kHLV 1FH5Z83svo2oEQFRPYmvgJI37fZjG2rwiJhy7HsneKGaOgZuYb3fpp4fzv6Ix79ZCyv5 5I2AVD8CGGlmj3N1cqi9jz2qnzK9izi7HjDKtDa3Vqjf5uD+LinpXpvg8RIiP4XhMxYE s6uH2RutFF9AlHkJB2CGpynPnTNUN3479IG47Vfhj+WuKF6leMwJxXcolQ8LICmPPHWi 1yOIRzD853qP4Gr5NfJKKKPM8hb65JCdhTwdW/CM1Y+Ehmvy5KBKoWl8VI14x1mo2TqY tUmQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; t=1748293984; s=strato-dkim-0002; d=strato.com; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=AVwR3RqXfhEXlhoGhmwUftz4FLAY7joifHIAsoI1opk=; b=OvFyu0tIjcvYj39AWAaPt7ni0JoeD6LVrR7n650YRZzHcdXAZCSXmod9hGXMHs/mmq jIe1OFQAEDuFeDJRoLAD+m1093+HWk/mYBEjJbUxkkG2osuNFgYTH0QVAXedz1tikvdc PwNVQIQSLaWW5HWAID7+0XwpLB4g6PYIVC3r/ZFsANctmHz89HIMjvdIIP3jf4qpo44V YxJdqXYBkI+sINKQ7vTFr+bdkBIqWdzBD/awRVrg6iI0p28vEyQvO411OKNxH/DwxaV5 3v2aonPLcLUu21m/zZTuMOC/vwOcZ8DQYjrp2EWhVRTgwWLh2cBpRGKzgsmIuHKfIFXD TGow== ARC-Authentication-Results: i=1; strato.com; arc=none; dkim=none X-RZG-CLASS-ID: mo01 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1748293984; s=strato-dkim-0002; d=fossekall.de; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=AVwR3RqXfhEXlhoGhmwUftz4FLAY7joifHIAsoI1opk=; b=JFM4E4uWQrv7IX07x09XDxvaleGSg846j1gzsnGypyamNDo0kP8UKj9pHKHgHS80G2 wFEaccJ6FLcGXlYnoT0gSshqnBWB2GBcTSrg598ktHY/O7mVfp0l9y36RGTU16m4qviK 7zvqx20kzI0G9brqLkMt2kn/TW9NPps/sR8ig786i/brSpbrRdDelRBSgkV6iZEhz9AE Bsi5qgRZztzaogMEu4sSnUnV5ixy2DY3dED28gZIm1eRj10N6umG5McmY8sCD916Tc8A fWPKzm+Whivq9kQmjl94VChlLILuGoHSj4VC9bwnpHgSONZ2yzZJOiXKA7GJZPm8L/5s W0nw== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; t=1748293984; s=strato-dkim-0003; d=fossekall.de; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=AVwR3RqXfhEXlhoGhmwUftz4FLAY7joifHIAsoI1opk=; b=avm+GovrShQDZ5jb9fbwW3Txdkh+3LcP/cFmWcO5Jq1kKqSnyvhDQH/VtyYZvzOHsG PvVYJwo4CElmkKKajlAQ== X-RZG-AUTH: ":O2kGeEG7b/pS1EzgE2y7nF0STYsSLflpbjNKxx7cGrBdao6FTL4AJcMdm+lap4JEHkzok9eyEg==" Received: from aerfugl by smtp.strato.de (RZmta 51.3.0 AUTH) with ESMTPSA id f28b3514QLD3ZDy (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Mon, 26 May 2025 23:13:03 +0200 (CEST) Received: from koltrast.home ([192.168.1.32] helo=a98shuttle.de) by aerfugl with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1uJf83-0001y1-0A; Mon, 26 May 2025 23:13:03 +0200 Date: Mon, 26 May 2025 23:13:01 +0200 From: Michael To: Maxime Ripard Cc: Chen-Yu Tsai , Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Jernej Skrabec , Samuel Holland , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, regressions@lists.linux.dev Subject: Re: [REGRESSION] [BISECTED] drm/sun4i: hdmi: No HDMI output with BananaPI M1 on 6.9 Message-ID: References: <20250526-refined-dog-of-blizzard-b48f11@houat> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <20250526-refined-dog-of-blizzard-b48f11@houat> Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250526_141323_081444_613487D7 X-CRM114-Status: GOOD ( 13.55 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, On Mon, May 26, 2025 at 07:30:35PM +0200, Maxime Ripard wrote: >On Mon, May 12, 2025 at 10:27:06PM +0200, Michael wrote: >> with v6.9 and later there is no output on the BananaPI HDMI connector. >> >> I have bisected the issue to the following commit: >> >> 358e76fd613a ("drm/sun4i: hdmi: Consolidate atomic_check and mode_valid") >> >> With this patch, sun4i_hdmi_connector_clock_valid() is occasionally called >> with clock=0, causing the function to return MODE_NOCLOCK. >> In the old sun4i_hdmi_mode_valid() before the patch, mode->clock is >> always!=0, maybe that gives someone a hint. > >This doesn't make sense to me, if only because the two callers of >tmds_char_rate_valid (hdmi_compute_clock and >drm_hdmi_connector_mode_valid) have, right before calling it, checks to >make sure the clock rate isn't 0, and would return MODE_ERROR or EINVAL >in such a case. > >https://elixir.bootlin.com/linux/v6.15/source/drivers/gpu/drm/display/drm_hdmi_state_helper.c#L234 >https://elixir.bootlin.com/linux/v6.15/source/drivers/gpu/drm/display/drm_hdmi_state_helper.c#L553 Before 6.14, sun4i_hdmi_connector_clock_valid() was also called from sun4i_hdmi_connector_atomic_check()... >Do you have some logs (with dri.devel=0xff)? Does it happen with 6.15 as >well? It does not happen with 6.15, as it was fixed in 6.14 with 84e541b1e58e ("drm/sun4i: use drm_atomic_helper_connector_hdmi_check()"). For completeness: 358e76fd613a: [ 1.091807] sun4i-drm display-engine: [drm:drm_atomic_check_only] checking (ptrval) [ 1.091850] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CRTC:47:crtc-0] mode changed [ 1.091876] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CRTC:47:crtc-0] enable changed [ 1.091898] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CRTC:47:crtc-0] active changed [ 1.091927] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] Updating routing for [CONNECTOR:66:HDMI-A-1] [ 1.091953] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CONNECTOR:66:HDMI-A-1] using [ENCODER:65:TMDS-65] on [CRTC:47:crtc-0] [ 1.092019] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CONNECTOR:66:HDMI-A-1] driver check failed [ 1.092042] sun4i-drm display-engine: [drm:drm_atomic_check_only] atomic driver check for (ptrval) failed: -22 [ 1.092070] sun4i-drm display-engine: [drm:drm_atomic_state_default_clear] Clearing atomic state (ptrval) 358e76fd613a^ : [ 14.402623] sun4i-drm display-engine: [drm] crtc=crtc-0 [ 14.402630] sun4i-drm display-engine: [drm] self_refresh_aware=0 [ 14.402638] sun4i-drm display-engine: [drm] max_requested_bpc=0 [ 14.402647] sun4i-drm display-engine: [drm] colorspace=Default [ 14.402657] sun4i-drm display-engine: [drm:drm_atomic_check_only] checking 31bd6e91 [ 14.402714] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] Updating routing for [CONNECTOR:66:HDMI-A-1] [ 14.402741] sun4i-drm display-engine: [drm:drm_atomic_helper_check_modeset] [CONNECTOR:66:HDMI-A-1] keeps [ENCODER:65:TMDS-65], now on [CRTC:47:crtc-0] [ 14.402767] sun4i-drm display-engine: [drm:drm_atomic_add_encoder_bridges] Adding all bridges for [encoder:65:TMDS-65] to 31bd6e91 [ 14.402792] sun4i-drm display-engine: [drm:drm_atomic_add_encoder_bridges] Adding all bridges for [encoder:65:TMDS-65] to 31bd6e91 [ 14.402826] [drm:sun4i_backend_atomic_check] Starting checking our planes [ 14.402846] [drm:sun4i_backend_atomic_check] Input size 1024x600, output size 1024x600 [ 14.402862] [drm:sun4i_backend_atomic_check] Input size 1024x600, output size 1024x600 [ 14.402877] [drm:sun4i_backend_atomic_check] Plane FB format is XR24 little-endian (0x34325258) [ 14.402894] [drm:sun4i_backend_atomic_check] Plane zpos is 0 [ 14.402906] [drm:sun4i_backend_atomic_check] State valid with 1 planes, 0 alpha, 0 video, 0 YUV [ 14.402922] [drm:sun4i_backend_atomic_check] Starting checking our planes Sorry for the noise :) > >Maxime Michael #regzbot fix: 84e541b1e58e