On Wed, May 10, 2017 at 2:59 PM, Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> wrote: >> @@ -841,6 +847,11 @@ static int gen9_init_workarounds(struct intel_engine_cs *engine) >> if (ret) >> return ret; >> >> + /* Allow the UMD to configure their own power clock state */ >> + ret = wa_ring_whitelist_reg(engine, GEN8_R_PWR_CLK_STATE); >> + if (ret) >> + return ret; > > This is not a workaround, so it should be part of the cmd parser and > have an userspace. Yeah, the "This allows userspace ..." start of the commit message should have been a dead giveaway that we do indeed need the userspace patch for this too. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
