From nobody Thu Dec 18 08:09:28 2025 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (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 B7DC72C374C for ; Mon, 14 Apr 2025 11:12:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.167.242.64 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744629158; cv=none; b=Xd/uH6UMUBUXauQzSdTt+M2Nb4ABt5QMAjgGSoZ/BQv7gv0thLaFVziIMMJZW3fwXZXaQGadCwGIy2s+tXs59L/rfWB7iQ+nrcDZYqlswHY9XJO7/UANaDabtSEBZTOoz+TtpUC6NA3hle5L1GYZVX3dLZHoKcG9cbgGV4luqck= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1744629158; c=relaxed/simple; bh=49nZeRMG+Wq88xPKlAPzjO8QNo3kDMMtOzbXmSftZ7w=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=kHq//n56oNUst7WLOLVvuzyCuXbvWQsGJWi+YmTcN0xn12uVPMN53Q2f5vTr2CFWQ3pM4YpTtFuLu1ZGMgpIpNcS94o5wiGlG/8poSIPFSXAw5iLbvFBT/IFmNaI/46v4yX93qa40TEzMUgm1/shJBVvu2bUvgxk3IT4IszvO1M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com; spf=pass smtp.mailfrom=ideasonboard.com; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b=mGSilwyd; arc=none smtp.client-ip=213.167.242.64 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ideasonboard.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="mGSilwyd" Received: from [127.0.1.1] (91-158-153-178.elisa-laajakaista.fi [91.158.153.178]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 9F6A62699; Mon, 14 Apr 2025 13:10:18 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1744629019; bh=49nZeRMG+Wq88xPKlAPzjO8QNo3kDMMtOzbXmSftZ7w=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=mGSilwydheDwKzTtEtco65ecJA+7vwfzRlcJfMv7idKihJd510xQmBEG3bWc/a/qc H4rRph89XPr7aIu/g/zAgbOgIVVqW137dZeO3LonlExdRp3wUbz0qqDJboPPNMxe51 Kjbbe0Y6MU3R/v8h+cgpW7CUXUw99ia1Zbnmn5bI= From: Tomi Valkeinen Date: Mon, 14 Apr 2025 14:11:23 +0300 Subject: [PATCH v3 14/17] drm/bridge: cdns-dsi: Use video mode and clean up cdns_dsi_mode2cfg() 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: <20250414-cdns-dsi-impro-v3-14-4e52551d4f07@ideasonboard.com> References: <20250414-cdns-dsi-impro-v3-0-4e52551d4f07@ideasonboard.com> In-Reply-To: <20250414-cdns-dsi-impro-v3-0-4e52551d4f07@ideasonboard.com> To: Jyri Sarha , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Vinod Koul , Kishon Vijay Abraham I , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Jayesh Choudhary Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-phy@lists.infradead.org, Francesco Dolcini , Aradhya Bhatia , Devarsh Thakkar , Tomi Valkeinen X-Mailer: b4 0.15-dev-c25d1 X-Developer-Signature: v=1; a=openpgp-sha256; l=4347; i=tomi.valkeinen@ideasonboard.com; h=from:subject:message-id; bh=49nZeRMG+Wq88xPKlAPzjO8QNo3kDMMtOzbXmSftZ7w=; b=kA0DAAgB+j2qjLyWHvUByyZiAGf87YGjCqSoqJaYVxyXLMypsIJ3rD4Zr+drjMHXmC5R3WHfy IkCMwQAAQgAHRYhBMQ4DD6WXv2BB5/zp/o9qoy8lh71BQJn/O2BAAoJEPo9qoy8lh710R8P/299 rBvpNadn3e+05rJ1IMPuigBP9rF3M9fWPm5DHAvoGki5cgvSXiAhiXzx/THDLZyentMs9YJfWqp 7ALQ0dQoQ4q8E5b+JbADRCldKt7BqOVvTtoUUzhmT992sNCJi1ZO/FchFdIoM3j+/8iO8DyGYz0 GTHvVoKCkMXUYRTmuJVkzWY9qU12Ka4P+13zEeAKV9STRmckwK9ha1/BRq/Q6aciZg/t7SKo2/M 8rm0kmnJZ90iDTPKNx9draATFHntQbblAp6JUvzNxoE+1rl5gtugHYzkufjbLaLm8zXEx9uc+ez pwZZqCfz84zIqtSozn2alIkaBGWSeWpvcgeHnQ7JchSLF+08Z7YLICuJcGr92h4727AU39ojB/S 4DkbG2diLRci1R3wy4WkR8jpH3ANmhIn2uDl/RRh8CRDNWzb3tvG3C4IxMK8GLLf1rkZ0VvtY9u ln6Z41/JtlzXsv2PywgXXdoTssIaPNQqTGmBpXi1GhV+GBZpoaXrRYux/47IxurBpOEIMiqBrRR SJoBNs/rJW9ezt8Y417lxSsfSTy0HXtMDWfrVH6rFmu0TwirBz6C8CuOb4Klxasl4zJx1GNeFog Lol5i+Tfex/ZNgMVV/BxVSfZzmax/6XBuJFqW3W30gMolfVvesFE8+FREA+Gw8Uv8w6ioI1j+Yk 1hWL3 X-Developer-Key: i=tomi.valkeinen@ideasonboard.com; a=openpgp; fpr=C4380C3E965EFD81079FF3A7FA3DAA8CBC961EF5 The driver does all the calculations and programming with video timings (hftp, hbp, etc.) instead of the modeline values (hsync_start, ...). Thus it makes sense to use struct videomode instead of struct drm_display_mode internally. Switch to videomode and do some cleanups in cdns_dsi_mode2cfg() along the way. Signed-off-by: Tomi Valkeinen Reviewed-by: Aradhya Bhatia --- drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c | 45 ++++++++++++++--------= ---- 1 file changed, 24 insertions(+), 21 deletions(-) diff --git a/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c b/drivers/gpu/d= rm/bridge/cadence/cdns-dsi-core.c index fb0623d3f854..a55f851711f0 100644 --- a/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c +++ b/drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c @@ -9,6 +9,7 @@ #include #include #include