RDKEMW-18673 - edidIGNORE Fallback

[hgfell683]

No description provided.

[Copilot]

Pull request overview

Updates the video port RPC service to make dsGetIgnoreEDIDStatus more resilient by applying a fallback behavior when the underlying HAL call fails.

Changes:

• Capture dsGetIgnoreEDIDStatus return code and detect HAL failures.
• On HAL error, log the failure and force ignoreEDID = true as a fallback.

Comments suppressed due to low confidence (1)

rpc/srv/dsVideoPort.c:2336

• param->ignoreEDID is not explicitly initialized in this handler (despite the “Default status is false” comment). If this IARM call is invoked with a non-zeroed dsEdidIgnoreParam_t, the function can return a stale/undefined ignoreEDID value when func is NULL or before the HAL writes it. Consider setting param->ignoreEDID = false before attempting the HAL call (and/or in an else branch when func is NULL) so the output is deterministic.

    dsEdidIgnoreParam_t *param = (dsEdidIgnoreParam_t*) arg;
    dsError_t ret = dsERR_GENERAL;
    _DEBUG_ENTER();
    IARM_BUS_Lock(lock);
    //Default status is false
    typedef dsError_t (*dsGetIgnoreEDIDStatus_t)(intptr_t handleArg, bool* statusArg);

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.