summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* docs: Reword create_{read/write}_window command specSuraj Jitindar Singh2017-05-031-19/+15
| | | | | | | | | Reword the create_{read/write}_window command specification to make it easier to understand and remove sentences which add no value or are unnecessarily verbose. Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com> Change-Id: Idb1ac8da82d4a1ab379c55ff650ef410de86e715
* configure: Download aspeed-lpc-ctrl.h if not presentAndrew Jeffery2017-05-031-2/+3
| | | | | | | | | | However, don't download it if the file is already present in the local directory. The AC_CHECK_HEADER macro can't test this case, so the condition will always trigger if the header isn't installed on the system. Change-Id: I7de8ff24823fe2ce54add9a33125cee5ad0b53f0 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add sequence_numbersAndrew Jeffery2017-04-262-1/+112
| | | | | Change-Id: Ida6123bf27445f0fb65099fbf6d9520344d33f10 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add read_window_cycleAndrew Jeffery2017-04-262-1/+95
| | | | | Change-Id: Iaa967b7e0732ebc0322ea89e74861a5bedb34867 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add invalid_commandAndrew Jeffery2017-04-262-1/+96
| | | | | Change-Id: I42bc056e5c6b6fdd078bc622e40db39583b53c64 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add write_window_dirty_eraseAndrew Jeffery2017-04-262-1/+172
| | | | | Change-Id: I16d2488e75dde619da39d755e00666bc24778217 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add read_window_mark_write_erasedAndrew Jeffery2017-04-262-1/+87
| | | | | Change-Id: I40e3e29e8064f7ec156cdfd579071563d1cc35c3 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add read_window_write_flushAndrew Jeffery2017-04-262-1/+86
| | | | | Change-Id: I17b237f66e57a40df136d47e2e963eae4c3de535 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add mark_read_dirtyAndrew Jeffery2017-04-262-1/+85
| | | | | Change-Id: I7ab43db7d69b0f085574b40701d48dfa99f00856 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add request_low_versionAndrew Jeffery2017-04-262-1/+67
| | | | | Change-Id: Ia2ce45977b2735663e9745e1da72378df5819a01 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add request_high_versionAndrew Jeffery2017-04-262-1/+65
| | | | | | | | | The test identified a flaw in the implementation and lack of clarity in the documentation when compared with Ben's intent and what was discussed with AMI. Change-Id: I1722b1cc8b005cef77fc48f6583ca0e2553ed2f1 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add implicit_flushAndrew Jeffery2017-04-262-1/+171
| | | | | Change-Id: Ib64fe711e7e7922ad90cb8e9a84972df4cbc5b2d Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add create_zero_size_windowAndrew Jeffery2017-04-262-1/+78
| | | | | Change-Id: I59f3c1e92b519d4f52b6d4fe031d425f161e4dd3 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add create_oversize_windowAndrew Jeffery2017-04-262-1/+78
| | | | | Change-Id: I944046be71b6973e8bd1760678fcfaea39d7bc12 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add bmc_event_ack_v2Andrew Jeffery2017-04-262-1/+79
| | | | | Change-Id: Id6966ce54ddfdc2c2ec3933c0dde12451223a61c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add mark_write_erased_v2Andrew Jeffery2017-04-262-1/+111
| | | | | Change-Id: I74f98b63f9971b53a90b4cc8e81ebeddaec1431d Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add write_flush_v2Andrew Jeffery2017-04-262-1/+166
| | | | | Change-Id: I567be3c016f6d96bb5e7b36342e162dfdff2f836 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add mark_write_dirty_v2Andrew Jeffery2017-04-262-1/+87
| | | | | Change-Id: I9a6fe86eff739c8bf377e7a700e79e8d28b90361 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add close_window_v2Andrew Jeffery2017-04-262-1/+129
| | | | | Change-Id: I2ca90440333f1b06c408884f59af22ab0f9c596d Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add create_write_window_v2Andrew Jeffery2017-04-263-1/+82
| | | | | Change-Id: I26992306b7de4ece0b14b51a4a9c443ef47ce302 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add create_read_window_v2Andrew Jeffery2017-04-262-1/+81
| | | | | Change-Id: I2b1295faea3df207cce5430653b4e0a78176e13c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add get_flash_info_v2Andrew Jeffery2017-04-262-1/+73
| | | | | Change-Id: I55916bb3b8538701b0f54f9ce259b1ac4715941c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add reset_stateAndrew Jeffery2017-04-262-1/+73
| | | | | Change-Id: Ia18a5e7c472d8fe55fb90a13be21f731e407e6e9 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add get_mbox_info_v2Andrew Jeffery2017-04-262-3/+70
| | | | | Change-Id: Ibd4d9c7880bc427edcecafcc2e9abeb487620fdb Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add system interface to define MTD and LPC characteristicsAndrew Jeffery2017-04-262-0/+134
| | | | | | | | | | | | | The primary entry point to the system abstraction is through its ioctl() implementation, which handles ioctl calls for the MTD and LPC subsystems. The MBOX portion needs no support as it is driven purely by read() and write() calls. Two helpers are exposed to configure the MTD and LPC subsystems respectively. Change-Id: Id50abdb967f6ea2ece2e3b21288412d276439c36 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add mbox helpers for unit/integration testsAndrew Jeffery2017-04-262-0/+349
| | | | | | | | | The helpers enable initialising the mbox context to the point that dispatch_mbox() can be usefully called in a controlled environment, and then testing any side-effects that resulted. Change-Id: I590841f70dd51e0993b491e537ed4a73ab696e14 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add write_flashAndrew Jeffery2017-04-262-1/+123
| | | | | Change-Id: I887d79db9f1fd5e63c16a717a88cd76a1f800986 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add erase_flashAndrew Jeffery2017-04-262-1/+228
| | | | | Change-Id: Idbe0e8a9d7c62eeb5ac90fce055fc87de80ab1fa Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add copy_flashAndrew Jeffery2017-04-262-1/+97
| | | | | Change-Id: Iac6b6207c01f1db322803218231ca97c210a583c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add NDEBUG sanity checkAndrew Jeffery2017-04-262-0/+14
| | | | | | | Hopefully stops people tripping over themselves if NDEBUG gets defined. Change-Id: Iafb81d01f6451619f49a5030e5154c8f30ffebdb Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* bootstrap: Add 'dev' parameter to run ./configure with additional CFLAGSAndrew Jeffery2017-04-261-1/+12
| | | | | | | | | | | | Also, add -DDEBUG as CODE_COVERAGE_CPPFLAGS has -DNDEBUG, and we set the coverage flags globally. Maybe that's not a good idea, but we definitely want to trigger asserts in the test cases. Change-Id: Ied39252678f1b28284024cd4435996416bd3eb22 Signed-off-by: Andrew Jeffery <andrew@aj.id.au> Change-Id: I9255e7300b783e536a687af6d7efdb54d41a86b1 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* autotools: Add code coverage supportAndrew Jeffery2017-04-263-0/+10
| | | | | Change-Id: I7d11c64ffa3e68973081cfb1f3fdc1817a64e8cd Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_lpc: Make subjective improvements to log messagesAndrew Jeffery2017-04-261-3/+2
| | | | | | | The bikeshed is red. Change-Id: Ibb99da24cc20436d0fc0c5028270fecb04a10d23 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_flash: strerror() requires errno for argumentAndrew Jeffery2017-04-261-2/+2
| | | | | Change-Id: I68381ed74ab8b75dac195b53d61fd8ccabcf6fb1 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_flash: Initialise flash size if necessaryAndrew Jeffery2017-04-261-0/+25
| | | | | Change-Id: I0587088f72df0664e03977c704da042314580f8a Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* test: Add temporary file helperAndrew Jeffery2017-04-262-0/+82
| | | | | Change-Id: Ib102d87232f897ef4c5e7267a56892d22f368f7f Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_lpc: Split init_lpc_dev() to accomodate custom pathsAndrew Jeffery2017-04-261-5/+10
| | | | | | | | This enables "dependency injection" of the device path, allowing tests to supply their own. Change-Id: Ieffdcc75c26ff5b43782ce5884dd6510b056d985 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* common: Move get_dev_mtd() to mtd.cAndrew Jeffery2017-04-263-38/+44
| | | | | | | | Allows tests to link in alternative implementations of get_dev_mtd() without losing other functions available in common.c. Change-Id: I606a83aa9bc59b5fbab1fb11ee1e2574c7f6d127 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* gitignore: Update to cover test outputAndrew Jeffery2017-04-261-0/+5
| | | | | Change-Id: I1cff96b64a4bef13a829b1f5898e18a739275b6c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_windows: Include mbox.h to make header self-containedAndrew Jeffery2017-04-261-0/+2
| | | | | Change-Id: I48ff8175a1e9f6f012e5bed4f7268c5e9b0745e0 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_flash: Include mbox.h to make header self-containedAndrew Jeffery2017-04-261-0/+2
| | | | | Change-Id: Ia8b1c514c2a102db462b376e5dc90c656655420c Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mbox: Add includes to header for used typesAndrew Jeffery2017-04-261-0/+2
| | | | | | | | mbox.h makes use of struct pollfd and bool types, so lets include the appropriate headers to make the mbox header self-contained. Change-Id: Ia9e1ce4477a510fb9a0fa841f6542327930e02bb Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* common: Add includes for used typesAndrew Jeffery2017-04-261-0/+4
| | | | | | | | common.h makes use of va_list, stdbool and stdint types, so lets include them to make the header self-contained. Change-Id: I3a62bfd80d4f138edee749587330c498e7232f38 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_flash: Fix infinite loop on EOFAndrew Jeffery2017-04-261-4/+6
| | | | | Change-Id: If3684d8c386dd41ce7f7a40353e55267e13d5b07 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_msg: Remove unreachable break statementAndrew Jeffery2017-04-261-1/+0
| | | | | Change-Id: Iaa73c1c02286f9f706e45eda8c1e45f3aa346675 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mboxd_msg: Split init function to enable testingAndrew Jeffery2017-04-261-3/+8
| | | | | | | Allows passing a dev node path through. Change-Id: Ice3fd8157366b4687fd7f3aac72619a1f0a67fff Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mbox: Clarify sequence number constraintsAndrew Jeffery2017-04-263-22/+46
| | | | | | | And implement the specified behaviour. Change-Id: I268d5896aa8dda3875cd79f4ff18929c8e3aea49 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mbox_msg: Cleanup error strings and whitespaceAndrew Jeffery2017-04-241-3/+4
| | | | | Change-Id: I6f432ef1b8c6b490c368cccc52044bef300bde84 Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mbox: 'regsiter' typo in protocol documentationAndrew Jeffery2017-04-241-1/+1
| | | | | Change-Id: Ic78359102aae580dadb4de782f7ea025aaa3f4ad Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
* mbox: Clarify protocol version negotionAndrew Jeffery2017-04-242-13/+16
| | | | | | | | | | | | | The specification intended that the mbox daemon return its highest supported protocol version less-than-or-equal to the version requested by the host. This was not clear in the documentation and was not the behaviour implemented by the daemon. Fix both the documentation and the daemon to perform to this expectation. Change-Id: I6f9cf0b16d2b9319d69656eb26defb46dccd4ddf Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
OpenPOWER on IntegriCloud