[PATCH v5] drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp

2024-07-15 Thread Mitul Golani
Describe newly added parameter target_rr_divider in struct drm_dp_as_sdp. -v2: Remove extra line from commit message.(Lucas) -v3: Rebase. Fixes: a20c6d954d75 ("drm/dp: Add refresh rate divider to struct representing AS SDP") Cc: Mitul Golani Cc: Arun R Murthy Cc: Suraj Kandpal

[PATCH v3] drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp

2024-07-14 Thread Mitul Golani
Describe newly added parameter target_rr_divider in struct drm_dp_as_sdp. -v2: Remove extra line from commit message.(Lucas) -v3: Rebase. Fixes: a20c6d954d75 ("drm/dp: Add refresh rate divider to struct representing AS SDP") Cc: Mitul Golani Cc: Arun R Murthy Cc: Suraj Kandpal

[PATCH v5] drm/i915/display: WA for Re-initialize dispcnlunitt1 xosc clock

2024-07-08 Thread Mitul Golani
: - Rebase. --v3: - Correct HSD number in commit message. --v4: - Reformat commit message. - Use intel_de_rmw instead of intel_de_write --v5: - Build Fixes. Signed-off-by: Mitul Golani Reviewed-by: Nemesa Garg --- drivers/gpu/drm/i915/display/intel_display_power.c | 8 1 file changed, 8

[PATCH v4] drm/i915/display: WA to Re-initialize dispcnlunitt1 xosc clock

2024-07-07 Thread Mitul Golani
: - Rebase. --v3: - Correct HSD number in commit message. --v4: - Reformat commit message. - Use intel_de_rmw instead of intel_de_write Signed-off-by: Mitul Golani Reviewed-by: Nemesa Garg --- drivers/gpu/drm/i915/display/intel_display_power.c | 8 1 file changed, 8 insertions(+) diff --git

[PATCH v2 1/1] drm/i915/display: Cache adpative sync caps to use it later

2024-07-04 Thread Mitul Golani
Add new member to struct intel_dp to cache support of Adaptive Sync SDP capabilities and use it whenever required to avoid HW access to read capability during each atomic commit. -v2: - Squash both the patches Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_alpm.c | 2

[PATCH v2 0/1] Cache SDP caps during connector detection

2024-07-04 Thread Mitul Golani
Cache connector caps during connector detection instead of reading capabilities each time during atomic commit. -v2: - Squash both the patches(Jani). Mitul Golani (1): drm/i915/display: Cache adpative sync caps to use it later drivers/gpu/drm/i915/display/intel_alpm.c | 2 +- .../drm

[PATCH 2/2] drm/i915/display: Cache Adaptive Sync SDP caps

2024-07-01 Thread Mitul Golani
Cache as sdp caps during connector detection itself and remove intel_dp_as_sdp_supported usage as it is being taken care by already caching caps. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 19 +++ drivers/gpu/drm/i915/display/intel_dp.h | 1 - 2

[PATCH 0/2] Cache SDP caps during connector detection

2024-07-01 Thread Mitul Golani
Cache connector caps during connector detection instead of reading capabilities each time during atomic commit. Mitul Golani (2): drm/i915/display: Avoid reading as sdp caps during each atomic commit drm/i915/display: Cache Adaptive Sync SDP caps drivers/gpu/drm/i915/display/intel_alpm.c

[PATCH 1/2] drm/i915/display: Avoid reading as sdp caps during each atomic commit

2024-07-01 Thread Mitul Golani
Add new member to struct intel_dp to cache support of Adaptive Sync SDP capabilities and use it whenever required to avoid HW access to read capability during each atomic commit. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_alpm.c | 2 +- drivers/gpu/drm/i915

[PATCH] drm/i915/display: Consider adjusted_pixel_rate to be u64

2024-06-21 Thread Mitul Golani
Consider adjusted_pixel_rate to be a u64 to match the return type of mul_u32_u32() and avoid any compiler dependency for do_div. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Rodrig

[PATCH v3] drm/i915/display: WA for Re-initialize dispcnlunitt1 xosc clock

2024-06-19 Thread Mitul Golani
-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display_power.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c index e288a1b21d7e..aef54c1a2ba9 100644 --- a/drivers/gpu/drm

[PATCH v2 1/1] drm/i915/display: WA for Re-initialize dispcnlunitt1 xosc clock

2024-06-19 Thread Mitul Golani
: - Update workaround number in commit message. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display_power.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c index

[PATCH v2 0/1] drm/i915/display: WA for Re-initialize dispcnlunitt1 xosc clock

2024-06-19 Thread Mitul Golani
The dispcnlunit1_cp_xosc_clk should be de-asserted in display off and only asserted in display on. But during observation it found clk remains active in display OFF. As workaround, Display driver shall execute set-reset sequence at the end of the Initialize Sequence. Wa_14020225554 Mitul Golani

[PATCH v2] drm/i915/display: Update calculation to avoid overflow

2024-06-13 Thread Mitul Golani
Update calculation to avoid overflow. -v2: Remove extra line between cc and signed-off. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Stephen Rothwell Signed-off-by: Mitul Golani

[PATCH v2 0/2] CMRR patch fixes

2024-06-13 Thread Mitul Golani
Address following issues regarding CMRR 1. Describe target_rr_divider in struct drm_dp_as_sdp. 2. Use required macro to avoid overflow. -v2: - Remove extra line from commit message. Mitul Golani (2): drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp drm/i915/display: Update

[PATCH v2 1/2] drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp

2024-06-13 Thread Mitul Golani
Describe newly added parameter target_rr_divider in struct drm_dp_as_sdp. -v2: Remove extra line from commit message.(Lucas) Fixes: a20c6d954d75 ("drm/dp: Add refresh rate divider to struct representing AS SDP") Cc: Mitul Golani Cc: Arun R Murthy Cc: Suraj Kandpal Cc: Ankit Na

[PATCH v2 2/2] drm/i915/display: Update calculation to avoid overflow

2024-06-13 Thread Mitul Golani
Update calculation to avoid overflow. -v2: Remove extra line from commit message.(Lucas) Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Stephen Rothwell Signed-off-by: Mitul Golani

[PATCH] drm/i915/display: Update calculation to avoid overflow

2024-06-13 Thread Mitul Golani
Update calculation to avoid overflow. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Stephen Rothwell Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i9

[PATCH 2/2] drm/i915/display: Update calculation to avoid overflow

2024-06-12 Thread Mitul Golani
Update calculation to avoid overflow. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Stephen Rothwell Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i9

[PATCH 0/2] CMRR patch fixes

2024-06-12 Thread Mitul Golani
Address following issues regarding CMRR 1. Describe target_rr_divider in struct drm_dp_as_sdp. 3. Use required macro to avoid overflow. Mitul Golani (2): drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp drm/i915/display: Update calculation to avoid overflow drivers/gpu/drm/i915

[PATCH 1/2] drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp

2024-06-12 Thread Mitul Golani
Describe newly added parameter target_rr_divider in struct drm_dp_as_sdp. Fixes: a20c6d954d75 ("drm/dp: Add refresh rate divider to struct representing AS SDP") Cc: Mitul Golani Cc: Arun R Murthy Cc: Suraj Kandpal Cc: Ankit Nautiyal Cc: Jani Nikula Cc: Stephen Rothwell

[PATCH 2/3] drm/i915/display: Send vrr vsync params whne vrr is enabled

2024-06-12 Thread Mitul Golani
Compute trans vrr vsync params only when either VRR or CMRR is enabled. Fixes: 5922f45329cd ("drm/i915/display: Compute vrr vsync params") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gp

[PATCH 3/3] drm/i915/display: Update calculation to avoid overflow

2024-06-12 Thread Mitul Golani
Update calculation to avoid overflow. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Cc: Stephen Rothwell Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 9 ++

[PATCH 0/3] CMRR patch fixes

