[Freedreno] [PATCH v4 1/7] drm/msm: Remove unnecessary pm_runtime_get/put

2022-08-17 Thread Akhil P Oommen
We already enable gpu power from msm_gpu_submit(), so avoid a duplicate pm_runtime_get/put from msm_job_run(). Signed-off-by: Akhil P Oommen --- (no changes since v1) drivers/gpu/drm/msm/msm_ringbuffer.c | 4 1 file changed, 4 deletions(-) diff --git

[Freedreno] [PATCH v4 4/7] drm/msm: Fix cx collapse issue during recovery

2022-08-17 Thread Akhil P Oommen
There are some hardware logic under CX domain. For a successful recovery, we should ensure cx headswitch collapses to ensure all the stale states are cleard out. This is especially true to for a6xx family where we can GMU co-processor. Currently, cx doesn't collapse due to a devlink between gpu

[Freedreno] [PATCH v4 3/7] drm/msm: Correct pm_runtime votes in recover worker

2022-08-17 Thread Akhil P Oommen
In the scenario where there is one a single submit which is hung, gpu is power collapsed when it is retired. Because of this, by the time we call reover(), gpu state would be already clear. Fix this by correctly managing the pm runtime votes. Signed-off-by: Akhil P Oommen --- (no changes since

[Freedreno] [PATCH v2 0/5] clk/qcom: Support gdsc collapse polling using 'reset' inteface

2022-08-17 Thread Akhil P Oommen
Some clients like adreno gpu driver would like to ensure that its gdsc is collapsed at hardware during a gpu reset sequence. This is because it has a votable gdsc which could be ON due to a vote from another subsystem like tz, hyp etc or due to an internal hardware signal. To allow this, gpucc

[Freedreno] [PATCH v4 7/7] drm/msm/a6xx: Handle GMU prepare-slumber hfi failure

2022-08-17 Thread Akhil P Oommen
When prepare-slumber hfi fails, we should follow a6xx_gmu_force_off() sequence. Signed-off-by: Akhil P Oommen --- (no changes since v1) drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c

[Freedreno] [PATCH v4 0/7] Improve GPU Recovery

2022-08-17 Thread Akhil P Oommen
Recently, I debugged a few device crashes which occured during recovery after a hangcheck timeout. It looks like there are a few things we can do to improve our chance at a successful gpu recovery. First one is to ensure that CX GDSC collapses which clears the internal states in gpu's CX

[Freedreno] [PATCH v4 5/7] drm/msm/a6xx: Ensure CX collapse during gpu recovery

2022-08-17 Thread Akhil P Oommen
Because there could be transient votes from other drivers/tz/hyp which may keep the cx gdsc enabled, we should poll until cx gdsc collapses. We can use the reset framework to poll for cx gdsc collapse from gpucc clk driver. This feature requires support from the platform's gpucc driver.

[Freedreno] [PATCH v2 4/5] clk: qcom: gpucc-sc7280: Add cx collapse reset support

2022-08-17 Thread Akhil P Oommen
Allow a consumer driver to poll for cx gdsc collapse through Reset framework. Signed-off-by: Akhil P Oommen --- Changes in v2: - Minor update to use the updated custom reset ops implementation drivers/clk/qcom/gpucc-sc7280.c | 10 ++ 1 file changed, 10 insertions(+) diff --git

[Freedreno] [PATCH v2 5/5] arm64: dts: qcom: sc7280: Add Reset support for gpu

2022-08-17 Thread Akhil P Oommen
Add support for Reset using GPUCC driver for GPU. This helps to ensure that GPU state is reset by making sure that CX head switch is collapsed. Signed-off-by: Akhil P Oommen --- (no changes since v1) arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +++ 1 file changed, 3 insertions(+) diff --git

[Freedreno] [PATCH v4 2/7] drm/msm: Take single rpm refcount on behalf of all submits

2022-08-17 Thread Akhil P Oommen
Instead of separate refcount for each submit, take single rpm refcount on behalf of all the submits. This makes it easier to drop the rpm refcount during recovery in an upcoming patch. Signed-off-by: Akhil P Oommen --- (no changes since v3) Changes in v3: - New patch

[Freedreno] [PATCH v4 6/7] drm/msm/a6xx: Improve gpu recovery sequence

2022-08-17 Thread Akhil P Oommen
We can do a few more things to improve our chance at a successful gpu recovery, especially during a hangcheck timeout: 1. Halt CP and GMU core 2. Do RBBM GBIF HALT sequence 3. Do a soft reset of GPU core Signed-off-by: Akhil P Oommen --- (no changes since v1)

[Freedreno] [PATCH v2 3/5] clk: qcom: gdsc: Add a reset op to poll gdsc collapse

2022-08-17 Thread Akhil P Oommen
Add a reset op compatible function to poll for gdsc collapse. Signed-off-by: Akhil P Oommen --- Changes in v2: - Minor update to function prototype drivers/clk/qcom/gdsc.c | 23 +++ drivers/clk/qcom/gdsc.h | 7 +++ 2 files changed, 26 insertions(+), 4 deletions(-)

Re: [Freedreno] [RFC PATCH] drm/edid: Make 144 Hz not preferred on Sharp LQ140M1JW46

2022-08-17 Thread Doug Anderson
Hi, On Sun, Aug 14, 2022 at 11:46 PM Maxime Ripard wrote: > > On Fri, Jul 29, 2022 at 12:57:40PM -0700, Doug Anderson wrote: > > Hi, > > > > On Fri, Jul 29, 2022 at 9:41 AM Maxime Ripard wrote: > > > > > > On Fri, Jul 29, 2022 at 07:50:20AM -0700, Doug Anderson wrote: > > > > On Fri, Jul 29,

Re: [Freedreno] [PATCH] drm/msm/dsi: Set panel orientation when directly connected

2022-08-17 Thread Doug Anderson
Hi, On Wed, Jul 20, 2022 at 3:42 PM Doug Anderson wrote: > > Hi, > > On Wed, Jul 20, 2022 at 1:46 PM Rob Clark wrote: > > > > On Fri, Jul 8, 2022 at 8:25 AM Doug Anderson wrote: > > > > > > Hi, > > > > > > On Wed, Jul 6, 2022 at 12:14 PM Stephen Boyd wrote: > > > > > > > > Set the panel

Re: [Freedreno] [PATCH v2 4/7] regulator: core: Allow specifying an initial load w/ the bulk API

2022-08-17 Thread Yongqin Liu
Hi, Douglas On Wed, 17 Aug 2022 at 22:26, Doug Anderson wrote: > > Hi, > > On Tue, Aug 16, 2022 at 5:58 AM Yongqin Liu wrote: > > > > HI, Douglas > > > > With this change, I get one kernel panic with my hikey960 > > android-mainline based Android build, > > if it's reverted, then the build

[Freedreno] [linux-next:master] BUILD SUCCESS WITH WARNING 95d10484d66e54d5c01e36389e9318221fb8f60b

2022-08-17 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: 95d10484d66e54d5c01e36389e9318221fb8f60b Add linux-next specific files for 20220817 Warning reports: https://lore.kernel.org/linux-doc/202208162058.7appivkl-...@intel.com https

Re: [Freedreno] [PATCH v2 4/7] regulator: core: Allow specifying an initial load w/ the bulk API

2022-08-17 Thread Doug Anderson
Hi, On Tue, Aug 16, 2022 at 5:58 AM Yongqin Liu wrote: > > HI, Douglas > > With this change, I get one kernel panic with my hikey960 > android-mainline based Android build, > if it's reverted, then the build could boot to the home screen successfully. > From the log information I shared here,

[Freedreno] [PATCH v2 4/5] dt-bindings: display/msm: dpu-sc7280: add missing DPU opp-table

2022-08-17 Thread Krzysztof Kozlowski
The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses opp-table, so reference it which allows restricting DPU schema to fixed list of properties. Fixes: 57fd4f34ddac ("dt-bindings: msm: add DT bindings for sc7280") Signed-off-by: Krzysztof Kozlowski --- Cc: Dmitry Baryshkov

[Freedreno] [PATCH v2 1/5] dt-bindings: display/msm: dpu-msm8998: add missing DPU opp-table

2022-08-17 Thread Krzysztof Kozlowski
The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses opp-table, so reference it which allows restricting DPU schema to fixed list of properties. Fixes: 6e986a8f1cf1 ("dt-bindings: display: msm: Add binding for msm8998 dpu") Signed-off-by: Krzysztof Kozlowski --- Cc: Dmitry

[Freedreno] [PATCH v2 3/5] dt-bindings: display/msm: dpu-sc7180: add missing DPU opp-table

2022-08-17 Thread Krzysztof Kozlowski
The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses opp-table, so reference it which allows restricting DPU schema to fixed list of properties. Fixes: 3d7a0dd8f39b ("dt-bindings: msm: disp: add yaml schemas for DPU bindings") Signed-off-by: Krzysztof Kozlowski --- Cc: Dmitry

[Freedreno] [PATCH v2 2/5] dt-bindings: display/msm: dpu-qcm2290: add missing DPU opp-table

2022-08-17 Thread Krzysztof Kozlowski
The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses opp-table, so reference it which allows restricting DPU schema to fixed list of properties. Fixes: 164f69d9d45a ("dt-bindings: msm: disp: add yaml schemas for QCM2290 DPU bindings") Signed-off-by: Krzysztof Kozlowski ---

[Freedreno] [PATCH v2 0/5] dt-bindings: display/msm: dpu: opp-table fixes

2022-08-17 Thread Krzysztof Kozlowski
Hi, Changes since v1 1. Use opp-table:object (Rob) Merging/other work == There is a conflicting series of https://lore.kernel.org/all/20220710090040.35193-1-dmitry.barysh...@linaro.org/ but I think this one here should go before to fix the issue before the

Re: [Freedreno] [PATCH 1/5] dt-bindings: display/msm: dpu-msm8998: add missing DPU opp-table

2022-08-17 Thread Krzysztof Kozlowski
On 15/08/2022 00:25, Rob Herring wrote: >> properties: >>compatible: >> @@ -105,6 +106,8 @@ patternProperties: >> maxItems: 1 >> >>operating-points-v2: true >> + opp-table: true > > type: object > > Otherwise, 'opp-table;' would be accepted. > Thanks, I'll