diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-10-27 12:54:09 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-10-27 16:10:07 -0400 |
commit | 230dc94a2a6e5786b8529e3218979719758f1fa0 (patch) | |
tree | c7ecc101b417489bce65a338830edf315088eb9a /drivers/media | |
parent | 8e1803f4cd4bab0a66a24945a00e2c3a89457a32 (diff) |
[media] nuvoton-cir: carrier detect support is broken - remove it
The carrier detect return value is never used on nuvoton driver:
drivers/media/rc/nuvoton-cir.c: In function 'nvt_process_rx_ir_data':
drivers/media/rc/nuvoton-cir.c:623:6: warning: variable 'carrier' set but not used [-Wunused-but-set-variable]
Also, this would be called only if a boolean variable is enabled,
but there's no condition that enables it inside the driver. So,
comment the carrier detection code, as it might be useful later,
and remove the unused glue code.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/rc/nuvoton-cir.c | 7 | ||||
-rw-r--r-- | drivers/media/rc/nuvoton-cir.h | 1 |
2 files changed, 2 insertions, 6 deletions
diff --git a/drivers/media/rc/nuvoton-cir.c b/drivers/media/rc/nuvoton-cir.c index 44ba834c363d..e4ea89a11eed 100644 --- a/drivers/media/rc/nuvoton-cir.c +++ b/drivers/media/rc/nuvoton-cir.c | |||
@@ -472,6 +472,7 @@ static void nvt_enable_wake(struct nvt_dev *nvt) | |||
472 | nvt_cir_wake_reg_write(nvt, 0, CIR_WAKE_IREN); | 472 | nvt_cir_wake_reg_write(nvt, 0, CIR_WAKE_IREN); |
473 | } | 473 | } |
474 | 474 | ||
475 | #if 0 /* Currently unused */ | ||
475 | /* rx carrier detect only works in learning mode, must be called w/nvt_lock */ | 476 | /* rx carrier detect only works in learning mode, must be called w/nvt_lock */ |
476 | static u32 nvt_rx_carrier_detect(struct nvt_dev *nvt) | 477 | static u32 nvt_rx_carrier_detect(struct nvt_dev *nvt) |
477 | { | 478 | { |
@@ -504,7 +505,7 @@ static u32 nvt_rx_carrier_detect(struct nvt_dev *nvt) | |||
504 | 505 | ||
505 | return carrier; | 506 | return carrier; |
506 | } | 507 | } |
507 | 508 | #endif | |
508 | /* | 509 | /* |
509 | * set carrier frequency | 510 | * set carrier frequency |
510 | * | 511 | * |
@@ -620,7 +621,6 @@ static void nvt_dump_rx_buf(struct nvt_dev *nvt) | |||
620 | static void nvt_process_rx_ir_data(struct nvt_dev *nvt) | 621 | static void nvt_process_rx_ir_data(struct nvt_dev *nvt) |
621 | { | 622 | { |
622 | DEFINE_IR_RAW_EVENT(rawir); | 623 | DEFINE_IR_RAW_EVENT(rawir); |
623 | u32 carrier; | ||
624 | u8 sample; | 624 | u8 sample; |
625 | int i; | 625 | int i; |
626 | 626 | ||
@@ -629,9 +629,6 @@ static void nvt_process_rx_ir_data(struct nvt_dev *nvt) | |||
629 | if (debug) | 629 | if (debug) |
630 | nvt_dump_rx_buf(nvt); | 630 | nvt_dump_rx_buf(nvt); |
631 | 631 | ||
632 | if (nvt->carrier_detect_enabled) | ||
633 | carrier = nvt_rx_carrier_detect(nvt); | ||
634 | |||
635 | nvt_dbg_verbose("Processing buffer of len %d", nvt->pkts); | 632 | nvt_dbg_verbose("Processing buffer of len %d", nvt->pkts); |
636 | 633 | ||
637 | init_ir_raw_event(&rawir); | 634 | init_ir_raw_event(&rawir); |
diff --git a/drivers/media/rc/nuvoton-cir.h b/drivers/media/rc/nuvoton-cir.h index 0d5e0872a2ea..7c3674ff5ea2 100644 --- a/drivers/media/rc/nuvoton-cir.h +++ b/drivers/media/rc/nuvoton-cir.h | |||
@@ -103,7 +103,6 @@ struct nvt_dev { | |||
103 | 103 | ||
104 | /* rx settings */ | 104 | /* rx settings */ |
105 | bool learning_enabled; | 105 | bool learning_enabled; |
106 | bool carrier_detect_enabled; | ||
107 | 106 | ||
108 | /* track cir wake state */ | 107 | /* track cir wake state */ |
109 | u8 wake_state; | 108 | u8 wake_state; |