Re: [PATCH v2 09/17] drm/msm/dpu: add an API to reset the encoder related hw blocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 21/04/2022 00:28, Abhinav Kumar wrote:


On 4/20/2022 12:23 AM, Dmitry Baryshkov wrote:
On 20/04/2022 04:46, Abhinav Kumar wrote:
Add an API to reset the encoder related hw blocks to ensure
a proper teardown of the pipeline. At the moment this is being
used only for the writeback encoder but eventually we can start
using this for all interfaces.

changes in v2:
    - split the writeback part to another commit

Signed-off-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>

A minor question: do we need to also reset the DSPP alongside resetting the LM?

So this function is mostly doing what the downstream equivalent of it does which is to disable all the sspp blend stages, the ping-pong binding and 3d-merge connection.

For DSPP, there is no ability to *bind* it or connect it. Its a hard-wired connection. Its just a question of whether to enable it or not.

When the CTL path connections are removed, there is no need to explicitly disable the DSPP.

Thats why even downstream doesnt do it today.

Ack, thanks.




---
  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c      | 82 ++++++++++++++++++++++++
  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h |  7 ++
  2 files changed, 89 insertions(+)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
index 4523693..0e31ad3 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
@@ -1,5 +1,6 @@
  // SPDX-License-Identifier: GPL-2.0-only
  /*
+ * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.    * Copyright (c) 2014-2018, 2020-2021 The Linux Foundation. All rights reserved.
   * Copyright (C) 2013 Red Hat
   * Author: Rob Clark <robdclark@xxxxxxxxx>
@@ -22,6 +23,7 @@
  #include "dpu_hw_ctl.h"
  #include "dpu_hw_dspp.h"
  #include "dpu_hw_dsc.h"
+#include "dpu_hw_merge3d.h"
  #include "dpu_formats.h"
  #include "dpu_encoder_phys.h"
  #include "dpu_crtc.h"
@@ -1838,6 +1840,86 @@ void dpu_encoder_kickoff(struct drm_encoder *drm_enc)
      DPU_ATRACE_END("encoder_kickoff");
  }
+static void dpu_encoder_helper_reset_mixers(struct dpu_encoder_phys *phys_enc)
+{
+    struct dpu_hw_mixer_cfg mixer;
+    int i, num_lm;
+    u32 flush_mask = 0;
+    struct dpu_global_state *global_state;
+    struct dpu_hw_blk *hw_lm[2];
+    struct dpu_hw_mixer *hw_mixer[2];
+    struct dpu_hw_ctl *ctl = phys_enc->hw_ctl;
+
+    memset(&mixer, 0, sizeof(mixer));
+
+    /* reset all mixers for this encoder */
+    if (phys_enc->hw_ctl->ops.clear_all_blendstages)
+        phys_enc->hw_ctl->ops.clear_all_blendstages(phys_enc->hw_ctl);
+
+    global_state = dpu_kms_get_existing_global_state(phys_enc->dpu_kms);
+
+    num_lm = dpu_rm_get_assigned_resources(&phys_enc->dpu_kms->rm, global_state, +        phys_enc->parent->base.id, DPU_HW_BLK_LM, hw_lm, ARRAY_SIZE(hw_lm));
+
+    for (i = 0; i < num_lm; i++) {
+        hw_mixer[i] = to_dpu_hw_mixer(hw_lm[i]);
+        flush_mask = phys_enc->hw_ctl->ops.get_bitmask_mixer(ctl, hw_mixer[i]->idx);
+        if (phys_enc->hw_ctl->ops.update_pending_flush)
+            phys_enc->hw_ctl->ops.update_pending_flush(ctl, flush_mask);
+
+        /* clear all blendstages */
+        if (phys_enc->hw_ctl->ops.setup_blendstage)
+            phys_enc->hw_ctl->ops.setup_blendstage(ctl, hw_mixer[i]->idx, NULL);
+    }
+}
+
+void dpu_encoder_helper_phys_cleanup(struct dpu_encoder_phys *phys_enc)
+{
+    struct dpu_hw_ctl *ctl = phys_enc->hw_ctl;
+    struct dpu_hw_intf_cfg intf_cfg = { 0 };
+    int i;
+    struct dpu_encoder_virt *dpu_enc;
+
+    dpu_enc = to_dpu_encoder_virt(phys_enc->parent);
+
+    phys_enc->hw_ctl->ops.reset(ctl);
+
+    dpu_encoder_helper_reset_mixers(phys_enc);
+
+    for (i = 0; i < dpu_enc->num_phys_encs; i++) {
+        if (dpu_enc->phys_encs[i] && phys_enc->hw_intf->ops.bind_pingpong_blk)
+            phys_enc->hw_intf->ops.bind_pingpong_blk(
+                    dpu_enc->phys_encs[i]->hw_intf, false,
+                    dpu_enc->phys_encs[i]->hw_pp->idx);
+
+        /* mark INTF flush as pending */
+        if (phys_enc->hw_ctl->ops.update_pending_flush_intf)
+ phys_enc->hw_ctl->ops.update_pending_flush_intf(phys_enc->hw_ctl,
+                    dpu_enc->phys_encs[i]->hw_intf->idx);
+    }
+
+    /* reset the merge 3D HW block */
+    if (phys_enc->hw_pp->merge_3d) {
+ phys_enc->hw_pp->merge_3d->ops.setup_3d_mode(phys_enc->hw_pp->merge_3d,
+                BLEND_3D_NONE);
+        if (phys_enc->hw_ctl->ops.update_pending_flush_merge_3d)
+            phys_enc->hw_ctl->ops.update_pending_flush_merge_3d(ctl,
+                    phys_enc->hw_pp->merge_3d->idx);
+    }
+
+    intf_cfg.stream_sel = 0; /* Don't care value for video mode */
+    intf_cfg.mode_3d = dpu_encoder_helper_get_3d_blend_mode(phys_enc);
+    if (phys_enc->hw_pp->merge_3d)
+        intf_cfg.merge_3d = phys_enc->hw_pp->merge_3d->idx;
+
+    if (ctl->ops.reset_intf_cfg)
+        ctl->ops.reset_intf_cfg(ctl, &intf_cfg);
+
+    ctl->ops.trigger_flush(ctl);
+    ctl->ops.trigger_start(ctl);
+    ctl->ops.clear_pending_flush(ctl);
+}
+
  void dpu_encoder_prepare_commit(struct drm_encoder *drm_enc)
  {
      struct dpu_encoder_virt *dpu_enc;
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
index 706b566..544a9a4 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h
@@ -1,5 +1,6 @@
  /* SPDX-License-Identifier: GPL-2.0-only */
  /*
+ * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved.
   * Copyright (c) 2015-2018 The Linux Foundation. All rights reserved.
   */
@@ -350,4 +351,10 @@ int dpu_encoder_helper_wait_for_irq(struct dpu_encoder_phys *phys_enc,
          void (*func)(void *arg, int irq_idx),
          struct dpu_encoder_wait_info *wait_info);
+/**
+ * dpu_encoder_helper_phys_cleanup - helper to cleanup dpu pipeline
+ * @phys_enc: Pointer to physical encoder structure
+ */
+void dpu_encoder_helper_phys_cleanup(struct dpu_encoder_phys *phys_enc);
+
  #endif /* __dpu_encoder_phys_H__ */




--
With best wishes
Dmitry



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux