Re: [PATCH v4 5/9] drm/msm/hdmi: turn mode_set into atomic_enable

2024-06-03 Thread Maxime Ripard
On Fri, 31 May 2024 23:07:28 +0300, Dmitry Baryshkov wrote: > The mode_set callback is deprecated, it doesn't get the > drm_bridge_state, just mode-related argumetns. Turn it into the > atomic_enable callback as suggested by the documentation. > > Signed-off-by: Dmitry Baryshkov > > [ ... ] Ack

Re: [PATCH v4 7/9] drm/msm/hdmi: get rid of hdmi_mode

2024-06-03 Thread Maxime Ripard
On Fri, 31 May 2024 23:07:30 +0300, Dmitry Baryshkov wrote: > Use connector->display_info.is_hdmi instead of manually using > drm_detect_hdmi_monitor(). > > Signed-off-by: Dmitry Baryshkov Acked-by: Maxime Ripard Thanks! Maxime

Re: [PATCH v4 1/9] drm/connector: hdmi: accept NULL for Audio Infoframe

2024-06-03 Thread Maxime Ripard
Hi, Sorry for not answering your mail on the previous version sooner. On Fri, May 31, 2024 at 11:07:24PM GMT, Dmitry Baryshkov wrote: > Allow passing NULL as audio infoframe as a way to disable Audio > Infoframe generation. > > Signed-off-by: Dmitry Baryshkov > --- > drivers/gpu/drm/display/dr

Re: [PATCH v4 3/9] drm/bridge-connector: implement glue code for HDMI connector