2024-06-12 Thread Mitul Golani
Address following issues regarding CMRR 1. Describe target_rr_divider in struct drm_dp_as_sdp. 2. Compute vrr_vsync params when vrr.enable is set. 3. Use required macro to avoid overflow. Mitul Golani (3): drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp drm/i915/display: Send vrr

[PATCH 1/3] drm/dp: Describe target_rr_divider in struct drm_dp_as_sdp

2024-06-12 Thread Mitul Golani
Describe newly added parameter target_rr_divider in struct drm_dp_as_sdp. Fixes: a20c6d954d75 ("drm/dp: Add refresh rate divider to struct representing AS SDP") Cc: Mitul Golani Cc: Arun R Murthy Cc: Suraj Kandpal Cc: Ankit Nautiyal Cc: Jani Nikula Cc: Stephen Rothwell

[PATCH] drm/i915/display: Update vtotal math to address 32b build

2024-06-11 Thread Mitul Golani
Fix vtotal division calculation which works for 32b systems. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 3 +

[RFC] drm/i915/display: Update vtotal math to address 32b build

2024-06-11 Thread Mitul Golani
Fix few divisions which may not work on 32b builds. Use DIV_ROUND_UP, with that expecting deviate params from +/- 1 accuracy in value. Fixes: 1676ecd303ac ("drm/i915: Compute CMRR and calculate vtotal") Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 10 +++

[PATCH] drm/i915/display: Send vrr vsync params whne vrr is enabled

2024-06-11 Thread Mitul Golani
Compute trans vrr vsync params only when either VRR or CMRR is enabled. Fixes: 5922f45329cd ("drm/i915/display: Compute vrr vsync params") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel

[PATCH] drm/i915/display: Send vrr vsync params whne vrr is enabled

2024-06-11 Thread Mitul Golani
Compute trans vrr vsync params only when either VRR or CMRR is enabled. Fixes: 5922f45329cd ("drm/i915/display: Compute vrr vsync params") Cc: Mitul Golani Cc: Ankit Nautiyal Cc: Suraj Kandpal Cc: Jani Nikula Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel

[PATCH v16 6/9] drm/i915/display: Add support for pack and unpack

2024-06-10 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v16 9/9] drm/i915: Compute CMRR and calculate vtotal

2024-06-10 Thread Mitul Golani
cmrr.enable.(Ankit) - Add TODO: for to address target refresh rate precision as future enhancement. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_display.c | 1 + .../drm/i915/display/intel_display_device.h | 1 + drivers/gpu/drm/i915/display

[PATCH v16 7/9] drm/i915/display: Compute Adaptive sync SDP params

2024-06-10 Thread Mitul Golani
intel_dp_compute_as_sdp. --v4: - Use drm_mode_vrefresh instead of manual calculation (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915

[PATCH v16 8/9] drm/i915/display: Compute vrr vsync params

2024-06-10 Thread Mitul Golani
) - Replace vrr.enable flag to cmrr.enable, mistakenly added. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c

[PATCH v16 5/9] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-06-10 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy Acked-by: Maxime Ripard --- include/drm/display

[PATCH v16 2/9] drm/i915: Separate VRR related register definitions

2024-06-10 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 111 ++ drivers/gpu/drm/i915

[PATCH v16 1/9] drm/i915: Update indentation for VRR registers and bits

2024-06-10 Thread Mitul Golani
. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/i915_reg.h | 174 1 file changed, 87 insertions(+), 87 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 7daf902772e4..3fbf639e6aa0

[PATCH v16 4/9] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-06-10 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/

[PATCH v16 3/9] drm/i915: Define and compute Transcoder CMRR registers

2024-06-10 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) --v7: - Remove adding CMRR flag to vrr_ctl register during set_transcoder_timing, as it is already being done during intel_vrr_enable. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v16 0/9] Implement CMRR Support

