diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2018-12-10 16:52:55 -0600 |
---|---|---|
committer | Christian R. Geddes <crgeddes@us.ibm.com> | 2018-12-19 15:42:02 -0600 |
commit | cdcc9c270584c2c5154c4d6015b831b6fc88f9f5 (patch) | |
tree | fa72647eb1cbcdc132d5a981446404e145910dd3 | |
parent | e2eaa810c17ab0052bbe412d6853a06f6419d3e6 (diff) | |
download | talos-hostboot-cdcc9c270584c2c5154c4d6015b831b6fc88f9f5.tar.gz talos-hostboot-cdcc9c270584c2c5154c4d6015b831b6fc88f9f5.zip |
Fix order of parameters for mmioscomdd
The device driver for register access to OCBM chips over MMIO was
calling validateInputs with the parameters passed in the wrong order
Change-Id: I2cbe8d12e62083d5071ccd81eeb07c104334f87c
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69636
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Matthew Raybuck <mraybuc@us.ibm.com>
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
-rw-r--r-- | src/usr/expscom/mmioscomdd.C | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/usr/expscom/mmioscomdd.C b/src/usr/expscom/mmioscomdd.C index 6d2ca3e5d..58ed0a6c6 100644 --- a/src/usr/expscom/mmioscomdd.C +++ b/src/usr/expscom/mmioscomdd.C @@ -70,7 +70,7 @@ errlHndl_t mmioScomPerformOp(DeviceFW::OperationType i_opType, do { // First make sure the inputs are valid - l_err = EXPSCOM::validateInputs ( i_opType, i_target, l_expAddr, io_buflen); + l_err = EXPSCOM::validateInputs ( i_opType, i_target, io_buflen, l_expAddr); if(l_err) { |