diff options
| author | Ajay Singh Parmar <aparmar@codeaurora.org> | 2013-07-23 14:04:11 -0700 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:19:24 -0700 |
| commit | bd1a312bec47bec2c17ead56caf14ef9c07c8d56 (patch) | |
| tree | a17d2e9a42cc7f181a29a348ce93b3ea7dcbd0bf /drivers/video/fbdev | |
| parent | c7c328d2b31b9e4e865e60c8860242158e77008d (diff) | |
mdss: hdmi: Correct CEA format check
This change corrects the CEA format check to include all the supported
resolutions through which the audio should be routed to HDMI TV.
CRs-Fixed: 517950
Change-Id: Icd5c7503b2dc3a184bbf52205b0386eee9daca62
Signed-off-by: Ajay Singh Parmar <aparmar@codeaurora.org>
Diffstat (limited to 'drivers/video/fbdev')
| -rw-r--r-- | drivers/video/fbdev/msm/mdss_hdmi_tx.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/drivers/video/fbdev/msm/mdss_hdmi_tx.c b/drivers/video/fbdev/msm/mdss_hdmi_tx.c index 7b4cf0aa6cbe..5e1ee38b7891 100644 --- a/drivers/video/fbdev/msm/mdss_hdmi_tx.c +++ b/drivers/video/fbdev/msm/mdss_hdmi_tx.c @@ -193,12 +193,18 @@ static const struct hdmi_tx_audio_acr_arry hdmi_tx_audio_acr_lut[] = { {20480, 247500} } }, }; -static bool is_cea_format(int mode) +static bool hdmi_tx_is_cea_format(int mode) { - if ((mode > 0) && (mode < HDMI_EVFRMT_END)) - return true; + bool cea_fmt; + + if ((mode > 0) && (mode <= HDMI_EVFRMT_END)) + cea_fmt = true; else - return false; + cea_fmt = false; + + DEV_DBG("%s: %s\n", __func__, cea_fmt ? "Yes" : "No"); + + return cea_fmt; } const char *hdmi_tx_pm_name(enum hdmi_tx_power_module_type module) @@ -2211,7 +2217,7 @@ static int hdmi_tx_start(struct hdmi_tx_ctrl *hdmi_ctrl) } if (!hdmi_tx_is_dvi_mode(hdmi_ctrl) && - is_cea_format(hdmi_ctrl->video_resolution)) { + hdmi_tx_is_cea_format(hdmi_ctrl->video_resolution)) { rc = hdmi_tx_audio_setup(hdmi_ctrl); if (rc) { DEV_ERR("%s: hdmi_msm_audio_setup failed. rc=%d\n", |