Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

gh #268 CompositeIn L3 testcase for new API #269

Merged
merged 4 commits into from
Dec 24, 2024
Merged

gh #268 CompositeIn L3 testcase for new API #269

merged 4 commits into from
Dec 24, 2024

Conversation

thanushreevani
Copy link
Contributor

Added L3 test case
Updated the spec and test procedure doc

@thanushreevani thanushreevani self-assigned this Nov 28, 2024
@thanushreevani thanushreevani linked an issue Nov 28, 2024 that may be closed by this pull request
dsCompositeIn: Handle L3 for new API dsCompositeInRegisterVideoModeUpdateCB #268
Closed
@thanushreevani thanushreevani added IF Updates HAL Interface file updates. L3 Work Element labels Nov 28, 2024
@TB-1993 TB-1993 force-pushed the feature/issue268_CompositeIn_L3_videomodeCBSupport branch from a5ab5d6 to ba7c503 Compare November 29, 2024 15:03
srinivasgtl
srinivasgtl reviewed Dec 16, 2024
View reviewed changes
Copy link
Contributor

@srinivasgtl srinivasgtl left a comment

Choose a reason for hiding this comment

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

suggestion to improve the code readability.

host/tests/L3_TestCases/dsCompositeIn/dsCompositeIn_test5_VerifyVideoMode_Callback.py Outdated Show resolved Hide resolved
Anbukannadhasan
Anbukannadhasan reviewed Dec 17, 2024
View reviewed changes
Copy link
Contributor

@Anbukannadhasan Anbukannadhasan left a comment

Choose a reason for hiding this comment

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

Added review comments

docs/pages/dsCompositeIn/ds-compositeIn-L3-Low-Level-TestSpec.md Outdated Show resolved Hide resolved
host/tests/dsClasses/dsCompositeIn.py Show resolved Hide resolved
@hari22yuva hari22yuva force-pushed the feature/issue268_CompositeIn_L3_videomodeCBSupport branch from cf1b81c to 0481130 Compare December 23, 2024 18:08
srinivasgtl
srinivasgtl approved these changes Dec 23, 2024
View reviewed changes
Copy link
Contributor

@srinivasgtl srinivasgtl left a comment

Choose a reason for hiding this comment

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

ship it.

Anbukannadhasan
Anbukannadhasan approved these changes Dec 23, 2024
View reviewed changes
Copy link
Contributor

@Anbukannadhasan Anbukannadhasan left a comment

Choose a reason for hiding this comment

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

Approved

@hari22yuva hari22yuva merged commit 3829e63 into develop Dec 24, 2024
3 checks passed
@hari22yuva hari22yuva deleted the feature/issue268_CompositeIn_L3_videomodeCBSupport branch December 24, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Anbukannadhasan Anbukannadhasan Anbukannadhasan approved these changes

@srinivasgtl srinivasgtl srinivasgtl approved these changes

Assignees

@thanushreevani thanushreevani

Labels
IF Updates HAL Interface file updates. L3 Work Element
Projects
HAL Interface - Upgrades
Status: Resolved
Milestone
VTS L1/L2/L3 - Phase 4 updates and bu...
Development

Successfully merging this pull request may close these issues.

dsCompositeIn: Handle L3 for new API dsCompositeInRegisterVideoModeUpdateCB
4 participants
@thanushreevani @srinivasgtl @Anbukannadhasan @hari22yuva