| Message ID | 20240208-fd-migrate-mdp5-v4-0-945d08ef3fa8@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/msm: provide migration path from MDP5 to DPU driver | expand |
On Thu, 08 Feb 2024 17:01:07 +0200, Dmitry Baryshkov wrote: > Over the last several years the DPU driver has been actively developed, > while the MDP5 is mostly in the maintenance mode. This results in some > features being available only in the DPU driver. For example, bandwidth > scaling, writeback support, properly supported bonded DSI aka dual DSI > support, DSC (display stream compression). > > All the pre-SDM845 platforms were originally supported by the MDP5 > driver only. However it is possible and easy to support some of the > older SoCs in the DPU driver. For example in the v5.18 it got > support for MSM8998. This can not be considered as a proper migration, > since there msm8998.dtsi didn't describe the display hardware > beforehand. Instead new bindings were added, making MSM8998 just another > display hardware to be supported by the DPU driver. > > [...] Applied, thanks! [1/4] drm/msm/mdss: generate MDSS data for MDP5 platforms https://gitlab.freedesktop.org/lumag/msm/-/commit/d2570ee67a47 [2/4] drm/msm/dpu: support binding to the mdp5 devices https://gitlab.freedesktop.org/lumag/msm/-/commit/39b06ed6d4f3 [3/4] drm/msm: add a kernel param to select between MDP5 and DPU drivers https://gitlab.freedesktop.org/lumag/msm/-/commit/b8b123187035 [4/4] drm/msm/dpu: add support for SDM660 and SDM630 platforms https://gitlab.freedesktop.org/lumag/msm/-/commit/7204df5e7e68 Best regards,
Over the last several years the DPU driver has been actively developed, while the MDP5 is mostly in the maintenance mode. This results in some features being available only in the DPU driver. For example, bandwidth scaling, writeback support, properly supported bonded DSI aka dual DSI support, DSC (display stream compression). All the pre-SDM845 platforms were originally supported by the MDP5 driver only. However it is possible and easy to support some of the older SoCs in the DPU driver. For example in the v5.18 it got support for MSM8998. This can not be considered as a proper migration, since there msm8998.dtsi didn't describe the display hardware beforehand. Instead new bindings were added, making MSM8998 just another display hardware to be supported by the DPU driver. This series provides a way to gradually migrate support for several existing and well-supported SoCs from the MDP5 to the DPU driver without changing the DT. From the user experience point of view this is facilitated by the `msm.prefer_mdp5' kernel param. If the parameter is set to `true' (current default), all `shared' platforms will be handled by the MDP5 driver. If the switch is flipped to `false' (or if the MDP5 driver is disabled), these platforms will be handled by the DPU driver. Handling this by the modparam (rather than solely by kernel config) allows one to easly switch between the drivers, simplifying testing. This series implements support for two DPU 3.n platforms, SDM660 and SDM630. The MSM8996 support was a part of the previous iterations of this patchset, but it was removed in v3. It requires additional development and testing. In theory after additional testing we can drop most of migration code and some parts of MDP5 driver. The proposed boundary is to move all platforms supporting cursor planes to the DPU driver, while limiting MDP5 to support only the older platforms which implement cursor as a part of the LM hardware block (MSM8974, APQ8084, MSM8x26, MSM8x16 and MSM8x39). --- Changes in v4: - Fixed param name for the msm_disp_drv_should_bind() stub (Carl Vanderlip) - Fixed the SoC name for MDP5 version 0x1010 (1.0.16), it is MSM8953 (Abhinav) - Rewored commit message for the msm_mdss change to mention possible crash on the unclocked HW_REV read (Abhinav) - Fixed dpu_kms_mmap_mdp5() to call dev_is_platform() on the parent of the DPU device (Abhinav) - Link to v3: https://lore.kernel.org/r/20240106-fd-migrate-mdp5-v3-0-3d2750378063@linaro.org Changes in v3: - Rebased on top of linux-next - After additional consideration dropped MSM8996 patch. It will be reiterated later, once the generic migration framework is accepted (and after we implement scalers support for that platform). Changes in v2: - Dropped accepted patches - Rebased on top of updated [1] - Added defines for MDSS hw revisions (Stephen) - Changed msm_mdss_generate_mdp5_mdss_data() to return const struct pointer (Stephen) - Fixed error handling in msm_ioremap_mdss() (Stephen) [1] https://patchwork.freedesktop.org/series/119804/ --- Dmitry Baryshkov (4): drm/msm/mdss: generate MDSS data for MDP5 platforms drm/msm/dpu: support binding to the mdp5 devices drm/msm: add a kernel param to select between MDP5 and DPU drivers drm/msm/dpu: add support for SDM660 and SDM630 platforms .../gpu/drm/msm/disp/dpu1/catalog/dpu_3_2_sdm660.h | 291 +++++++++++++++++++++ .../gpu/drm/msm/disp/dpu1/catalog/dpu_3_3_sdm630.h | 225 ++++++++++++++++ drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 2 + drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 2 + drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 103 ++++++-- drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 3 + drivers/gpu/drm/msm/msm_drv.c | 33 +++ drivers/gpu/drm/msm/msm_drv.h | 4 + drivers/gpu/drm/msm/msm_io_utils.c | 13 + drivers/gpu/drm/msm/msm_mdss.c | 51 ++++ 10 files changed, 706 insertions(+), 21 deletions(-) --- base-commit: b1d3a0e70c3881d2f8cf6692ccf7c2a4fb2d030d change-id: 20240105-fd-migrate-mdp5-6a2aa51bc83b Best regards,