Skip to content

Conversation

aktamilbe
Copy link
Contributor

No description provided.

@aktamilbe aktamilbe requested a review from a team as a code owner July 15, 2025 12:09
int *max_offset, tvWBColor_t **color,
tvWBControl_t **control, size_t* num_color,
size_t* num_control, tvContextCaps_t ** context_caps);
#define HAL_NOT_READY 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove all HAL_NOT_READY flags

#define STRING_COLORTEMPERATURE "ColorTemperature."
#define CREATE_DIRTY(__X__) (__X__+=STRING_DIRTY)
#define CAPABLITY_FILE_NAME "pq_capabilities.ini"
#define CAPABILITY_FILE_NAMEV2 "/etc/pq_capabilities.json"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We dont need this in plugin

tvError_t m_MEMCStatus = tvERROR_NONE;

tvColorTemp_t* m_colortemp = nullptr;
size_t m_numColortemp = 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add correct indexing

tvError_t m_aspectRatioStatus = tvERROR_NONE;

tvDimmingMode_t* m_dimmingModes = nullptr;
size_t m_numdimmingModes = 0;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add correct indexing

tvContextCaps_t* m_dimmingModeCaps = nullptr;
tvError_t m_dimmingModeStatus = tvERROR_NONE;

tvPQModeIndex_t* m_pictureModes = nullptr;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add correct indexing

tvContextCaps_t* m_pictureModeCaps = nullptr;
tvError_t m_pictureModeStatus = tvERROR_NONE;

tvBacklightMode_t* m_backlightModes = nullptr;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add correct indexing

yuvaramachandran-gurusamy and others added 15 commits July 22, 2025 15:32
This reverts commit f4a230f.

Signed-off-by: yuvaramachandran_gurusamy <[email protected]>
* RDKEMW-6230: Retry logic used dsGetHDMIARCPortId in HdmiCecSink

Signed-off-by: yuvaramachandran_gurusamy <[email protected]>

* RDKEMW-6230: Fix L1 test build failure

Signed-off-by: yuvaramachandran_gurusamy <[email protected]>

---------

Signed-off-by: yuvaramachandran_gurusamy <[email protected]>
Reason For Change: platfromSupport is missing from getBacklightDimmingModeCaps
Test procedure: Mentioned in the ticket
Risks: Low
Signed-off-by: [email protected]
RDKEMW-6325: platfromSupport is missing from dimmingMode capabilities
* [RDKEMW-3513] PR action to run with latest change

* Modifying L1,L2 .yml

* Modifying artifacts job name

* resolving review comments

* modifying testframework branch name to develop

* pointing to develop

* Update tests-trigger.yml

---------

Co-authored-by: Prathyushakothuru <[email protected]>
Co-authored-by: kprathyusha <[email protected]>
@CLAassistant
Copy link

CLAassistant commented Jul 29, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
8 out of 9 committers have signed the CLA.

✅ suppal045
✅ yuvaramachandran-gurusamy
✅ srikanth-vv
✅ anjuraveendran2024
✅ PriyaPesala
✅ aktamilbe
✅ utkarshece14
✅ ssitar583
❌ skk451


skk451 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

PriyaPesala and others added 6 commits July 30, 2025 11:10
…e PR branch (#211)

* Reffering to current branch

* For PR purpose added github.head_ref

* Fixing copilot review comments

* Adding description to caller_branch

* Adding develop branch in ref

* Modifying L1, L2 yml files

* Modifying README in Tests folder

* Modifying Branch to Trigger Source
@utkarshece14 utkarshece14 deleted the branch pq_dvt1.1 September 17, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants