On Wed, Nov 30, 2011 at 11:26 AM, Jean Delvare <jdelvare@xxxxxxx> wrote:
> Use the proper macro to issue the debugging message in
> radeon_atif_call(). Otherwise we spam the log of many systems with a
> message which looks like an error message of unknown origin, and could
> thus confuse the user. Commit dc77de12dde95c8da39e4c417eb70c7d445cf84b
> was a first step in this direction, but was not sufficient IMHO.
>
> Signed-off-by: Jean Delvare <jdelvare@xxxxxxx>
> Cc: David Airlie <airlied@xxxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>
> ---
> Might be considered for stable, this is not a critical bug but it can
> waste time of users and developers alike.
>
> drivers/gpu/drm/radeon/radeon_acpi.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> --- linux-3.2-rc3.orig/drivers/gpu/drm/radeon/radeon_acpi.c 2011-11-29 17:47:02.000000000 +0100
> +++ linux-3.2-rc3/drivers/gpu/drm/radeon/radeon_acpi.c 2011-11-29 18:12:02.000000000 +0100
> @@ -35,7 +35,8 @@ static int radeon_atif_call(acpi_handle
>
> /* Fail only if calling the method fails and ATIF is supported */
> if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) {
> - printk(KERN_DEBUG "failed to evaluate ATIF got %s\n", acpi_format_exception(status));
> + DRM_DEBUG_DRIVER("failed to evaluate ATIF got %s\n",
> + acpi_format_exception(status));
> kfree(buffer.pointer);
> return 1;
> }
>
> --
> Jean Delvare
> Suse L3
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
