Re: [PATCH 1/2] drm/i915/guc: Load GuC on CFL

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

 



With this approach we need to have in mind that any new kbl firmware
version needs to be validated on both kbl and cfl by our QA before
publishing.

However the differences are really minimal if not 0. So publishing 2
identical files with different names maybe doesn't make any sense and
wouldn't save QA effort anyways.

So,
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>


On Wed, 2017-06-07 at 11:43 -0700, Anusha Srivatsa wrote:
> Coffeelake reuses Kabylake's GuC.
> 
> Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> Signed-off-by: Anusha Srivatsa <anusha.srivatsa@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_pci.c         | 1 +
>  drivers/gpu/drm/i915/intel_guc_loader.c | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index d90942c..c356646 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -435,6 +435,7 @@ static const struct intel_device_info intel_cannonlake_info = {

why is this below cnl?
ok, I need to rebase cfl on top and this will be above so a conflict
that it will probably appear when rebasing this patch either...


>  	BDW_FEATURES, \
>  	.gen = 9, \
>  	.platform = INTEL_COFFEELAKE, \
> +	.has_guc = 1, \
>  	.ddb_size = 896
>  
>  static const struct intel_device_info intel_coffeelake_info = {
> diff --git a/drivers/gpu/drm/i915/intel_guc_loader.c b/drivers/gpu/drm/i915/intel_guc_loader.c
> index d9045b6..8b0ae7f 100644
> --- a/drivers/gpu/drm/i915/intel_guc_loader.c
> +++ b/drivers/gpu/drm/i915/intel_guc_loader.c
> @@ -402,7 +402,7 @@ int intel_guc_select_fw(struct intel_guc *guc)
>  		guc->fw.path = I915_BXT_GUC_UCODE;
>  		guc->fw.major_ver_wanted = BXT_FW_MAJOR;
>  		guc->fw.minor_ver_wanted = BXT_FW_MINOR;
> -	} else if (IS_KABYLAKE(dev_priv)) {
> +	} else if (IS_KABYLAKE(dev_priv) || IS_COFFEELAKE(dev_priv)) {
>  		guc->fw.path = I915_KBL_GUC_UCODE;
>  		guc->fw.major_ver_wanted = KBL_FW_MAJOR;
>  		guc->fw.minor_ver_wanted = KBL_FW_MINOR;

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux