Commit 6686c30e authored by Jani Nikula's avatar Jani Nikula

drm/i915: move more of the display probe to display code

Initializing i915->display.info.__device_info and DISPLAY_RUNTIME_INFO()
really belongs in display code. Move them there.

Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230912120537.2158209-1-jani.nikula@intel.com
parent 2fad9e44
...@@ -850,8 +850,8 @@ probe_gmdid_display(struct drm_i915_private *i915, u16 *ver, u16 *rel, u16 *step ...@@ -850,8 +850,8 @@ probe_gmdid_display(struct drm_i915_private *i915, u16 *ver, u16 *rel, u16 *step
return &no_display; return &no_display;
} }
const struct intel_display_device_info * static const struct intel_display_device_info *
intel_display_device_probe(struct drm_i915_private *i915, bool has_gmdid, probe_display(struct drm_i915_private *i915, bool has_gmdid,
u16 *gmdid_ver, u16 *gmdid_rel, u16 *gmdid_step) u16 *gmdid_ver, u16 *gmdid_rel, u16 *gmdid_step)
{ {
struct pci_dev *pdev = to_pci_dev(i915->drm.dev); struct pci_dev *pdev = to_pci_dev(i915->drm.dev);
...@@ -876,6 +876,24 @@ intel_display_device_probe(struct drm_i915_private *i915, bool has_gmdid, ...@@ -876,6 +876,24 @@ intel_display_device_probe(struct drm_i915_private *i915, bool has_gmdid,
return &no_display; return &no_display;
} }
void intel_display_device_probe(struct drm_i915_private *i915)
{
u16 ver, rel, step;
/* Probe display support */
i915->display.info.__device_info = probe_display(i915, HAS_GMD_ID(i915),
&ver, &rel, &step);
memcpy(DISPLAY_RUNTIME_INFO(i915),
&DISPLAY_INFO(i915)->__runtime_defaults,
sizeof(*DISPLAY_RUNTIME_INFO(i915)));
if (HAS_GMD_ID(i915)) {
DISPLAY_RUNTIME_INFO(i915)->ip.ver = ver;
DISPLAY_RUNTIME_INFO(i915)->ip.rel = rel;
DISPLAY_RUNTIME_INFO(i915)->ip.step = step;
}
}
void intel_display_device_info_runtime_init(struct drm_i915_private *i915) void intel_display_device_info_runtime_init(struct drm_i915_private *i915)
{ {
struct intel_display_runtime_info *display_runtime = DISPLAY_RUNTIME_INFO(i915); struct intel_display_runtime_info *display_runtime = DISPLAY_RUNTIME_INFO(i915);
......
...@@ -149,9 +149,7 @@ struct intel_display_device_info { ...@@ -149,9 +149,7 @@ struct intel_display_device_info {
} color; } color;
}; };
const struct intel_display_device_info * void intel_display_device_probe(struct drm_i915_private *i915);
intel_display_device_probe(struct drm_i915_private *i915, bool has_gmdid,
u16 *ver, u16 *rel, u16 *step);
void intel_display_device_info_runtime_init(struct drm_i915_private *i915); void intel_display_device_info_runtime_init(struct drm_i915_private *i915);
void intel_display_device_info_print(const struct intel_display_device_info *info, void intel_display_device_info_print(const struct intel_display_device_info *info,
......
...@@ -424,7 +424,6 @@ void intel_device_info_driver_create(struct drm_i915_private *i915, ...@@ -424,7 +424,6 @@ void intel_device_info_driver_create(struct drm_i915_private *i915,
const struct intel_device_info *match_info) const struct intel_device_info *match_info)
{ {
struct intel_runtime_info *runtime; struct intel_runtime_info *runtime;
u16 ver, rel, step;
/* Setup INTEL_INFO() */ /* Setup INTEL_INFO() */
i915->__info = match_info; i915->__info = match_info;
...@@ -433,18 +432,7 @@ void intel_device_info_driver_create(struct drm_i915_private *i915, ...@@ -433,18 +432,7 @@ void intel_device_info_driver_create(struct drm_i915_private *i915,
runtime = RUNTIME_INFO(i915); runtime = RUNTIME_INFO(i915);
memcpy(runtime, &INTEL_INFO(i915)->__runtime, sizeof(*runtime)); memcpy(runtime, &INTEL_INFO(i915)->__runtime, sizeof(*runtime));
/* Probe display support */ intel_display_device_probe(i915);
i915->display.info.__device_info = intel_display_device_probe(i915, HAS_GMD_ID(i915),
&ver, &rel, &step);
memcpy(DISPLAY_RUNTIME_INFO(i915),
&DISPLAY_INFO(i915)->__runtime_defaults,
sizeof(*DISPLAY_RUNTIME_INFO(i915)));
if (HAS_GMD_ID(i915)) {
DISPLAY_RUNTIME_INFO(i915)->ip.ver = ver;
DISPLAY_RUNTIME_INFO(i915)->ip.rel = rel;
DISPLAY_RUNTIME_INFO(i915)->ip.step = step;
}
runtime->device_id = device_id; runtime->device_id = device_id;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment