- Feb 20, 2024
-
-
Jiri Slaby (SUSE) authored
execute_cb::signal is not used since commit 5ac545b8 (drm/i915/request: Remove the hook from await_execution). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
Jiri Slaby (SUSE) <jirislaby@kernel.org> Cc: Jani Nikula <jani.nikula@linux.intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com> Cc: intel-gfx@lists.freedesktop.org Acked-by:
Jani Nikula <jani.nikula@intel.com> Signed-off-by:
Rodrigo Vivi <rodrigo.vivi@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20240216065326.6910-20-jirislaby@kernel.org
-
Jiri Slaby (SUSE) authored
intel_gvt_irq::pending_events was never used since its addition in commit c8fe6a68 (drm/i915/gvt: vGPU interrupt virtualization.). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_gvt_event_info::policy was never used since its addition in commit c8fe6a68 (drm/i915/gvt: vGPU interrupt virtualization.). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_gvt_irq_info::warned was never used since its addition in commit c8fe6a68 (drm/i915/gvt: vGPU interrupt virtualization.). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
gvt_mmio_block::device is not used since commit e0f74ed4 (i915/gvt: Separate the MMIO tracking table from GVT-g). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vgpu::intx_trigger was never used since its addition in commit f30437c5 (drm/i915/gvt: add KVMGT support). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vgpu_opregion::mapped is not used since commit 36774806 (drm/i915/gvt: remove enum hypervisor_type). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vgpu_fence::base was never used since its addition in commit 28a60dee (drm/i915/gvt: vGPU HW resource management). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vgpu_gtt::active_ppgtt_mm_bitmap was never used since its addition in commit 2707e444 (drm/i915/gvt: vGPU graphics memory virtualization). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
i915_perf_stream::size_exponent was never used since its addition in commit a37f08a8 (drm/i915/perf: Refactor oa object to better manage resources). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
i915_drm_client::id is not used since commit e894b724 (drm/i915: Use the fdinfo helper). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_guc::ads_engine_usage_size was never used since its addition in commit 77cdd054 (drm/i915/pmu: Connect engine busyness stats from GuC to pmu). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vbt_panel_data::edp::initialized is not used since commit 9f0e7ff4 (drm/i915: fetch eDP configuration data from the VBT). Drop it. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_vgpu_workload::ring_context was never used since its addition in commit 28c4c6ca (drm/i915/gvt: vGPU workload submission) and ::restore_inhibit since its addition in commit e4734057 (drm/i915/gvt: vGPU workload scheduler). Drop them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_gvt_mmio_info::device is not used since commit e0f74ed4 (i915/gvt: Separate the MMIO tracking table from GVT-g) and ::addr_range was never used since its addition in commit 12d14cc4 (drm/i915/gvt: Introduce a framework for tracking HW registers.). Drop them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
intel_gvt_gtt::{mm_alloc_page_table,mm_free_page_table} are not used since commit ede9d0cf (drm/i915/gvt: Rework shadow graphic memory management code). Drop them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by: -
Jiri Slaby (SUSE) authored
intel_dsi::port_bits is unused since commit 369602d3 (drm/i915: Add support for port enable/disable for dual link configuration) and ::hs is unused likely since commit 063c86f6 (drm/i915/dsi: remove intel_dsi_cmd.c and the unused functions therein). Drop them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
Both struct intel_vgpu_pipe_format and intel_vgpu_fb_format were never used since its addition in commit 9f31d106 (drm/i915/gvt: Add framebuffer decoder support). Drop them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
Jiri Slaby (SUSE) authored
struct intel_dvo_dev_ops's ::create_resources(), ::prepare(), ::commit::, and get_modes() are all unused since their addition in 79e53945 (DRM: i915: add mode setting support). Drop all of them. Found by https://github.com/jirislaby/clang-struct . Signed-off-by:
-
- Feb 16, 2024
-
-
Ville Syrjälä authored
Add a bit of documentation to briefly explain the methods by which we can change the CDCLK frequency. Signed-off-by:
Ville Syrjälä <ville.syrjala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20240207013334.29606-5-ville.syrjala@linux.intel.com Reviewed-by:
Gustavo Sousa <gustavo.sousa@intel.com>
-
Ville Syrjälä authored
cdclk_compute_crawl_and_squash_midpoint() was still assuming that cd2x divider == 1 (ie. full divider == 2). Remove that assumption by computing the dividers properly. We'll also toss in a WARN in case the divider somehow ends up different between the old and new cdclk configs. That should never happen given we have div==2 in all the cdclk table entries for the affected platforms. If in the future we need a config where the divider also needs to be changed then we likely need to add an extra step into the cdclk programming sequence to make sure things stay within legal limits throughout the process. Signed-off-by:
-
Ville Syrjälä authored
Have cdclk_squash_waveform() return a u16 since that's how many bits we have in the waveform. We alreday use u16 everywhere else. Signed-off-by:
-
Ville Syrjälä authored
Extract the cdclk divider calculation into a helper. We'll have more users of this soon. We can now also get rid of the intermediate 'unsquashed_cdclk' variable. Signed-off-by:
-
- Feb 15, 2024
-
-
Ville Syrjälä authored
There's nothing magical about vlv+ platforms vs. fastboot. If it works somewhere it should work everywhere, assuming we've not missed any crucial state checks. That seems unlikely on older platforms with less state to check anyway. Just enable fastboot across the board, and the remove the remnants of the optional stuff (we already removed the modparam for fastboot anyway). Signed-off-by:
Mika Kahola <mika.kahola@intel.com>
-
Ville Syrjälä authored
Chunk up the humongous dpll_hw_state comparison check into per-platform variants, implemented in the dpll_mgr. This is step one in allowing each platform (or perhaps even PLL) type to have a custom hw state structure instead of having to smash it all into one. Signed-off-by:
-
Ville Syrjälä authored
GMCH platform DPLLs are similar to the IBX+ PCH DPLLs so we can just use the same state dump function for both. Signed-off-by:
-
Ville Syrjälä authored
Most crtc state mismatches include the CRTC id+name in the prints. Also include it in the ELD buffer mismatch prints. Signed-off-by:
-
Ville Syrjälä authored
GMCH DPLL state check was mistakenly removed in commit 87fc875a ("drm/i915/dg2: Skip shared DPLL handling"). Bring it back. Signed-off-by:
-
Gustavo Sousa authored
Extend the list of ADL-N PCI IDs to contain two new entries. Bspec: 68397 Signed-off-by:
Dnyaneshwar Bhadane <dnyaneshwar.bhadane@intel.com> Signed-off-by:
Matt Roper <matthew.d.roper@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20240214144629.106702-2-gustavo.sousa@intel.com
-
Ravi Kumar Vodapalli authored
DP/eDP and HDMI C20 PHY PLL values were updated for MTL platform Signed-off-by:
Ravi Kumar Vodapalli <ravi.kumar.vodapalli@intel.com> Reviewed-by:
-
- Feb 13, 2024
-
-
Imre Deak authored
Prevent accessing the HW from the get_modes hooks of connectors deriving the mode list from the display's EDID. drm_edid_connector_add_modes() will return the mode list based on the EDID which was cached during a previous detection/get_modes call. This also fixes the NULL deref problem (10085) which was introduced/revealed by commit bab87ef4 ("drm/i915: Disable hotplug detection handlers during driver init/shutdown") After the above change MST connectors will not change state during driver init/shutdown; thus some of these connectors with no I2C/DDC adapter registered for them (since the given MST port has no sink connected) may stay then in the 'unknown' connector status. The get_modes() hook should not try to use the I2C/DDC adapter in this state (which would lead to the above NULL deref) which this patch ensures. v2: - Remove the redundant check from intel_crt_ddc_get_modes(). - Rebase on latest drm-tip. - Add Fixes: line / related commit notes. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/10085 Fixes: bab87ef4 ("drm/i915: Disable hotplug detection handlers during driver init/shutdown") Signed-off-by:
Imre Deak <imre.deak@intel.com> Reviewed-by:
Jouni Högander <jouni.hogander@intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20240212175237.2625812-2-imre.deak@intel.com
-
Imre Deak authored
Prevent accessing the HW from the SDVO/TV get_modes connector hook. Returning 0 from the hook will make the caller - drm_helper_probe_single_connector_modes() - return a default/EDID override mode list to users. This matches the case where intel_sdvo_get_tv_modes() fails to retrieve the current mode list due to a HW access failure. v2: Clarify the commit message wrt. which modes get_modes() returns. (Jouni) Signed-off-by:
-
Stanislav Lisovskiy authored
For validation purposes, it might be useful to be able to force Bigjoiner mode, even if current dotclock/resolution do not require that. Lets add such to option to debugfs. v2: - Apparently intel_dp_need_bigjoiner can't be used, when debugfs entry is created so lets just check manually the DISPLAY_VER. v3: - Switch to intel_connector from drm_connector(Jani Nikula) - Remove redundant modeset lock(Jani Nikula) - Use kstrtobool_from_user for boolean value(Jani Nikula) v4: - Apply the changes to proper function(Jani Nikula) v5: - Removed unnecessary check from i915_bigjoiner_enable_show (Ville Syrjälä) - Added eDP connector check to intel_connector_debugfs_add (Ville Syrjälä) - Removed debug message in order to prevent dmesg flooding (Ville Syrjälä) v6: - Assume now always that m->private is intel_connector - Fixed other similar conflicts v7: - Move bigjoiner force option to intel_connector(Ville Syrjälä) - Use DEFINE_SHOW_STORE_ATTRIBUTE instead of defining fops manually.(Ville Syrjälä) v8: - Pass intel_connector to debugfs_create_file, instead of drm_connector. (Jani Nikula) Signed-off-by:Stanislav Lisovskiy <stanislav.lisovskiy@intel.com> Acked-by:
-
- Feb 09, 2024
-
-
Ville Syrjälä authored
Prefer sizeof(*variable) to sizeof(type) to make it a bit harder to screw things up. Signed-off-by:
-
Ville Syrjälä authored
The dvo 'dev_priv' is void* so no need for an explicit cast. Signed-off-by:
-
Ville Syrjälä authored
Switch to drm_dbg_kms() in the ilk wm code so we see which device generated the debugs. Need to plumb i915 a bit deeper to make that happen. Signed-off-by:
-
Ville Syrjälä authored
Switch to drm_dbg_kms() in the pre-ilk wm code so we see which device generated the debugs. Need to plumb i915 a bit deeper to make that happen. Signed-off-by:
-
Ville Syrjälä authored
Just pass the whole i915 to intel_get_cxsr_latency() instead of having each caller dig out bits and pieces. Signed-off-by:
-
Ville Syrjälä authored
Switch to per-device debugs in the hdcp code so we see at least which device is involved. Should proably also print the connector/encoder/etc. in there, but left that for the future. Signed-off-by:
-
Ville Syrjälä authored
Switch to drm_dbg_kms() in the VBT code so we see which device generated the debugs. Need to plumb i915 a bit deeper to make that happen. Signed-off-by:
-
