On Fri, 31 Jan 2014, Ville Syrjälä <[email protected]> wrote: > On Fri, Jan 31, 2014 at 01:41:08PM +0200, Jani Nikula wrote: >> The WARN is a bit too verbose, make it a DRM_INFO that gets printed >> once. >> >> While at it, fix the typo in DSLP. >> >> Suggested-by: Chris Wilson <[email protected]> >> Signed-off-by: Jani Nikula <[email protected]> >> --- >> drivers/gpu/drm/i915/intel_opregion.c | 6 +++++- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_opregion.c >> b/drivers/gpu/drm/i915/intel_opregion.c >> index 46d288055345..0fae017c4c7b 100644 >> --- a/drivers/gpu/drm/i915/intel_opregion.c >> +++ b/drivers/gpu/drm/i915/intel_opregion.c >> @@ -263,7 +263,11 @@ static int swsci(struct drm_device *dev, u32 function, >> u32 parm, u32 *parm_out) >> dslp = 50; >> } else if (dslp > 1500) { >> /* Hey bios, trust must be earned. */ >> - WARN_ONCE(1, "excessive driver sleep timeout (DSPL) %u\n", >> dslp); >> + static bool warned; >> + if (!warned) { >> + DRM_INFO("excessive driver sleep timeout (DSLP) %u\n", >> dslp); >> + warned = true; >> + } > > This is the second "print once" thing I've seen in a few days. Seems > like we should add DRM_INFO_ONCE() & co. to hide the ugliness a bit.
I'm adding DRM_INFO_ONCE() in the next version, but if you're referring to commit 53a4c6b26ddef1f2969f8bc17178bcda4782d18d Author: Chris Wilson <[email protected]> Date: Thu Jan 30 14:38:15 2014 +0000 drm/i915: Only print information for filing bug reports once it's probably better to keep that as it is. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
