Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-13 Thread Ville Syrjälä
On Tue, Oct 13, 2015 at 04:16:47PM +0300, Jani Nikula wrote: > On Wed, 07 Oct 2015, Jani Nikula wrote: > > On Tue, 06 Oct 2015, Ville Syrjälä wrote: > >> On Tue, Oct 06, 2015 at 04:43:11PM +0300, Jani Nikula wrote: > >>> On Tue, 06 Oct 2015, Ville Syrjälä wrote: > >>> > On Tue, Oct 06, 2015 at 0

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-13 Thread Jani Nikula
On Wed, 07 Oct 2015, Jani Nikula wrote: > On Tue, 06 Oct 2015, Ville Syrjälä wrote: >> On Tue, Oct 06, 2015 at 04:43:11PM +0300, Jani Nikula wrote: >>> On Tue, 06 Oct 2015, Ville Syrjälä wrote: >>> > On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: >>> >> Prefer inclusive ranges for

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-07 Thread Jani Nikula
On Tue, 06 Oct 2015, Ville Syrjälä wrote: > On Tue, Oct 06, 2015 at 04:43:11PM +0300, Jani Nikula wrote: >> On Tue, 06 Oct 2015, Ville Syrjälä wrote: >> > On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: >> >> Prefer inclusive ranges for revision checks rather than "below B0". Per >>

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-06 Thread Ville Syrjälä
On Tue, Oct 06, 2015 at 04:43:11PM +0300, Jani Nikula wrote: > On Tue, 06 Oct 2015, Ville Syrjälä wrote: > > On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: > >> Prefer inclusive ranges for revision checks rather than "below B0". Per > >> specs A2 is not used, so revid <= A1 matches r

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-06 Thread Vivi, Rodrigo
On Tue, 2015-10-06 at 16:43 +0300, Jani Nikula wrote: > On Tue, 06 Oct 2015, Ville Syrjälä > wrote: > > On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: > > > Prefer inclusive ranges for revision checks rather than "below > > > B0". Per > > > specs A2 is not used, so revid <= A1 matc

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-06 Thread Jani Nikula
On Tue, 06 Oct 2015, Ville Syrjälä wrote: > On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: >> Prefer inclusive ranges for revision checks rather than "below B0". Per >> specs A2 is not used, so revid <= A1 matches revid < B0. > > The w/a db would say UNTIL_B0 etc., so might be easier

Re: [Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-06 Thread Ville Syrjälä
On Tue, Oct 06, 2015 at 02:41:15PM +0300, Jani Nikula wrote: > Prefer inclusive ranges for revision checks rather than "below B0". Per > specs A2 is not used, so revid <= A1 matches revid < B0. The w/a db would say UNTIL_B0 etc., so might be easier to check against it if we keep to the same conven

[Intel-gfx] [PATCH 2/3] drm/i915/bxt: add revision id for A1 stepping and use it

2015-10-06 Thread Jani Nikula
Prefer inclusive ranges for revision checks rather than "below B0". Per specs A2 is not used, so revid <= A1 matches revid < B0. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/i915_drv.h| 1 + drivers/gpu/drm/i915/i915_gem.c| 2 +- drivers/gpu/drm/i915/i915_guc_submi