Re: [PATCH 5/6] drm/i915: drop intel_device_info_dump()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 02 Jan 2019, Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> wrote:
> 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?

I'm thinking the device info dumpers will need some refactoring in the
future anyway to best suit all the needs of dmesg logging, debugfs, and
error state dumping. The last one especially wants to use data copied to
the error state instead of drm_i915_private.

I admittedly took the easy way out here, and removed the function
altogether because there's currently only one user. I didn't want to
change now, and then change again later.

BR,
Jani.

>
> Regards,
>
> Tvrtko
>
>> 
>> Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx>
>> Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx>
>> ---
>>   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,
>> 

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux