Remove ODM specific APIs from TvSettings HAL to make it generic #5894
Conversation
|
|
arjunbinusky
force-pushed
the
ODM_Phase1_24Q4
branch
from
38b62cb
to
8e6665c
Compare
AVOutput/AVOutputTV.h
Outdated
| opt_path.append(filename); | ||
|
|
||
| if(!boost::filesystem::exists( opt_path)) { | ||
| std::cout << "DS HAL : Using " << m_path <<std::endl; |
There was a problem hiding this comment.
Sure. Will change the print and update.
| public: | ||
| CIniFile(const std::string & filename, const std::string & filepath = "/etc/" ) | ||
| { | ||
| opt_path = "/opt/panel/"; |
There was a problem hiding this comment.
Does this override path holds good for tvsettings use case?
There was a problem hiding this comment.
Its generic path for config file overriding for PQ Module.
aml-pqserver, TvSettings,AVOuptut using this path for overriding.
| public: | ||
| CIniFile(const std::string & filename, const std::string & filepath = "/etc/" ) | ||
| { | ||
| opt_path = "/opt/panel/"; |
There was a problem hiding this comment.
Its generic path for config file overriding for PQ Module.
aml-pqserver, TvSettings,AVOuptut using this path for overriding.
AVOutput/AVOutputTV.h
Outdated
| opt_path.append(filename); | ||
|
|
||
| if(!boost::filesystem::exists( opt_path)) { | ||
| std::cout << "DS HAL : Using " << m_path <<std::endl; |
There was a problem hiding this comment.
Sure. Will change the print and update.
|
Removed redundant ODM APIs. Approved. |
No description provided.