[Intel-gfx] [PATCH 0/2] MTL Degamma implementation

2023-07-25 Thread Chaitanya Kumar Borah
continue supporting degamma on MTL. To avoid pipe config mismatch between 24 bit HW lut values and 16 bit userspace sent values, convert back the 24 bit lut values read from HW to 16 bit values. Chaitanya Kumar Borah (2): drm/i915/color: Upscale degamma values for MTL drm/i915/color: Downscale

[Intel-gfx] [PATCH v3 1/2] drm/i915/color: Upscale degamma values for MTL

2023-07-25 Thread Chaitanya Kumar Borah
te a helper function for upscaling values v3: Fix multi line comment style (Uma) Signed-off-by: Chaitanya Kumar Borah Reviewed-by: Uma Shankar --- drivers/gpu/drm/i915/display/intel_color.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/drivers/

[Intel-gfx] [PATCH v2 2/2] drm/i915/color: Downscale degamma lut values read from hardware

2023-07-25 Thread Chaitanya Kumar Borah
For MTL and beyond, convert back the 24 bit lut values read from HW to 16 bit values to maintain parity with userspace values. This way we avoid pipe config mismatch for pre-csc lut values. v2: Add helper function to downscale values (Jani) Signed-off-by: Chaitanya Kumar Borah Reviewed-by: Uma

[Intel-gfx] [PATCH v4 1/2] drm/i915/color: Upscale degamma values for MTL

2023-07-27 Thread Chaitanya Kumar Borah
te a helper function for upscaling values v3: Fix multi line comment style (Uma) v4: Remove extra line(Ankit) Signed-off-by: Chaitanya Kumar Borah Reviewed-by: Uma Shankar --- drivers/gpu/drm/i915/display/intel_color.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-)

[Intel-gfx] [PATCH] drm/i915/rpl: Add new RPL PCI-IDs

2023-08-24 Thread Chaitanya Kumar Borah
Add newly added PCI-IDs for RPL BSpec: 55376 Signed-off-by: Chaitanya Kumar Borah --- include/drm/i915_pciids.h | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h index 1661f9e552d2..1256770d3827 100644 --- a

[Intel-gfx] [PATCH] drm/i915: Add wrapper for getting display step

2023-10-02 Thread Chaitanya Kumar Borah
Add a wrapper around intel_step_name that takes in driver data as an argument. This wrapper will help maintain compatibility with the proposed xe driver. Signed-off-by: Chaitanya Kumar Borah Signed-off-by: Animesh Manna --- drivers/gpu/drm/i915/display/intel_dmc.c | 2 +- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 1/2] drm/i915/color: Add function to load degamma LUT in MTL

2023-06-25 Thread Chaitanya Kumar Borah
continue supporting degamma on MTL. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_color.c | 42 -- 1 file changed, 40 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm/i915/display/intel_color.c

[Intel-gfx] [PATCH 2/2] drm/i915/color: For MTL convert 24 bit lut values to 16 bit

2023-06-25 Thread Chaitanya Kumar Borah
For MTL and beyond, convert back the 24 bit lut values read from HW to 16 bit values to maintain parity with userspace values. This way we avoid pipe config mismatch for pre-csc lut values. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_color.c | 8 1 file

[Intel-gfx] [PATCH 0/2] MTL Degamma implementation

2023-06-25 Thread Chaitanya Kumar Borah
continue supporting degamma on MTL. To avoid pipe config mismatch between 24 bit HW lut values and 16 bit userspace sent values, convert back the 24 bit lut values read from HW to 16 bit values. Chaitanya Kumar Borah (2): drm/i915/color: Add function to load degamma LUT in MTL drm/i915/color: For

[Intel-gfx] [PATCH] drm/i915/display: Add header file for struct seq_file

2023-06-25 Thread Chaitanya Kumar Borah
With change [1], visibility of struct seq_file is lost in intel_display_power.h leading to build errors. Add header file explicitly to restore visibility. [1] ef104443bffa ("procfs: consolidate arch_report_meminfo declaration") Signed-off-by: Chaitanya Kumar Borah --- drivers/gp

[Intel-gfx] [PATCH] drm/i915/display: Add forward declaration for struct seq_file

2023-06-26 Thread Chaitanya Kumar Borah
Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_display_power.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/display/intel_display_power.h b/drivers/gpu/drm/i915/display/intel_display_power.h index be1a87bde0c9..0ba268e566b0 100644 --- a/drivers/gpu/drm/i9

[Intel-gfx] [PATCH 0/2] MTL Degamma implementation

2023-07-10 Thread Chaitanya Kumar Borah
continue supporting degamma on MTL. To avoid pipe config mismatch between 24 bit HW lut values and 16 bit userspace sent values, convert back the 24 bit lut values read from HW to 16 bit values. Chaitanya Kumar Borah (2): drm/i915/color: Upscale degamma values for MTL drm/i915/color: Downscale

[Intel-gfx] [PATCH v2 1/2] drm/i915/color: Upscale degamma values for MTL

2023-07-10 Thread Chaitanya Kumar Borah
te a helper function for upscaling values Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_color.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm/i915/display

[Intel-gfx] [PATCH v2 2/2] drm/i915/color: Downscale degamma lut values read from hardware

2023-07-10 Thread Chaitanya Kumar Borah
For MTL and beyond, convert back the 24 bit lut values read from HW to 16 bit values to maintain parity with userspace values. This way we avoid pipe config mismatch for pre-csc lut values. v2: Add helper function to downscale values (Jani) Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu

[Intel-gfx] [PATCH] drm/i915/display: Set correct voltage level for 480MHz CDCLK

2023-03-07 Thread Chaitanya Kumar Borah
According to Bspec, the voltage level for 480MHz is to be set as 1 instead of 2. BSpec: 49208 Fixes: 06f1b06dc5b7 ("drm/i915/display: Add 480 MHz CDCLK steps for RPL-U") Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 30 +++-

[Intel-gfx] [PATCH v2] drm/i915/display: Set correct voltage level for 480MHz CDCLK

2023-05-28 Thread Chaitanya Kumar Borah
According to Bspec, the voltage level for 480MHz is to be set as 1 instead of 2. BSpec: 49208 Fixes: 06f1b06dc5b7 ("drm/i915/display: Add 480 MHz CDCLK steps for RPL-U") v2: rebase Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cd

[Intel-gfx] [RFC] fbcon: Reschedule cursor worker if try lock fails

2023-06-08 Thread Chaitanya Kumar Borah
ave, particularly with the "Fix me" comment which indicates a bigger underlying problem. Hence, the RFC. [1] 3b0fb6ab25("fbcon: Use delayed work for cursor") [2] https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13243/shard-glk2/igt@kms_fbcon_...@fbc-suspend.html Signed-off-by

[Intel-gfx] [PATCH] drm/i915/mtl: Support HBR3 rate with C10 phy and eDP in MTL

2023-10-18 Thread Chaitanya Kumar Borah
eDP specification supports HBR3 link rate since v1.4a. Moreover, C10 phy can support HBR3 link rate for both DP and eDP. Therefore, do not clamp the supported rates for eDP at 6.75Gbps. Cc: BSpec: 70073 74224 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_dp.c

[Intel-gfx] [topic/core-for-CI] Revert "debugfs: annotate debugfs handlers vs. removal with lockdep"

2023-12-05 Thread Chaitanya Kumar Borah
From: Johannes Berg This reverts commit f4acfcd4deb1 ("debugfs: annotate debugfs handlers vs. removal with lockdep"), it appears to have false positives and really shouldn't have been in the -rc series with the fixes anyway. topic/core-for-CI note: cherry-picked from https://patchwork.kernel.org

[Intel-gfx] [topic/core-for-CI] Revert "debugfs: annotate debugfs handlers vs. removal with lockdep"

2023-12-05 Thread Chaitanya Kumar Borah
From: Johannes Berg This reverts commit f4acfcd4deb1 ("debugfs: annotate debugfs handlers vs. removal with lockdep"), it appears to have false positives and really shouldn't have been in the -rc series with the fixes anyway. Link:https://patchwork.kernel.org/project/linux-fsdevel/patch/202312021

[Intel-gfx] [RFC 0/2] Add new CDCLK step for RPL-U

2022-11-29 Thread Chaitanya Kumar Borah
dding this distinction since they are currently in use. However, here the question arises if we keep this change in upstream or not as this could just be dead code down the line. Feedbacks are appreciated on this. Chaitanya Kumar Borah (2): drm/i915: Add RPL-U CDCLK table drm/i915: Add additional check

[Intel-gfx] [RFC 2/2] drm/i915: Add additional check for 480Mhz step CDCLK

2022-11-29 Thread Chaitanya Kumar Borah
Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index 9bfeb1abba47..1890e5135cfc 100644 --- a/drivers/gp

[Intel-gfx] [RFC 1/2] drm/i915: Add RPL-U CDCLK table

2022-11-29 Thread Chaitanya Kumar Borah
A new step of 480MHz has been added on SKUs that have a RPL-U device id. Add a new table which include this new CDCLK step. BSpec: 55409 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 39 ++ 1 file changed, 39 insertions(+) diff --git

[Intel-gfx] [PATCH] drm/i915/color: Add function to load degamma LUT in MTL

2022-12-21 Thread Chaitanya Kumar Borah
writing into the HW to continue supporting degamma on MTL. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_color.c | 42 -- 1 file changed, 40 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_color.c b/drivers/gpu/drm

[Intel-gfx] [RFC 0/4] Add new CDCLK step for RPL-U

