From nobody Mon Jun 15 07:33:49 2026 Received: from mx.nabladev.com (mx.nabladev.com [178.251.229.89]) (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 6829B23AE62 for ; Thu, 9 Apr 2026 02:49:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=178.251.229.89 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775702982; cv=none; b=VLpKwk5zYMNUZku7KuQEgufKk8Jz1PxcZJkbBVh7NPBQBgxtCxDxgcfv27HP1DilTeTvzzzjBfamslgpYQXsj/d4HPVvMp78rGmzuczLHTq1PCCaRquhZJrev519LFojxI0KrBUdV5jZHaZyw3fwKgVlZIG5iM9Z9uuuWZ6Gg3w= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775702982; c=relaxed/simple; bh=s0B+wG6PNs2jlaIKK88S35+xXtsZ3hsiW0ZpOi2z2zI=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=bDy9wXp5upagw11/2/DPUX5UTJu4zdbGZzF0LesYN/blbageq00LIUAJsn4X44/nrp5f1UWEvitNsnp7pl1RppJEexFScjZZtDzHEE4StUf9+BUSm46PzDY1mdNCNSpkO5q9iSnA04J31OV8VHl6uMLFxBy9qoiyTBniuXsy2OE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nabladev.com; spf=pass smtp.mailfrom=nabladev.com; dkim=pass (2048-bit key) header.d=nabladev.com header.i=@nabladev.com header.b=fbd4W5wC; arc=none smtp.client-ip=178.251.229.89 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=nabladev.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nabladev.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=nabladev.com header.i=@nabladev.com header.b="fbd4W5wC" Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 9084D11276A; Thu, 9 Apr 2026 04:49:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nabladev.com; s=dkim; t=1775702977; h=from:subject:date:message-id:to:cc:mime-version: content-transfer-encoding; bh=Y/Vs+lajGNLuHVTZi9YuxQObAJW7yBV0C3O93uFX/q8=; b=fbd4W5wCyZNMs4RQdmQV/nWZ2NrK4FZ9nr5hlDhgtMMdsQCYm4IE3dhIv+WqKy056IZ+3m XRfHE9ZwPjNhSG9Kd9BNJ5kkzrqMulsMJ5Sb+pSFKp4xbG6YHzeRnwJgzY1/DuqNUTLVXD 0JmR4dqn3ZyP+R0Cg4nlY+SC4t7k5CUVa8LVvKNWVaKslG9Vn05rwD8QhH87FnVXv1H2s2 i2Aa5GoPBPOqJPHPVSbIkDe1nWA0+jQwLox3uKzdpFRMoBNqcydS3SSrScQ4dGuV7k1vrt IFCNbBEYNo/orIfYT0YIpIYSVmzkcU+MboZdVnhpjVM9AsCwH8KOFs/nqg0VoQ== From: Marek Vasut To: dri-devel@lists.freedesktop.org Cc: Marek Vasut , Alexandre Torgue , David Airlie , Maarten Lankhorst , Maxime Coquelin , Maxime Ripard , Philippe Cornu , Raphael Gallais-Pou , Simona Vetter , Thomas Zimmermann , Yannick Fertre , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com Subject: [PATCH] drm/bridge: stm_lvds: Do not fail atomic_check on disabled connector Date: Thu, 9 Apr 2026 04:48:41 +0200 Message-ID: <20260409024928.344010-1-marex@nabladev.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 Content-Type: text/plain; charset="utf-8" If the connector is disabled, the new connector state has .crtc field set to NULL and there is nothing more to validate after that point. The .crtc field being NULL is not an error. Test for .crtc being NULL, and if it is NULL, exit early with return 0. This fixes a failure in suspend/resume path, where the connector is already disabled, but .atomic_check is called, fails, returns -EINVAL and blocks the suspend entry. Fixes: aca1cbc1c986 ("drm/stm: lvds: add new STM32 LVDS Display Interface T= ransmitter driver") Signed-off-by: Marek Vasut Acked-by: Rapha=C3=ABl Gallais-Pou --- Cc: Alexandre Torgue Cc: David Airlie Cc: Maarten Lankhorst Cc: Maxime Coquelin Cc: Maxime Ripard Cc: Philippe Cornu Cc: Raphael Gallais-Pou Cc: Simona Vetter Cc: Thomas Zimmermann Cc: Yannick Fertre Cc: dri-devel@lists.freedesktop.org Cc: linux-arm-kernel@lists.infradead.org Cc: linux-kernel@vger.kernel.org Cc: linux-stm32@st-md-mailman.stormreply.com --- drivers/gpu/drm/stm/lvds.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/stm/lvds.c b/drivers/gpu/drm/stm/lvds.c index fe38c0984b2b5..25e2ba98f36ae 100644 --- a/drivers/gpu/drm/stm/lvds.c +++ b/drivers/gpu/drm/stm/lvds.c @@ -897,14 +897,14 @@ static int lvds_connector_atomic_check(struct drm_con= nector *connector, if (!conn_state) return -EINVAL; =20 + if (!conn_state->crtc) + return 0; + if (list_empty(&connector->modes)) { drm_dbg(connector->dev, "connector: empty modes list\n"); return -EINVAL; } =20 - if (!conn_state->crtc) - return -EINVAL; - panel_mode =3D list_first_entry(&connector->modes, struct drm_display_mode, head); =20 --=20 2.53.0