Hi,
On 20/10/2020 15:22, Thomas Monjalon wrote:
19/10/2020 16:53, David Marchand:
On Fri, Oct 16, 2020 at 12:54 AM Honnappa Nagarahalli
<honnappa.nagaraha...@arm.com> wrote:
IMO, this is not a performance critical path and it is not a difficult solution
to
store these values in the application. My suggestion is to skip adding the API
and store the values in the application.
Vladimir, what's your opinion?
Agree. Global vars or part of a global configuration could be used here.
Thank you. I think we are fine to go ahead with merging this patch.
I saw Honnappa and Kevin acks, are other techboard members aware and
okay with this change?
This is an API change for better maintenance.
I'm not sure the benefit is big enough,
but the cost for the applications is said to be workable.
Better to do it now than later, so
Acked-by: Thomas Monjalon <tho...@monjalon.net>
(at the condition it is added in release notes)
What others think?
This changes LGTM (agree with Thomas about release notes).
Acked-by: Vladimir Medvedkin <vladimir.medved...@intel.com>
--
Regards,
Vladimir