2023-01-01 Thread Chaitanya Kumar Borah
recated in future. Feedbacks are welcome. Chaitanya Kumar Borah (4): drm/i915/quirks: Add quirk for 480MHz CDCLK step drm/i915/display: Add 480 MHz CDCLK steps for RPL-U drm/i915: Initialize intel quirks before CDCLK initialization drm/i915: Apply CDCLK quirk only on QS parts drivers/g

[Intel-gfx] [RFC 1/4] drm/i915/quirks: Add quirk for 480MHz CDCLK step

2023-01-01 Thread Chaitanya Kumar Borah
. Therefore, quirks are a good way to achieve the same. BSpec: 55409 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_quirks.c | 14 ++ drivers/gpu/drm/i915/display/intel_quirks.h | 1 + 2 files changed, 15 insertions(+) diff --git a/drivers/gpu/drm/i915

[Intel-gfx] [RFC 2/4] drm/i915/display: Add 480 MHz CDCLK steps for RPL-U

2023-01-01 Thread Chaitanya Kumar Borah
A new step of 480MHz has been added on SKUs that have a RPL-U device id to support 120Hz displays more efficiently. Use a new quirk to identify the machine for which this change needs to be applied. BSpec: 55409 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display

[Intel-gfx] [RFC 3/4] drm/i915: Initialize intel quirks before CDCLK initialization

2023-01-01 Thread Chaitanya Kumar Borah
With addition of new quirk QUIRK_480MHZ_CDCLK_STEP, it is imperative that quirks should be initialized before CDCLK initialization. Refactor the code accordingly. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_display.c | 2 -- drivers/gpu/drm/i915/i915_driver.c

[Intel-gfx] [RFC 4/4] drm/i915: Apply CDCLK quirk only on QS parts

2023-01-01 Thread Chaitanya Kumar Borah
n Intel(R) Core(TM) i5-1345U" BSpec: 55409 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_quirks.c | 32 +++-- 1 file changed, 29 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_quirks.c b/drivers/gpu/drm/i915/di

[Intel-gfx] [PATCH] drm/i915/dg1: Remove require_force_probe protection

2022-10-20 Thread Chaitanya Kumar Borah
Remove force probe protection from DG1 platform as testing suggests that the platform is stable enough. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/i915_pci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c

[Intel-gfx] [PATCH] drm/i915/color: Fix typo for Plane CSC indexes

2023-03-30 Thread Chaitanya Kumar Borah
Replace _PLANE_INPUT_CSC_RY_GY_2_* with _PLANE_CSC_RY_GY_2_* for Plane CSC Fixes: 6eba56f64d5d ("drm/i915/pxp: black pixels on pxp disabled") Cc: Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/i915_reg.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

[Intel-gfx] [RFC 0/2] Add new CDCLK step for RPL-U

2023-01-06 Thread Chaitanya Kumar Borah
decision if this particular part needs to be upstreamed.(see comments on previous versions) Chaitanya Kumar Borah (2): drm/i915: Add rplu sub platform drm/i915/display: Add 480 MHz CDCLK steps for RPL-U arch/x86/kernel/early-quirks.c | 1 + drivers/gpu/drm/i915/display/intel_cdclk.c

[Intel-gfx] [RFC 1/2] drm/i915: Add rplu sub platform

2023-01-06 Thread Chaitanya Kumar Borah
Adding RPL-U as a sub platform. In RPL-U a new CDCLK step has been added so we need to make a distinction between RPL-P and RPL-U while CDCLK initialization. Adding a sub-platform, enables us to make this differentiation in the code. Signed-off-by: Chaitanya Kumar Borah --- arch/x86/kernel

[Intel-gfx] [RFC 2/2] drm/i915/display: Add 480 MHz CDCLK steps for RPL-U

2023-01-06 Thread Chaitanya Kumar Borah
A new step of 480MHz has been added on SKUs that have a RPL-U device id to support 120Hz displays more efficiently. Use a new quirk to identify the machine for which this change needs to be applied. BSpec: 55409 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display

[Intel-gfx] [RFC 0/2] Add new CDCLK step for RPL-U

2023-01-12 Thread Chaitanya Kumar Borah
decision if this particular part needs to be upstreamed.(see comments on revision 2) Chaitanya Kumar Borah (2): drm/i915: Add sub platform for 480MHz CDCLK step drm/i915/display: Add 480 MHz CDCLK steps for RPL-U drivers/gpu/drm/i915/display/intel_cdclk.c | 26 ++ drivers

[Intel-gfx] [RFC v2 1/2] drm/i915: Add sub platform for 480MHz CDCLK step

2023-01-12 Thread Chaitanya Kumar Borah
RPL-U part of RPL subplatform Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/i915_drv.h | 2 ++ drivers/gpu/drm/i915/i915_pci.c | 1 + drivers/gpu/drm/i915/intel_device_info.c | 8 drivers/gpu/drm/i915/intel_device_info.h | 2 ++ include/drm

[Intel-gfx] [RFC 2/2] drm/i915/display: Add 480 MHz CDCLK steps for RPL-U

