-
Notifications
You must be signed in to change notification settings - Fork 6
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
base: develop
Are you sure you want to change the base?
Conversation
5627ff1
to
62770fa
Compare
62770fa
to
084051b
Compare
d83b2b8
to
940dd14
Compare
There was a problem hiding this 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; |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this 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.
There was a problem hiding this 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; |
There was a problem hiding this comment.
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.
Remove ODM specific APIs - phase2
Phase 1 PR was as follows:
https://github.com/rdkcentral/rdkv-halif-tvsettings/pull/43/files