drivers/gpu/drm/tidss/tidss_dispc.c | 85 +++++++++++++---------------- drivers/gpu/drm/tidss/tidss_dispc.h | 1 - drivers/gpu/drm/tidss/tidss_drv.h | 11 ++++ drivers/gpu/drm/tidss/tidss_oldi.c | 27 +++++++++ 4 files changed, 75 insertions(+), 49 deletions(-)
In an effort to make the existing compatibles more usable, we are removing the max_pclk_khz form dispc_features structure and doing the supported pixel clock checks using "max_successful_rate[]" and "max_attempted_rate[]". Changes are fully backwards compatible. After integration of OLDI support[0], we need additional patches in OLDI to identify the VP that has OLDI. We have to do this since OLDI driver owns the VP clock (its serial clock) and we cannot perform clock operations on those VP clock from tidss driver. This issue was also reported upstream when DSI fixes[1] had some clock related calls in tidss driver. When "clk_round_rate()" is called, ideally it should have gone to "sci_clk_determine_rate()" to query DM but it doesn't since clock is owned by OLDI not tidss. So add a member is_ext_vp_clk[] in tidss_device structure to identify this and avoid performing clock operations for VP if it has OLDI panel. For the same checks in OLDI driver, atomic_check() hook is added to its bridge_funcs. In the atomic_check() chain, first the bridge_atomic_check() is called and then crtc_atomic_check() is called. So mode clock is first checked in oldi driver and then skipped in tidss driver. Had the tidss_oldi structure been exposed to tidss_dispc.c, we could have directly checked VP type in dispc but since the structure is defined in tidss_oldi.c , we have to add additional member to tidss_device structure. [0]: https://lore.kernel.org/all/20250528122544.817829-1-aradhya.bhatia@linux.dev/ [1]: https://lore.kernel.org/all/DA6TT575Z82D.3MPK8HG5GRL8U@kernel.org/ Changelog v4->v5 - Rename is_oldi_vp[] to is_ext_vp_clk[] - Store both pixel clock round_rate and attempted_rate to reduce clk_round_rate() calls while validating modes - Code changes suggested by Tomi[2] - Minor fixes in comments and commit message [2]: https://lore.kernel.org/all/8cd9d1c4-2e9f-4766-b224-21925c4f991d@ideasonboard.com/ v4 patch link: https://lore.kernel.org/all/20250704094851.182131-1-j-choudhary@ti.com/ Changelog v3->v4: - Minor cosmetic fixes in code, comments and commit message - Pick up R-by and add Fixes tag v3 patch link: https://lore.kernel.org/all/20250701095541.190422-1-j-choudhary@ti.com/ Changelog v2->v3: - Add changes for OLDI - Rename max_pclk as it is misleading - Change commit message to make it more appropriate - Drop unnecessary zero initialization v2 patch link: https://lore.kernel.org/all/20250618100509.20386-1-j-choudhary@ti.com/ Changelog v1->v2: - Rebase it on linux-next after OLDI support series as all of its patches are reviewed and tested and it touches one of the functions used. v1 patch link: https://lore.kernel.org/all/20250618075804.139844-1-j-choudhary@ti.com/ Jayesh Choudhary (3): drm/tidss: oldi: Add property to identify OLDI supported VP drm/tidss: Remove max_pclk_khz from tidss display features drm/tidss: oldi: Add atomic_check hook for oldi bridge drivers/gpu/drm/tidss/tidss_dispc.c | 85 +++++++++++++---------------- drivers/gpu/drm/tidss/tidss_dispc.h | 1 - drivers/gpu/drm/tidss/tidss_drv.h | 11 ++++ drivers/gpu/drm/tidss/tidss_oldi.c | 27 +++++++++ 4 files changed, 75 insertions(+), 49 deletions(-)
On Tue Aug 19, 2025 at 9:21 PM CEST, Swamil Jain wrote: > In an effort to make the existing compatibles more usable, we are > removing the max_pclk_khz form dispc_features structure and doing the > supported pixel clock checks using "max_successful_rate[]" and > "max_attempted_rate[]". > > Changes are fully backwards compatible. > > After integration of OLDI support[0], we need additional patches in > OLDI to identify the VP that has OLDI. We have to do this since > OLDI driver owns the VP clock (its serial clock) and we cannot perform > clock operations on those VP clock from tidss driver. This issue was > also reported upstream when DSI fixes[1] had some clock related calls > in tidss driver. When "clk_round_rate()" is called, ideally it should > have gone to "sci_clk_determine_rate()" to query DM but it doesn't since > clock is owned by OLDI not tidss. > > So add a member is_ext_vp_clk[] in tidss_device structure to identify > this and avoid performing clock operations for VP if it has OLDI panel. > For the same checks in OLDI driver, atomic_check() hook is added to its > bridge_funcs. > In the atomic_check() chain, first the bridge_atomic_check() is called > and then crtc_atomic_check() is called. So mode clock is first checked > in oldi driver and then skipped in tidss driver. > > Had the tidss_oldi structure been exposed to tidss_dispc.c, we could > have directly checked VP type in dispc but since the structure is defined > in tidss_oldi.c , we have to add additional member to tidss_device > structure. > > [0]: https://lore.kernel.org/all/20250528122544.817829-1-aradhya.bhatia@linux.dev/ > [1]: https://lore.kernel.org/all/DA6TT575Z82D.3MPK8HG5GRL8U@kernel.org/ Since that wasn't picked up from v4: Tested-by: Michael Walle <mwalle@kernel.org> # on am67a
On 8/21/25 17:39, Michael Walle wrote: > On Tue Aug 19, 2025 at 9:21 PM CEST, Swamil Jain wrote: >> In an effort to make the existing compatibles more usable, we are >> removing the max_pclk_khz form dispc_features structure and doing the >> supported pixel clock checks using "max_successful_rate[]" and >> "max_attempted_rate[]". >> >> Changes are fully backwards compatible. >> >> After integration of OLDI support[0], we need additional patches in >> OLDI to identify the VP that has OLDI. We have to do this since >> OLDI driver owns the VP clock (its serial clock) and we cannot perform >> clock operations on those VP clock from tidss driver. This issue was >> also reported upstream when DSI fixes[1] had some clock related calls >> in tidss driver. When "clk_round_rate()" is called, ideally it should >> have gone to "sci_clk_determine_rate()" to query DM but it doesn't since >> clock is owned by OLDI not tidss. >> >> So add a member is_ext_vp_clk[] in tidss_device structure to identify >> this and avoid performing clock operations for VP if it has OLDI panel. >> For the same checks in OLDI driver, atomic_check() hook is added to its >> bridge_funcs. >> In the atomic_check() chain, first the bridge_atomic_check() is called >> and then crtc_atomic_check() is called. So mode clock is first checked >> in oldi driver and then skipped in tidss driver. >> >> Had the tidss_oldi structure been exposed to tidss_dispc.c, we could >> have directly checked VP type in dispc but since the structure is defined >> in tidss_oldi.c , we have to add additional member to tidss_device >> structure. >> >> [0]: https://lore.kernel.org/all/20250528122544.817829-1-aradhya.bhatia@linux.dev/ >> [1]: https://lore.kernel.org/all/DA6TT575Z82D.3MPK8HG5GRL8U@kernel.org/ > > Since that wasn't picked up from v4: > > Tested-by: Michael Walle <mwalle@kernel.org> # on am67a Hi Michael, Thanks for testing the patches. Sorry, I missed T-By. Will add in v6. Regards, Swamil
© 2016 - 2025 Red Hat, Inc.