2023-01-12 Thread Chaitanya Kumar Borah
-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index 0c107a38f9d0..ba365ef17abc 100644 --- a/drivers

[Intel-gfx] [PATCH] drm/i915/display: Fix typo for reference clock

2023-01-12 Thread Chaitanya Kumar Borah
Fix typo for reference clock from 24400 to 24000 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index

[Intel-gfx] [RFC 0/2] Add new CDCLK step for RPL-U

2023-01-16 Thread Chaitanya Kumar Borah
decision if this particular part needs to be upstreamed.(see comments on revision 2) Chaitanya Kumar Borah (2): drm/i915: Add RPL-U sub platform drm/i915/display: Add 480 MHz CDCLK steps for RPL-U drivers/gpu/drm/i915/display/intel_cdclk.c | 27 ++ drivers/gpu/drm/i915

[Intel-gfx] [RFC 2/2] drm/i915/display: Add 480 MHz CDCLK steps for RPL-U

2023-01-16 Thread Chaitanya Kumar Borah
: - Revert to RPL-U subplatform Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 27 ++ 1 file changed, 27 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c index 0c107a38f9d0

[Intel-gfx] [RFC 1/2] drm/i915: Add RPL-U sub platform

2023-01-16 Thread Chaitanya Kumar Borah
: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/i915_drv.h | 2 ++ drivers/gpu/drm/i915/i915_pci.c | 1 + drivers/gpu/drm/i915/intel_device_info.c | 8 drivers/gpu/drm/i915/intel_device_info.h | 2 ++ include/drm/i915_pciids.h| 11 +++ 5 files

[Intel-gfx] [RFC 0/2] Add new CDCLK step for RPL-U

2023-01-30 Thread Chaitanya Kumar Borah
decision if this particular part needs to be upstreamed.(see comments on revision 2) Chaitanya Kumar Borah (2): drm/i915: Add RPL-U sub platform drm/i915/display: Add 480 MHz CDCLK steps for RPL-U drivers/gpu/drm/i915/display/intel_cdclk.c | 26 ++ drivers/gpu/drm/i915

[Intel-gfx] [RFC v4 1/2] drm/i915: Add RPL-U sub platform

2023-01-30 Thread Chaitanya Kumar Borah
) - Add RPL-U ids to RPL-P platform - Remove redundant comment Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/i915_drv.h | 2 ++ drivers/gpu/drm/i915/intel_device_info.c | 7 +++ drivers/gpu/drm/i915/intel_device_info.h | 1 + include/drm/i915_pciids.h

[Intel-gfx] [RFC v4 2/2] drm/i915/display: Add 480 MHz CDCLK steps for RPL-U

2023-01-30 Thread Chaitanya Kumar Borah
: - Revert to RPL-U subplatform (Jani) v4: - Remove Bspec reference from code (Jani) Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_cdclk.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers

[core-for-CI PATCH] usb: typec: ucsi: Fix a potential deadlock in ucsi_send_command_common()

2024-08-06 Thread Chaitanya Kumar Borah
From: Heikki Krogerus The function returns with the ppm_lock held if there's an error or the PPM reports BUSY condition. This is a core-for-ci patch for [1] [1] https://lore.kernel.org/linux-usb/20240806112029.2984319-1-heikki.kroge...@linux.intel.com/ Reported-by: Luciano Coelho Fixes: 5e9c

[PATCH] drm/i915: Preserve value of fec_enable calculated before DSC compute config

2024-08-12 Thread Chaitanya Kumar Borah
dware. Therefore, change operator to AND. While at it, add a comment explaining why we don't enable FEC in eDP v1.5. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_dp.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i9

[PATCH] drm/i915: Do not explicilty enable FEC in DP_TP_CTL for UHBR rates

2024-08-21 Thread Chaitanya Kumar Borah
across MST and SST use-cases for UHBR rates. While at it, add a comment explaining why we don't enable FEC in eDP v1.5. v2: Correct logic to cater to SST use-cases (Jani) Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_dp.c | 8 +++- 1 file changed, 7 inser

[PATCH 1/5] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2024-02-04 Thread Chaitanya Kumar Borah
From: Ville Syrjälä The cursor hardware only does sync updates, and thus the hardware will be scanning out from the old fb until the next start of vblank. So in order to make the legacy cursor fastpath actually safe we should not unpin the old fb until we're sure the hardware has ceased accessing

[RFC PATCH 0/5] Cursor Fault Fixes

2024-02-04 Thread Chaitanya Kumar Borah
a discussion to arrive at one. [1] https://patchwork.freedesktop.org/series/126934/ Chaitanya Kumar Borah (3): drm/i915: do not destroy plane state if cursor unpin worker is scheduled drm/i915: Add sanity checks before accessing fb buffer object drm/i915: do not defer cleanup work

[PATCH 3/5] drm/i915: do not destroy plane state if cursor unpin worker is scheduled

