* [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW @ 2014-01-14 19:04 Jesse Barnes 2014-01-14 22:05 ` Ben Widawsky 0 siblings, 1 reply; 5+ messages in thread From: Jesse Barnes @ 2014-01-14 19:04 UTC (permalink / raw) To: intel-gfx It ought to work ok in 3.14. We have some fun stuff coming after that, but all the basics are in place now and seem relatively stable. Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> --- drivers/gpu/drm/i915/i915_drv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index 61fb9fc..df4d350 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c @@ -342,7 +342,6 @@ static const struct intel_device_info intel_haswell_m_info = { }; static const struct intel_device_info intel_broadwell_d_info = { - .is_preliminary = 1, .gen = 8, .num_pipes = 3, .need_gfx_hws = 1, .has_hotplug = 1, .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, @@ -351,7 +350,6 @@ static const struct intel_device_info intel_broadwell_d_info = { }; static const struct intel_device_info intel_broadwell_m_info = { - .is_preliminary = 1, .gen = 8, .is_mobile = 1, .num_pipes = 3, .need_gfx_hws = 1, .has_hotplug = 1, .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW 2014-01-14 19:04 [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW Jesse Barnes @ 2014-01-14 22:05 ` Ben Widawsky 2014-01-14 22:35 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Ben Widawsky @ 2014-01-14 22:05 UTC (permalink / raw) To: Jesse Barnes; +Cc: intel-gfx On Tue, Jan 14, 2014 at 11:04:16AM -0800, Jesse Barnes wrote: > It ought to work ok in 3.14. We have some fun stuff coming after that, > but all the basics are in place now and seem relatively stable. > > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> > --- > drivers/gpu/drm/i915/i915_drv.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 61fb9fc..df4d350 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -342,7 +342,6 @@ static const struct intel_device_info intel_haswell_m_info = { > }; > > static const struct intel_device_info intel_broadwell_d_info = { > - .is_preliminary = 1, > .gen = 8, .num_pipes = 3, > .need_gfx_hws = 1, .has_hotplug = 1, > .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, > @@ -351,7 +350,6 @@ static const struct intel_device_info intel_broadwell_d_info = { > }; > > static const struct intel_device_info intel_broadwell_m_info = { > - .is_preliminary = 1, > .gen = 8, .is_mobile = 1, .num_pipes = 3, > .need_gfx_hws = 1, .has_hotplug = 1, > .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, I had been hoping to at least get the following in or checked out before marking as not preliminary: semaphores - patch series has one regression, but I would have liked it merged and turned off RC6 well tested - I am not sure what the state of our HW is in this regard. I was told the HW we have doesn't support it, but I haven't tried. RPS - see RC6 4G GGTT - I guess this is currently blocked on PPGTT, which is unfortunate. I can live without this for non-preilm. I've been trying my best to keep the bug count low, and it has been pretty good thanks to you and Paulo - so I think it should be safe enough to call the HW supported, even with my gripes above. So while I prefer to get the stuff above in/tested, if you want to move ahead with this, it's Acked by: Ben Widawsky <ben@bwidawsk.net> -- Ben Widawsky, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW 2014-01-14 22:05 ` Ben Widawsky @ 2014-01-14 22:35 ` Daniel Vetter 2014-01-16 16:08 ` Damien Lespiau 0 siblings, 1 reply; 5+ messages in thread From: Daniel Vetter @ 2014-01-14 22:35 UTC (permalink / raw) To: Ben Widawsky; +Cc: intel-gfx On Tue, Jan 14, 2014 at 02:05:45PM -0800, Ben Widawsky wrote: > On Tue, Jan 14, 2014 at 11:04:16AM -0800, Jesse Barnes wrote: > > It ought to work ok in 3.14. We have some fun stuff coming after that, > > but all the basics are in place now and seem relatively stable. > > > > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> > > --- > > drivers/gpu/drm/i915/i915_drv.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > > index 61fb9fc..df4d350 100644 > > --- a/drivers/gpu/drm/i915/i915_drv.c > > +++ b/drivers/gpu/drm/i915/i915_drv.c > > @@ -342,7 +342,6 @@ static const struct intel_device_info intel_haswell_m_info = { > > }; > > > > static const struct intel_device_info intel_broadwell_d_info = { > > - .is_preliminary = 1, > > .gen = 8, .num_pipes = 3, > > .need_gfx_hws = 1, .has_hotplug = 1, > > .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, > > @@ -351,7 +350,6 @@ static const struct intel_device_info intel_broadwell_d_info = { > > }; > > > > static const struct intel_device_info intel_broadwell_m_info = { > > - .is_preliminary = 1, > > .gen = 8, .is_mobile = 1, .num_pipes = 3, > > .need_gfx_hws = 1, .has_hotplug = 1, > > .ring_mask = RENDER_RING | BSD_RING | BLT_RING | VEBOX_RING, > > I had been hoping to at least get the following in or checked out before > marking as not preliminary: > > semaphores - patch series has one regression, but I would have liked it > merged and turned off > > RC6 well tested - I am not sure what the state of our HW is in this > regard. I was told the HW we have doesn't support it, but I haven't > tried. > > RPS - see RC6 > > 4G GGTT - I guess this is currently blocked on PPGTT, which is > unfortunate. I can live without this for non-preilm. > > > I've been trying my best to keep the bug count low, and it has been > pretty good thanks to you and Paulo - so I think it should be safe > enough to call the HW supported, even with my gripes above. So while I > prefer to get the stuff above in/tested, if you want to move ahead with > this, it's > > Acked by: Ben Widawsky <ben@bwidawsk.net> The target for dropping the preliminary hw support tag is "works at least as good as vesa". That means - modesetting works with the boot config (broken hotplug in corner cases is meh, but imo ok) - suspend/resume doesn't blow up - userspace isn't in such a bad shape to take down the entire machine All the things you're listing are goodies on top (of course we want them for the shipping driver, but that's not the criterion here at all). We have the preliminary hw support thing since at least on hsw and on byt we've had entire clases of outputs that didn't work at all (DP on hsw and mipi on byt). The goal here is that users can always at least install a distro (and then upgrade to a decent kernel) with whatever gets burned into the images, without the need to resort to stuff like adding nomodeset. Patch merged for 3.14, thanks. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW 2014-01-14 22:35 ` Daniel Vetter @ 2014-01-16 16:08 ` Damien Lespiau 2014-01-16 16:23 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Damien Lespiau @ 2014-01-16 16:08 UTC (permalink / raw) To: Daniel Vetter; +Cc: Ben Widawsky, intel-gfx On Tue, Jan 14, 2014 at 11:35:22PM +0100, Daniel Vetter wrote: > On Tue, Jan 14, 2014 at 02:05:45PM -0800, Ben Widawsky wrote: > > On Tue, Jan 14, 2014 at 11:04:16AM -0800, Jesse Barnes wrote: > > > It ought to work ok in 3.14. We have some fun stuff coming after that, > > > but all the basics are in place now and seem relatively stable. > > > > > > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> > > Patch merged for 3.14, thanks. > -Daniel There's a: WARN(!i915_preliminary_hw_support, "GEN8_CENTROID_PIXEL_OPT_DIS not be needed for production\n"); in gen8_init_clock_gating() that now always triggers. -- Damien ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW 2014-01-16 16:08 ` Damien Lespiau @ 2014-01-16 16:23 ` Daniel Vetter 0 siblings, 0 replies; 5+ messages in thread From: Daniel Vetter @ 2014-01-16 16:23 UTC (permalink / raw) To: Damien Lespiau; +Cc: Ben Widawsky, intel-gfx On Thu, Jan 16, 2014 at 5:08 PM, Damien Lespiau <damien.lespiau@intel.com> wrote: > On Tue, Jan 14, 2014 at 11:35:22PM +0100, Daniel Vetter wrote: >> On Tue, Jan 14, 2014 at 02:05:45PM -0800, Ben Widawsky wrote: >> > On Tue, Jan 14, 2014 at 11:04:16AM -0800, Jesse Barnes wrote: >> > > It ought to work ok in 3.14. We have some fun stuff coming after that, >> > > but all the basics are in place now and seem relatively stable. >> > > >> > > Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org> >> >> Patch merged for 3.14, thanks. >> -Daniel > > There's a: > > WARN(!i915_preliminary_hw_support, > "GEN8_CENTROID_PIXEL_OPT_DIS not be needed for > production\n"); > > in gen8_init_clock_gating() that now always triggers. That just needs to die, since prelimary_hw_support != pre-production hw support. This was just lost in the final cleanups when upstreaming stage1 bdw patches. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-01-16 16:23 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-01-14 19:04 [PATCH] drm/i915/bdw: remove preliminary_hw_support flag from BDW Jesse Barnes 2014-01-14 22:05 ` Ben Widawsky 2014-01-14 22:35 ` Daniel Vetter 2014-01-16 16:08 ` Damien Lespiau 2014-01-16 16:23 ` Daniel Vetter
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.