2024-06-10 Thread Mitul Golani
. -v12: - Add patch to fix check patch issues for VRR related registers in i915_reg.h then move them to intel_vrr_regs.h with separate patch. -v13: - Reverted unrelated patches while rebase. -v14: - Fix all indentations for VRR related registes in Patch#1 -v15: - Rebase. Mitul Golani (9): drm/i915

[PATCH v15 7/9] drm/i915/display: Compute Adaptive sync SDP params

2024-06-09 Thread Mitul Golani
intel_dp_compute_as_sdp. --v4: - Use drm_mode_vrefresh instead of manual calculation (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915

[PATCH v15 6/9] drm/i915/display: Add support for pack and unpack

2024-06-09 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v15 3/9] drm/i915: Define and compute Transcoder CMRR registers

2024-06-09 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) --v7: - Remove adding CMRR flag to vrr_ctl register during set_transcoder_timing, as it is already being done during intel_vrr_enable. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v15 9/9] drm/i915: Compute CMRR and calculate vtotal

2024-06-09 Thread Mitul Golani
cmrr.enable.(Ankit) - Add TODO: for to address target refresh rate precision as future enhancement. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_display.c | 1 + .../drm/i915/display/intel_display_device.h | 1 + drivers/gpu/drm/i915/display

[PATCH v15 8/9] drm/i915/display: Compute vrr vsync params

2024-06-09 Thread Mitul Golani
) - Replace vrr.enable flag to cmrr.enable, mistakenly added. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c

[PATCH v15 5/9] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-06-09 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy Acked-by: Maxime Ripard --- include/drm/display

[PATCH v15 4/9] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-06-09 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/

[PATCH v15 2/9] drm/i915: Separate VRR related register definitions

2024-06-09 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 111 ++ drivers/gpu/drm/i915/i915_reg.h | 100

[PATCH v15 0/9] Implement CMRR Support

2024-06-09 Thread Mitul Golani
. -v12: - Add patch to fix check patch issues for VRR related registers in i915_reg.h then move them to intel_vrr_regs.h with separate patch. -v13: - Reverted unrelated patches while rebase. -v14: - Fix all indentations for VRR related registes in Patch#1 -v15: - Rebase. Mitul Golani (9): gpu/drm

[PATCH v15 1/9] gpu/drm/i915: Update indentation for VRR registers and bits

2024-06-09 Thread Mitul Golani
. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/i915_reg.h | 174 1 file changed, 87 insertions(+), 87 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 7daf902772e4..a10591424338

[PATCH v14 2/9] drm/i915: Separate VRR related register definitions

2024-06-07 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 114 ++ drivers/gpu/drm/i915/i915_reg.h | 103

[PATCH v14 1/9] gpu/drm/i915: Update indentation for VRR registers and bits

2024-06-07 Thread Mitul Golani
. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/i915_reg.h | 183 1 file changed, 93 insertions(+), 90 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index 15ad35178f1a..295689d163c5

[PATCH v13 8/9] drm/i915/display: Compute vrr vsync params

2024-06-05 Thread Mitul Golani
) - Replace vrr.enable flag to cmrr.enable, mistakenly added. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c

[PATCH v13 7/9] drm/i915/display: Compute Adaptive sync SDP params

2024-06-05 Thread Mitul Golani
intel_dp_compute_as_sdp. --v4: - Use drm_mode_vrefresh instead of manual calculation (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915

[PATCH v13 4/9] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-06-05 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/

[PATCH v13 9/9] drm/i915: Compute CMRR and calculate vtotal

2024-06-05 Thread Mitul Golani
cmrr.enable.(Ankit) - Add TODO: for to address target refresh rate precision as future enhancement. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_display.c | 1 + .../drm/i915/display/intel_display_device.h | 1 + drivers/gpu/drm/i915/display

[PATCH v13 5/9] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-06-05 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy --- include/drm/display/drm_dp_helper.h | 1 + 1

[PATCH v13 6/9] drm/i915/display: Add support for pack and unpack

2024-06-05 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v13 3/9] drm/i915: Define and compute Transcoder CMRR registers

2024-06-05 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) --v7: - Remove adding CMRR flag to vrr_ctl register during set_transcoder_timing, as it is already being done during intel_vrr_enable. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v13 2/9] drm/i915: Separate VRR related register definitions

2024-06-05 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 112 ++ drivers/gpu/drm/i915/i915_reg.h | 101

[PATCH v13 1/9] gpu/drm/i915: Update indentation for VRR registers and bits

2024-06-05 Thread Mitul Golani
Update the indentation for the VRR register definition and its bits, and fix checkpatch issues to ensure smooth movement of registers and bits. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/i915_reg.h | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH v13 0/9] Implement CMRR Support

2024-06-05 Thread Mitul Golani
. -v12: - Add patch to fix check patch issues for VRR related registers in i915_reg.h then move them to intel_vrr_regs.h with separate patch. -v13: - Reverted unrelated patches while rebase. Mitul Golani (9): gpu/drm/i915: Update indentation for VRR registers and bits drm/i915: Separate VRR

[PATCH v12 8/9] drm/i915/display: Add support for pack and unpack

2024-06-05 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v12 9/9] drm/i915/display: Compute Adaptive sync SDP params

2024-06-05 Thread Mitul Golani
intel_dp_compute_as_sdp. --v4: - Use drm_mode_vrefresh instead of manual calculation (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_dp.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915

[PATCH v12 7/9] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-06-05 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy --- include/drm/display/drm_dp_helper.h | 1 + 1

[PATCH v12 5/9] drm/i915: Define and compute Transcoder CMRR registers

2024-06-05 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) --v7: - Remove adding CMRR flag to vrr_ctl register during set_transcoder_timing, as it is already being done during intel_vrr_enable. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v12 6/9] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-06-05 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/

[PATCH v12 4/9] drm/i915: Separate VRR related register definitions

2024-06-05 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 112 ++ drivers/gpu/drm/i915/i915_reg.h | 101

[PATCH v12 3/9] gpu/drm/i915: Update indentation for VRR registers and bits

2024-06-05 Thread Mitul Golani
Update the indentation for the VRR register definition and its bits, and fix checkpatch issues to ensure smooth movement of registers and bits. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/i915_reg.h | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH v12 2/9] drm-tip: 2024y-06m-05d-09h-52m-50s UTC integration manifest

2024-06-05 Thread Mitul Golani
From: Ville Syrjälä --- integration-manifest | 28 1 file changed, 28 insertions(+) create mode 100644 integration-manifest diff --git a/integration-manifest b/integration-manifest new file mode 100644 index ..d840964a2208 --- /dev/null +++

[PATCH v12 1/9] drm/i915: Protect CRC reg macro arguments for consistency

2024-06-05 Thread Mitul Golani
From: Ville Syrjälä It's probably a good idea to start protecting all macro arguments to avoid any cargo-cult mistakes when people go looking for examples of how to define these things. Signed-off-by: Ville Syrjälä Link:

[PATCH v12 0/9] Implement CMRR Support

2024-06-05 Thread Mitul Golani
. -v12: - Add patch to fix check patch issues for VRR related registers in i915_reg.h then move them to intel_vrr_regs.h with separate patch. - Use drm_mode_vrefresh instead of manual refresh rate calculation. Mitul Golani (7): gpu/drm/i915: Update indentation for VRR registers and bits drm/i915

[PATCH v11 6/8] drm/i915/display: Compute Adaptive sync SDP params

2024-06-02 Thread Mitul Golani
intel_dp_compute_as_sdp. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index ac81b172b1ec..be3b9ba943a5

[PATCH v11 8/8] drm/i915: Compute CMRR and calculate vtotal

2024-06-02 Thread Mitul Golani
cmrr.enable.(Ankit) - Add TODO: for to address target refresh rate precision as future enhancement. Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_display.c | 1 + .../drm/i915/display/intel_display_device.h | 1 + drivers/gpu/drm/i915/display

[PATCH v11 7/8] drm/i915/display: Compute vrr vsync params

