Re: [Freedreno] [PATCH] drm/msm/dp: Use the connector passed to dp_debug_get()

2021-10-12 Thread abhinavk
On 2021-10-12 16:38, Bjorn Andersson wrote: On Tue 12 Oct 16:03 PDT 2021, abhin...@codeaurora.org wrote: On 2021-10-09 20:04, Bjorn Andersson wrote: > The debugfs code is provided an array of a single drm_connector. Then to > access the connector, the list of all connectors of the DRM device

Re: [Freedreno] [PATCH] drm/msm/dp: Use the connector passed to dp_debug_get()

2021-10-12 Thread Bjorn Andersson
On Tue 12 Oct 16:03 PDT 2021, abhin...@codeaurora.org wrote: > On 2021-10-09 20:04, Bjorn Andersson wrote: > > The debugfs code is provided an array of a single drm_connector. Then to > > access the connector, the list of all connectors of the DRM device is > > traversed and all non-DisplayPort

Re: [Freedreno] [PATCH] drm/msm/dp: Use the connector passed to dp_debug_get()

2021-10-12 Thread abhinavk
On 2021-10-09 20:04, Bjorn Andersson wrote: The debugfs code is provided an array of a single drm_connector. Then to access the connector, the list of all connectors of the DRM device is traversed and all non-DisplayPort connectors are skipped, to find the one and only DisplayPort connector.

Re: [Freedreno] [PATCH] drm/msm/dp: Use the connector passed to dp_debug_get()

2021-10-12 Thread Stephen Boyd
Quoting Bjorn Andersson (2021-10-09 20:04:35) > The debugfs code is provided an array of a single drm_connector. Then to > access the connector, the list of all connectors of the DRM device is > traversed and all non-DisplayPort connectors are skipped, to find the > one and only DisplayPort

Re: [Freedreno] [PATCH v2] drm/msm/dsi: Use division result from div_u64_rem in 7nm and 14nm PLL

2021-10-12 Thread Stephen Boyd
Quoting Marijn Suijten (2021-10-11 13:16:40) > div_u64_rem provides the result of the division and additionally the > remainder; don't use this function to solely calculate the remainder > while calculating the division again with div_u64. > > A similar improvement was applied earlier to the 10nm

Re: [Freedreno] [PATCH] drm: Remove redundant 'flush_workqueue()' calls

2021-10-12 Thread Rob Clark
On Sun, Oct 10, 2021 at 7:07 AM Christophe JAILLET wrote: > > 'destroy_workqueue()' already drains the queue before destroying it, so > there is no need to flush it explicitly. > > Remove the redundant 'flush_workqueue()' calls. > > This was generated with coccinelle: > > @@ > expression E; > @@

Re: [Freedreno] [PATCH] drm: Remove redundant 'flush_workqueue()' calls

2021-10-12 Thread Jyri Sarha
On 2021-10-10 16:59, Christophe JAILLET wrote: 'destroy_workqueue()' already drains the queue before destroying it, so there is no need to flush it explicitly. Remove the redundant 'flush_workqueue()' calls. This was generated with coccinelle: @@ expression E; @@ - flush_workqueue(E);

Re: [Freedreno] [PATCH] drm/msm/dpu: Add CRC support for DPU

2021-10-12 Thread kernel test robot
' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Jessica-Zhang/drm-msm-dpu-Add-CRC-support-for-DPU/20211012-074357 base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 64570fbc14f8d7cb3fe3995f20e26bc25ce4b2cc config: arm

[Freedreno] [bug report] drm/msm/a6xx: Fix llcc configuration for a660 gpu

2021-10-12 Thread Dan Carpenter
Hello Akhil P Oommen, The patch a6f24383f6c0: "drm/msm/a6xx: Fix llcc configuration for a660 gpu" from Jul 30, 2021, leads to the following Smatch static checker warning: drivers/gpu/drm/msm/adreno/a6xx_gpu.c:1480 a6xx_llc_activate() error: uninitialized symbol 'gpu_scid'.