On (2017-09-08 14:19), Jonathan Gray wrote: > On Thu, Sep 07, 2017 at 09:21:27AM -0600, Robert Nagy wrote: > > CVSROOT: /cvs > > Module name: ports > > Changes by: rob...@cvs.openbsd.org 2017/09/07 09:21:27 > > > > Log message: > > import of the inteldrm port for creating the firmware package that > > includes > > firmwares for i915/inteldrm > > > > ok ajacoutot@ > > This does not package the correct guc files. > > The code in tree has: > intel_guc_loader.c:#define I915_SKL_GUC_UCODE "i915/skl_guc_ver4.bin" > > proposed kaby lake patch has > > #define I915_KBL_GUC_UCODE "i915/kbl_guc_ver9_14.bin" > > but only i915/skl_guc_ver6.bin is packaged. > > Naming the files i915-filename or inteldrm-filename without a > .ucode/.bin suffix instead of putting them in a i915 directory would be > consistent with how the other firmware is handled. >
Hi I know the file names are not the same now. jcs@ has a patch to load the guc and I was testing with that, but obviously I will change the port to have all the firmware installed.