2024-06-02 Thread Mitul Golani
) - Replace vrr.enable flag to cmrr.enable, mistakenly added. (Ankit) Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c

[PATCH v11 5/8] drm/i915/display: Add support for pack and unpack

2024-06-02 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/drm/i915/display

[PATCH v11 4/8] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-06-02 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy --- include/drm/display/drm_dp_helper.h | 1 + 1

[PATCH v11 3/8] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-06-02 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani Reviewed-by: Ankit Nautiyal --- drivers/gpu/

[PATCH v11 1/8] drm/i915: Separate VRR related register definitions

2024-06-02 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 113 ++ drivers/gpu/drm/i915/i915_reg.h | 100

[PATCH v11 2/8] drm/i915: Define and compute Transcoder CMRR registers

2024-06-02 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) --v7: - Remove adding CMRR flag to vrr_ctl register during set_transcoder_timing, as it is already being done during intel_vrr_enable. (Ankit) Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display.c | 23

[PATCH v11 0/8] Implement CMRR Support

2024-06-02 Thread Mitul Golani
. Mitul Golani (8): drm/i915: Separate VRR related register definitions drm/i915: Define and compute Transcoder CMRR registers drm/i915: Update trans_vrr_ctl flag when cmrr is computed drm/dp: Add refresh rate divider to struct representing AS SDP drm/i915/display: Add support for pack

[PATCH v10 6/8] drm/i915/display: Add support for pack and unpack

