Re: [PATCH] libata: add human-readable error value decoding

[Mark Lord]

Tejun Heo wrote:
> Robert Hancock wrote:
> > This adds human-readable decoding of the ATA status and error registers
> > (similar
> > to what drivers/ide does) as well as the SATA Serror register to libata
...
> I'm not really convinced whether this is necessary.  The human readable
> form is also a bit cryptic and can get quite long.  So, mild NACK from me.

Same here, but I would like to see it in there under a CONFIG_DEBUG_LIBATA
kernel build option or something.  Kind of like the "FANCY_STATUS_DUMPS" flag
that drivers/ide used to have for this kind of stuff.

Cheers

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