2024-02-04 Thread Chaitanya Kumar Borah
The plane destroy hook can be called asynchronously even when vblank worker responsible for unpinning the cursor fb is scheduled. Since the vblank worker destroys the plane state, do not destroy the plane state if it is scheduled. Signed-off-by: Chaitanya Kumar Borah --- .../gpu/drm/i915

[PATCH 4/5] drm/i915: Add sanity checks before accessing fb buffer object

2024-02-04 Thread Chaitanya Kumar Borah
Now that cursor plane fb unpinning can be deferred to vblank work check if plane state and corresponding fb pointers are still valid before cleanup. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++- 1 file changed, 7 insertions(+), 1

[PATCH 5/5] drm/i915: do not defer cleanup work

2024-02-04 Thread Chaitanya Kumar Borah
0x18c/0x350 [ 278.750452] ? __pfx_worker_thread+0x10/0x10 [ 278.750455] kthread+0xfe/0x130 [ 278.750460] ? __pfx_kthread+0x10/0x10 [ 278.750464] ret_from_fork+0x2c/0x50 [ 278.750468] ? __pfx_kthread+0x10/0x10 [ 278.750472] ret_from_fork_asm+0x1b/0x30 Signed-off-by: Chaitanya

[PATCH 2/5] drm/i915: Use the same vblank worker for atomic unpin

2024-02-04 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In case of legacy cursor update, the cursor VMA needs to be unpinned only after vblank. This exceeds the lifetime of the whole atomic commit. Any trick I attempted to keep the atomic commit alive didn't work, as drm_atomic_helper_setup_commit() force throttles on any old

[PATCH 3/6] drm/i915: Use the same vblank worker for atomic unpin

2024-02-04 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In case of legacy cursor update, the cursor VMA needs to be unpinned only after vblank. This exceeds the lifetime of the whole atomic commit. Any trick I attempted to keep the atomic commit alive didn't work, as drm_atomic_helper_setup_commit() force throttles on any old

[PATCH 4/6] drm/i915: do not destroy plane state if cursor unpin worker is scheduled

2024-02-04 Thread Chaitanya Kumar Borah
The plane destroy hook can be called asynchronously even when vblank worker responsible for unpinning the cursor fb is scheduled. Since the vblank worker destroys the plane state, do not destroy the plane state if it is scheduled. Signed-off-by: Chaitanya Kumar Borah --- .../gpu/drm/i915

[PATCH 5/6] drm/i915: Add sanity checks before accessing fb buffer object

2024-02-04 Thread Chaitanya Kumar Borah
Now that cursor plane fb unpinning can be deferred to vblank work check if plane state and corresponding fb pointers are still valid before cleanup. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 8 +++- 1 file changed, 7 insertions(+), 1

[PATCH 1/6] drm: Add drm_vblank_work_flush_all().

2024-02-04 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In some cases we want to flush all vblank work, right before vblank_off for example. Add a simple function to make this possible. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_vblank_work.c | 22 ++ include/drm/drm_vblank_work.h | 2 +

[PATCH 2/6] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2024-02-04 Thread Chaitanya Kumar Borah
From: Ville Syrjälä The cursor hardware only does sync updates, and thus the hardware will be scanning out from the old fb until the next start of vblank. So in order to make the legacy cursor fastpath actually safe we should not unpin the old fb until we're sure the hardware has ceased accessing

[PATCH 6/6] drm/i915: do not defer cleanup work

2024-02-04 Thread Chaitanya Kumar Borah
0x18c/0x350 [ 278.750452] ? __pfx_worker_thread+0x10/0x10 [ 278.750455] kthread+0xfe/0x130 [ 278.750460] ? __pfx_kthread+0x10/0x10 [ 278.750464] ret_from_fork+0x2c/0x50 [ 278.750468] ? __pfx_kthread+0x10/0x10 [ 278.750472] ret_from_fork_asm+0x1b/0x30 Signed-off-by: Chaitanya

[PATCH 0/6] Cursor Fault Fixes

2024-02-04 Thread Chaitanya Kumar Borah
a discussion to arrive at one. [1] https://patchwork.freedesktop.org/series/126934/ v2: Add missing patch Chaitanya Kumar Borah (3): drm/i915: do not destroy plane state if cursor unpin worker is scheduled drm/i915: Add sanity checks before accessing fb buffer object drm/i915: do not

[PATCH 0/6] Cursor Fault Fixes

2024-02-04 Thread Chaitanya Kumar Borah
a discussion to arrive at one. [1] https://patchwork.freedesktop.org/series/126934/ v2: Add missing patch v3: Remove misleading error log Change condition to access fb object Chaitanya Kumar Borah (3): drm/i915: do not destroy plane state if cursor unpin worker is scheduled drm/i915

[PATCH 1/6] drm: Add drm_vblank_work_flush_all().

2024-02-04 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In some cases we want to flush all vblank work, right before vblank_off for example. Add a simple function to make this possible. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_vblank_work.c | 22 ++ include/drm/drm_vblank_work.h | 2 +

