summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLucas Stach <dev@lynxeye.de>2014-04-21 22:15:25 +0200
committerSascha Hauer <s.hauer@pengutronix.de>2014-04-23 09:05:52 +0200
commit3e74fd9d059d96ec0d4cce0e8e7dc0dd2dd3d148 (patch)
tree13e797d71131e661eb03ead15d1302d9231e5676
parent4d5ad7c9c64dd2e96e18cdf6ce5334f8f47de59c (diff)
downloadbarebox-3e74fd9d059d96ec0d4cce0e8e7dc0dd2dd3d148.tar.gz
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.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/video/of_display_timing.c b/drivers/video/of_display_timing.c
index 5dfd5b3..eb29ec6 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) {