2024-06-03 Thread Maxime Ripard
Hi, On Fri, May 31, 2024 at 11:07:26PM GMT, Dmitry Baryshkov wrote: > +static int drm_bridge_connector_clear_infoframe(struct drm_connector > *connector, > + enum hdmi_infoframe_type type) > +{ > + struct drm_bridge_connector *bridge_connector = > +

Re: [PATCH v4 6/9] drm/msm/hdmi: make use of the drm_connector_hdmi framework

2024-06-03 Thread Maxime Ripard
On Fri, 31 May 2024 23:07:29 +0300, Dmitry Baryshkov wrote: > Setup the HDMI connector on the MSM HDMI outputs. Make use of > atomic_check hook and of the provided Infoframe infrastructure. > > Signed-off-by: Dmitry Baryshkov Acked-by: Maxime Ripard Thanks! Maxime

Re: [PATCH v4 1/9] drm/connector: hdmi: accept NULL for Audio Infoframe

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 11:09:40AM +0200, Maxime Ripard wrote: > Hi, > > Sorry for not answering your mail on the previous version sooner. > > On Fri, May 31, 2024 at 11:07:24PM GMT, Dmitry Baryshkov wrote: > > Allow passing NULL as audio infoframe as a way to disable Audio > > Infoframe generati

Re: [PATCH 1/9] drm/msm/dpu: drop dpu_format_check_modified_format

2024-06-03 Thread Abhinav Kumar
On 4/22/2024 4:06 AM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 07:32:35PM -0700, Abhinav Kumar wrote: On 4/19/2024 6:26 PM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 04:43:20PM -0700, Abhinav Kumar wrote: On 3/19/2024 6:21 AM, Dmitry Baryshkov wrote: The msm_kms_funcs::che

Re: [PATCH v2 1/9] drm/msm/dpu: drop dpu_format_check_modified_format

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The msm_kms_funcs::check_modified_format() callback is not used by the driver. Drop it completely. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_formats.c | 43 - drivers/gpu/drm/msm/disp/dpu1

Re: [PATCH v2 2/9] drm/msm/dpu: drop dpu_format_populate_layout from dpu_plane_sspp_atomic_update

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The dpu_plane_prepare_fb() already calls dpu_format_populate_layout(). Store the generated layour in the plane state and drop this call from dpu_plane_sspp_update(). noticed a small typo (missed it in v1), layour --> layout. No need to resend,

Re: [PATCH 4/9] drm/msm/dpu: move dpu_format_populate_plane_sizes to atomic_check

2024-06-03 Thread Abhinav Kumar
On 4/22/2024 5:22 AM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 07:37:44PM -0700, Abhinav Kumar wrote: On 4/19/2024 6:34 PM, Dmitry Baryshkov wrote: On Fri, Apr 19, 2024 at 05:14:01PM -0700, Abhinav Kumar wrote: On 3/19/2024 6:22 AM, Dmitry Baryshkov wrote: Move a call to dpu_for

Re: [PATCH v2 4/9] drm/msm/dpu: move dpu_format_populate_plane_sizes to atomic_check

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Move a call to dpu_format_populate_plane_sizes() to the atomic_check step, so that any issues with the FB layout can be reported as early as possible. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 12 ++-

Re: [PATCH v2 6/9] drm/msm/dpu: drop call to _dpu_crtc_setup_lm_bounds from atomic_begin

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: The dpu_crtc_atomic_check() already calls _dpu_crtc_setup_lm_bounds(). There is no need to call it from dpu_crtc_atomic_begin(). Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 2 -- 1 file changed, 2 deletions(-)

[PATCH] drm/msm/adreno: request memory region

2024-06-03 Thread Kiarash Hajian
The driver's memory regions are currently just ioremap()ed, but not reserved through a request. That's not a bug, but having the request is a little more robust. Implement the region-request through the corresponding managed devres-function. Signed-off-by: Kiarash Hajian --- drivers/gpu/drm/msm

[PATCH v3 2/2] drm/msm/a6xx: request memory region

2024-06-03 Thread Kiarash Hajian
The devm_iounmap function is being used unnecessarily, managed resource mechanisms (devres) are handling resource cleanup automatically This commit removes the calls to devm_iounmap and relies on devres Signed-off-by: Kiarash Hajian --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 18 ---

Re: [PATCH] drm/msm/adreno: De-spaghettify the use of memory barriers

2024-06-03 Thread Andrew Halaney
On Wed, May 15, 2024 at 12:08:49AM GMT, Akhil P Oommen wrote: > On Wed, May 08, 2024 at 07:46:31PM +0200, Konrad Dybcio wrote: > > Memory barriers help ensure instruction ordering, NOT time and order > > of actual write arrival at other observers (e.g. memory-mapped IP). > > On architectures employ

Re: [PATCH] drm/msm/dpu: guard ctl irq callback register/unregister

2024-06-03 Thread Barnabás Czémán
There was some previously sent patch series made by Dmitry for these soc, msm8996 was sent together by sdm660 but in the last version it was dropped. I have recreated msm8996 DPU support from that series but it will need some more test. I am testing msm8953 series but it is depend on msm8996.

Re: [PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-06-03 Thread Steven Rostedt
On Fri, 17 May 2024 10:36:37 -0700 Guenter Roeck wrote: > Building csky:allmodconfig (and others) ... failed > -- > Error log: > In file included from include/trace/trace_events.h:419, > from include/trace/define_trace.h:102, > from drivers/cxl/core/t

[PATCH] drm/msm/adreno: Add A306A support

2024-06-03 Thread Barnabás Czémán
From: Otto Pflüger Add support for Adreno 306A GPU what is found in MSM8917 SoC. This GPU marketing name is Adreno 308. Signed-off-by: Otto Pflüger [use internal name of the GPU, reword the commit message] Signed-off-by: Barnabás Czémán --- drivers/gpu/drm/msm/adreno/a3xx_gpu.c | 14

[PATCH v4] drm/msm/a6xx: request memory region

2024-06-03 Thread Kiarash Hajian
The driver's memory regions are currently just ioremap()ed, but not reserved through a request. That's not a bug, but having the request is a little more robust. Implement the region-request through the corresponding managed devres-function. Signed-off-by: Kiarash Hajian --- Changes in v4: - Com

[PATCH] drm/msm/dpu: drop duplicate drm formats from wb2_formats arrays

2024-06-03 Thread Junhao Xie
There are duplicate items in wb2_formats_rgb and wb2_formats_rgb_yuv, which cause weston assertions failed. weston: libweston/drm-formats.c:131: weston_drm_format_array_add_format: Assertion `!weston_drm_format_array_find_format(formats, format)' failed. Signed-off-by: Junhao Xie --- drivers/gp

Re: [PATCH] drm/msm/adreno: De-spaghettify the use of memory barriers

2024-06-03 Thread Andrew Halaney
On Thu, May 16, 2024 at 08:20:05PM GMT, Akhil P Oommen wrote: > On Thu, May 16, 2024 at 08:15:34AM -0500, Andrew Halaney wrote: > > On Wed, May 15, 2024 at 12:08:49AM GMT, Akhil P Oommen wrote: > > > On Wed, May 08, 2024 at 07:46:31PM +0200, Konrad Dybcio wrote: > > > > Memory barriers help ensure

[PATCH v2] arm64: dts: qcom: msm8998: enable adreno_smmu by default

2024-06-03 Thread Marc Gonzalez
15 qcom platform DTSI files define an adreno_smmu node. msm8998 is the only one with adreno_smmu disabled by default. There's no reason why this SMMU should be disabled by default, it doesn't need any further configuration. Bring msm8998 in line with the 14 other platforms. This fixes GPU init f

[PATCH] drm/msm/dpu: fix encoder irq wait skip

2024-06-03 Thread Barnabás Czémán
The irq_idx is unsigned so it cannot be lower than zero, better to change the condition to check if it is equal with zero. It could not cause any issue because a valid irq index starts from one. Signed-off-by: Barnabás Czémán --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 2 +- 1 file changed

Re: [PATCH] drm/msm: log iommu init failure

2024-06-03 Thread Marc Gonzalez
On 16/05/2024 10:43, Marijn Suijten wrote: > On 2024-05-15 17:09:02, Marc Gonzalez wrote: > >> When create_address_space() fails (e.g. when smmu node is disabled) >> msm_gpu_init() silently fails: >> >> msm_dpu c901000.display-controller: failed to load adreno gpu >> msm_dpu c901000.display-contro

[PATCH v2] drm/msm/a6xx: request memory region

2024-06-03 Thread Kiarash Hajian
The driver's memory regions are currently just ioremap()ed, but not reserved through a request. That's not a bug, but having the request is a little more robust. Implement the region-request through the corresponding managed devres-function. Signed-off-by: Kiarash Hajian --- Changes in v2: - upd

[PATCH v3 0/2] drm/msm/a6xx: request memory region

2024-06-03 Thread Kiarash Hajian
Signed-off-by: Kiarash Hajian --- Changes in v3: - Remove redundant devm_iounmap calls, relying on devres for automatic resource cleanup. Changes in v2: - update the subject prefix to "drm/msm/a6xx:", to match the majority of other changes to this file. --- Kiarash Hajian (2): drm/msm/a6

[PATCH] drm/msm/dpu: guard ctl irq callback register/unregister

2024-06-03 Thread Barnabás Czémán
CTLs on older qualcomm SOCs like msm8953 and msm8996 has not got interrupts, so better to skip CTL irq callback register/unregister make dpu_ctl_cfg be able to define without intr_start. Signed-off-by: Barnabás Czémán --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 4 ++-- 1 file chan

[PATCH] arm64: dts: qcom: msm8998: enable adreno_smmu

2024-06-03 Thread Marc Gonzalez
Right now, GPU init fails: [2.756363] [drm:adreno_bind] Found GPU: 5.4.0.1 [2.767183] [drm:a5xx_gpu_init] [2.767422] [drm:adreno_gpu_init] fast_rate=71097, slow_rate=2700 [3.003869] [drm:msm_gpu_init] ebi1_clk: fffe [3.004002] adreno 500.gpu: supply vdd

[PATCH v3 1/2] drm/msm/a6xx: request memory region

2024-06-03 Thread Kiarash Hajian
The driver's memory regions are currently just ioremap()ed, but not reserved through a request. That's not a bug, but having the request is a little more robust. Implement the region-request through the corresponding managed devres-function. Signed-off-by: Kiarash Hajian --- drivers/gpu/drm/msm

[PATCH v2] drm/msm/dpu: fix encoder irq wait skip

2024-06-03 Thread Barnabás Czémán
The irq_idx is unsigned so it cannot be lower than zero, better to change the condition to check if it is equal with zero. It could not cause any issue because a valid irq index starts from one. Fixes: 5a9d50150c2c ("drm/msm/dpu: shift IRQ indices by 1") Signed-off-by: Barnabás Czémán --- Changes

[RFC PATCH v1] drm/msm: add msm8998 hdmi phy/pll support

2024-06-03 Thread Marc Gonzalez
From: Arnaud Vrac Ported from the downstream driver. Signed-off-by: Arnaud Vrac Signed-off-by: Marc Gonzalez --- drivers/gpu/drm/msm/Makefile | 1 + drivers/gpu/drm/msm/hdmi/hdmi.c | 1 + drivers/gpu/drm/msm/hdmi/hdmi.h | 8 + drivers/gpu/drm/msm/hdmi/hdmi.

[PATCH] drm/msm: log iommu init failure

2024-06-03 Thread Marc Gonzalez
When create_address_space() fails (e.g. when smmu node is disabled) msm_gpu_init() silently fails: msm_dpu c901000.display-controller: failed to load adreno gpu msm_dpu c901000.display-controller: failed to bind 500.gpu (ops a3xx_ops): -19 Log create_address_space() failure. Signed-off-by:

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing LM requirements. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff -

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Dmitry Baryshkov
On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar wrote: > > > > On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: > > Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing > > LM requirements. > > > > Signed-off-by: Dmitry Baryshkov > > --- > > drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c |

Re: [PATCH] drm/msm/dpu: drop duplicate drm formats from wb2_formats arrays

2024-06-03 Thread Dmitry Baryshkov
On Fri, May 24, 2024 at 11:01:12PM +0800, Junhao Xie wrote: > There are duplicate items in wb2_formats_rgb and wb2_formats_rgb_yuv, > which cause weston assertions failed. > > weston: libweston/drm-formats.c:131: weston_drm_format_array_add_format: > Assertion `!weston_drm_format_array_find_format

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Abhinav Kumar
On 6/3/2024 2:16 PM, Dmitry Baryshkov wrote: On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar wrote: On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Check in _dpu_crtc_setup_lm_bounds() that CRTC width is not overflowing LM requirements. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/ms

Re: [PATCH v2 9/9] drm/msm/dpu: sync mode_config limits to the FB limits in dpu_plane.c

2024-06-03 Thread Abhinav Kumar
On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: Lift mode_config limits set by the DPU driver to the actual FB limits as handled by the dpu_plane.c. Move 2*max_lm_width check where it belongs, to the drm_crtc_helper_funcs::mode_valid() callback. Signed-off-by: Dmitry Baryshkov --- drivers/gpu

Re: [PATCH v2 7/9] drm/msm/dpu: check for overflow in _dpu_crtc_setup_lm_bounds()

2024-06-03 Thread Dmitry Baryshkov
On Mon, Jun 03, 2024 at 02:24:27PM -0700, Abhinav Kumar wrote: > > > On 6/3/2024 2:16 PM, Dmitry Baryshkov wrote: > > On Mon, 3 Jun 2024 at 23:57, Abhinav Kumar > > wrote: > > > > > > > > > > > > On 6/2/2024 2:39 PM, Dmitry Baryshkov wrote: > > > > Check in _dpu_crtc_setup_lm_bounds() that C

Re: [PATCH v2] drm/msm/dpu: fix encoder irq wait skip

2024-06-03 Thread Abhinav Kumar
On 5/9/2024 12:40 PM, Barnabás Czémán wrote: The irq_idx is unsigned so it cannot be lower than zero, better to change the condition to check if it is equal with zero. It could not cause any issue because a valid irq index starts from one. Fixes: 5a9d50150c2c ("drm/msm/dpu: shift IRQ indices

[PATCH] drm/msm/adreno: Add support for Adreno 505 GPU

2024-06-03 Thread Barnabás Czémán
tatic inline int adreno_is_a506(const struct adreno_gpu *gpu) { return adreno_is_revn(gpu, 506); --- base-commit: 861a3cb5a2a8480d361fa6708da24747d6fa72fe change-id: 20240603-a505-e57389b61d2a Best regards, -- Barnabás Czémán