[PATCH 2/6] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2024-02-04 Thread Chaitanya Kumar Borah
From: Ville Syrjälä The cursor hardware only does sync updates, and thus the hardware will be scanning out from the old fb until the next start of vblank. So in order to make the legacy cursor fastpath actually safe we should not unpin the old fb until we're sure the hardware has ceased accessing

[PATCH 3/6] drm/i915: Use the same vblank worker for atomic unpin

2024-02-04 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In case of legacy cursor update, the cursor VMA needs to be unpinned only after vblank. This exceeds the lifetime of the whole atomic commit. Any trick I attempted to keep the atomic commit alive didn't work, as drm_atomic_helper_setup_commit() force throttles on any old

[PATCH 4/6] drm/i915: do not destroy plane state if cursor unpin worker is scheduled

2024-02-04 Thread Chaitanya Kumar Borah
The plane destroy hook can be called asynchronously even when vblank worker responsible for unpinning the cursor fb is scheduled. Since the vblank worker destroys the plane state, do not destroy the plane state if it is scheduled. Signed-off-by: Chaitanya Kumar Borah --- .../gpu/drm/i915

[PATCH v2 5/6] drm/i915: Add sanity check before accessing fb buffer object

2024-02-04 Thread Chaitanya Kumar Borah
Now that cursor plane fb unpinning can be deferred to vblank work access the fb object only when vblank unpin worker is not scheduled. v2: - Remove misleading error log - Change condition for accessing fb object Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display

[PATCH 6/6] drm/i915: do not defer cleanup work

2024-02-04 Thread Chaitanya Kumar Borah
0x18c/0x350 [ 278.750452] ? __pfx_worker_thread+0x10/0x10 [ 278.750455] kthread+0xfe/0x130 [ 278.750460] ? __pfx_kthread+0x10/0x10 [ 278.750464] ret_from_fork+0x2c/0x50 [ 278.750468] ? __pfx_kthread+0x10/0x10 [ 278.750472] ret_from_fork_asm+0x1b/0x30 Signed-off-by: Chaitanya

[PATCH 1/6] drm: Add drm_vblank_work_flush_all().

2024-02-05 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In some cases we want to flush all vblank work, right before vblank_off for example. Add a simple function to make this possible. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_vblank_work.c | 22 ++ include/drm/drm_vblank_work.h | 2 +

[PATCH 0/6] Cursor Fault Fixes

2024-02-05 Thread Chaitanya Kumar Borah
a discussion to arrive at one. [1] https://patchwork.freedesktop.org/series/126934/ v2: Add missing patch v3: Remove misleading error log Change condition to access fb object v4: Remove unused variables Chaitanya Kumar Borah (3): drm/i915: do not destroy plane state if cursor unpin worker

[PATCH 2/6] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2024-02-05 Thread Chaitanya Kumar Borah
From: Ville Syrjälä The cursor hardware only does sync updates, and thus the hardware will be scanning out from the old fb until the next start of vblank. So in order to make the legacy cursor fastpath actually safe we should not unpin the old fb until we're sure the hardware has ceased accessing

[PATCH 3/6] drm/i915: Use the same vblank worker for atomic unpin

2024-02-05 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In case of legacy cursor update, the cursor VMA needs to be unpinned only after vblank. This exceeds the lifetime of the whole atomic commit. Any trick I attempted to keep the atomic commit alive didn't work, as drm_atomic_helper_setup_commit() force throttles on any old

[PATCH 4/6] drm/i915: do not destroy plane state if cursor unpin worker is scheduled

2024-02-05 Thread Chaitanya Kumar Borah
The plane destroy hook can be called asynchronously even when vblank worker responsible for unpinning the cursor fb is scheduled. Since the vblank worker destroys the plane state, do not destroy the plane state if it is scheduled. Signed-off-by: Chaitanya Kumar Borah --- .../gpu/drm/i915

[PATCH 5/6] drm/i915: Add sanity check before accessing fb buffer object

2024-02-05 Thread Chaitanya Kumar Borah
Now that cursor plane fb unpinning can be deferred to vblank work access the fb object only when vblank unpin worker is not scheduled. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git

[PATCH 6/6] drm/i915: do not defer cleanup work

2024-02-05 Thread Chaitanya Kumar Borah
0x18c/0x350 [ 278.750452] ? __pfx_worker_thread+0x10/0x10 [ 278.750455] kthread+0xfe/0x130 [ 278.750460] ? __pfx_kthread+0x10/0x10 [ 278.750464] ret_from_fork+0x2c/0x50 [ 278.750468] ? __pfx_kthread+0x10/0x10 [ 278.750472] ret_from_fork_asm+0x1b/0x30 Signed-off-by: Chaitanya

[PATCH] drm/i915/audio: Fix audio time stamp programming for DP

