diff mbox

[v2,3/5] mxl5005: Fix warning caused by new entries in an enum

Message ID 1304882240-23044-4-git-send-email-steve@stevekerrison.com (mailing list archive)
State Rejected, archived
Headers

Commit Message

Steve Kerrison May 8, 2011, 7:17 p.m. UTC 
  Additional bandwidth modes have been added in frontend.h
mxl5005s.c had no default case so the compiler was warning about
a non-exhausive switch statement.

Signed-off-by: Steve Kerrison <steve@stevekerrison.com>
---
 drivers/media/common/tuners/mxl5005s.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
diff mbox

Patch

diff --git a/drivers/media/common/tuners/mxl5005s.c b/drivers/media/common/tuners/mxl5005s.c
index 0d6e094..d80e6f3 100644
--- a/drivers/media/common/tuners/mxl5005s.c
+++ b/drivers/media/common/tuners/mxl5005s.c
@@ -4020,6 +4020,10 @@  static int mxl5005s_set_params(struct dvb_frontend *fe,
 			case BANDWIDTH_7_MHZ:
 				req_bw  = MXL5005S_BANDWIDTH_7MHZ;
 				break;
+			default:
+				dprintk(1,"%s: Unsupported bandwidth mode %u, reverting to default\n",
+					__func__,params->u.ofdm.bandwidth);
+				/* Fall back to auto */
 			case BANDWIDTH_AUTO:
 			case BANDWIDTH_8_MHZ:
 				req_bw  = MXL5005S_BANDWIDTH_8MHZ;