Re: [alsa-cvslog] CVS: alsa-kernel/usb usbaudio.c,1.172,1.173

[Lee Revell <rlrevell@joe-job.com>]

On Wed, 2006-03-08 at 23:58 -0800, Clemens Ladisch wrote:
> +static const char *usb_error_string(int err)
> +{
> +	switch (err) {
> +	case -ENODEV:
> +		return "no device";
> +	case -ENOENT:
> +		return "endpoint not enabled";
> +	case -EPIPE:
> +		return "endpoint stalled";
> +	case -ENOSPC:
> +		return "not enough bandwidth";
> +	case -ESHUTDOWN:
> +		return "device disabled";
> +	case -EHOSTUNREACH:
> +		return "device suspended";
> +	case -EINVAL:
> +	case -EAGAIN:
> +	case -EFBIG:
> +	case -EMSGSIZE:
> +		return "internal error";
> +	default:
> +		return "unknown error";
> +	}
> +}

Shouldn't a generic facility be created for this?  After all these are
standard error codes and it seem like other parts of the kernel might
want to do user friendly error reporting.

Lee

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/