aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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
| * XmlValidator toolMattijs Korpershoek2014-06-254-0/+184
|/ | | | | | | | | | | | This tool can be used to validate a set of xml files against w3c schemas. It scans a directory recursively and checks for .xml files in it. For each file, it extracts the current schema file based on a convention (the corresponding schema file must have the same name as the root node) which is used in the parameter-framework. Change-Id: I27e32028e39a450ed18f86cb095348f73d59ade5
* Merge pull request #5 from kir0gawa/tutorial-changesDavid Wagner2014-06-252-0/+11
|\ | | | | Added cmake rules for installing tools in bin dir
| * Added cmake rules for installing tools in bin dirMattijs Korpershoek2014-06-252-0/+11
|/ | | | | | | | | When performing the tutorials, I wanted to use the tools/xmlGenerator/hostDomainGenerator.sh script. In order to do that, i had to copy manually my scripts files to the pfwInstalledDir/bin directory. This patchs completes the install target by copying the needed files
* Merge pull request #4 from kir0gawa/help-messagesDavid Wagner2014-06-252-7/+24
|\ | | | | Help messages
| * Implemented -h command for test-platformMattijs Korpershoek2014-03-071-6/+23
| | | | | | | | | | | | | | For new users, it was not very clear that you had to give an .xml to the test-platform executable This patch implements -h an explicits that
| * Merge branch 'master' of https://github.com/01org/parameter-frameworkMattijs Korpershoek2014-03-067-0/+369
| |\
| * | Changed failmessage when entering unkown commandMattijs Korpershoek2014-03-051-1/+1
| | | | | | | | | | | | | | | | | | New users should be aware of the existence of the "help" command when they try the pfw for their first time
* | | Merge pull request #7 from kir0gawa/skeleton-plugin-fixesDavid Wagner2014-06-252-1/+41
|\ \ \ | | | | | | | | Skeleton plugin fixes
| * | | Added comments for new plugin developersMattijs Korpershoek2014-03-211-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As a new pfw plugin developer, i encountered an undocumented (in the plugin) naming issue related to how the pfw loads subsystem builders. This patchs just added a comment to inform other developers not to make the same mistake
| * | | Fixed SubsystemBuilder function in Skeleton pluginMattijs Korpershoek2014-03-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | The system class convention needs the get<TYPE>SubsystemBuilder function to have type in caps lock, otherwise the pfw will loop infinitely when attempting to load plugin.
| * | | Added CmakeLists.txt for building the pluginMattijs Korpershoek2014-03-211-0/+31
| | |/ | |/|
* | | Align the CMakeFiles with the recent code changesDavid Wagner2014-06-251-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | We have forgotten to update the list of headers to be installed after some recent developments. Change-Id: Iaeadde5bd566021e9ee34c5fd49f3649f1509e3e Signed-off-by: David Wagner <david.wagner@intel.com>
* | | Merge pull request #9 from 01org/dev-featuresDavid Wagner2014-06-2555-308/+2346
|\ \ \ | |/ / |/| | Realign from internal tree
| * | Create specific targets for headers exportMattijs Korpershoek2014-06-251-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 203091 Hard coded relative or absolute paths shall be avoided. To allow other components to import necessary headers we created two new targets. Change-Id: I95e8f13765537f431f5730a7cdde4f85f4b7d06a Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | Fix bug when using setConfigurationParameter commandFrédéric Boisnard2014-06-251-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 189941 There is an issue when trying to set parameter value in a configuration using the setConfigurationParameter command. The error returned is "Unable to forward synchronize configurable element, Failed to write value in mixer control". The root cause is a recent update concerning the auto synchronization mode which is now turned on by default. The setConfigurationParameter command first updates the local Configuration blackboard, and then the Main blackboard if this Configuration is the one applicable for the domain. However, this local blackboard should not be synchronized, only the main blackboard should be. This patch deactivates synchronization for the local blackboard. Change-Id: I7ca71806309d19ae482399f474d32a0c1b70ae6c Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com>
| * | Removed phony package to copy .xsd filesMattijs Korpershoek2014-06-252-14/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 192861 When hostDomainGenerator.sh is called by the build system, it needs to copy schema files to /tmp/tmp.XXXXXXX. This is because parameter-framework needs to have the schema files next to the .xml files in order to apply validation. In order to do that, hostDomainGenerator.sh needs to know where he can copy the .xsd files from. Currently, the .xsd files are copied from $HOST_OUT/etc/parameter-framework/Schemas via a dedicated makefile. This used a phony package to group all the files that have to be copied. The problem is that phony packages the phony package is not calling the prebuildt rules we need. So the files we want to export to /etc/parameter-framework/Schemas are not copied. This patch removes the phony package in order to produce the expected behaviour. Change-Id: I96da56f85a8c3d512fcfb97244181dc6c1cb9401 Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | Refactored schema export makefileMattijs Korpershoek2014-06-251-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 192861 The makefile contained a lot of redundancy regarding the source files. This patch reorders some variables and uses a less error prone way to declare the LOCAL_SRC_FILES variable by avoiding the repeat of the source files. Change-Id: Idfff93ef4a9d4aff91d2bc864abf38bf904df685 Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | Fixed point test scriptMattijs Korpershoek2014-06-251-0/+240
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 197723 This python script performs several checks on fixed point parameters: - Bound check: Can we set a value? - Sanity check: If we get this previously set parameter, is the value approximately the same? There should be at maximum a quantum difference between the original value and the returned value. - Consistency check: Can we set the value we got from the sanity check? - Bijectivity check: If we get the parameter we set a second time, is the result the same as the value we got from the set we did in the Sanity check? Change-Id: I7b4d61ec740139b0ee70a44b6b38009507a569ee Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | Fixed point test configurationMattijs Korpershoek2014-06-254-0/+726
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 197723 This contains the configuration files which contains a lot of fixedpoint parameters which can be used for testing. Change-Id: I261120d16eb6f4d28b6b61a5b2845f1fdc18e91f Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | FixedPointParameter: Rename conversion functionsMattijs Korpershoek2014-06-252-18/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 197723 Give a few functions more meaningful names. Add some cast fixes and doxygen to the renamed functions. Change-Id: I002a3fa9b397294153c23b9883c3ad3b1c136e54 Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
| * | Fix fixed-point parameter range checkMattijs Korpershoek2014-06-251-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BZ: 197723 When adding parameters which are far out of bounds, the range check does not refuse to set the value. This is unexpected behaviour. This patch adds a correct range check. Change-Id: I325920bcabc9485bd8f95aadcaff36a67182b97c Signed-off-by: Mattijs Korpershoek <mattijsx.korpershoek@intel.com>
generated by cgit v1.1 at 2025-01-21 21:49:48 +0000