kernel/linux
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ synced 2026-10-18 19:12:56 +00:00
Code Issues Releases Packages Activity
linux/drivers/media/firewire
History
Nathan Chancellor e6256cb198 media: firewire: Fix app_info parameter type in avc_ca{,_app}_info 
[ Upstream commit b2e9a4eda1 ]

Clang warns:

drivers/media/firewire/firedtv-avc.c:999:45: warning: implicit
conversion from 'int' to 'char' changes value from 159 to -97
[-Wconstant-conversion]
        app_info[0] = (EN50221_TAG_APP_INFO >> 16) & 0xff;
                    ~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
drivers/media/firewire/firedtv-avc.c:1000:45: warning: implicit
conversion from 'int' to 'char' changes value from 128 to -128
[-Wconstant-conversion]
        app_info[1] = (EN50221_TAG_APP_INFO >>  8) & 0xff;
                    ~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
drivers/media/firewire/firedtv-avc.c:1040:44: warning: implicit
conversion from 'int' to 'char' changes value from 159 to -97
[-Wconstant-conversion]
        app_info[0] = (EN50221_TAG_CA_INFO >> 16) & 0xff;
                    ~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
drivers/media/firewire/firedtv-avc.c:1041:44: warning: implicit
conversion from 'int' to 'char' changes value from 128 to -128
[-Wconstant-conversion]
        app_info[1] = (EN50221_TAG_CA_INFO >>  8) & 0xff;
                    ~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
4 warnings generated.

Change app_info's type to unsigned char to match the type of the
member msg in struct ca_msg, which is the only thing passed into the
app_info parameter in this function.

Link: https://github.com/ClangBuiltLinux/linux/issues/105

Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
History 2019-01-26 09:20:44 +01:00
..
firedtv-avc.c media: firewire: Fix app_info parameter type in avc_ca{,_app}_info 2019-01-26 09:20:44 +01:00
firedtv-ci.c vfs: do bulk POLL* -> EPOLL* replacement 2018-02-11 14:34:03 -08:00
firedtv-dvb.c media: move dvb kAPI headers to include/media 2017-12-28 13:16:01 -05:00
firedtv-fe.c media: replace strcpy() by strscpy() 2018-09-11 13:32:17 -04:00
firedtv-fw.c media: move dvb kAPI headers to include/media 2017-12-28 13:16:01 -05:00
firedtv-rc.c [media] firewire: use dev_dbg() instead of printk() 2016-10-21 10:05:34 -02:00
firedtv.h media: firewire: Fix app_info parameter type in avc_ca{,_app}_info 2019-01-26 09:20:44 +01:00
Kconfig [media] firewire: move it one level up 2012-08-13 23:20:36 -03:00
Makefile media: move dvb kAPI headers to include/media 2017-12-28 13:16:01 -05:00