aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | New remote command for exporting a single domainDavid Wagner2015-01-223-23/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The command is used as follows: getDomainWithSettingsXML <domain> and outputs the requested domain to the standard output. In that end, the const version of CConfigurableDomains::findConfigurableDomain is made public. This somewhat breaks encapsulation but since the returned domain is const, this has limited impact. Change-Id: I9b854040a5d59913b9b405c8e38d29a3018a6079 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Pass the SystemClass as deserialization context for DomainsDavid Wagner2015-01-223-12/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows diminishing the dependency on the parent element when creating a Domain element from an XML document: the parent was used to get a handle to the system class in order to check the existence of ConfigurableElements included in the Domain. Change-Id: Icba7c3c4db2b9728c0fb7c6840a254de9775f6a4 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Extract and specialize domain import- and export-specific contextsDavid Wagner2015-01-2211-161/+242
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import and Export of ConfigurableDomains have different context needs: e.g. the value representation is only used for export; auto-validation is only meaningful for import whereas the "with settings" context is common to both. We create two new classes, derived from XmlDomainSerializingContext and move most of its content to each class it belongs to. Change-Id: I56589cdb3a8ea417e11d2ed98ccd055d7cdead67 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Schemas: Split ConfigurableDomain from ConfigurableDomainsDavid Wagner2015-01-223-114/+130
| | | | | | | | | | | | | | | | | | | | | | | | This is done in preparation for future work making it possible to export/import domains in standalone and separate files. Change-Id: I35ae66361d18e2de603cedee6b1701defcc9d357 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Remove const/non-const code duplication in CConfigurableDomainsDavid Wagner2015-01-221-11/+4
| | | | | | | | | | | | | | | | | | | | | | | | Implement the non-const method of findConfigurableDomain by calling the const version, as described in Effective C++ Item 3. Change-Id: Ib7b58100dc817d02092299923edf2e6c7ce2f117 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Allow more flexibility in overriding CElement::toXmlDavid Wagner2015-01-225-8/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The toXml method has a default implementation that recursively calls toXml on the children and set their "Name" attribute. With this approach, the toplevel element's name isn't set unless its overrided implementation explicitly does so. A first approach for fixing that is to set the XmlElement's name and recursively call toXml on children (without setting their name because it will be done by their toXml method). However, the "CXmlElement" being created may or may not be the object on which the toXml method is called, in which case the name will be set on the wrong XmlElement. Instead, in CElement::toXml, we set the XmlElement's name and call a new virtual method, childrenToXml which only recursively call toXml on children. This gives full flexibility to elements to choose how they want to serialize themselves and how they want to serialize their children. CConfigurableDomain::toXml is modified to take that change into account. Change-Id: Id12a1023e91cb000e55c242c13643b1db7d46871 Signed-off-by: David Wagner <david.wagner@intel.com>
* | | Merge pull request #28 from krocard/ctest_configDavid Wagner2015-01-223-0/+43
|\ \ \ | | | | | | | | Integrate fix point parameter test in cmake
| * | | Integrate fix point parameter test in cmakeKevin Rocard2014-12-103-0/+43
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Cmake can manage test with ctest. Running them when make test is run. Unfortunately the only automatic tests the pfw has (fix point parameter test) are not declared in cmake. Add a test target and declare the fix point parameter test in cmake. Thus a simple make test after the make install will run all tests. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* | | Merge pull request #30 from krocard/explicit_dependency_on_dlDavid Wagner2015-01-221-1/+2
|\ \ \ | | | | | | | | Link parameter-framework.so with dl
| * | | Link parameter-framework.so with dlKevin Rocard2014-12-051-1/+2
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | The parameter-framework was using dl{open,close,error} but was not requesting to link with dl. This leaded to compilation failure on non implicit dl distributions. Add explict dependency to dl. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* | | Merge pull request #35 from 01org/fix-test-platform-boolean-gettersDavid Wagner2015-01-221-2/+1
|\ \ \ | |_|/ |/| | test-platform: fix boolean remote getter commands output
| * | test-platform: fix boolean remote getter commands outputDavid Wagner2015-01-021-2/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Getter commands return a boolean value ("true" or "false") were implemented by a generic method. However, this method returned a "Done" result, which suppresses the normal output and prints "Done" instead. This is fixed by making this method return a "Success" result, which is interpreted as "give the output to the user". In the same time, we remove a useless "cast to void" which is usually a workaround for unused variables. Change-Id: I49115d3b29967fd455ffc0b703dfb4f4d438280e Signed-off-by: David Wagner <david.wagner@intel.com>
* | Release v2.3.0David Wagner2014-12-121-1/+1
|/ | | | | | | It's been a long time we haven't changed this version number. We will start making releases and this one is the first of a series. Signed-off-by: David Wagner <david.wagner@intel.com>
* Merge pull request #26 from 01org/schemas-readme-fixesDavid Wagner2014-11-261-8/+48
|\ | | | | Schemas readme fixes
| * Schemas/Readme: more details for each type of configuration fileDavid Wagner2014-11-251-7/+47
| | | | | | | | | | | | | | The scope of this readme is widened to document the hierarchy of configuration files and details on the content of each of them. Signed-off-by: David Wagner <david.wagner@intel.com>
| * Schemas/Readme: replace 'TuningMode' with 'TuningAllowed'David Wagner2014-11-251-2/+2
|/ | | | | | | There is no such "TuningMode" attribute; it was a mistake and the attribute described in this documentation corresponds to "TuningAllowed". Signed-off-by: David Wagner <david.wagner@intel.com>
* Merge pull request #25 from 01org/fix-fixed-point-testDavid Wagner2014-11-032-708/+715
|\ | | | | Fix fixed point test
| * fixed-point-tests: strip the '_host' suffix on executablesDavid Wagner2014-11-031-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This only makes sense on the Android build system. However, these tests are broken on Android because libremote-processor is compiled as "libremote-processor_host.so" on Android whereas libparameter will only try to load "libremote-processor.so" anyway. For now, let's ignore the Android build system and only care about how we do it with CMake: CMake builds libremote-processor.so, test-platform and remote-process so let's remove the "_host" suffixes that were added for compatibility with the output of the Android makefiles. Signed-off-by: David Wagner <david.wagner@intel.com>
| * fixed-point-tests: Run the tests for each possible parameter sizeDavid Wagner2014-11-032-706/+713
| | | | | | | | | | | | | | | | | | | | | | | | | | | | A lot of parameters were duplicated because we had one parameter for each size (8, 16 and 32) and for each integral/fractional combination. However, they were all at the root of the subsystem and a lot overlapped, e.g., q1.2 for 32bits and q1.2 for 16 bits. We fix this by: - Adding a ParameterBlock for each size - Modifying the tests to account for the change in the parameter paths - Run the tests for 8, 16 and 32 bits Signed-off-by: David Wagner <david.wagner@intel.com>
* | Merge pull request #24 from 01org/fix-android-l-buildDavid Wagner2014-11-032-68/+2
|\ \ | | | | | | Remove libxml2-schemas target since it is now covered by the aosp
| * | Audio: Remove the hack to buid a libxml2 with schemaspafonsoX2014-11-032-68/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 218524 The "Schemas" feature is enabled in Android-L, so there is no need for this out-of-tree compilation anymore. So this patch deletes its compilation in parameter-framework project. Change-Id: I111afa3e12fa80c2871bae475d550bc49c4440cc Signed-off-by: pafonsoX <philippex.afonso@intel.com> Signed-off-by: David Wagner <david.wagner@intel.com>
* | Merge pull request #22 from krocard/Update_.pfw_syntaxDavid Wagner2014-11-031-1/+4
|\ \ | | | | | | Update .pfw gedit syntax grammar
| * | Update .pfw gedit syntax grammarKevin Rocard2014-10-301-1/+4
| | | | | | | | | | | | | | | | | | | | | Add the new Extended Domain Description keywords to gedit syntax grammar. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* | | Merge pull request #23 from 01org/clarify-non-accesibble-errorDavid Wagner2014-10-311-1/+1
|\ \ \ | |_|/ |/| | Clarify the "Non accessible element" error
| * | Clarify the "Non accessible element" errorDavid Wagner2014-10-311-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | This error happens, among others, when trying to set an element as if it was a parameter. The error message was not very helpful. Previous error message: Non accessible element New error message: Can't set /A/b/c because it is not a parameter Signed-off-by: David Wagner <david.wagner@intel.com>
* | Merge pull request #19 from krocard/Remove_useless_log_on_xml_generation_failureDavid Wagner2014-10-241-9/+0
|\ \ | | | | | | Remove flood debug trace on xml generation error
| * | Remove flood debug trace on xml generation errorKevin Rocard2014-10-241-9/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | The patch 0b3ff88 "Add log in case of failure" added system process and network log on failure. It was planed to be used to debug a tricky socket error bug. The bug was since fixed and the traces now only flood build log, making xml generation error root cause spotting a lot harder. Remove extra debug message (netstat and ps invocation) on error. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* | Merge pull request #18 from Makohoek/tmp_dir_name_fixDavid Wagner2014-10-241-1/+1
|\ \ | | | | | | hostDomainGenerator: Fix the stripping of the temporary directory name
| * | hostDomainGenerator: Fix the stripping of the temporary directory nameDavid Wagner2014-10-031-1/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | We didn't properly remove the name of the temporary directory in the generated files which caused two generation from identical sources result in different generated files. We know the full name of the temporary directory so we can simply completly remove it from the generated file. Issue: GMINL-1953 Change-Id: I1f360cea249328d30318a24643b524c87424475e Signed-off-by: David Wagner <david.wagner@intel.com>
* | Merge pull request #17 from Makohoek/static_analysis_fixDavid Wagner2014-10-241-2/+2
|\ \ | | | | | | Fix a use-after-free issue
| * | Fix a use-after-free issueDavid Wagner2014-10-021-2/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | In CBinaryStream, a member gets used after it has been destroyed. This was caught by a static code analyser. Issue: GMINL-1702 Change-Id: I5c93855f12a8f392e1f4249a15048816fb0f13ce Signed-off-by: David Wagner <david.wagner@intel.com> Reviewed-on: https://android.intel.com/278483 Reviewed-by: cactus <cactus@intel.com> Tested-by: cactus <cactus@intel.com> Reviewed-by: Beatty, Robert M <robert.m.beatty@intel.com> Tested-by: Beatty, Robert M <robert.m.beatty@intel.com>
* | Merge pull request #15 from krocard/Remove_non_ascii_charDavid Wagner2014-10-243-38/+38
|\ \ | | | | | | Remove non ascii char
| * | Remove non ASCII char in tree representationKevin Rocard2014-10-022-29/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some comments representing a file tree were using non ASCII characters. They are not readable on many editors leading to incoherent display. Replace them by plain ASCII characters. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
| * | Remove non ASCII char from log stringKevin Rocard2014-10-011-9/+9
| |/ | | | | | | | | | | | | | | | | Some string printed in log were containing Non-breaking spaces leading to terminal corruption. Replace them with normal ASCII spaces. Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
* | Merge pull request #20 from 01org/fix-build-issues-bisDavid Wagner2014-10-248-11/+47
|\ \ | | | | | | Fix build issues bis
| * | Android: Don't build libparameter with ClangDavid Wagner2014-10-241-0/+2
| | | | | | | | | | | | | | | | | | It does not compile yet; this issue will have to be tackled. Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Android: xmlserializer: Update the path to the unicode includesDavid Wagner2014-10-241-2/+2
| | | | | | | | | | | | | | | | | | | | | The unicode library in AOSP's master branch has moved. libxml2 has been updated and xmlserializer needs to be too. Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Android.mk: declare intel as module ownerDavid Wagner2014-10-248-0/+31
| | | | | | | | | | | | | | | | | | | | | This seems to be mandatory. Change-Id: Idba8c7a3ec9e2f05b28ee593d1d15a420a0bd9ed Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Fix pthread compilation flagsDavid Wagner2014-10-242-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the sem_init(3) manpage, we must pass the -pthread flag at link-time when using it. When using pthread_join(3), this flag must also be passed at compile time. The flags were wrong for libremote-processor and were missing for test-platform_host. Change-Id: Iec2797592d6d25297c87ceb2983e87e75624a576 Signed-off-by: David Wagner <david.wagner@intel.com>
| * | Fix multi-arch build issueQiming Shi2014-10-241-2/+0
| |/ | | | | | | | | | | | | | | | | LOCAL_MODULE_PATH is deprecated and in this case, useless. Change-Id: I72054e3a5c70c2779d170f07f24734d75fdf48f8 Signed-off-by: Qiming Shi <qiming.shi@intel.com> Reviewed-on: https://android.intel.com/211229 Reviewed-by: cactus <cactus@intel.com>
* | Merge pull request #13 from 01org/OznOg-bisDavid Wagner2014-10-24200-1422/+1190
|\ \ | | | | | | some C++ cleanup
| * | Remove using std::XXX from headersSebastien Gonzalve2014-09-11182-971/+1125
| | | | | | | | | | | | | | | This is a bad practice to have using in headers because it pollutes the namespace of any user of that header.
| * | Remove useless cpp'sSebastien Gonzalve2014-09-1125-457/+71
| |/ | | | | | | | | Those files are empty or almost empty, put remaining stuff in header if necessary.
* | Merge pull request #14 from Makohoek/libs_use_std_locationDavid Wagner2014-10-241-3/+5
|\ \ | |/ |/| Allow the plugin libraries to have empty path for location
| * Allow the plugin libraries to have empty path for locationLouis Le Gall2014-10-011-3/+5
|/ | | | | | | | | | | | | | | If the xml defines the path of plugin library as empty, as a slash is always added between the folder path and the library name, the parameter framework will look for the library at the root of the file system. This patch prevents from adding the slash except if the path is not empty. Issue: GMINL-1748 Change-Id: I14ed5a5ddbef7e0305300479f4fd44bf611f3442 Signed-off-by: Louis Le Gall <louis.le.gall@intel.com>
* More README filesDavid Wagner2014-09-1010-12/+480
| | | | | | | - More details in the root README.md file; add a nice diagram - Add a lot of README files in subdirectories Signed-off-by: David Wagner <david.wagner@intel.com>
* Merge pull request #10 from 01org/internal-treeDavid Wagner2014-06-3016-30/+294
|\ | | | | Realign from internal tree
| * Add 3-clause BSD license to CMake makefiles.Mattijs Korpershoek2014-06-3010-0/+280
| | | | | | | | | | | | | | | | | | | | | | BZ: 207083 License headers were missing in CMake makefiles. This patch adds the 3-clause BSD license header to each CMakeLists.txt. Change-Id: Ia4da92e139e02d6348d655d0d88a16166d8431ca Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * Fix stlport includes in makefilesMattijs Korpershoek2014-06-306-30/+14
|/ | | | | | | | | | | | | | BZ: 207083 According to libstlport.mk, to use stlport for a given target, we shall "include external/stlport/libstlport.mk" in the target. This was not done in the parameter-framework's makefiles. This patch change all targets which require stlport to include it in the correct way. Change-Id: Ie9b75af6269f172ad6a6c753e15efd0c7baafdd9 Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
* Merge pull request #6 from kir0gawa/xmlValidatorDavid Wagner2014-06-254-0/+184
|\ | | | | Xml validator script
generated by cgit v1.1 at 2024-06-09 02:00:04 +0000