summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Convert build process to autotoolsMatthew Barth2016-10-195-82/+104
| | | | | | | | | Replaced the use of a manual Makefile with the use of autotools to automatically verify and generate the necessary build files. Follow the steps outlined within the README.md file to build the package. Change-Id: Iaef09f17006ee51ac45f84c6c59826dcc567e279 Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Change H->hpp & C->cppMatthew Barth2016-10-187-6/+6
| | | | | Change-Id: I4bb8cf7a58c517f348a524d3e027ebcd1dcd0dea Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Ensure all tools are building with -Wall and -WerrorMatthew Barth2016-08-302-20/+12
| | | | | | | | | Added -Werror to CFLAGS & CXXFLAGS. Also corrected resulting compiler errors, including commenting out possible future-use variables. Change-Id: If70c7fdbf1d02f29e3d2cab5fa12b51bf5302cd5 Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Fix warnings in printf formatPatrick Williams2016-08-181-7/+7
| | | | | | | | | A number of arguments to printf were 'size_t', which is the same size as 'int' only on 32-bit architectures. Replace all %d format strings with %zd where size_t is the type passed. Change-Id: Ida863312d692f47ca8240f32b75b8c55a8accc19 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Fix compile failurePatrick Williams2016-08-181-1/+3
| | | | | | | | Improper const was added in 804077 to one of the variables used as a parameter to mapper_get_service. Change-Id: I29eeb255794af381c48b505e12a7d68db6d5c54a Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Improper use of non-const c-string.Patrick Williams2016-08-081-2/+3
| | | | | | | | | | | | | | | | sd_bus_message_read(..."s"...) gives a pointer to a char* from within the message. Even though the message is in writable memory, it is generally bad to write to the char* from within it. Also, as a reviewer it is less of a concern to see a const char* that is lacking a free than it is a char*. Changing to const char* hopefully avoids the confusion as to why the c-string does not have a corresponding free. Eventually this code will likely transition over to the sdbusplus interfaces and this will be irrelevant. Change-Id: I0dfe948bb2f0eee03042ba9b9dc7563995f59027 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Use mapper binding for busname queriesBrad Bishop2016-07-272-8/+40
| | | | | | | | | Prefer mapper queries to hardcoded busnames. React to /org/openbmc/managers/system GetObjectById API change. Change-Id: I651b820622c3bff51c05365b811b41e73fde997c Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Merge pull request #18 from vishwabmc/masterPatrick Williams2016-06-151-5/+46
|\ | | | | Fixes openbmc/ipmi-fru-parser#3
| * Update BINARY encoded IPMI FRU area data to FRU Inventoryvishwa2016-06-151-5/+46
|/ | | | | | | | | | | | | | | | IPMI fru record format can contain BINARY and ASCII encodings. There was an error in the way FRU parser was handling the BINARY encoded data and that resulted in incorrect updates to FRU Inventory object. The reason why it was done like that was to workaround a limitation encountered while creating the dictionary having a name and value pair where value is an array of bytes. This proposed fix will convert the array of binary encoded data into ascii string and updates the FRU inventory as a string. Since string is a single element, it fits in within the limits of dictionary. Fixes openbmc/ipmi-fru-parser#3
* Merge pull request #20 from bradbishop/versionPatrick Williams2016-06-142-9/+19
|\ | | | | Add shared library versioning
| * Add shared library versioningBrad Bishop2016-06-131-5/+10
| | | | | | | | | | | | Use -soname and install a library symlink. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Makefile whitespace fixesBrad Bishop2016-06-131-5/+5
| | | | | | | | | | | | A couple spaces at the end of line. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
| * Added gitignoreBrad Bishop2016-06-131-0/+5
|/ | | | Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Replacing std::ifstream by access(2) for checking file existencevishwa2016-06-061-1/+1
| | | | Fixes openbmc/ipmi-fru-parser#12.
* Merge pull request #14 from adamliyi/fix_lookup_0411Patrick Williams2016-05-241-5/+90
|\ | | | | Using new getFRUOjbect() API to fix 'not found in lookup' error message
| * Using new getFRUArea() API to fix 'not found in lookup' error messageYi Li2016-04-281-5/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ipmi-fru-parser calls the dbus method SystemManager::getObjectFromId('FRU_STR', fru_area_name) for each fru areas to get the dbus object path of fru area, even if the fru area does not exist. Also ipmi_validate_fru_area() may be invoked multipul times before full fru data is parsed. This will generate duplicated error messages from SystemManager:, e.g: "ERROR SystemManager: dbus.String(u'INTERNAL_13') not found in lookup". In this case, "INTERNAL" area does not exists for FRU In another pull request for skeleton, a new method: getFRUArea(fru_id) is added specially for ipmi-fru-parser. See PR: https://github.com/openbmc/skeleton/pull/62. ipmi-fru-parser gets fru area names defined in Skeleton, and it calls getObjectFromId() only for fru_aream_name already defined. This way the 'not found in lookup' error message can be avoided. Signed-off-by: Yi Li <adamliyi@msn.com>
* | Merge pull request #15 from anoo1/frusizePatrick Williams2016-05-242-2/+2
|\ \ | |/ |/| Increase size of buffer length variable
| * Replace uint8_t buffer length variable with size_tAdriana Kobylak2016-05-162-2/+2
|/ | | | | | | | | | | The latest Barreleye vpd contains additional custom fields in one of the board sections, pushing the size of the buffer over 0x100. The current variable that stores the buffer size is set to be size_t, but the function using it crops it to uint8_t which doesn't fit the extended size. Changing the function to use size_t. Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* Merge pull request #10 from vishwabmc/masternkskjames2016-01-285-318/+652
|\ | | | | Set Fault and Present status while handling fru
| * Set Fault and Present status while handling fruvishwa2016-01-225-318/+652
|/
* Merge pull request #9 from vishwabmc/masterPatrick Williams2016-01-084-37/+79
|\ | | | | set present status True after updating inventory
| * set present status True after updating inventoryvishwa2015-12-024-37/+79
|/
* Merge pull request #8 from causten/buildingnkskjames2015-11-241-1/+1
|\ | | | | Makefile not running install correctly
| * Makefile not running install correctlyChris Austen2015-11-241-1/+1
|/
* Merge pull request #5 from vishwabmc/eepromPatrick Williams2015-11-247-115/+303
|\ | | | | eeprom read CLI
| * eeprom read CLIvishwa2015-11-247-115/+303
|/
* Merge pull request #6 from causten/reduceconnectionsPatrick Williams2015-11-131-8/+1
|\ | | | | Fru write was growing the number of connections on dbus
| * Fru write was growing the number of connections on dbusChris Austen2015-11-131-8/+1
|/
* keeping fru files foreverChris Austen2015-11-011-24/+17
| | | | | added correct ipmi response buffer cleaned up a incorrect rc
* Fix compile and linker problemsChris Austen2015-11-013-2/+13
|
* Clean up makefile.Patrick Williams2015-10-311-19/+51
|
* Merging IPMI FRU writer and parser.Vishwa2015-10-314-9/+639
|
* Use field type encoding for dbus message format.Hariharasubramanian R2015-10-312-116/+191
|
* Store time in ISO8601 format.Hariharasubramanian R2015-10-311-2/+25
|
* Add interface to parse fru data into a dictionary.Hariharasubramanian R2015-10-312-16/+423
|
* Checking in the basic IPMI FRU Parser Library.Hariharasubramanian R2015-10-313-0/+625
| | | | | | | | Parser has just enough intelligence to parse Chassis, Board & Product info areas. No support for multi-record in this version. This parser is a stripped down version of the parser in the FreeIPMI distribution.
* Initial commitPatrick Williams2015-10-081-0/+675
OpenPOWER on IntegriCloud