2024-04-30 Thread Chaitanya Kumar Borah
Maud and Naud SDPs on it's own. Cc: sta...@vger.kernel.org # v5.17 Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8097 Co-developed-by: Kai Vehmanen Signed-off-by: Kai Vehmanen Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_audio.c

[PATCH] drm/i915/audio: Fix audio time stamp programming for DP

2024-04-30 Thread Chaitanya Kumar Borah
Maud and Naud SDPs on it's own. Cc: sta...@vger.kernel.org # v5.17 Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8097 Co-developed-by: Kai Vehmanen Signed-off-by: Kai Vehmanen Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_audio.c

[PATCH 0/4] Cursor Fault Fix

2024-05-08 Thread Chaitanya Kumar Borah
-2.html Chaitanya Kumar Borah (1): drm: Add helper to check if there are pending vblank work Maarten Lankhorst (2): drm: Add drm_vblank_work_flush_all(). drm/i915: Use the same vblank worker for atomic unpin Ville Syrjälä (1): drm/i915: Use vblank worker to unpin old legacy cursor fb

[PATCH 1/4] drm: Add drm_vblank_work_flush_all().

2024-05-08 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In some cases we want to flush all vblank work, right before vblank_off for example. Add a simple function to make this possible. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/drm_vblank_work.c | 22 ++ include/drm/drm_vblank_work.h | 2 +

[PATCH 2/4] drm: Add helper to check if there are pending vblank work

2024-05-08 Thread Chaitanya Kumar Borah
Add helper to check if there are pending vblank work for a crtc which are yet to be scheduled. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/drm_vblank_work.c | 24 include/drm/drm_vblank_work.h | 1 + 2 files changed, 25 insertions(+) diff --git a

[PATCH 3/4] drm/i915: Use vblank worker to unpin old legacy cursor fb safely

2024-05-08 Thread Chaitanya Kumar Borah
crtc_vblank_off(), therefore, wait one vblank if there are pending vblank workers. This patch is slightly reworked by Maarten. Cc: Maarten Lankhorst Signed-off-by: Ville Syrjälä Signed-off-by: Maarten Lankhorst Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_

[PATCH 4/4] drm/i915: Use the same vblank worker for atomic unpin

2024-05-08 Thread Chaitanya Kumar Borah
From: Maarten Lankhorst In case of legacy cursor update, the cursor VMA needs to be unpinned only after vblank. This exceeds the lifetime of the whole atomic commit. Any trick I attempted to keep the atomic commit alive didn't work, as drm_atomic_helper_setup_commit() force throttles on any old

[core-for-CI PATCH] Revert "e1000e: move force SMBUS near the end of enable_ulp function"

2024-06-10 Thread Chaitanya Kumar Borah
References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11305 Signed-off-by: Chaitanya Kumar Borah --- drivers/net/ethernet/intel/e1000e/ich8lan.c | 22 - drivers/net/ethernet/intel/e1000e/netdev.c | 18 + 2 files changed, 18 insertions(+), 22 deletions

[PATCH] drm/i915/audio: Fix audio time stamp programming for DP

2024-04-18 Thread Chaitanya Kumar Borah
Maud and Naud SDPs on it's own. Fixes: 6014ac122ed0 ("drm/i915/audio: set proper N/M in modeset") Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/8097 Co-developed-by: Kai Vehmanen Signed-off-by: Kai Vehmanen Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/d

[v2] drm/i915/dp: Add FEC Enable Retry mechanism

2024-10-08 Thread Chaitanya Kumar Borah
nth] Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_ddi.c| 79 + drivers/gpu/drm/i915/display/intel_ddi.h| 2 +- drivers/gpu/drm/i915/display/intel_dp_mst.c | 4 +- 3 files changed, 67 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/

[v2] drm/i915: Allow fastset for change in HDR infoframe

2024-10-09 Thread Chaitanya Kumar Borah
intel_hdmi_fastset_infoframes(). v2: - Update HDMI AVI infoframe during fastset. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_ddi.c | 3 +++ drivers/gpu/drm/i915/display/intel_display.c | 3 ++- drivers/gpu/drm/i915/display/intel_hdmi.c| 19

[v3] drm/i915/dp: Add FEC Enable Retry mechanism

2024-10-21 Thread Chaitanya Kumar Borah
nth] - Wait for FEC_ENABLE_LIVE_STATUS to be cleared before re-enabling FEC [Srikanth] Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_ddi.c | 57 +++- drivers/gpu/drm/i915/display/intel_ddi.h | 6 +-- 2 files changed, 50 insertions(+), 13 deleti

[PATCH] drm/i915: Allow fastset for change in HDR infoframe

2024-10-22 Thread Chaitanya Kumar Borah
intel_hdmi_fastset_infoframes(). v3: - Create a wrapper intel_ddi_update_pipe_hdmi to stick to uniform naming (Jani) - Do not disable HDMI AVI infoframe if already disabled (Uma) v2: - Update HDMI AVI infoframe during fastset. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display

