| commit | c98300a0e8cf160aaea60bc05d2cd156a7666173 | |
| author | Mauro Carvalho Chehab <m.chehab@samsung.com> | |
| Sat, 2 Nov 2013 11:13:02 +0000 (2 08:13 -0300) | ||
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | |
| Fri, 8 Nov 2013 11:45:46 +0000 (8 09:45 -0200) | ||
| tree | 2a1bae8d40adc7cb472361e0a50ab48a188a1dd7 | treesnapshot (tar.gz zip) |
| parent | 7760e148350bf6df95662bc0db3734e9d991cb03 | commitdiff |
[media] mxl111sf: Don't use dynamic static allocation
Dynamic static allocation is evil, as Kernel stack is too low, and
compilation complains about it on some archs:
drivers/media/usb/dvb-usb-v2/mxl111sf.c:74:1: warning: 'mxl111sf_ctrl_msg' uses dynamic stack allocation [enabled by default]
Instead, let's enforce a limit for the buffer to be the max size of
a control URB payload data (64 bytes).
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Reviewed-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Dynamic static allocation is evil, as Kernel stack is too low, and
compilation complains about it on some archs:
drivers/media/usb/dvb-usb-v2/mxl111sf.c:74:1: warning: 'mxl111sf_ctrl_msg' uses dynamic stack allocation [enabled by default]
Instead, let's enforce a limit for the buffer to be the max size of
a control URB payload data (64 bytes).
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Reviewed-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
| drivers/media/usb/dvb-usb-v2/mxl111sf.c | diffblobblamehistory |