[Intel-gfx] [PATCH] drm/i915: make prelim hw msg in log a bit more harsh

[Jani Nikula]

On Fri, 18 Sep 2015, Rob Clark <robdclark at gmail.com> wrote:
> Apparently some people see this message and try to turn on
> preliminary_hw_support.  And then for some reason are surprised
> when it doesn't work.  So let's set expectations a bit lower.

I think the message goes wrong already at the beginning, "hardware
requires preliminary hardware support". The hardware really requires
full support, but preliminary is all we've got so far, and that's been
disabled by default...

How about something along the lines of,

"""
This kernel version does not properly support the hardware.
Preliminary support disabled by default.
See CONFIG_DRM_I915_PRELIMINARY_HW_SUPPORT or i915.preliminary_hw_support.
"""

BR,
Jani.



>
> Signed-off-by: Rob Clark <robdclark at gmail.com>
> ---
>  drivers/gpu/drm/i915/i915_drv.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index 8edcec8..5e87fe1 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -926,7 +926,8 @@ static int i915_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  
>  	if (IS_PRELIMINARY_HW(intel_info) && !i915.preliminary_hw_support) {
>  		DRM_INFO("This hardware requires preliminary hardware support.\n"
> -			 "See CONFIG_DRM_I915_PRELIMINARY_HW_SUPPORT, and/or modparam preliminary_hw_support\n");
> +			 "See CONFIG_DRM_I915_PRELIMINARY_HW_SUPPORT, and/or modparam preliminary_hw_support.\n"
> +			 "Note that preliminary hw support is unsupported, use at your own risk!\n");
>  		return -ENODEV;
>  	}
>  
> -- 
> 2.4.3
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Jani Nikula, Intel Open Source Technology Center