Skip to content

Commit 9559afc

Browse files
author
skk451
committed
Revert "RDKEMW-8202: Temporarily comment out set/get/reset CMS due to missing GetCMSCaps implementation"
This reverts commit d229924.
1 parent d229924 commit 9559afc

File tree

1 file changed

+1
-24
lines changed

1 file changed

+1
-24
lines changed

AVOutput/AVOutputTV.cpp

Lines changed: 1 addition & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -5020,7 +5020,6 @@ namespace Plugin {
50205020
}
50215021
else
50225022
{
5023-
#ifdef HAS_HAL_GETCMSCAPS
50245023
// Extract color and component from input parameters
50255024
std::string color = parameters.HasLabel("color") ? parameters["color"].String() : "";
50265025
std::string component = parameters.HasLabel("component") ? parameters["component"].String() : "";
@@ -5064,12 +5063,7 @@ namespace Plugin {
50645063
LOGERR("Failed to get CMS param from local storage");
50655064
returnResponse(false);
50665065
}
5067-
#else
5068-
LOGINFO("Exit: GetCMSCaps is not implemented in the HAL.");
5069-
returnResponse(false);
5070-
#endif
50715066
}
5072-
returnResponse(false);
50735067
}
50745068

50755069
uint32_t AVOutputTV::setCMS(const JsonObject& parameters, JsonObject& response)
@@ -5161,7 +5155,6 @@ namespace Plugin {
51615155
}
51625156
else
51635157
{
5164-
#ifdef HAS_HAL_GETCMSCAPS
51655158
bool status = setCMSParam(parameters);
51665159
if (status) {
51675160
LOGINFO("setCMS success");
@@ -5170,12 +5163,7 @@ namespace Plugin {
51705163
LOGERR("setCMS failed");
51715164
returnResponse(false);
51725165
}
5173-
#else
5174-
LOGINFO("Exit: GetCMSCaps is not implemented in the HAL.");
5175-
returnResponse(false);
5176-
#endif
51775166
}
5178-
returnResponse(false);
51795167
}
51805168

51815169
uint32_t AVOutputTV::resetCMS(const JsonObject& parameters, JsonObject& response)
@@ -5285,7 +5273,6 @@ namespace Plugin {
52855273
}
52865274
else
52875275
{
5288-
#ifdef HAS_HAL_GETCMSCAPS
52895276
if (isSetRequiredForParam(parameters, "CMS")) {
52905277
LOGINFO("Proceed with SetCMSState \n");
52915278
tvError_t ret = SetCMSState(false);
@@ -5301,18 +5288,13 @@ namespace Plugin {
53015288
returnResponse(false);
53025289
}
53035290
returnResponse(true);
5304-
#else
5305-
LOGINFO("Exit: GetCMSCaps is not implemented in the HAL.");
5306-
returnResponse(false);
5307-
#endif
53085291
}
5309-
returnResponse(false);
53105292
}
53115293

53125294
uint32_t AVOutputTV::getCMSCapsV2(const JsonObject& parameters, JsonObject& response)
53135295
{
53145296
LOGINFO("Entry: getCMSCapsV2");
5315-
#ifdef HAS_HAL_GETCMSCAPS
5297+
53165298
int max_hue = 0, max_saturation = 0, max_luma = 0;
53175299
tvDataComponentColor_t* colorArray = nullptr;
53185300
tvComponentType_t* componentArray = nullptr;
@@ -5360,11 +5342,6 @@ namespace Plugin {
53605342

53615343
LOGINFO("Exit: getCMSCapsV2");
53625344
returnResponse(true);
5363-
#else
5364-
response["platformSupport"] = false;
5365-
LOGINFO("Exit: GetCMSCaps is not implemented in the HAL.");
5366-
returnResponse(false);
5367-
#endif
53685345
}
53695346

53705347
uint32_t AVOutputTV::getCMSCaps(const JsonObject& parameters, JsonObject& response)

0 commit comments

Comments
 (0)