diff options
| author | Hans Verkuil <hverkuil@xs4all.nl> | 2015-06-05 19:09:31 +0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-06-06 04:22:05 +0300 |
| commit | c70316f2a193fcd62232cddc1b2d44997ca2c930 (patch) | |
| tree | ebe90ed1a3247fa9ea022f4c11da342b41e07517 /drivers/media/usb | |
| parent | 64d5702229d86deacf42a43bc893a981f72d4908 (diff) | |
| download | linux-c70316f2a193fcd62232cddc1b2d44997ca2c930.tar.xz | |
[media] vivid: move PRINTSTR to separate functions
Commit 84cb7be43cec12868e94163c99fdc34c0297c3b8 broke vivid-tpg
(uninitialized variable p).
This patch takes a different approach: four different functions are
created, one for each PRINTSTR version.
In order to avoid the 'the frame size of 1308 bytes is larger than 1024
bytes' warning I had to mark those functions with 'noinline'. For
whatever reason gcc seems to inline this aggressively and it is doing
weird things with the stack.
I tried to read the assembly code, but I couldn't see what exactly it
was doing on the stack.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/usb')
0 files changed, 0 insertions, 0 deletions