From patchwork Sun Dec 31 00:43:47 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Baryshkov X-Patchwork-Id: 759257 Received: from mail-lf1-f54.google.com (mail-lf1-f54.google.com [209.85.167.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 18ED0658 for ; Sun, 31 Dec 2023 00:43:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="PdDoZz9K" Received: by mail-lf1-f54.google.com with SMTP id 2adb3069b0e04-50e835800adso4086107e87.0 for ; Sat, 30 Dec 2023 16:43:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1703983437; x=1704588237; darn=vger.kernel.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=uFdcEklglVMCc5SvkcvBwXoblALE59NOZweHT3jRNTA=; b=PdDoZz9K5uy3jNNsNqneW2TQRBrQDu/yBUqyfnISl6WyJLI8qSvFIoqr8rlkuQYI7e 4FCggR+cx0G9mh4tFi6q5Hti5QP+41IwWDcUNZ/yoakCfn0zG/5mb1WErHg1RaKnTsiW Y9kMPol2j0mkfEe59EBBlfhLcjVGFbLetMqyUf5Hcbeb56w0xeq5gyxhzvhvP8r59Fqy AcqvUFOfzkGHkKvNg+qA5o3RKXyI02XYgK07XE3jplVxHgfthtoIptv5/toZNrzT7/9v ELpVWL+UARRrAtyD3KlqV1RRp0P7AmtB6XMpzpv2vE4Z6O6k6NtlADxuH8aVpvT8033c oBqw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703983437; x=1704588237; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uFdcEklglVMCc5SvkcvBwXoblALE59NOZweHT3jRNTA=; b=fnRP1wQKYxMJr2GfR00mlRCkja4GNgnlgxi4JDgfg/qpFLclMIzFIgMnvkHSzL5F7I EohRe8K9UXXX6TfcY81o6TJQ/RL3HJGmtdXYDYjVg4qrk7agtLwL/I+O4/EpW2oRHVY8 w7tB5D3BvJXweZtTfy0ftIIrIjgBC2QMY3QUSGC+4G34sIFUfpZbKQ/Qrq42zUb1VQk9 viGCCvtCoIrUNA9GDrYdwuksvH3cyP6I5iqf2uZAKpnyoP4XQ8hto9VHcBSeKkB46E32 O+d9zXaGQ5YxcKyxOuY2YQA6Zgt49zUDzfJQ3il3QTRsIq0AbXL0RV/NXM2q6MH0tnWU s3wg== X-Gm-Message-State: AOJu0Yx8jnbcaZq4iLxuCbxPWfZn6s3CflVOjHfllJrRArThKj1U75Y/ alm8NzR1rv/McZz3f9EZrgz2SWEEl6XKJw== X-Google-Smtp-Source: AGHT+IEbsxU1LPMV0eSBJNc42YNvs7W6aovH4q706qKA5nwASqqB5dYIqDRug//C3SBrEw1aEysHJA== X-Received: by 2002:a05:6512:10cf:b0:50e:7dbe:eba9 with SMTP id k15-20020a05651210cf00b0050e7dbeeba9mr3149107lfg.68.1703983436543; Sat, 30 Dec 2023 16:43:56 -0800 (PST) Received: from umbar.lan ([192.130.178.91]) by smtp.gmail.com with ESMTPSA id n13-20020a05651203ed00b0050e84be8127sm1295995lfq.101.2023.12.30.16.43.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 30 Dec 2023 16:43:56 -0800 (PST) From: Dmitry Baryshkov Subject: [PATCH v2 00/15] drm/msm/dp: clear power and parser submodules away Date: Sun, 31 Dec 2023 02:43:47 +0200 Message-Id: <20231231-dp-power-parser-cleanup-v2-0-fc3e902a6f5b@linaro.org> Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAEO5kGUC/53OsQ6CMBDG8VchnT1CS4rWyfcwDBUO2ghtc0WUE N7dwuRscst/+H65lUUki5Fds5URzjZa71KIU8Yao12PYNvUTBSi5OmgDRD8GwmCprSFZkDtXgE Ullp2SlcXwVlaB8LOfg75XqfuyI8wGUL94wklhKxkkSsuzpJD4kc70ZI/NC3RPP18G6zT5HNP/ a4aGydPy/HuzHf7L6netu0LMCVjjPsAAAA= To: Rob Clark , Abhinav Kumar , Dmitry Baryshkov , Sean Paul , Marijn Suijten , David Airlie , Daniel Vetter , Kuogee Hsieh , Stephen Boyd Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, Bjorn Andersson , Konrad Dybcio X-Mailer: b4 0.12.4 X-Developer-Signature: v=1; a=openpgp-sha256; l=2654; i=dmitry.baryshkov@linaro.org; h=from:subject:message-id; bh=gSCAT2zSZoYG95yfZ9I4UoUJFs7cxpxpX5tE2/TTf4A=; b=owEBbQGS/pANAwAKAYs8ij4CKSjVAcsmYgBlkLlJHdQmsFHeqgB7p3Ofz4CG5hT6HXzfxtpoJ Tj40LfsVqiJATMEAAEKAB0WIQRMcISVXLJjVvC4lX+LPIo+Aiko1QUCZZC5SQAKCRCLPIo+Aiko 1cLeCACqY9y+0CFMS+yYXYGemMBrdncF6UuJwK71u686FfT0eeYXBYJzqMynEBMkGUEnuq+723E 36665tTjYtfYpI53w8kYnDG+yKNxEVxnjIj1drQZX7JhKipjMIXcvbQbbDeSX5+/NGUhtLP23+w ISonrc2AHGd/3qF/qKr7SDksr3dzyE+62Ut4pVgTfdgeNDN0xWOPqWlZda64W9essWo4XAiOiYG nCsENN+D36cLzJZUTYUTuaDOAgmkFXKjpC2ZExTz21pZcMYJN0qmSMBKj7+mIQRJyJrcWKLsG89 iFxPguaSqwb9wLuZbFcfG5G8vfIqL3SlCpA0ozF0zt1PS8v6 X-Developer-Key: i=dmitry.baryshkov@linaro.org; a=openpgp; fpr=8F88381DD5C873E4AE487DA5199BF1243632046A Reshuffle code in the DP driver, cleaning up clocks and DT parsing and dropping the dp_power and dp_parser submodules. Initially I started by looking onto stream_pixel clock handling only to find several wrapping layers around a single clocks. After inlining and/or dropping them (and thus dp_power submodule), it was more or less natural to continue cleaning up the dp_parser until it got removed completely. --- Changes in v2: - Fixed unrelated power->ctrl change in comment (Konrad) - Made sure that all functions use reverse-Christmas-tree flow (Konrad) - Fixed indents in several moved functions - Added a patch splitting dp_ctlr_clk_enable - Link to v1: https://lore.kernel.org/r/20231229225650.912751-1-dmitry.baryshkov@linaro.org --- Dmitry Baryshkov (15): drm/msm/dp: drop unused parser definitions drm/msm/dp: drop unused fields from dp_power_private drm/msm/dp: parse DT from dp_parser_get drm/msm/dp: inline dp_power_(de)init drm/msm/dp: fold dp_power into dp_ctrl module drm/msm/dp: simplify stream clocks handling drm/msm/dp: stop parsing clock names from DT drm/msm/dp: split dp_ctrl_clk_enable into four functuions drm/msm/dp: move phy_configure_opts to dp_ctrl drm/msm/dp: remove PHY handling from dp_catalog.c drm/msm/dp: handle PHY directly in dp_ctrl drm/msm/dp: move all IO handling to dp_catalog drm/msm/dp: move link property handling to dp_panel drm/msm/dp: move next_bridge handling to dp_display drm/msm/dp: drop dp_parser drivers/gpu/drm/msm/Makefile | 2 - drivers/gpu/drm/msm/dp/dp_aux.c | 9 +- drivers/gpu/drm/msm/dp/dp_aux.h | 2 + drivers/gpu/drm/msm/dp/dp_catalog.c | 156 +++++++++++----- drivers/gpu/drm/msm/dp/dp_catalog.h | 6 +- drivers/gpu/drm/msm/dp/dp_ctrl.c | 358 ++++++++++++++++++++++++------------ drivers/gpu/drm/msm/dp/dp_ctrl.h | 17 +- drivers/gpu/drm/msm/dp/dp_debug.c | 1 - drivers/gpu/drm/msm/dp/dp_display.c | 101 +++------- drivers/gpu/drm/msm/dp/dp_display.h | 3 +- drivers/gpu/drm/msm/dp/dp_panel.c | 66 +++++++ drivers/gpu/drm/msm/dp/dp_parser.c | 327 -------------------------------- drivers/gpu/drm/msm/dp/dp_parser.h | 155 ---------------- drivers/gpu/drm/msm/dp/dp_power.c | 183 ------------------ drivers/gpu/drm/msm/dp/dp_power.h | 95 ---------- 15 files changed, 464 insertions(+), 1017 deletions(-) --- base-commit: 39676dfe52331dba909c617f213fdb21015c8d10 change-id: 20231231-dp-power-parser-cleanup-9e3a5f9a6821 Best regards,