Re: [PATCH 2/4] drm/msm/hdmi: use new helper for ACR tables

2025-03-09 Thread kernel test robot
Hi Dmitry, kernel test robot noticed the following build errors: [auto build test ERROR on 0a2f889128969dab41861b6e40111aa03dc57014] url: https://github.com/intel-lab-lkp/linux/commits/Dmitry-Baryshkov/drm-display-hdmi-provide-central-data-authority-for-ACR-params/20250309-161610 base

Re: [PATCH 4/4] drm: bridge: dw-hdmi: use new helper to get ACR values

2025-03-09 Thread kernel test robot
Hi Dmitry, kernel test robot noticed the following build errors: [auto build test ERROR on 0a2f889128969dab41861b6e40111aa03dc57014] url: https://github.com/intel-lab-lkp/linux/commits/Dmitry-Baryshkov/drm-display-hdmi-provide-central-data-authority-for-ACR-params/20250309-161610 base

[pull] drm/msm: drm-msm-next-2025-03-09 for v6.15

2025-03-09 Thread Rob Clark
Hi Dave, Simona, Pull for v6.15, as described below. The following changes since commit 72d0af4accd965dc32f504440d74d0a4d18bf781: drm/msm/dp: Add support for LTTPR handling (2025-02-25 18:34:14 +0200) are available in the Git repository at: https://gitlab.freedesktop.org/drm/msm.git tags/d

[PATCH v2] drm/msm/dpu: reorder pointer operations after sanity checks to avoid NULL deref

2025-03-09 Thread Qasim Ijaz
_dpu_encoder_trigger_start dereferences "struct dpu_encoder_phys *phys" before the sanity checks which can lead to a NULL pointer dereference if phys is NULL. Fix this by reordering the dereference after the sanity checks. Fixes: 8144d17a81d9 ("drm/msm/dpu: Skip trigger flush and start for CWB"

[PATCH] drm/msm/dpu: reorder pointer operations after sanity checks to avoid NULL deref

2025-03-09 Thread Qasim Ijaz
_dpu_encoder_trigger_start dereferences "struct dpu_encoder_phys *phys" before the sanity checks which can lead to a NULL pointer dereference if phys is NULL. Fix this by reordering the dereference after the sanity checks. Signed-off-by: Qasim Ijaz Fixes: 8144d17a81d9 ("drm/msm/dpu: Skip trig

Re: [PATCH v2] drm/msm/dpu: reorder pointer operations after sanity checks to avoid NULL deref

2025-03-09 Thread Dmitry Baryshkov
On Sun, Mar 09, 2025 at 09:55:25AM +, Qasim Ijaz wrote: > _dpu_encoder_trigger_start dereferences "struct dpu_encoder_phys *phys" > before the sanity checks which can lead to a NULL pointer dereference if > phys is NULL. > > Fix this by reordering the dereference after the sanity checks. >

[PATCH 0/4] drm/display: hdmi: provide common code to get Audio Clock Recovery params

2025-03-09 Thread Dmitry Baryshkov
HDMI standards define a recommended set of values to be used for Audio Clock Regeneration. Nevertheless, each HDMI driver dealing with audio implements its own way to determine those values. Implement a common helper and use it for MSM HDMI (tested), VC4 and DW-HDMI (compile-tested only) drivers.

Re: [PATCH 08/11] arm64: dts: qcom: sc8180x: Rename AOSS_QMP to power-management

2025-03-09 Thread Dmitry Baryshkov
On Thu, Mar 06, 2025 at 07:11:20PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio > > The node is currently named power-controller, which requires the device > underneath is a power domain provider. Rename it to align with other > SoCs and resolve this sort of warnings: > > power-controller@c

[PATCH 3/4] drm/vc4: use new helper to get ACR values

2025-03-09 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Use drm_hdmi_acr_get_n_cts() helper instead of calculating N and CTS values in the VC4 driver. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/vc4/vc4_hdmi.c | 10 +++--- drivers/gpu/drm/vc4/vc4_hdmi.h | 7 +++ 2 files changed, 10 insertions(+), 7 deletions(

Re: [PATCH] drm/msm/dpu: reorder pointer operations after sanity checks to avoid NULL deref

2025-03-09 Thread Dmitry Baryshkov
On Sat, Mar 08, 2025 at 02:48:39PM +, Qasim Ijaz wrote: > _dpu_encoder_trigger_start dereferences "struct dpu_encoder_phys *phys" > before the sanity checks which can lead to a NULL pointer dereference if > phys is NULL. > > Fix this by reordering the dereference after the sanity checks. >

[PATCH 4/4] drm: bridge: dw-hdmi: use new helper to get ACR values

2025-03-09 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Use drm_hdmi_acr_get_n_cts() helper instead of calculating N and CTS values in the DW-HDMI driver. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 90 +++ 1 file changed, 8 insertions(+), 82 deletions(-) diff -

[PATCH 1/4] drm/display: hdmi: provide central data authority for ACR params

2025-03-09 Thread Dmitry Baryshkov
From: Dmitry Baryshkov HDMI standard defines recommended N and CTS values for Audio Clock Regeneration. Currently each driver implements those, frequently in somewhat unique way. Provide a generic helper for getting those values to be used by the HDMI drivers. The helper is added to drm_hdmi_hel

[PATCH 2/4] drm/msm/hdmi: use new helper for ACR tables

2025-03-09 Thread Dmitry Baryshkov
From: Dmitry Baryshkov Use new drm_hdmi_acr_get_n_cts() helper instead of hand-coding the tables. Instead of storing the rate 'index', store the audio sample rate in hdmi->audio.rate, removing the need for even more defines. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/hdmi/hdmi_aud