summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* hello_world: run p9 mambo testsStewart Smith2016-12-222-0/+70
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: install systemsim-p8 on centos7Stewart Smith2016-12-221-0/+2
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: install systemsim-p8 on centos6Stewart Smith2016-12-221-0/+2
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: install systemsim-p9Stewart Smith2016-12-227-0/+14
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: enable fedora25Stewart Smith2016-12-222-0/+8
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: use curl rather than wgetStewart Smith2016-12-228-13/+13
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* travis: always pull new docker imageStewart Smith2016-12-221-1/+1
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Makefile: Disable stack protector due to gcc problemsBenjamin Herrenschmidt2016-12-221-3/+9
| | | | | | | | | | | | | | Depending on how it was built, gcc will use the canary from a global (works for us) or from the TLS (doesn't work for us and accesses random stuff instead). Fixing that would be tricky. There are talks of adding a gcc option to force use of globals, but in the meantime, disable the stack protector Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> [stewart@linux.vnet.ibm.com: add -fno-stack-protector] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* stack: Don't recurse into __stack_chk_failBenjamin Herrenschmidt2016-12-221-2/+7
| | | | | Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Makefile: Use -ffixed-r13Benjamin Herrenschmidt2016-12-221-0/+1
| | | | | | | We use r13 for our own stuff, make sure it's properly fixed Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* phb3: Lock the PHB on set_xive callbacksBenjamin Herrenschmidt2016-12-221-0/+8
| | | | | | | | | | Those are called by the interrupts core and thus skip the locking implicit in the PCI opal calls. However IODA table access can be racy, so make sure we lock the PHB. Signed-off-by: Benjamin Herrenschmidt Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* gitignore: Add Secure/Trusted Boot build filesMichael Neuling2016-12-211-0/+2
| | | | | Signed-off-by: Michael Neuling <mikey@neuling.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tree-wide: use dt_add_property_u64s() where we canOliver O'Halloran2016-12-2113-44/+19
| | | | | | | | | | | | A few places (mostly old code) were using: add_property_cells(hi32(number), lo32(number)); This patch converts them to use the helper rather than doing it manually. Signed-off-by: Oliver O'Halloran <oohall@gmail.com> Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Add fwts annotation for duplicate DT node entries.ppaidipe@linux.vnet.ibm.com2016-12-211-1/+19
| | | | | | | | Reference bug: https://github.com/open-power/op-build/issues/751 Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com> [stewart@linux.vnet.ibm.com: tweak message slightly] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* arch_flash_arm: Don't assume mtd labels are shortJoel Stanley2016-12-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | pflash relies on arch_flash_arm parsing /proc/mtd to discover the pnor partition. It helpfully uses strcasestr so it can handle the string changing, which is what has happened as we moved to upstream compliant mtd device tree bindings. We currently have a string like this: dev: size erasesize name mtd0: 00060000 00001000 "u-boot" mtd1: 00020000 00001000 "u-boot-env" mtd2: 00280000 00001000 "kernel" mtd3: 001c0000 00001000 "initramfs" mtd4: 01740000 00001000 "rofs" mtd5: 00400000 00001000 "rwfs" mtd6: 02000000 00001000 "1e620000.flash-controller:flash@1" mtd7: 08000000 00001000 "1e630000.flash-controller:pnor@0" Unfortunately arch_flash_arm assumes the string will be at most 50 characters. That's right before the label we're looking for starts so we ignore that line and keep searching. Fix it by allowing for a 255 character line. Fixes: 48ab7ce09504 (external/pflash: Add --mtd) Signed-off-by: Joel Stanley <joel@jms.id.au> Reviewed-by: Cyril Bur <cyril.bur@au1.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Always set ibm,occ-functional-state correctlyPridhiviraj Paidipeddi2016-12-212-7/+14
| | | | | | | | | | | | | | | | | | | | | | | Currently chip->occ_functional data field set's to true always whether ibm,occ-functional-state property exists or not. ibm,occ-functional-state property of xscom node's are populated only in ibm,fsp systems by detecting the OCC state in OPAL/SPIRA. In non-fsp(i.e OpenPower) systems this property is not populating. But OPAL sets chip->occ_functional field incorrectly to true without acually knowing the occ state. OCC Spec says: When OCC is told to go active it will populate OCC-OPAL shared memory interface with ‘valid’ and all Pstate data. So incase of OpenPower systems OPAL can add DT property ibm,occ-functional-state and set the chip->occ_functional when OPAL finds the valid homer region and corresponding occ_valid byte is set to 0x1. Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com> [stewart@linux.vnet.ibm.com: squash commits, tweak commit message] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* opal/fast-reboot: set fw_progress sensor status with IPMI_FW_PCI_INIT.Pridhiviraj Paidipeddi2016-12-211-0/+3
| | | | | | | | In fast-reboot path, OPAL is re-initializing the PCI subsystem. Accordingly set firmware progress sensor status with IPMI_FW_PCI_INIT. Signed-off-by: Pridhiviraj Paidipeddi <ppaidipe@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hdat: Add PSI HB xscom detailsVasant Hegde2016-12-211-0/+5
| | | | | | | | | Add PSI HB XSCOM address. SCOM address and length is taken from p9 example dts (doc/device-tree/examples/power9-phb4.dts). Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> CC: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hdat: Add new fields to proc_init_data structureVasant Hegde2016-12-211-0/+2
| | | | | | | | Latest spec added two new fields to proc_init_data structure. I don't think we have to pass any data in these new fields. Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hdat: Add processor version check for hs service ntupleVasant Hegde2016-12-211-0/+4
| | | | | | | Host services ntuple is deprecated on P9. Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hdat: add_iplparams_serial - Validate HDIF_get_iarray_size() return valueVasant Hegde2016-12-201-1/+1
| | | | | Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Stop using 3-operand cmp[l][i] for latest binutilsAlexey Kardashevskiy2016-12-201-2/+2
| | | | | | | | | | | Since a5721ba270, binutils does not support 3-operand cmp[l][i]. This adds (previously optional) parameter L. Hre is the binutils commit: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff;h=a5721ba270ddf860e0e5a45bba456214e8eac2be;hp=b82317dd347991288e4cca4772e951c672fca8cc Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* psi: fix the xive registers initialization on P8Cédric Le Goater2016-12-161-12/+8
| | | | | | | | | | | | | | | | | | | | | | | When skiboot initializes PSIHB, it fills the xive registers with server=0, prio=0xff. The source (irq) value is left shifted by 29 bits and the last two xive registers (irq 4 and 5) are set with a bogus value : write 0x30 val 0x000000ff00000000 write 0x60 val 0x000000ff20000000 write 0x68 val 0x000000ff40000000 write 0x70 val 0x000000ff60000000 write 0x78 val 0xffffffff80000000 write 0x80 val 0xffffffffa0000000 which seems to be fine for real HW but causes a lof of pain under qemu. Let's use an 'unsigned' type to initialize the xive registers and also use a loop on the PSI irq numbers, like it is done in psi_cleanup_irq. Signed-off-by: Cédric Le Goater <clg@kaod.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* core/cpu.c: Use a device-tree node to detect nest mmu presenceAlistair Popple2016-12-161-14/+22
| | | | | | | | | | | | | | | | | | | The nest mmu address scom was hardcoded which could lead to boot failure on POWER9 systems without a nest mmu. For example Mambo doesn't model the nest mmu which results in the following failure when calling opal_nmmu_set_ptcr() during kernel load: WARNING: 20856759: (20856757): Invalid address 0x0000000028096258 in XSCOM range, SCOM=0x00280962b WARNING: 20856759: (20856757): Attempt to store non-existent address 0x00001A0028096258 20856759: (20856757): 0x000000003002DA08 : stdcix r26,r0,r3 FATAL ERROR: 20856759: (20856757): Check Stop for 0:0: Machine Check with ME bit of MSR off This patch instead reads the address from the device-tree and makes opal_nmmu_set_ptcr return OPAL_UNSUPPORTED on systems without a nest mmu. Signed-off-by: Alistair Popple <alistair@popple.id.au> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hw/npu: Remove obsolete config space indexesGavin Shan2016-12-141-9/+0
| | | | | | | | | | The PCI config space indexes NPU_DEV_CFG_{NORMAL, RDONLY, W1CR, MAX} are useless. This removes them. Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Acked-By: Alistair Popple <alistair@popple.id.au> Reviewed-By: Alistair Popple <alistair@popple.id.au> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* include/npu.h: Remove unused struct npu_dev_trapAlistair Popple2016-12-141-17/+0
| | | | | | | | | | | | Commit "0736df5 hw/npu: Use PCI virtual device" converted the npu over to using some generic infrastructure to filter/intercept PCIe config space writes. This made the original trap implementation redundant. However it seems removal of the old struct definitions was missed during the cleanup, so do that now. Signed-off-by: Alistair Popple <alistair@popple.id.au> Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* external/fwts: Add 'last-tag' to FWTS olog outputJack Miller2016-12-141-5/+12
| | | | | | | | This isn't so useful at the moment, but this will make cleaning out crufty old error definitions much easier. Signed-off-by: Jack Miller <jack@codezen.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* external/fwts: Add FWTS olog merge scriptJack Miller2016-12-141-0/+115
| | | | | | | | | | A script to merge olog error definitions from multiple skiboot versions into a single olog JSON file. Will prompt when conflicting patterns are found to update the pattern, or add both. Signed-off-by: Jack Miller <jack@codezen.org> [stewart@linux.vnet.ibm.com: add copyright notice] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* pci: Always print PHB and PE numbers as hexadecimalRussell Currey2016-12-144-25/+25
| | | | | | | | | | | PHB and PE numbers are sometimes printed as decimal and sometimes as hexadecimal, which is confusing. Standardise on hexadecimal. This is especially useful now that PHB numbers in Linux always match their OPAL ID. Signed-off-by: Russell Currey <ruscur@russell.cc> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* slw: Set PSSCR value for idle statesShilpasri G Bhat2016-12-132-14/+93
| | | | | | | | | | | | | | | | | | | | PSSCR(Processor Stop Status and Control Register) is an SPR whose contents control the operation of stop instruction. Currently in DT, the value of PSSCR only included RL(Requested Level) for each idle state. This patch adds MTL(Maximum Transistion Level), TR(Transistion Rate), ESL(Enable State Loss) and EC(Exit Criterion) bits for idle state in the PSSCR configuration. It also adds new idle states stop0_lite, stop1_lite and stop2_lite which are no state loss versions of stop0, stop1 and stop2 respectively. The lite variant has EC and ESL bits of PSSCR set to 0. The thread entering a lite variant of stop state will not lose any state and will wakeup at next instruction following stop if MSR.EE=0 or at the corresponding interrupt handler if MSR.EE=1. This will reduce the exit latency of the idle state and hardware will not allocate the thread resources to other threads while in power saving mode. Signed-off-by: Shilpasri G Bhat <shilpa.bhat@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* mambo: fake NVRAM supportChris Smart2016-12-132-18/+24
| | | | | | | | | | | | | | | | | | | | | | | This re-configures the Mambo platform to use the new fake NVRAM introduced by Jack Miller <jack@codezen.org> in commit: mambo: Add Fake NVRAM driver An existing NVRAM file can be loaded by pointing SKIBOOT_NVRAM environment variable to the file when running Mambo. If no NVRAM file is provided, the default is set to 256Kb and will be formatted automatically by Skiboot on boot, e.g.: [ 0.000975501,5 ] NVRAM: Size is 256 KB [ 0.002292860,3 ] NVRAM: Partition at offset 0x0 has incorrect 0 length [ 0.002298792,3 ] NVRAM: Re-initializing (size: 0x00040000) This has been tested in Mambo, on bare metal Linux, as well as OpenPower BMC machines. Signed-off-by: Chris Smart <chris@distroguy.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* mambo: Add Fake NVRAM driverJack Miller2016-12-134-0/+71
| | | | | | | | | Implement a fake NVRAM device based on arbitrary memory reserves. This allows NVRAM images to be preloaded without the FSP. Signed-off-by: Jack Miller <jack@codezen.org> Tested-by: Chris Smart <chris@distroguy.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* BMC/PCI: Check slot tables against detected devicesStewart Smith2016-12-137-0/+94
| | | | | | | | | | | | | | | | | | On BMC machines, we have slot tables of built in PHBs, slots and devices that are physically present in the system (such as the BMC itself). We can use these tables to check what we *detected* against what *should* be in the system and throw an error if they differ. We have seen this occur a couple of times while still booting, giving the user just an empty petitboot screen and not much else to go on. This patch helps in that we get a skiboot error message, and at some point in the future when we pump them up to the OS we could get a big friendly error message telling you you're having a bad day. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com> Acked-by: Russell Currey <ruscur@russell.cc> [stewart@linux.vnet.ibm.com: add barreleye] Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* hw/phb3: fix error handling in complete resetAndrew Donnellan2016-12-131-2/+1
| | | | | | | | | | | | | | | During a complete reset, when we get a timeout waiting for pending transaction in state PHB3_STATE_CRESET_WAIT_CQ, we mark the PHB as permanently broken. Set the state to PHB3_STATE_FENCED so that the kernel can retry the complete reset. Reported-by: Pradipta Ghosh <pradghos@in.ibm.com> Suggested-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com> Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* doc/sensors: be explicit about the encoding of sensor ID/handle not being ABIStewart Smith2016-12-082-3/+5
| | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* sensor: add a family field in the handlerCédric Le Goater2016-12-085-21/+34
| | | | | | | | | | | | | | Currently, we are hijacking the last bit of the resource field of the sensor handler to differentiate the sensor families and route the opal_sensor_read() call to the appropriate component. Let's reserve the last 3bits and provide an API to set the sensor family for current use and future use. This gives us a maximum of 8 families and 32 resource classes. The FSP uses 15, so we should be fine for a while. Signed-off-by: Cédric Le Goater <clg@kaod.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* phb3: Trick to allow control of the PCIe link width and speedBenjamin Herrenschmidt2016-12-082-0/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This implements a hook inside OPAL that catches 16 and 32 bit writes to the link status register of the PHB. It allows you to write a new speed or a new width, and OPAL will then cause the PHB to renegociate. Example: First read the link status on PHB4: setpci -s 0004:00:00.0 0x5a.w a103 It's at x16 Gen3 speed (8GT/s) bits 0x0ff0 are the width and 0x000f the speed. The width can be 1 to 16 and the speed 1 to 3 (2.5, 5 and 8GT/s) Then try to bring it down to 1x Gen1 : setpci -s 0004:00:00.0 0x5a.w=0xa011 Observe the result in the PHB: / # lspci -s 0004:00:00.0 -vv 0004:00:00.0 PCI bridge: IBM Device 03dc (prog-if 00 [Normal decode]) .../... LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk- DLActive+ BWMgmt- ABWMgmt+ And in the device: / # lspci -s 0004:01:00.0 -vv .../... LnkSta: Speed 2.5GT/s, Width x1, TrErr- Train- SlotClk+ DLActive- BWMgmt- ABWMgmt- Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* Add skiboot-5.4.2 release notesStewart Smith2016-12-021-0/+15
| | | | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com> (cherry picked from commit 6ed71166d3d02400c2e315923f2bc6953349d7f6) Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* add skiboot-5.4.1 release notesStewart Smith2016-12-021-0/+27
| | | | | | Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com> (cherry picked from commit 8b5cf36bae7c3cf6510a5c77fd0d247f750c641a) Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* i2c: Add nuvoton quirk, disallowing i2cdetect as it locks TPMStewart Smith2016-12-023-0/+42
| | | | | | | | | | | | | | In TPM 2.0 Firmware 1.3.0.1 and 1.3.1.0 (at least) there exists a bug where if you send the wrong thing to the TPM it may lock the bus, with no way of recovery except powering the TPM off/on. On our current systems, the only way to power the TPM off/on is to pull the power on the system (*NOT* just power off/on to host from BMC). So, this patch adds the ability to do things to the i2c request really early on, well before it hits any hardware, such as quickly drop it. Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* p8-i2c reset things manually in some error conditionsStewart Smith2016-12-021-22/+157
| | | | | | | | | | | | | | | | | | | | | It appears that our reset code wasn't entirely correct, and what we're meant to do is reset each port and wait for command complete. In the event where that fails, we can then bitbang things to recover to a state where at least the i2c engine isn't in a weird state. Practically, this means that "i2cdetect -y 10; i2cdetect -y 10" (where 10 is the bus where a TPM is attached, typically p8e1p2) doesn't hard lock the machine (things are still bad and you won't reboot successfully, but it's *better*). one downside to this patch is that we spend a *long* time in OPAL (tens of ms) when doing the reset. This is something that we really need to fix, as it's not at all nice. The full fix for this though will involve changing a decent chunk of the p8-i2c code, as we don't want to write *any* registers while doing this extended reset (while existing code checks status a bit later). Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* phb4: Add ability to log config space accessMichael Neuling2016-11-301-0/+13
| | | | | | | Useful for debugging Signed-off-by: Michael Neuling <mikey@neuling.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: work out the polling time using mftb()Claudio Carvalho2016-11-291-26/+45
| | | | | | | | | | | Currently, the polling time is calculated by adding the sleep time to it. This calculates the polling time by taking timestamps with mftb() before calling the i2c-interface to send an i2c request to the tpm. Thus having a much more accurate polling time. Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: handle errors after reading the tpm fifoClaudio Carvalho2016-11-291-1/+11
| | | | | | | | | This adds code to handle errors after reading the tpm fifo in tpm_read_fifo(). Fixes: 56ad053c3e8bf0764ad5878cb018f00a389d30cf Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: cleanup variables in tpm_read_fifo()Claudio Carvalho2016-11-291-17/+11
| | | | | | | | | The tpm_read_fifo() has unnecessary and not so intuitive variables. This cleans up these variables. Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: handle errors after writting the tpm fifoClaudio Carvalho2016-11-291-2/+11
| | | | | | | | | This adds code to handle errors after writting the tpm fifo in tpm_write_fifo(). Fixes: 56ad053c3e8bf0764ad5878cb018f00a389d30cf Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: cleanup variables in tpm_write_fifo()Claudio Carvalho2016-11-291-27/+23
| | | | | | | | | The tpm_write_fifo() has unnecessary and not so intuitive variables. This cleans up these variables. Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: handle errors after writing sts.commandReady in step 5Claudio Carvalho2016-11-291-1/+11
| | | | | | | | | This adds code to handle errors after writting the sts.commandReady to release the tpm. Fixes: 56ad053c3e8bf0764ad5878cb018f00a389d30cf Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: handle errors after writing sts.goClaudio Carvalho2016-11-291-2/+11
| | | | | | | | | This handles errors returned by the tpm-i2c interface after writing sts.go in tpm_transmit(). Fixes: 56ad053c3e8bf0764ad5878cb018f00a389d30cf Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
* tpm_i2c_nuvoton: handle errors after checking the tpm fifo statusClaudio Carvalho2016-11-291-66/+86
| | | | | | | | | | | | | The functions tpm_is_expecting() and tpm_is_data_avail() ignore the errors returned by the tpm-i2c interface. This adds code to handle erros after checking the tpm fifo status. The tpm_is_expecting() and tpm_is_data_avail() functions are replaced by tpm_wait_for_fifo_status(). Fixes: 56ad053c3e8bf0764ad5878cb018f00a389d30cf Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
OpenPOWER on IntegriCloud