[Openpvrsgx-devgroup] [PATCH] pvrsgx: set correct SGX version for Poulsbo in Makefile

[H. Nikolaus Schaller]

Hi Eloy,

> Am 26.10.2020 um 00:00 schrieb Eloy Degen <degeneloy at gmail.com>:
> 
> The SGX used in the GMA500 is the 535, but the Makefile incorrectly used 530.
> 
> Signed-off-by: Eloy Degen <degeneloy at gmail.com>
> ---
> drivers/gpu/drm/pvrsgx/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/pvrsgx/Makefile b/drivers/gpu/drm/pvrsgx/Makefile
> index c528d928d99e..6c3f99ce29bf 100644
> --- a/drivers/gpu/drm/pvrsgx/Makefile
> +++ b/drivers/gpu/drm/pvrsgx/Makefile
> @@ -133,7 +133,7 @@ endif # CONFIG_SGX_OMAP
> ifneq ($(CONFIG_SGX_POULSBO),)
> 	@$(kecho) "  MAKE    $(obj): poulsbo"
> # Fix versions
> -	$(Q)SOC_VENDOR=intel SOC_FAMILY=poulsbo SOC=gma500 SGX=sgx530
> SGX_FLAG=SGX535 SGX_REV=126 $(MAKE) TARGET_TYPE=$(CONFIG_SGX_POULSBO)
> $(build)=$(SOURCE)
> +	$(Q)SOC_VENDOR=intel SOC_FAMILY=poulsbo SOC=gma500 SGX=sgx535
> SGX_FLAG=SGX535 SGX_REV=126 $(MAKE) TARGET_TYPE=$(CONFIG_SGX_POULSBO)
> $(build)=$(SOURCE)
> 	$(Q)if [ -r $(SOURCE)/modules.order ]; then cat
> $(SOURCE)/modules.order >>$(obj)/modules.order; fi
> endif # CONFIG_SGX_POULSBO
> 
> -- 
> 2.25.1
> _______________________________________________

Thanks for the patch.
With this it works for me and I get:

  LD [M]  drivers/gpu/drm/pvrsgx/1.17.4948957/eurasia_km/pvrsrvkm_gma3600_sgx545_10141.o - due to target missing
  LD [M]  drivers/gpu/drm/pvrsgx/1.17.4948957/eurasia_km/pvrsrvkm_gma500_sgx535_126.o - due to target missing
  LD [M]  drivers/gpu/drm/gma500/gma500_gfx.o - due to target missing

What I have also done is to take your defconfig as a new letux_defconfig.
I had to disable two CONFIGS to get it compile on my compiler (disable RETPOLINE
and remove reference of SYSTEM_TRUSTED_KEYS to the debian predefined keys).

My plan is to have a letux-5.9.2 which will include these changes. Depends
on how quickly v5.9.2 appears upstream. Maybe today or tomorrow.

BR,
Nikolaus