diff options
author | Ole Reinhardt <ole.reinhardt@kernelconcepts.de> | 2005-12-13 20:03:38 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-12-14 00:18:16 -0500 |
commit | fb79ffa4ddbe3f6f30fdb8429b5bad84d25ae6ef (patch) | |
tree | 2118328f37af59e65e024e34d09d9c902aea47bd /drivers | |
parent | 118c71bcacce82a4317c9bd99c6a15af14020aee (diff) |
[PATCH] fbdev: make pxafb more robust to errors with CONFIG_FB_PXA_PARAMETERS
pxafb.c runs into an oops if CONFIG_FB_PXA_PARAMETERS is enabled and no
parameters are set in command line. The following patch avoids this
problem.
Signed-off-by: Nicolas Pitre <nico@cam.org>
Cc: "Antonino A. Daplas" <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/video/pxafb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/video/pxafb.c b/drivers/video/pxafb.c index 7b4cd250bec8..9fc10b9e6f57 100644 --- a/drivers/video/pxafb.c +++ b/drivers/video/pxafb.c | |||
@@ -1396,7 +1396,8 @@ static struct platform_driver pxafb_driver = { | |||
1396 | int __devinit pxafb_setup(char *options) | 1396 | int __devinit pxafb_setup(char *options) |
1397 | { | 1397 | { |
1398 | # ifdef CONFIG_FB_PXA_PARAMETERS | 1398 | # ifdef CONFIG_FB_PXA_PARAMETERS |
1399 | strlcpy(g_options, options, sizeof(g_options)); | 1399 | if (options) |
1400 | strlcpy(g_options, options, sizeof(g_options)); | ||
1400 | # endif | 1401 | # endif |
1401 | return 0; | 1402 | return 0; |
1402 | } | 1403 | } |