diff options
author | Lucas Stach <dev@lynxeye.de> | 2014-04-21 22:15:25 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2014-04-23 09:05:52 +0200 |
commit | 3e74fd9d059d96ec0d4cce0e8e7dc0dd2dd3d148 (patch) | |
tree | 13e797d71131e661eb03ead15d1302d9231e5676 | |
parent | 4d5ad7c9c64dd2e96e18cdf6ce5334f8f47de59c (diff) | |
download | barebox-3e74fd9d059d96ec0d4cce0e8e7dc0dd2dd3d148.tar.gz barebox-3e74fd9d059d96ec0d4cce0e8e7dc0dd2dd3d148.tar.xz |
video: displaytimings: remove two broken error messages
The error messages would dereference the just checked NULL
ptr. As those messages don't add much value without further
info just remove them.
Signed-off-by: Lucas Stach <dev@lynxeye.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
-rw-r--r-- | drivers/video/of_display_timing.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/video/of_display_timing.c b/drivers/video/of_display_timing.c index 5dfd5b3f1c..eb29ec699e 100644 --- a/drivers/video/of_display_timing.c +++ b/drivers/video/of_display_timing.c @@ -109,10 +109,8 @@ int of_get_display_timing(struct device_node *np, const char *name, { struct device_node *timing_np; - if (!np) { - pr_err("%s: no devicenode given\n", np->full_name); + if (!np) return -EINVAL; - } timing_np = of_get_child_by_name(np, name); if (!timing_np) { @@ -136,10 +134,8 @@ struct display_timings *of_get_display_timings(struct device_node *np) struct device_node *native_mode; struct display_timings *disp; - if (!np) { - pr_err("%s: no device node given\n", np->full_name); + if (!np) return NULL; - } timings_np = of_get_child_by_name(np, "display-timings"); if (!timings_np) { |