[v4] drm/i915/dp: Add FEC Enable Retry mechanism

2024-10-24 Thread Chaitanya Kumar Borah
ble checks [Jani] v2: - Refactor code to avoid duplication and improve readability [Jani] - In case of PTL, wait for FEC status directly after FEC enable [Srikanth] - Wait for FEC_ENABLE_LIVE_STATUS to be cleared before re-enabling FEC [Srikanth] Signed-off-by: Chaitanya Kumar Borah --- driv

[core-for-CI] cpufreq: intel_pstate: Rearrange locking in hybrid_init_cpu_capacity_scaling()

2024-11-11 Thread Chaitanya Kumar Borah
bf22f8a90fc3a3edc8b9...@sj1pr11mb6129.namprd11.prod.outlook.com/ Signed-off-by: Rafael J. Wysocki References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12763 Link: https://lore.kernel.org/linux-pm/12554508.o9o76zd...@rjwysocki.net/ Signed-off-by: Chaitanya Kumar Borah --- drivers/cp

[PATCH] drm/i915/dp: Add FEC Enable Retry mechanism

2024-09-22 Thread Chaitanya Kumar Borah
PRX does not respond with an FEC_ENABLE within the stipulated 5ms. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_ddi.c | 61 +++- 1 file changed, 59 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/

[PATCH] drm/i915: Allow fastset for change in HDR infoframe

2024-09-30 Thread Chaitanya Kumar Borah
Changes in Dynamic Range and Mastering infoframe should not trigger a full modeset. Therefore, allow fastset. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_display.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display

[core-for-CI] nvme-pci: don't use dma_alloc_noncontiguous with 0 merge boundary

2024-12-03 Thread Chaitanya Kumar Borah
w the MAX_ORDER of the page allocator, but blows up with a warning if it is too large. Fixes: 63a5c7a4b4c4 ("nvme-pci: use dma_alloc_noncontigous if possible") Reported-by: Leon Romanovsky Reported-by: Chaitanya Kumar Borah Signed-off-by: Christoph Hellwig References: https://gitlab.freedeskt

[RFC PATCH] drm/xe/display: Program double buffered LUT registers

2024-12-11 Thread Chaitanya Kumar Borah
LUT programming is also disabled in the vblank worker. Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_color.c | 28 +++- drivers/gpu/drm/i915/display/intel_crtc.c| 4 ++- drivers/gpu/drm/i915/display/intel_display.c | 2 +- 3 files changed,

[core-for-CI] Revert "panel/auo-a030jtn01: Use refcounted allocation in place of devm_kzalloc()"

2025-04-08 Thread Chaitanya Kumar Borah
/ References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14039 Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/panel/panel-auo-a030jtn01.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/panel/panel-auo-a030jtn01.c b/drivers/gpu

[PATCH v2 09/11] drm/i915: Program DB LUT registers before vblank

2025-04-08 Thread Chaitanya Kumar Borah
failure for this is probably less drastic than for plane programming. (Ville) Signed-off-by: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_display.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_display.c b

[PATCH 02/11] drm/i915/dsb: Extract assert_dsb_tail_is_aligned()

2025-04-08 Thread Chaitanya Kumar Borah
From: Ville Syrjälä Extract the DSB tail alignment checks into helper. We already have two uses of this, and soo we'll get a third. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_dsb.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a

[PATCH 00/11] drm/xe/display: Program double buffered LUT registers

2025-04-08 Thread Chaitanya Kumar Borah
>From PTL, LUT registers are made double buffered. This helps us to program them in the active region without any concern of tearing. This particulary helps in case of displays with high refresh rates where vblank periods are shorter. Add MMIO and DSB path to program them. Chaitanya Kumar Bo

[PATCH 04/11] drm/i915/dsb: Implement intel_dsb_gosub()

2025-04-08 Thread Chaitanya Kumar Borah
From: Ville Syrjälä Add support for the new GOSUB DSB instruction (available on ptl+), which instructs the DSB to jump to a different buffer, executie the commands there, and then return execution to the next instruction in the original buffer. There are a few alignment related workarounds that

[PATCH v2 05/11] drm/i915/dsb: add intel_dsb_gosub_finish()

2025-04-08 Thread Chaitanya Kumar Borah
: Chaitanya Kumar Borah --- drivers/gpu/drm/i915/display/intel_dsb.c | 13 + drivers/gpu/drm/i915/display/intel_dsb.h | 1 + 2 files changed, 14 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dsb.c b/drivers/gpu/drm/i915/display/intel_dsb.c index 2cda6fc7857b..97ea3c655590

[PATCH 01/11] drm/i915/dsb: Extract intel_dsb_ins_align()

2025-04-08 Thread Chaitanya Kumar Borah
From: Ville Syrjälä Extract the code that alings the next instruction to the next QW boundary into a small helper. I'll have some more uses for this later. Also explain why we don't have to zero out the extra DW. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_dsb.c | 16 +

  1   2   >