From nobody Mon Feb 9 02:47:22 2026 Received: from bali.collaboradmins.com (bali.collaboradmins.com [148.251.105.195]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8B9CF22A4FF for ; Mon, 13 Jan 2025 13:36:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=148.251.105.195 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736775393; cv=none; b=GeieQK9X8kxflnO5ynnMNN03f8BFuh9OGSZ8FLQnMP6alpJ6l8XVPD8+MfRYib25drG3qLymRk0suSkXisfS5PN/MR46VH4UNSrDSO54CwkT6/QWvlWSUJKAR2tKp7TmA4i05Gyr05FPiHBnGXP13JeoHNbi2HcQ4Gbx9pGBuRI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736775393; c=relaxed/simple; bh=Y1kWdYZ1cSP52tdJOWKKUUGp+uBr6UO+JoyQye7joaU=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=SLn7pkE4VK5xzg5zvKhUJdocg80bryZrMHcWtgsUWQD1/EV+nO9IOjlV5oLzwe/3XZQ9nsL6kQnWPxztVY4J2J9qeN/TPy5qR0yK4PM1OHlw7zONxEIka4vND8Vbzp6ddGjU/jQlBMeBZ9ueXvHJtJumibJQ81fzabT9etwM7Qg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=hk0K63DF; arc=none smtp.client-ip=148.251.105.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="hk0K63DF" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1736775389; bh=Y1kWdYZ1cSP52tdJOWKKUUGp+uBr6UO+JoyQye7joaU=; h=From:Date:Subject:To:Cc:From; b=hk0K63DFA8qXE6U7cFUFPj8FHLUREna+H3fh77hdU6eK7XwIp3WxJv9kETjqeht0c AUGR/oQUBnXA/1P79emngYpbU5UTdpOgURYUDOR8WLzaq0CMgh2OHbTEiBvRxyROt0 BSoNMbBvP9QhVRzJEOSQC3j3yBVNUIuE9qIpEbyiSDEh7aZQtzhZIsqmhBmUVIiK2c knVN8A5tQob6V6FBVj2BmjjhfMHJu5OjzXFpukDCnqPOhMEbMYKUSF6NkxSInR0sVP DmWEoQrekJWhI2wmV4f1RJl1NXWclH+bc/8fA35b7AlNNKA3tSurBP7RXoGfUyY0EW EY5CpnGjbUOdQ== Received: from localhost (unknown [82.76.59.196]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: cristicc) by bali.collaboradmins.com (Postfix) with ESMTPSA id 7569F17E0C20; Mon, 13 Jan 2025 14:36:29 +0100 (CET) From: Cristian Ciocaltea Date: Mon, 13 Jan 2025 15:36:18 +0200 Subject: [PATCH v2] drm/display: hdmi: Do not read EDID on disconnected connectors Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250113-hdmi-conn-edid-read-fix-v2-1-d2a0438a44ab@collabora.com> X-B4-Tracking: v=1; b=H4sIANEWhWcC/3WNywrCMBBFf6XM2pFMah+48j+kizQZ7UCbSCJFK fl3Y8Gly3PgnrtB4iic4FxtEHmVJMEX0IcK7GT8nVFcYdBKN4pI4eQWQRu8R3biMLJxeJMXUtc 3VFs9dmygrB+Ri97L16HwJOkZ4ns/Wulrf03621wJCV3b1y21+lT36mLDPJsxRHO0YYEh5/wBe HFKzcIAAAA= X-Change-ID: 20250110-hdmi-conn-edid-read-fix-178513c2b7ea To: Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Dmitry Baryshkov Cc: kernel@collabora.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.14.2 The recently introduced hotplug event handler in the HDMI Connector framework attempts to unconditionally read the EDID data, leading to a bunch of non-harmful, yet quite annoying DDC/I2C related errors being reported. Ensure the operation is done only for connectors having the status connected or unknown. Additionally, perform an explicit reset of the connector information when dealing with a disconnected status. Fixes: ab716b74dc9d ("drm/display/hdmi: implement hotplug functions") Signed-off-by: Cristian Ciocaltea --- Changes in v2: - Moved drm_edid_connector_update() after drm_connector_hdmi_audio_plugged_notify(), per Dmitry's review - Updated commit description to mention the explicit connector information reset (Maxime) - Link to v1: https://lore.kernel.org/r/20250111-hdmi-conn-edid-read-fix-v1= -1-d68361624380@collabora.com --- drivers/gpu/drm/display/drm_hdmi_state_helper.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/display/drm_hdmi_state_helper.c b/drivers/gpu/= drm/display/drm_hdmi_state_helper.c index 2691e8b3e480131ac6e4e4b74b24947be55694bd..ca3c99b3805fbd78fb3162818b3= 877450d72b178 100644 --- a/drivers/gpu/drm/display/drm_hdmi_state_helper.c +++ b/drivers/gpu/drm/display/drm_hdmi_state_helper.c @@ -788,6 +788,8 @@ drm_atomic_helper_connector_hdmi_update(struct drm_conn= ector *connector, if (status =3D=3D connector_status_disconnected) { // TODO: also handle CEC and scramber, HDMI sink disconnected. drm_connector_hdmi_audio_plugged_notify(connector, false); + drm_edid_connector_update(connector, NULL); + return; } =20 if (connector->hdmi.funcs->read_edid) --- base-commit: 1854df7087be70ad54e24b2e308d7558ebea9f27 change-id: 20250110-hdmi-conn-edid-read-fix-178513c2b7ea