From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Fri, 14 Oct 2016 10:40:12 +0200 Move the setting for the local variables "mask", "match" and "rc6_csl" behind the source code for a condition check by this function at the beginning. Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> --- drivers/media/rc/winbond-cir.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-cir.c index fd997f0..9d05e17 100644 --- a/drivers/media/rc/winbond-cir.c +++ b/drivers/media/rc/winbond-cir.c @@ -702,17 +702,17 @@ wbcir_shutdown(struct pnp_dev *device) bool do_wake = true; u8 match[11]; u8 mask[11]; - u8 rc6_csl = 0; + u8 rc6_csl; int i; - memset(match, 0, sizeof(match)); - memset(mask, 0, sizeof(mask)); - if (wake_sc == INVALID_SCANCODE || !device_may_wakeup(dev)) { do_wake = false; goto finish; } + rc6_csl = 0; + memset(match, 0, sizeof(match)); + memset(mask, 0, sizeof(mask)); switch (protocol) { case IR_PROTOCOL_RC5: if (wake_sc > 0xFFF) { -- 2.10.1 -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html