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.

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

gh #44 Remove ODM specific apis - phase2 #45

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

arjunbinu
Copy link
Contributor

Remove ODM specific APIs - phase2
Phase 1 PR was as follows:
https://github.com/rdkcentral/rdkv-halif-tvsettings/pull/43/files

@arjunbinu arjunbinu force-pushed the feature/gh44_odm_api_removal_phase2 branch from 5627ff1 to 62770fa Compare November 25, 2024 16:29
@arjunbinu arjunbinu self-assigned this Nov 25, 2024
@arjunbinu arjunbinu added the enhancement New feature or request label Nov 25, 2024
@arjunbinu arjunbinu closed this Dec 3, 2024
@arjunbinu arjunbinu force-pushed the feature/gh44_odm_api_removal_phase2 branch from 62770fa to 084051b Compare December 3, 2024 17:19
@arjunbinu arjunbinu reopened this Dec 3, 2024
@arjunbinu arjunbinu closed this Dec 3, 2024
@arjunbinu arjunbinu force-pushed the feature/gh44_odm_api_removal_phase2 branch from d83b2b8 to 940dd14 Compare December 3, 2024 17:27
@arjunbinu arjunbinu reopened this Dec 3, 2024
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 comment

COMPONENT_ENABLE, //!< State is Enable
COMPONENT_RESET, //!< State is Reset
COMPONENT_MAX //!< End of enum
}tvcomponent_state_t;
Copy link
Contributor

@Anbukannadhasan Anbukannadhasan Dec 5, 2024

Choose a reason for hiding this comment

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

Are tvcomponent_state_t, fp_mfrColorTemp_t, and input_source_offset_t supposed to be removed? This information is not present in the email conversations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All types in these ODM headers are marked as deprecated and supposed to be removed. I couldn't listing each type in the email, I provided a general description since the changes were just removal of deprecated types. Only for DolbyVision APIs, I have provided an explanation as it demanded some level of explanation in the way it shall be used for multiple modes. Please excuse if there was a lack of clarity.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for the clarification. Please let me know once your testing is complete.

Copy link
Contributor Author

@arjunbinu arjunbinu left a comment

Choose a reason for hiding this comment

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

Please go through my reply.

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 comments

COMPONENT_ENABLE, //!< State is Enable
COMPONENT_RESET, //!< State is Reset
COMPONENT_MAX //!< End of enum
}tvcomponent_state_t;
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for the clarification. Please let me know once your testing is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Resolved
Development

Successfully merging this pull request may close these issues.

Remove ODM specific apis from tvsettings HAL to make it generic - Phase 2
2 participants