* [PATCH] drm: Fix EDID dump format
@ 2011-06-27 13:32 Jean Delvare
2011-06-27 18:04 ` Tormod Volden
0 siblings, 1 reply; 2+ messages in thread
From: Jean Delvare @ 2011-06-27 13:32 UTC (permalink / raw)
To: dri-devel
* print_hex_dump_bytes() already includes a log level, so we shouldn't
add one. That log level is KERN_DEBUG so use the same for our header
for consistency.
* print_hex_dump_bytes() properly puts a newline at the end of the
last line, so there is no reason why we would add another one.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: David Airlie <airlied@linux.ie>
---
drivers/gpu/drm/drm_edid.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
--- linux-3.0-rc4.orig/drivers/gpu/drm/drm_edid.c 2011-06-22 16:55:11.000000000 +0200
+++ linux-3.0-rc4/drivers/gpu/drm/drm_edid.c 2011-06-27 15:11:53.000000000 +0200
@@ -184,9 +184,9 @@ drm_edid_block_valid(u8 *raw_edid)
bad:
if (raw_edid) {
- printk(KERN_ERR "Raw EDID:\n");
- print_hex_dump_bytes(KERN_ERR, DUMP_PREFIX_NONE, raw_edid, EDID_LENGTH);
- printk(KERN_ERR "\n");
+ printk(KERN_DEBUG "Raw EDID:\n");
+ print_hex_dump_bytes("", DUMP_PREFIX_OFFSET,
+ raw_edid, EDID_LENGTH);
}
return 0;
}
--
Jean Delvare
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] drm: Fix EDID dump format
2011-06-27 13:32 [PATCH] drm: Fix EDID dump format Jean Delvare
@ 2011-06-27 18:04 ` Tormod Volden
0 siblings, 0 replies; 2+ messages in thread
From: Tormod Volden @ 2011-06-27 18:04 UTC (permalink / raw)
To: Jean Delvare; +Cc: dri-devel
On Mon, Jun 27, 2011 at 3:32 PM, Jean Delvare wrote:
> * print_hex_dump_bytes() already includes a log level, so we shouldn't
> add one. That log level is KERN_DEBUG so use the same for our header
> for consistency.
> * print_hex_dump_bytes() properly puts a newline at the end of the
> last line, so there is no reason why we would add another one.
>
> Signed-off-by: Jean Delvare <khali@linux-fr.org>
> Cc: David Airlie <airlied@linux.ie>
Please also consider the alternative patch I posted here a week ago:
http://lists.freedesktop.org/archives/dri-devel/2011-June/012264.html
It preserves KERN_ERR, however it does not remove the extra newline.
Cheers,
Tormod
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-27 18:04 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-27 13:32 [PATCH] drm: Fix EDID dump format Jean Delvare
2011-06-27 18:04 ` Tormod Volden
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.