On to, 2015-04-16 at 14:22 +0530, Animesh Manna wrote: > [...] > +#include <linux/firmware.h> > +#include "i915_drv.h" > +#include "i915_reg.h" > + > +#define I915_CSR_SKL "i915/skl_dmc_ver1.bin" The latest version on the FW download page is skl_dmc_ver4.bin as Damien pointed out and skl_dmc_ver1.bin is not available there. So why did you choose ver1? If ver4 is not compatible with the current driver code, then ver1 must be made available too on the download page, otherwise we should use here ver4. The patch otherwise looks ok to me. --Imre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx