On Tue, Oct 31, 2017 at 09:30:27AM -0700, Oscar Mateo wrote: > > > On 10/31/2017 07:14 AM, Joonas Lahtinen wrote: > > On Mon, 2017-10-30 at 13:17 -0700, Oscar Mateo wrote: > > > By doing this, we can dump these workarounds in debugfs for validation (which, > > > at the moment, we are only able to do for the contexts WAs). > > > > > > v2: > > > - Wrong macro used for MMIO set bit masked > > > - Improved naming > > > - Rebased > > > > > > v3: > > > - GT instead of MMIO (Chris, Mika) > > > - Leave L3_PRIO_CREDITS_MASK for a separate patch > > > - Rebased > > > > > > v4: Carry the init_early nomenclature over (Chris) > > > > > > Signed-off-by: Oscar Mateo <oscar.mateo@xxxxxxxxx> > > > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxxxxxxxx> > > > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > This and the following patch are still a no-go and won't be merged. The > > required changes for the series to be accepted (to make it more > > declarative) were clearly described previously. If there are further > > questions, we should discuss those instead wasting time looking at > > respins that do not address the input given. > > > > Respins that ignore review are a waste of everyone's time. So, please > > react to the review instead of trying to brute force a series in as you > > see fit. > > > > Regards, Joonas > > My apologies, I wasn't trying to "brute force" anything. I haven't been able > to work on this for the last two weeks and I didn't want the issue to go > cold, so I sent a version to attend to Ville's and Chris' review comments > (this is useful anyway, because the same comments apply whether you go with > functions or with a static table). I am working on a version that implements > your suggestions and I will send it as soon as it's ready. Typically when submitting a changeset that isn't ready to be merged, you'd tag it RFC or similar. Kind regards, David Weinehall _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
