From d13232eecd83055c535ce278e23d4d01684fb7d5 Mon Sep 17 00:00:00 2001 From: hgfell683 <107510770+hgfell683@users.noreply.github.com> Date: Tue, 21 Apr 2026 11:29:25 -0400 Subject: [PATCH 1/3] RDKEMW-17349 : setColor failure --- ds/frontPanelIndicator.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/ds/frontPanelIndicator.cpp b/ds/frontPanelIndicator.cpp index 68bae9a9..fe40620c 100644 --- a/ds/frontPanelIndicator.cpp +++ b/ds/frontPanelIndicator.cpp @@ -476,10 +476,12 @@ uint32_t FrontPanelIndicator::getColor() */ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool toPersist) { + INT_INFO("[%s:%d] Color id = %d, color name = %s", __FUNCTION__, __LINE__, color.getId(), color.getName().c_str()); bool IsPersist = toPersist; if(_colorMode == 0 || _colorMode == 1) { + INT_INFO("[%s:%d] Color mode is %d, API not supported", __FUNCTION__, __LINE__, _colorMode); throw UnsupportedOperationException("This API not supported for the color mode"); } else if(_colorMode == 2) @@ -488,6 +490,7 @@ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool const List supportedColors = FrontPanelConfig::getInstance().getColors(); for (uint j = 0; j < supportedColors.size(); j++) { + INT_INFO("[%s:%d] Supported color id = %d, color id = %d", __FUNCTION__, __LINE__, supportedColors.at(j).getId(), color.getId()); if( supportedColors.at(j).getId() == color.getId()) { isValidColor = true; @@ -496,6 +499,7 @@ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool } if(!isValidColor) { + INT_INFO("[%s:%d] Invalid color id = %d", __FUNCTION__, __LINE__, color.getId()); throw IllegalArgumentException("Invalid color object"); } } From c17544ee8850e9230ff3671ac248d174e3ba3855 Mon Sep 17 00:00:00 2001 From: nanimatta <84963246+nanimatta@users.noreply.github.com> Date: Wed, 22 Apr 2026 16:51:08 -0400 Subject: [PATCH 2/3] Update frontPanelIndicator.cpp --- ds/frontPanelIndicator.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/ds/frontPanelIndicator.cpp b/ds/frontPanelIndicator.cpp index fe40620c..061cc95d 100644 --- a/ds/frontPanelIndicator.cpp +++ b/ds/frontPanelIndicator.cpp @@ -529,6 +529,7 @@ void FrontPanelIndicator::setColor(uint32_t color,bool toPersist) bool isValidColor = false; if(_colorMode == 0 || _colorMode == 2) { + INT_INFO("[%s:%d] setLED: This API not supported for the color mode", __FUNCTION__, __LINE__); throw UnsupportedOperationException("This API not supported for the color mode"); } if (dsERR_NONE != dsSetFPDColor((dsFPDIndicator_t)_id, (dsFPDColor_t) color,IsPersist) ) From fd8f5ce8ca0407bd77fce71718b6d4424145c7f7 Mon Sep 17 00:00:00 2001 From: nanimatta <84963246+nanimatta@users.noreply.github.com> Date: Wed, 22 Apr 2026 16:53:56 -0400 Subject: [PATCH 3/3] Update frontPanelIndicator.cpp --- ds/frontPanelIndicator.cpp | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/ds/frontPanelIndicator.cpp b/ds/frontPanelIndicator.cpp index 061cc95d..fc2a6a35 100644 --- a/ds/frontPanelIndicator.cpp +++ b/ds/frontPanelIndicator.cpp @@ -476,12 +476,12 @@ uint32_t FrontPanelIndicator::getColor() */ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool toPersist) { - INT_INFO("[%s:%d] Color id = %d, color name = %s", __FUNCTION__, __LINE__, color.getId(), color.getName().c_str()); + INT_INFO("[%s:%d] 17349:Color id = %d, color name = %s", __FUNCTION__, __LINE__, color.getId(), color.getName().c_str()); bool IsPersist = toPersist; if(_colorMode == 0 || _colorMode == 1) { - INT_INFO("[%s:%d] Color mode is %d, API not supported", __FUNCTION__, __LINE__, _colorMode); + INT_INFO("[%s:%d] 17349:Color mode is %d, API not supported", __FUNCTION__, __LINE__, _colorMode); throw UnsupportedOperationException("This API not supported for the color mode"); } else if(_colorMode == 2) @@ -490,7 +490,7 @@ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool const List supportedColors = FrontPanelConfig::getInstance().getColors(); for (uint j = 0; j < supportedColors.size(); j++) { - INT_INFO("[%s:%d] Supported color id = %d, color id = %d", __FUNCTION__, __LINE__, supportedColors.at(j).getId(), color.getId()); + INT_INFO("[%s:%d] 17349:Supported color id = %d, color id = %d", __FUNCTION__, __LINE__, supportedColors.at(j).getId(), color.getId()); if( supportedColors.at(j).getId() == color.getId()) { isValidColor = true; @@ -499,7 +499,7 @@ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool } if(!isValidColor) { - INT_INFO("[%s:%d] Invalid color id = %d", __FUNCTION__, __LINE__, color.getId()); + INT_INFO("[%s:%d] 17349:Invalid color id = %d", __FUNCTION__, __LINE__, color.getId()); throw IllegalArgumentException("Invalid color object"); } } @@ -507,6 +507,7 @@ void FrontPanelIndicator::setColor(const FrontPanelIndicator::Color & color,bool if (dsERR_NONE != dsSetFPDColor((dsFPDIndicator_t)_id, (dsFPDColor_t)colorValue,IsPersist) ) { + INT_INFO("[%s:%d] 17349: dsSetFPDColor failed--", __FUNCTION__, __LINE__); throw IllegalArgumentException("dsSetFPDColor failed"); } _color_rgb32 = colorValue; @@ -529,11 +530,12 @@ void FrontPanelIndicator::setColor(uint32_t color,bool toPersist) bool isValidColor = false; if(_colorMode == 0 || _colorMode == 2) { - INT_INFO("[%s:%d] setLED: This API not supported for the color mode", __FUNCTION__, __LINE__); + INT_INFO("[%s:%d] 17349: This API not supported for the color mode", __FUNCTION__, __LINE__); throw UnsupportedOperationException("This API not supported for the color mode"); } if (dsERR_NONE != dsSetFPDColor((dsFPDIndicator_t)_id, (dsFPDColor_t) color,IsPersist) ) { + INT_INFO("[%s:%d] 17349: dsSetFPDColor failed ", __FUNCTION__, __LINE__); throw IllegalArgumentException(); } _color_rgb32 = (dsFPDColor_t) color;