2024-05-30 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). --v3: - target_rr_divider is bools so set accordingly (Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 4 1 file

[PATCH v10 8/8] drm/i915/display: Compute vrr vsync params

2024-05-30 Thread Mitul Golani
Compute vrr vsync params in case of FAVT as well instead of only to AVT mode of operation. --v2: - Remove redundant computation for vrr_vsync_start and vrr_vsync_end(Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9

[PATCH v10 7/8] drm/i915/display: Compute Adaptive sync SDP params

2024-05-30 Thread Mitul Golani
Compute params for Adaptive Sync SDP when Fixed Average Vtotal mode is enabled. --v2: Since vrr.enable is set in case of cmrr also, handle accordingly(Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 17 + 1 file changed, 13 insertions(+), 4

[PATCH v10 3/8] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-05-30 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) --v9: - Revert removed line(Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vr

[PATCH v10 5/8] drm/dp: Add refresh rate divider to struct representing AS SDP

2024-05-30 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. --v2: - Update commit header and send patch to dri-devel. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy --- include/drm/display/drm_dp_helper.h | 1 + 1

[PATCH v10 2/8] drm/i915: Define and compute Transcoder CMRR registers

2024-05-30 Thread Mitul Golani
name to intel_vrr_regs.h instead of reg.h (Jani) Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display.c | 23 ++- .../drm/i915/display/intel_display_types.h| 6 + drivers/gpu/drm/i915/display/intel_vrr.c | 22 ++ drivers/gpu/drm

[PATCH v10 4/8] drm/i915: Compute CMRR and calculate vtotal

2024-05-30 Thread Mitul Golani
and cmrr.enable are not mutually exclusive, handle accordingly (Ankit). - is_edp is not required inside is_cmrr_frac_required function (Ankit). - Add video_mode_required flag for future enhancement. - Correct cmrr_m/cmrr_n calculation. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display

[PATCH v10 1/8] drm/i915: Separate VRR related register definitions

2024-05-30 Thread Mitul Golani
Move VRR related register definitions to a separate file called intel_vrr_regs.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + drivers/gpu/drm/i915/display/intel_vrr_regs.h | 117 ++ drivers/gpu/drm/i915/i915_reg.h | 100

[PATCH v10 0/8] Implement CMRR Support

2024-05-30 Thread Mitul Golani
accordingly. - is_edp is not required inside is_cmrr_frac_required function. - Add video_mode_required flag for future enhancement. - Correct cmrr_m/cmrr_n calculation. - target_rr_divider is bools so handle accordingly. Mitul Golani (8): drm/i915: Separate VRR related register definitions drm

[PATCH v1 1/1] drm/i915/display: WA for Re-initialize dispcnlunitt1 xosc clock

2024-05-28 Thread Mitul Golani
-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display_power.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c index a860d88a65da..af2960c7e5b8 100644 --- a/drivers/gpu/drm

[PATCH v1 0/1] WA for Re-initialize dispcnlunitt1 xosc clock

2024-05-28 Thread Mitul Golani
The dispcnlunit1_cp_xosc_clk should be de-asserted in display off and only asserted in display on. But during observation it found clk remains active in display OFF. As workaround, Display driver shall execute set-reset sequence at the end of the Initialize Sequence. Wa_15013987218 Mitul Golani

[PATCH v9 7/8] drm/i915/display: Compute vrr vsync params

2024-05-24 Thread Mitul Golani
Compute vrr vsync params in case of FAVT as well instead of only to AVT mode of operation. --v2: - Remove redundant computation for vrr_vsync_start and vrr_vsync_end(Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 17 + 1 file changed, 9

[PATCH v9 8/8] drm/i915/display: Compute cmrr.enable flag

2024-05-24 Thread Mitul Golani
Set cmrr.enable flag during intel_vrr_compute_config. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c b/drivers/gpu/drm/i915/display/intel_vrr.c index 07be70f7c536

[PATCH v9 6/8] drm/i915/display: Compute Adaptive sync SDP params

2024-05-24 Thread Mitul Golani
Compute params for Adaptive Sync SDP when Fixed Average Vtotal mode is enabled. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu

[PATCH v9 5/8] drm/i915/display: Add support for pack and unpack

2024-05-24 Thread Mitul Golani
Add support of pack and unpack for target_rr_divider. --v2: - Set Target Refresh Rate Divider bit when related AS SDP bit is set (Ankit). Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_dp.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/i915/display

[PATCH v9 0/8] Implement CMRR Support

2024-05-24 Thread Mitul Golani
. - Set cmrr.enable with a separate patch at last. Mitul Golani (8): drm/i915: Define and compute Transcoder CMRR registers drm/i915: Update trans_vrr_ctl flag when cmrr is computed drm/i915: Compute CMRR and calculate vtotal Add refresh rate divider to struct representing AS SDP drm/i915

[PATCH v9 2/8] drm/i915: Update trans_vrr_ctl flag when cmrr is computed

2024-05-24 Thread Mitul Golani
-v4: - Removing RFC tag. --v5: - CMRR handling in co-existatnce of LRR and DRRS. --v7: - Rebase on top of AS SDP merge. --v8: - Remove cmrr_enabling/disabling and update commit message. (Ankit) Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_vrr.c | 11 --- 1 file changed

[PATCH v9 1/8] drm/i915: Define and compute Transcoder CMRR registers

2024-05-24 Thread Mitul Golani
on register offset. [Jani] --v3: - Removing RFC tag. --v4: - Update place holder for CMRR register definition. (Jani) --v5: - Add CMRR register definitions to a separate file intel_vrr_reg.h. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display.c | 23

[PATCH v9 3/8] drm/i915: Compute CMRR and calculate vtotal

2024-05-24 Thread Mitul Golani
scanline precision. --v6: - Make CMRR a small subset of FAVT mode. --v7: - Update commit message to avoid confusion with Legacy VRR (Ankit). - Add cmrr.enable in last, so remove from this patch. Signed-off-by: Mitul Golani --- drivers/gpu/drm/i915/display/intel_display.c | 1 + .../drm/i915

[PATCH v9 4/8] Add refresh rate divider to struct representing AS SDP

2024-05-24 Thread Mitul Golani
Add target_rr_divider to structure representing AS SDP. It is valid only in FAVT mode, sink device ignores the bit in AVT mode. Signed-off-by: Mitul Golani Reviewed-by: Arun R Murthy --- include/drm/display/drm_dp_helper.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/drm/display

  1   2   3   4   >