[PATCH 03/10] drm/i915/gvt: Enable device info initialization for BXT.

Colin Xu Colin.Xu at intel.com
Fri Jun 1 06:48:49 UTC 2018


On 06/01/2018 01:28 PM, Zhenyu Wang wrote:
> On 2018.06.01 10:12:32 +0800, Colin Xu wrote:
>> Initialize BXT device info as SKL/KBL.
>>
>> Signed-off-by: Colin Xu <colin.xu at intel.com>
>> ---
>>   drivers/gpu/drm/i915/gvt/gvt.c | 6 ++++--
>>   1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/gvt/gvt.c b/drivers/gpu/drm/i915/gvt/gvt.c
>> index 22a3ddff38a3..4afae986cea3 100644
>> --- a/drivers/gpu/drm/i915/gvt/gvt.c
>> +++ b/drivers/gpu/drm/i915/gvt/gvt.c
>> @@ -238,8 +238,10 @@ static void init_device_info(struct intel_gvt *gvt)
>>   	struct intel_gvt_device_info *info = &gvt->device_info;
>>   	struct pci_dev *pdev = gvt->dev_priv->drm.pdev;
>>   
>> -	if (IS_BROADWELL(gvt->dev_priv) || IS_SKYLAKE(gvt->dev_priv)
>> -		|| IS_KABYLAKE(gvt->dev_priv)) {
>> +	if (IS_BROADWELL(gvt->dev_priv)
>> +		|| IS_SKYLAKE(gvt->dev_priv)
>> +		|| IS_KABYLAKE(gvt->dev_priv)
>> +		|| IS_BROXTON(gvt->dev_priv)) {
>>   		info->max_support_vgpus = 8;
>>   		info->cfg_space_size = PCI_CFG_SPACE_EXP_SIZE;
>>   		info->mmio_size = 2 * 1024 * 1024;
> As we have only one config for all hw, why just remove those chip check?

Yes since is_supported_device() will be the gate keeper, all these redundant
checks could be removed.

-- 

Best Regards,
Colin Xu



More information about the intel-gvt-dev mailing list