From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Rob Clark <robdclark@gmail.com>, Sean Paul <sean@poorly.run>,
Jonathan Marek <jonathan@marek.ca>
Cc: Stephen Boyd <sboyd@kernel.org>, David Airlie <airlied@linux.ie>,
Daniel Vetter <daniel@ffwll.ch>,
linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
freedreno@lists.freedesktop.org
Subject: [RFC 00/24] drm/msm/dsi: refactor MSM DSI PHY/PLL drivers
Date: Tue, 2 Mar 2021 02:24:43 +0300 [thread overview]
Message-ID: <20210301232506.3057401-1-dmitry.baryshkov@linaro.org> (raw)
This is an RFC attempt to restructure MSM DSI PHY drivers. What started
as an attempt to grok the overcomplicated PHY drivers, has lead up to
the idea of merging PHY and PLL code, reducing abstractions, code
duplication, dropping dead code, etc.
The patches were mainly tested on RB5 (sm8250, 7nm) and lightly tested on RB3
(sdm845, 10nm), DB410c (apq8016, 28nm-lp).
The following changes since commit fe07bfda2fb9cdef8a4d4008a409bb02f35f1bd8:
Linux 5.12-rc1 (2021-02-28 16:05:19 -0800)
are available in the Git repository at:
ssh://git@git.linaro.org/people/dmitry.baryshkov/kernel.git dsi-phy-rfc
for you to fetch changes up to 819a9071587f1acba14a6780f422d78fdb55d671:
arm64: dts: qcom: sdm845: assign DSI clock source parents (2021-03-02 02:23:14 +0300)
----------------------------------------------------------------
Dmitry Baryshkov (24):
drm/msm/dsi: fuse dsi_pll_* code into dsi_phy_* code
drm/msm/dsi: drop multiple pll enable_seq support
drm/msm/dsi: move all PLL callbacks into PHY config struct
drm/msm/dsi: move min/max PLL rate to phy config
drm/msm/dsi: remove msm_dsi_pll_set_usecase
drm/msm/dsi: stop setting clock parents manually
arm64: dts: qcom: sm8250: assign DSI clock source parents
drm/msm/dsi: push provided clocks handling into a generic code
clk: mux: provide devm_clk_hw_register_mux()
clk: divider: add devm_clk_hw_register_divider
drm/msm/dsi: use devm_clk_*register to registe DSI PHY clocks
drm/msm/dsi: use devm_of_clk_add_hw_provider
drm/msm/dsi: replace PHY's init callback with configurable data
drm/msm/dsi: make save/restore_state phy-level functions
drm/msm/dsi: limit vco_delay to 28nm PHY
drm/msi/dsi: inline msm_dsi_pll_helper_clk_prepare/unprepare
drm/msm/dsi: make save_state/restore_state callbacks accept msm_dsi_phy
drm/msm/dsi: drop msm_dsi_pll abstracton
drm/msm/dsi: drop PLL accessor functions
drm/msm/dsi: move ioremaps to dsi_phy_driver_probe
drm/msm/dsi: remove duplicate fields from dsi_pll_Nnm instances
drm/msm/dsi: remove temp data from global pll structure
drm/msm/dsi: drop global msm_dsi_phy_type enumaration
arm64: dts: qcom: sdm845: assign DSI clock source parents
arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 +
arch/arm64/boot/dts/qcom/sm8250.dtsi | 6 +
drivers/clk/clk-mux.c | 35 +
drivers/gpu/drm/msm/Kconfig | 8 -
drivers/gpu/drm/msm/Makefile | 9 -
drivers/gpu/drm/msm/dsi/dsi.h | 56 +-
drivers/gpu/drm/msm/dsi/dsi_host.c | 51 --
drivers/gpu/drm/msm/dsi/dsi_manager.c | 18 +-
drivers/gpu/drm/msm/dsi/phy/dsi_phy.c | 129 +--
drivers/gpu/drm/msm/dsi/phy/dsi_phy.h | 31 +-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c | 741 ++++++++++++++-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 925 ++++++++++++++++++-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_20nm.c | 3 +-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c | 662 +++++++++++++-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm_8960.c | 476 +++++++++-
drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c | 769 +++++++++++++++-
drivers/gpu/drm/msm/dsi/pll/dsi_pll.c | 184 ----
drivers/gpu/drm/msm/dsi/pll/dsi_pll.h | 130 ---
drivers/gpu/drm/msm/dsi/pll/dsi_pll_10nm.c | 881 ------------------
drivers/gpu/drm/msm/dsi/pll/dsi_pll_14nm.c | 1096 -----------------------
drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.c | 643 -------------
drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm_8960.c | 526 -----------
drivers/gpu/drm/msm/dsi/pll/dsi_pll_7nm.c | 912 -------------------
include/linux/clk-provider.h | 30 +
24 files changed, 3678 insertions(+), 4649 deletions(-)
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll.c
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll.h
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll_10nm.c
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll_14nm.c
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.c
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm_8960.c
delete mode 100644 drivers/gpu/drm/msm/dsi/pll/dsi_pll_7nm.c
next reply other threads:[~2021-03-02 7:58 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-01 23:24 Dmitry Baryshkov [this message]
2021-03-01 23:24 ` [RFC 02/24] drm/msm/dsi: drop multiple pll enable_seq support Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 03/24] drm/msm/dsi: move all PLL callbacks into PHY config struct Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 04/24] drm/msm/dsi: move min/max PLL rate to phy config Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 05/24] drm/msm/dsi: remove msm_dsi_pll_set_usecase Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 06/24] drm/msm/dsi: stop setting clock parents manually Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 07/24] arm64: dts: qcom: sm8250: assign DSI clock source parents Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 08/24] drm/msm/dsi: push provided clocks handling into a generic code Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 09/24] clk: mux: provide devm_clk_hw_register_mux() Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 10/24] clk: divider: add devm_clk_hw_register_divider Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 11/24] drm/msm/dsi: use devm_clk_*register to registe DSI PHY clocks Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 12/24] drm/msm/dsi: use devm_of_clk_add_hw_provider Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 13/24] drm/msm/dsi: replace PHY's init callback with configurable data Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 14/24] drm/msm/dsi: make save/restore_state phy-level functions Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 15/24] drm/msm/dsi: limit vco_delay to 28nm PHY Dmitry Baryshkov
2021-03-01 23:24 ` [RFC 16/24] drm/msi/dsi: inline msm_dsi_pll_helper_clk_prepare/unprepare Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 17/24] drm/msm/dsi: make save_state/restore_state callbacks accept msm_dsi_phy Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 18/24] drm/msm/dsi: drop msm_dsi_pll abstracton Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 19/24] drm/msm/dsi: drop PLL accessor functions Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 20/24] drm/msm/dsi: move ioremaps to dsi_phy_driver_probe Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 21/24] drm/msm/dsi: remove duplicate fields from dsi_pll_Nnm instances Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 22/24] drm/msm/dsi: remove temp data from global pll structure Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 23/24] drm/msm/dsi: drop global msm_dsi_phy_type enumaration Dmitry Baryshkov
2021-03-01 23:25 ` [RFC 24/24] arm64: dts: qcom: sdm845: assign DSI clock source parents Dmitry Baryshkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210301232506.3057401-1-dmitry.baryshkov@linaro.org \
--to=dmitry.baryshkov@linaro.org \
--cc=airlied@linux.ie \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=jonathan@marek.ca \
--cc=linux-arm-msm@vger.kernel.org \
--cc=robdclark@gmail.com \
--cc=sboyd@kernel.org \
--cc=sean@poorly.run \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).