aboutsummaryrefslogtreecommitdiffstats
path: root/parameter
Commit message (Collapse)AuthorAgeFilesLines
...
* Close all dynamic libraries when PFW is destroyedRenaud de Chivre2014-02-124-3/+23
| | | | | | | | | | | | | BZ: 159294 The PFW opens many dynamic libs but does not keep any track of them. This prevents to close them cleanly and have border effect when trying to do coverage measurement. The patch ensures that the PFW closes its libraries when destroyed. Change-Id: Idd1f69bfbc719e8e000914bef990c809ec5ed206 Signed-off-by: Renaud de Chivre <renaud.de.chivre@intel.com>
* Make "Path not found" error more verboseKevin Rocard2014-02-123-9/+11
| | | | | | | | | | | | | BZ: 156207 When an error occurs while using a path one does not know which path it refers to. Append the unfound path to the error : "Path not found: /the/path/that/was/not/found" Change-Id: If53c1a5c301906b464d38f604d6e97029f83ae3f Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* Modify include handling to comply with new plugins repoGuillaume Denneulin2014-02-121-0/+55
| | | | | | | | | | | BZ: 156675 Plugins are moving to new repositories. Modify include handling to comply inter-repo include dependency. Change-Id: I4f65ab4a7f1f524b59dffc567a679154e3394147 Signed-off-by: Guillaume Denneulin <guillaume.denneulin@intel.com>
* Adds default value pair for inclusive criterionFrancois Gaffie2014-02-123-2/+57
| | | | | | | | | | | | | | | | | | | | BZ: 99956 Inclusive criterion type does not allow to check if the bitfield is empty, i.e., if zero is not available. This patch adds the default value pair "none, 0" to allow check either if the criterion "Is" 0, and establishes consistency with inclusion rule. It also allows to convert a concatenated chain of string value, where "|" is considered as AND rule into the corresponding numerical value. Change-Id: I4ee7c0f69228a727964ab80050ac12c6babd4f58 Signed-off-by: Francois Gaffie <francois.gaffie@intel.com> Signed-off-by: Jorge Quintero <jorge.quintero@intel.com> Reviewed-on: http://android.intel.com:8080/138518 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com>
* Add getbitposition API for bitfieldFrancois Gaffie2014-02-121-0/+8
| | | | | | | | | | | | | | | | | | | BZ: 99956 Bitfield parameter type does not allow to retrieve the position of the bit. This patch introduces a new API to retrieve the bit position in the bitfield Change-Id: I54bf8a446e7bd53bed6acda274d32966a737c877 Signed-off-by: Francois Gaffie <francois.gaffie@intel.com> Reviewed-on: http://android.intel.com:8080/138517 Reviewed-by: Quintero, Jorge <jorge.quintero@intel.com> Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: Balestriere, VianneyX <vianneyx.balestriere@intel.com> Tested-by: Balestriere, VianneyX <vianneyx.balestriere@intel.com>
* fix build errorJin Wei2014-02-121-1/+1
| | | | | | | host version of lib libicuuc has changed to libicuuc-host Change-Id: I936a8e196d01d8cacf4ce0d64370b9cddccd8b3a Signed-off-by: Jin Wei <wei.a.jin@intel.com>
* Fix typo in PFW when loading pluginsFrédéric Boisnard2014-02-121-1/+1
| | | | | | | | | | | | BZ: 151782 There is a typo in the error message when a plugin is not found (folowings instead of following). This patch updates the error message. Change-Id: Ifdb749fd90d646d0c04522874e879a329520de0f Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com>
* Avoid integer overflow while calculating max valuesDmitry Shkurko2014-02-123-4/+12
| | | | | | | | | | | | BZ: 134249 The behavior is undefined in the case of signed integer overflow for enum and fixed point parameter types. Modify the behavior to handle correctly the signed integers. Change-Id: Idbd0798a39f826853ae1afcd05cebd897675b9a8 Signed-off-by: Dmitry Shkurko <Dmitry.V.Shkurko@intel.com>
* [PFW] Add possibility to put mapping info in component nodesKevin Rocard2014-02-121-2/+0
| | | | | | | | | | | | | BZ: 147490 Components did not support instantiation mapping (mapping that instantiate subsystem objects) which leaded to artificial parameter bloc use, ie a useless level. Treat the components as parameter block as for instantiation mappings. Change-Id: I64834dbdcbda9ab32a29d6c70dc12da5b5fe6d35 Signed-off-by: Georges-Henri Baron <georges-henrix.baron@intel.com>
* Handle key names in mapping contextRenaud de Chivre2014-02-124-10/+56
| | | | | | | | | | | | | | | | | | | | | BZ: 126002 Mapping context only allows to retrieve a mapping value from its corresponding key ID. We also want to do the same using its key as a string. This patch adds key name handling in context mapping. It also adds an API to retieve a mapping value from its corresponding key name. Note: it also removes an unused argument from handleMappingContext function. Change-Id: I5a18ad624a69272ea2796090692f2bfa8b373a52 Signed-off-by: Renaud de Chivre <renaud.de.chivre@intel.com> Reviewed-on: http://android.intel.com:8080/131571 Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add debug symbols in host buildRenaud de Chivre2014-02-121-1/+1
| | | | | | | | | | | | | | | | BZ: 126002 Host build is used for testing and debugging purpose. It shall contain debug symbols and shall no be optimized. This patch add required compilation flags to achieve this. Change-Id: I6dec6166ca4c705a3ccc7a9ddd6f5a955a91f910 Signed-off-by: Renaud de Chivre <renaud.de.chivre@intel.com> Reviewed-on: http://android.intel.com:8080/131570 Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add an option to continue on failed setting loadKevin Rocard2014-02-124-5/+89
| | | | | | | | | | | | | | | | | BZ: 122982 When the PFW starts it loads the configurable domains, if the load fails, it used to abort the parameterMgr global load. Add an option to continue on failed setting load, as if there were none. Change-Id: Ifd3cd859ad0eaef266be4c9245ebd28c9e9c0a74 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/119924 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add missing subsystem policyKevin Rocard2014-02-124-8/+79
| | | | | | | | | | | | | | | | | | BZ: 122982 When the PFW starts it loads subsystems using there corresponding plugin. If the requested plugin is not found, the start fails. This is a problem, as for host, the plugins are not compiled. Add a command in the public API to ignore missing subsystem. Change-Id: I7597e3fef33466638191ff70b76e7faa9f979418 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/118039 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Split subsystems loadingKevin Rocard2014-02-122-25/+55
| | | | | | | | | | | | | | | | BZ: 122982 The CSystemClass::loadSubsystems was too long. Create a subfunction to load subsystems defined in shared libraries. Change-Id: I8f40ee271f25d0996e1976d8ee2ef6c12581d1d4 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/118192 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Optional fallback on virtual subsystemKevin Rocard2014-02-122-60/+66
| | | | | | | | | | | | | | | | | | | BZ: 122982 When the PFW starts, it loads subsystems using their corresponding plugins. If a requested plugin is not found, the start fails. This is a problem, as for host, plugins are not compiled. Add an option to fallback on virtual subsystem if a plugin is not found during load. Change-Id: Id408873fdc904612c0c741524b33025d7d199fe9 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/117709 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Add default element fallbackKevin Rocard2014-02-125-23/+106
| | | | | | | | | | | | | | | | | | | BZ: 122982 The element library class is a factory that receives an xml node and instanciates the corresponding element. In some case, it would be usefull not to fail if no matching builder is found but use a default one. Add a setDefaultElementBuilder method to set the default builder to fallback to, in case no matching builder is found. Change-Id: I58f0ada3450195a3ca7d878e4b666b0a9359b499 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/118042 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Change the element builder semanticKevin Rocard2014-02-1211-82/+45
| | | | | | | | | | | | | | | | | | | | BZ: 122982 To parse XML, the PFW uses factories containing a map of XML tags and their associated builder builder. The builders used to contain the XML tag of the XML element that it will use in the context of the factory. A builder and an XML tag are already linked by the factory, thus the tag member in the builder is redundant. Remove the XML tag builder attribute and provide it on factory filling. Change-Id: I827c3b34dffb23bc850316d0b092b31c21987ac2 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/124214 Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Fix issue in PFW when loading host librariesFrédéric Boisnard2014-02-121-13/+18
| | | | | | | | | | | | | | | | | | | | | BZ: 127874 Default entry point for plugins libraries must follow the pattern "lib*-subsystem.so". However, when compiling on host, the plugins are named "lib*-subsystem_host.so", so the PFW is unable to load them. This patch modifies the way plugins symbols are detected, so that host libraries can be loaded successfully. Change-Id: I079162c3e47a975641b3d7cb1fc054f95c86028e Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/122812 Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* Fix bug when retrieving a boolean through a handleFrédéric Boisnard2014-02-122-2/+11
| | | | | | | | | | | | | | | | | | | | | BZ: 127872 There is a bug in the CParameterHandle::getAsBoolean() method: instead of accepting a reference to a bool in argument, it takes a copy of the boolean. The fix consists in accepting a reference as parameter. Change-Id: Id1078534620b4cd37b81efa28471ecf8f7e644b5 Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/122811 Reviewed-by: buildbot <buildbot@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* PFW: Update CFLAGS to add Wall, Werror, Wextra everywhereFrédéric Boisnard2014-02-121-0/+10
| | | | | | | | | | | | | | | | | | | BZ: 129048 Some Makefiles of the PFW projects are missing the regular CFLAGS -Wall, -Werror and -Wextra. This patch aims to update the CLFAGS of these Makefiles. Some source code had to be updated because warnings are now considered as errors. Change-Id: Icc91f7bdc649118f822411d9b4693776a84037e9 Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/123977 Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com>
* [PFW] Fix typo in the naming of the SubsystemBuilder APIFrédéric Boisnard2014-02-123-10/+10
| | | | | | | | | | | | | | | | | | BZ: 124779 There is a typo in the naming of the SubsystemBuilder API, "Susbystem" is used instead of "Subsystem". This patch replaces all occurences of "Susbystem" with "Subsystem". Change-Id: Ic132bdc685013257a138465545b99d001c46535a Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/120166 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Add showMapping command to the PFWFrederic Boisnard2014-02-1214-96/+652
| | | | | | | | | | | | | | | | | | | | BZ: 99822 Developers often need to get the mapping corresponding to a specific parameter. However, the PFW doesn't provide a command to achieve this goal. This patch aims to add the showMapping command to the PFW. Change-Id: I05af64a408abe4ceb7f5d177a0ff5fa0461034d0 Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/105145 Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW] Add getFormattedMapping for InstanceConfigurableElementsFrederic Boisnard2014-02-1211-5/+90
| | | | | | | | | | | | | | | | | | | | | BZ: 99822 The mapping data need to be printed, however helper methods are missing to achieve this easily. This patch aims to add a new method "getFormattedMapping()" for CMappingData, allowing to format the mapping data as a string. Change-Id: Ia030c6b88905fcb7591ad45339712051eb88d1c1 Signed-off-by: Frederic Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/109301 Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* parameter-framework: Remove the hardcoding from TOP of treeChetan Rao2014-02-121-1/+1
| | | | | | | | | | | | | | | | | | | BZ: 120651 Remove the hard coding of path from TOP of android build tree. pathmap_INCL has been extended to add AOSP include dirs used by parameter framework. Need to use the MACRO include-path-for defined in build/core/pathmap.mk to include the AOSP paths Change-Id: Ic7c9eb4bcf39db5e31d14c785d8d2e37ab53af78 Signed-off-by: Chetan Rao <chetan.rao@intel.com> Reviewed-on: http://android.intel.com:8080/105311 Reviewed-by: Dixon, CharlesX <charlesx.dixon@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Fix useless CFLAGSKevin Rocard2014-02-121-4/+0
| | | | | | | | | | | | | | | | | | | BZ: 115743 The patch e2c35a01 deleted -Wno-non-virtual-dtor flags by deleting the common_cflags definition but it was still used in the rest of the makefiles. Delete the common_cflags occurrences. Change-Id: If4301a774c45594b36962ce5b82ae5656c6a5bb1 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/115292 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Format variables to lower_snake_caseKevin Rocard2014-02-121-26/+26
| | | | | | | | | | | | | | | | | BZ: 115743 Some variables where not formatted as requested by the coding style. Format them to lower_snake_case. Change-Id: I31c152f161cde469bcd4c00953e95274c60868ca Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/115175 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Makefile] Append "_host" to host modules namesKevin Rocard2014-02-121-4/+4
| | | | | | | | | | | | | | | | | | | | | BZ: 115743 In some parameter framework makefiles, host and target modules (with the same name) are defined in the same file. External prebuilt framework properly handle this case when modules are defined in different Android.mk, but there is a limitation when modules are defined in same Android.mk. Append _host to all host modules. Change-Id: Ib594dabd86617cc40993066338f095b49afb2ea2 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/113537 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Update the PFW version due to api changeKevin Rocard2014-02-121-1/+1
| | | | | | | | | | | | | | | | | | | BZ: 116973 As, the PFW public api changed (deprecated commands), external tools must be aware of it. Increment the PFW version number to let users detect api version. Change-Id: I61860b651c2559242c5eb32ec5d8afaf9bc6b0c6 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/114688 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Mark the getDomainsXML command as deprecatedKevin Rocard2014-02-121-1/+5
| | | | | | | | | | | | | | | | | | | | | | BZ: 116973 The patch 15db1a7d renamed the command getDomainsXML to getDomainsWithSettingsXML. This changes the PFW API and breaks the PFW backward compatibility. Put the command back marked as deprecated. Both getDomainsXML and getDomainsWithSettingsXML are now supported. getDomainsXML output an deprecated message when used. Change-Id: Id838b6e02f003c789bb9cbe7a534e4985a4b348f Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/114071 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Add setDomainsXML command to import domains from stringGeorges-Henri Baron2014-02-122-82/+196
| | | | | | | | | | | | | | | | | | | | | BZ: 55694 Unable possibility to import the configurable domains from a string Create one command that can be called through the command line of the parameter-framework. The command setDomainXML allows to import configurable domains with settings from a string provided as first argument containing the xml description. Change-Id: I6db7ccb8ba61b5e4c8ba81579a68f40e887d1534 Signed-off-by: Georges-Henri Baron <georges-henrix.baron@intel.com> Reviewed-on: http://android.intel.com:8080/65408 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Remove no-non-virtual-dtor flagSebastien Gonzalve2014-02-1210-3/+23
| | | | | | | | | | | | | | BZ: 111473 The warning non-virtual-dtor should not be ignored for the whole project, but only in the few cases where it it necessary. Change-Id: I0ede82a30f2cd055b789f7fb56104fb227934e6f Signed-off-by: Sebastien Gonzalve <sebastien.gonzalve@intel.com> Signed-off-by: Guilhem IMBERTON <guilhem.imberton@intel.com> Reviewed-on: http://android.intel.com:8080/110191 Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [Parameter] Move public header to include dirKevin Rocard2014-02-125-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | BZ: 107991 The parameter-framework core public headers were mixed up with all other private headers. Making it dificult for other components to include PFW public headers. Move public headers - ParameterHandle - ParameterMgrPlatformConnector - SelectionCriterionInterface - SelectionCriterionTypeInterface to the "include" dir. Change-Id: I3a4840345a26ec9bd848179bdd4966d44d97bede Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109757 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Change TARGET_ERROR_FLAGS to LOCAL_CFLAGSKevin Rocard2014-02-121-3/+3
| | | | | | | | | | | | | | | | | | BZ: 107991 All plugin Android.mk were copy pasted resulting in use of TARGET_ERROR_FLAGS instead of LOCAL_CFLAGS. Change TARGET_ERROR_FLAGS to LOCAL_CFLAGS Change-Id: I792164e8c056be5dbfee35d07e7dd21b3137f4f6 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109723 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Delete useless libstdc++ includesKevin Rocard2014-02-121-1/+0
| | | | | | | | | | | | | | | | | | BZ: 107991 All plugin Android.mk were copy pasted resulting in duplicated useless libstdc++ includes. Delete the useless makefiles libstdc++ includes. Change-Id: I42bd991c261cd9fe1287b572668c677c42baad8e Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109279 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [xml serializer] Make xmlserializer lib staticKevin Rocard2014-02-121-2/+2
| | | | | | | | | | | | | | | | | | BZ: 107991 The lib xmlserializer is used only in the PFW. As a result it is useless to have it dynamic. Make the lib static. Change-Id: I5fe91509485781136ecd2a85338f2229c1aebace Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/107597 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW core] Add support for host buildKevin Rocard2014-02-121-19/+59
| | | | | | | | | | | | | | | | | | BZ: 107991 In order to generate setting xml at build time, one need to build and run PFW environement on host. Add an host compilation target. Change-Id: I7e3c9731ba855585b9d85e96400ace0a861910e1 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/107272 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Fix missing space before := in makefilesKevin Rocard2014-02-121-5/+5
| | | | | | | | | | | | | | | | | | BZ: 107991 Some makefile were missing a space before := in makefiles Add a space before := Change-Id: Ie04cd85e3b21a0628f612a374b5c3d294fd724db Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109305 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Wrap command table linesKevin Rocard2014-02-121-57/+133
| | | | | | | | | | | | | | | | | | BZ: 101806 The coding style request all lines to be shorter than 100 caracters. Wrap modified table lines. Change-Id: Ib3ec331a162fff11f41ad82eedb9e84656dde2f7 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/109776 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Add an optional csv flag to the listCriteria commandKevin Rocard2014-02-121-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 101806 The listCriteria command output is very dificult to parse. The default output format contains a lot of useless formating character: $ listCriteria BandRinging: ============ Possible states (Exclusive): {NetworkGenerated, PhoneGenerated} Current state = <none> Add a optional csv flag. With this flag, each criterion info fits in one line, with each field separated by commas: $ listCriteria csv Criterion name: BandRinging, type kind: exclusive, current state: <none>, states: {NetworkGenerated, PhoneGenerated} Change-Id: I7f2b0a86405846b03cab7f4a3023503b69c6e29b Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/102724 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: avoid warning in SubsystemObjectAndrei V. FOMITCHEV2014-02-121-3/+4
| | | | | | | | | | | | | | | | | | BZ: 103961 In current implementation of PFW, if a subsytem does not implement full interface of SubsystemObject, a warning message is logged. The purpose of this patch is to change the default behaviour and do not more log a warning. Change-Id: I51bc61a418956553d3c880aafa622ca6c2f68520 Signed-off-by: Andrei V. FOMITCHEV <andreix.fomitchev@intel.com> Reviewed-on: http://android.intel.com:8080/106320 Reviewed-by: Barthes, FabienX <fabienx.barthes@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Reduce parameter-framework log on startKevin Rocard2014-02-122-5/+1
| | | | | | | | | | | | | | | | | | BZ: 102528 The PFW used to make a lot of useless log during init. Delete those useless logs. Change-Id: I9e4cf6461d4eb643444443ea57e90f97f79571a3 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/103882 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Fix the useless subsystem resync on PFW loadKevin Rocard2014-02-123-2/+29
| | | | | | | | | | | | | | | | | | | | BZ: 97808 The PFW used to resynchronise some subsystems at start before back/forward syncronisation. Which was useless. Reset subsystems 'needToResync' flag during PFW start before sync to avoid this dummy resync. Change-Id: Id0c0bb786f10e2724222fc18f2085164e95f963f Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/98452 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: [Klockwork issue] Fix const operator=Kevin Rocard2014-02-122-2/+2
| | | | | | | | | | | | | | | | | | BZ: 92744 Klockwork issue. operator = should not return a const value. Change-Id: I6525e7969295866b8d3defe1f0d38ad774edb9b9 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/100072 Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: Boisnard, FredericX <fredericx.boisnard@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Coding style violation correctionKevin Rocard2014-02-123-11/+32
| | | | | | | | | | | | | | | | | | BZ: 98166 Some code is not following the coding style. Wrap too long lines according to coding style, add doxygen annotations. Change-Id: I1c160c5f2b4bedab159baadc034042ec9956cd0e Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/99545 Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: cactus <cactus@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* PFW: Parameter bloc can handle any mapping depthKevin Rocard2014-02-123-52/+99
| | | | | | | | | | | | | | | | | | | BZ: 85878 Needed for subsystem object definition enhancement. In a PFW structure file, if a parameter bloc is defined in another parameter bloc with context mapping only (no instanciable mapping), the PFW will no longer stop diving on the first parameter bloc. Change-Id: If4f704f01e20eb348d931c0baab1b85936fc75a4 Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com> Reviewed-on: http://android.intel.com:8080/88959 Reviewed-by: Dixon, CharlesX <charlesx.dixon@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: Boisnard, FredericX <fredericx.boisnard@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW] Fix display error when using getConfigurationParameter with ↵Frédéric Boisnard2014-02-121-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BooleanParameters BZ: 96140 When using getConfigurationParameter on a BooleanParameter, the displayed value is duplicated on the output if the configuration is applicable. The method CParameterMgr::accessConfigurationValue() accesses the element twice when the configuration is the last one applied (in order to update the parameter value in both the configuration blackboard and the main blackboard). The output result is managed using a string passed as reference to ParameterMgr::accessValue() and then to CBooleanParameterType::fromBlackboard(). In the latest method, the boolean value is appended to the output string, however no check is performed to make sure that the string is empty before being used. As the method is called twice with the same output string, we end up with a dupplicated output. This fix updates CBooleanParameterType::fromBlackboard() in order to make sure that the output string is overwritten with the current result. Change-Id: I394366910b1ddc0c8bbe570aa357e007745613db Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/98648 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Barthes, FabienX <fabienx.barthes@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW] Fix issue when setting a parameter of a configuration not applicableFrédéric Boisnard2014-02-123-14/+35
| | | | | | | | | | | | | | | | | | | | | BZ: 93374 This patch aims to fix an issue when using the command setConfigurationParameter to update the value of a parameter of a configuration. The autosync shall be turned off when updating the value of a parameter in a configuration that is not currently applicable. Change-Id: I12f9d37e544dc7b54f1bed0470783fdfc8568b7c Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/96832 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: De Chivre, Renaud <renaud.de.chivre@intel.com> Reviewed-by: Benavoli, Patrick <patrick.benavoli@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW] Access Configurations directlyFrédéric Boisnard2014-02-1210-11/+257
| | | | | | | | | | | | | | | | | | | | | BZ: 88357 This patch aims to enable direct access to Configurations without using the main blackboard. 2 additional commands are available: - getConfigurationParameter <domain> <configuration> <param path>: Get value for parameter at given path from configuration. - setConfigurationParameter <domain> <configuration> <param path> <value> Set value for parameter at given path to configuration. Change-Id: I9357ba5141feee558fa3f7c10f62db14406433b6 Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/92325 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Gonzalve, Sebastien <sebastien.gonzalve@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* [PFW] Take into account base offset when accessing value of parametersFrédéric Boisnard2014-02-128-34/+37
| | | | | | | | | | | | | | | | | | | BZ: 88357 This patch aims to offer the possibility of accessing the value of parameters from a Configuration blackboard instead of Main blackboard. The read/write operation is now relative to the base offset of the ancestor element associated to the Configuration. The base offset is stored in the ParameterAccessContext. Change-Id: I158df63d2ef3bd8cbacb84b7d2f51723ed1074f5 Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com> Reviewed-on: http://android.intel.com:8080/93671 Reviewed-by: cactus <cactus@intel.com> Reviewed-by: Dixon, CharlesX <charlesx.dixon@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
* Exposes API to dump inclusive criteriaFrancois Gaffie2014-02-122-1/+2
| | | | | | | | | | | | | | | | | | | | BZ: 83285 To avoid implement in route manager an helper function to dump inclusive criteria, expose an already existing API through the CriterionTypeInterface. Signed-off-by: Francois Gaffie <francois.gaffie@intel.com> Change-Id: Ib500f23872fd4da47c5d17666f75708e1627f749 Reviewed-on: http://android.intel.com:8080/90783 Reviewed-by: Gaffie, Francois <francois.gaffie@intel.com> Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com> Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com> Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com> Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com> Reviewed-by: buildbot <buildbot@intel.com> Tested-by: buildbot <buildbot@intel.com>
generated by cgit v1.1 at 2025-01-09 08:11:46 +0000