Commit 3d351531 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

media: rc-main: clean-up two warnings

While correct, the code is too complex for smatch to undersdand
that protocol will always be initialized:

	drivers/media/rc/rc-main.c:1531 store_wakeup_protocols() error: uninitialized symbol 'protocol'.
	drivers/media/rc/rc-main.c:1541 store_wakeup_protocols() error: uninitialized symbol 'protocol'.

So, change it a little bit in order to avoid such warning.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Acked-by: default avatarSean Young <sean@mess.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent f4930887
...@@ -1502,7 +1502,7 @@ static ssize_t store_wakeup_protocols(struct device *device, ...@@ -1502,7 +1502,7 @@ static ssize_t store_wakeup_protocols(struct device *device,
const char *buf, size_t len) const char *buf, size_t len)
{ {
struct rc_dev *dev = to_rc_dev(device); struct rc_dev *dev = to_rc_dev(device);
enum rc_proto protocol; enum rc_proto protocol = RC_PROTO_UNKNOWN;
ssize_t rc; ssize_t rc;
u64 allowed; u64 allowed;
int i; int i;
...@@ -1511,9 +1511,7 @@ static ssize_t store_wakeup_protocols(struct device *device, ...@@ -1511,9 +1511,7 @@ static ssize_t store_wakeup_protocols(struct device *device,
allowed = dev->allowed_wakeup_protocols; allowed = dev->allowed_wakeup_protocols;
if (sysfs_streq(buf, "none")) { if (!sysfs_streq(buf, "none")) {
protocol = RC_PROTO_UNKNOWN;
} else {
for (i = 0; i < ARRAY_SIZE(protocols); i++) { for (i = 0; i < ARRAY_SIZE(protocols); i++) {
if ((allowed & (1ULL << i)) && if ((allowed & (1ULL << i)) &&
sysfs_streq(buf, protocols[i].name)) { sysfs_streq(buf, protocols[i].name)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment