[Intel-gfx] [PATCH 5/6] drm/i915: drop intel_device_info_dump()
Tvrtko Ursulin
tvrtko.ursulin at linux.intel.com
Wed Jan 2 10:02:43 UTC 2019
On 31/12/2018 14:56, Jani Nikula wrote:
> The debugfs, error state and regular dmesg logging dump needs seem to be
> different. Remove the generic dump function only used for the welcome
> message. This may be added back later when better abstractions are
> identified, but at the moment this seems to be the simplest considering
> the device info rework in progress. No longer rely on device info being
> a substruct of dev_priv.
Why not just make intel_device_info_dump take dev_priv?
Regards,
Tvrtko
>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>
> Signed-off-by: Jani Nikula <jani.nikula at intel.com>
> ---
> drivers/gpu/drm/i915/i915_drv.c | 8 +++++++-
> drivers/gpu/drm/i915/intel_device_info.c | 15 ---------------
> drivers/gpu/drm/i915/intel_device_info.h | 2 --
> 3 files changed, 7 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 8d7a3a852c10..fe01d090f9bb 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -1636,7 +1636,13 @@ static void i915_welcome_messages(struct drm_i915_private *dev_priv)
> if (drm_debug & DRM_UT_DRIVER) {
> struct drm_printer p = drm_debug_printer("i915 device info:");
>
> - intel_device_info_dump(INTEL_INFO(dev_priv), &p);
> + drm_printf(&p, "pciid=0x%04x rev=0x%02x platform=%s gen=%i\n",
> + INTEL_DEVID(dev_priv),
> + INTEL_REVID(dev_priv),
> + intel_platform_name(INTEL_INFO(dev_priv)->platform),
> + INTEL_GEN(dev_priv));
> +
> + intel_device_info_dump_flags(INTEL_INFO(dev_priv), &p);
> intel_device_info_dump_runtime(RUNTIME_INFO(dev_priv), &p);
> }
>
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index f35e8cff4b99..e0ce0c9791fc 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -113,21 +113,6 @@ void intel_device_info_dump_runtime(const struct intel_runtime_info *info,
> info->cs_timestamp_frequency_khz);
> }
>
> -void intel_device_info_dump(const struct intel_device_info *info,
> - struct drm_printer *p)
> -{
> - struct drm_i915_private *dev_priv =
> - container_of(info, struct drm_i915_private, info);
> -
> - drm_printf(p, "pciid=0x%04x rev=0x%02x platform=%s gen=%i\n",
> - INTEL_DEVID(dev_priv),
> - INTEL_REVID(dev_priv),
> - intel_platform_name(info->platform),
> - info->gen);
> -
> - intel_device_info_dump_flags(info, p);
> -}
> -
> void intel_device_info_dump_topology(const struct sseu_dev_info *sseu,
> struct drm_printer *p)
> {
> diff --git a/drivers/gpu/drm/i915/intel_device_info.h b/drivers/gpu/drm/i915/intel_device_info.h
> index f0e6d374d4ec..76735869e32d 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.h
> +++ b/drivers/gpu/drm/i915/intel_device_info.h
> @@ -267,8 +267,6 @@ static inline void sseu_set_eus(struct sseu_dev_info *sseu,
> const char *intel_platform_name(enum intel_platform platform);
>
> void intel_device_info_runtime_init(struct drm_i915_private *dev_priv);
> -void intel_device_info_dump(const struct intel_device_info *info,
> - struct drm_printer *p);
> void intel_device_info_dump_flags(const struct intel_device_info *info,
> struct drm_printer *p);
> void intel_device_info_dump_runtime(const struct intel_runtime_info *info,
>
More information about the Intel-gfx
mailing list