* [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified. @ 2015-01-19 20:13 Rodrigo Vivi 2015-01-20 8:51 ` Jani Nikula 2015-01-22 1:24 ` shuang.he 0 siblings, 2 replies; 4+ messages in thread From: Rodrigo Vivi @ 2015-01-19 20:13 UTC (permalink / raw) To: intel-gfx; +Cc: Rodrigo Vivi We have BDW that PCH might not be propperly identified and we force it to be LPT. However when we correctly identify it is LPT we incorrectly WARN that this isn't a Haswell. For now we have many platforms out there we still need to force LPT so let's keep the force below, but let's remove the WARN when everything is propperly identified. For reference we started receiving logs with this uncessary and incorrect WARN. Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Reference: https://bugs.freedesktop.org/attachment.cgi?id=110972 --- drivers/gpu/drm/i915/i915_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c index 308774f..d4ed3a9 100644 --- a/drivers/gpu/drm/i915/i915_drv.c +++ b/drivers/gpu/drm/i915/i915_drv.c @@ -462,7 +462,7 @@ void intel_detect_pch(struct drm_device *dev) } else if (id == INTEL_PCH_LPT_DEVICE_ID_TYPE) { dev_priv->pch_type = PCH_LPT; DRM_DEBUG_KMS("Found LynxPoint PCH\n"); - WARN_ON(!IS_HASWELL(dev)); + WARN_ON(!IS_HASWELL(dev) || !IS_BROADWELL(dev)); WARN_ON(IS_HSW_ULT(dev)); } else if (IS_BROADWELL(dev)) { dev_priv->pch_type = PCH_LPT; -- 2.1.0 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified. 2015-01-19 20:13 [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified Rodrigo Vivi @ 2015-01-20 8:51 ` Jani Nikula 2015-01-20 18:17 ` Rodrigo Vivi 2015-01-22 1:24 ` shuang.he 1 sibling, 1 reply; 4+ messages in thread From: Jani Nikula @ 2015-01-20 8:51 UTC (permalink / raw) To: intel-gfx; +Cc: Rodrigo Vivi On Mon, 19 Jan 2015, Rodrigo Vivi <rodrigo.vivi@intel.com> wrote: > We have BDW that PCH might not be propperly identified and we force it > to be LPT. However when we correctly identify it is LPT we > incorrectly WARN that this isn't a Haswell. For now we have many > platforms out there we still need to force LPT so let's keep the force > below, but let's remove the WARN when everything is propperly > identified. Before this we haven't tested the BDW + non-LP LPT combination. Maybe we should do some review of any checks against INTEL_PCH_LPT_LP_DEVICE_ID_TYPE in BDW code. Those have always matched before. If the pch ids are getting identified right, shouldn't we then move INTEL_PCH_LPT_LP_DEVICE_ID_TYPE check higher up in the pch identification, and accept BDW for that too, and make IS_BROADWELL(dev) a fallback after both the LPT variants have been checked? Finally the condition below is busted! > For reference we started receiving logs with this uncessary and incorrect WARN. > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> > Reference: https://bugs.freedesktop.org/attachment.cgi?id=110972 > --- > drivers/gpu/drm/i915/i915_drv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 308774f..d4ed3a9 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -462,7 +462,7 @@ void intel_detect_pch(struct drm_device *dev) > } else if (id == INTEL_PCH_LPT_DEVICE_ID_TYPE) { > dev_priv->pch_type = PCH_LPT; > DRM_DEBUG_KMS("Found LynxPoint PCH\n"); > - WARN_ON(!IS_HASWELL(dev)); > + WARN_ON(!IS_HASWELL(dev) || !IS_BROADWELL(dev)); This will warn always... BR, Jani. > WARN_ON(IS_HSW_ULT(dev)); > } else if (IS_BROADWELL(dev)) { > dev_priv->pch_type = PCH_LPT; > -- > 2.1.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified. 2015-01-20 8:51 ` Jani Nikula @ 2015-01-20 18:17 ` Rodrigo Vivi 0 siblings, 0 replies; 4+ messages in thread From: Rodrigo Vivi @ 2015-01-20 18:17 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx, Rodrigo Vivi Thanks Jani. Please ignore this patch... I'm testing the propper clean on BDWs here and submiting another patch, On Tue, Jan 20, 2015 at 12:51 AM, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Mon, 19 Jan 2015, Rodrigo Vivi <rodrigo.vivi@intel.com> wrote: >> We have BDW that PCH might not be propperly identified and we force it >> to be LPT. However when we correctly identify it is LPT we >> incorrectly WARN that this isn't a Haswell. For now we have many >> platforms out there we still need to force LPT so let's keep the force >> below, but let's remove the WARN when everything is propperly >> identified. > > Before this we haven't tested the BDW + non-LP LPT combination. Maybe we > should do some review of any checks against > INTEL_PCH_LPT_LP_DEVICE_ID_TYPE in BDW code. Those have always matched > before. > > If the pch ids are getting identified right, shouldn't we then move > INTEL_PCH_LPT_LP_DEVICE_ID_TYPE check higher up in the pch > identification, and accept BDW for that too, and make IS_BROADWELL(dev) > a fallback after both the LPT variants have been checked? > > Finally the condition below is busted! > >> For reference we started receiving logs with this uncessary and incorrect WARN. >> >> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> >> Reference: https://bugs.freedesktop.org/attachment.cgi?id=110972 >> --- >> drivers/gpu/drm/i915/i915_drv.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c >> index 308774f..d4ed3a9 100644 >> --- a/drivers/gpu/drm/i915/i915_drv.c >> +++ b/drivers/gpu/drm/i915/i915_drv.c >> @@ -462,7 +462,7 @@ void intel_detect_pch(struct drm_device *dev) >> } else if (id == INTEL_PCH_LPT_DEVICE_ID_TYPE) { >> dev_priv->pch_type = PCH_LPT; >> DRM_DEBUG_KMS("Found LynxPoint PCH\n"); >> - WARN_ON(!IS_HASWELL(dev)); >> + WARN_ON(!IS_HASWELL(dev) || !IS_BROADWELL(dev)); > > This will warn always... > > BR, > Jani. > >> WARN_ON(IS_HSW_ULT(dev)); >> } else if (IS_BROADWELL(dev)) { >> dev_priv->pch_type = PCH_LPT; >> -- >> 2.1.0 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx > > -- > Jani Nikula, Intel Open Source Technology Center > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified. 2015-01-19 20:13 [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified Rodrigo Vivi 2015-01-20 8:51 ` Jani Nikula @ 2015-01-22 1:24 ` shuang.he 1 sibling, 0 replies; 4+ messages in thread From: shuang.he @ 2015-01-22 1:24 UTC (permalink / raw) To: shuang.he, ethan.gao, intel-gfx, rodrigo.vivi Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com) Task id: 5607 -------------------------------------Summary------------------------------------- Platform Delta drm-intel-nightly Series Applied PNV 353/353 353/353 ILK 353/353 353/353 SNB 400/422 400/422 IVB 487/487 487/487 BYT 296/296 296/296 HSW +21 487/508 508/508 BDW 401/402 401/402 -------------------------------------Detailed------------------------------------- Platform Test drm-intel-nightly Series Applied HSW igt_kms_cursor_crc_cursor-size-change NSPT(1, M19)TIMEOUT(1, M40)PASS(8, M20M19M40) PASS(1, M40) HSW igt_kms_fence_pin_leak NSPT(1, M19)DMESG_WARN(1, M40)PASS(8, M20M19M40) PASS(1, M40) HSW igt_kms_mmio_vs_cs_flip_setcrtc_vs_cs_flip NSPT(1, M19)TIMEOUT(1, M40)PASS(8, M20M19M40) PASS(1, M40) HSW igt_kms_mmio_vs_cs_flip_setplane_vs_cs_flip NSPT(1, M19)TIMEOUT(1, M40)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_lpsp_non-edp NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_cursor NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_cursor-dpms NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_dpms-mode-unset-non-lpsp NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_dpms-non-lpsp NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_drm-resources-equal NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_fences NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_fences-dpms NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_gem-execbuf NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_gem-mmap-cpu NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_gem-mmap-gtt NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_gem-pread NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_i2c NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_modeset-non-lpsp NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_modeset-non-lpsp-stress-no-wait NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_pci-d3-state NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) HSW igt_pm_rpm_rte NSPT(1, M19)PASS(8, M20M19M40) PASS(1, M40) Note: You need to pay more attention to line start with '*' _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-01-22 1:24 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-19 20:13 [PATCH] drm/i915: Don't WARN on BDW when PCH is propperly identified Rodrigo Vivi 2015-01-20 8:51 ` Jani Nikula 2015-01-20 18:17 ` Rodrigo Vivi 2015-01-22 1:24 ` shuang.he
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox