diff options
author | Ben Dooks <ben-fbdev@fluff.org> | 2008-05-23 16:04:56 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-05-24 12:56:12 -0400 |
commit | d585dfe840c93ea800afc124333b6ac04722d359 (patch) | |
tree | 9e3824920e47db054f13dea3a13a029d4be926e5 /drivers/video/s3c2410fb.c | |
parent | 673b4600e3b3cc6689025e6a6fc6909b6e53dd5e (diff) |
S3C2410: add error print if we cannot add attribute
Fix the following warning by checking the result of device_create_file and
printing an error but not removing the device (loss of debug registers is
not fatal).
drivers/video/s3c2410fb.c:905: warning: ignoring return value of 'device_create_file', declared with attribute warn_unused_result
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Cc: Arnaud Patard <arnaud.patard@rtp-net.org>
Acked-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video/s3c2410fb.c')
-rw-r--r-- | drivers/video/s3c2410fb.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/video/s3c2410fb.c b/drivers/video/s3c2410fb.c index 2219ae56a0e6..5fea847acd1c 100644 --- a/drivers/video/s3c2410fb.c +++ b/drivers/video/s3c2410fb.c | |||
@@ -972,7 +972,10 @@ static int __init s3c24xxfb_probe(struct platform_device *pdev, | |||
972 | } | 972 | } |
973 | 973 | ||
974 | /* create device files */ | 974 | /* create device files */ |
975 | device_create_file(&pdev->dev, &dev_attr_debug); | 975 | ret = device_create_file(&pdev->dev, &dev_attr_debug); |
976 | if (ret) { | ||
977 | printk(KERN_ERR "failed to add debug attribute\n"); | ||
978 | } | ||
976 | 979 | ||
977 | printk(KERN_INFO "fb%d: %s frame buffer device\n", | 980 | printk(KERN_INFO "fb%d: %s frame buffer device\n", |
978 | fbinfo->node, fbinfo->fix.id); | 981 | fbinfo->node, fbinfo->fix.id); |