Skip to content

Commit

Permalink
Remove Redundant LPC Bus Check
Browse files Browse the repository at this point in the history
Current implementation of lpcWrite contains a check for errors
on LPC bus before the attempt to writeLPC, and writeLPC is not
attempted if an error is found on the bus. However, the error
check performs bus reset, and the same check is performed in
writeLPC with the difference that the lpcWrite is still attempted
after the bus reset, and only if the write fails is the error
returned.

This commit removes the first check for LPC errors to give lpcWrite
a chance to run after the LPC bus reset due to errors.

Change-Id: I6375e6575f771e33bd3b866534eb3cd8656d38d8
CQ: SW455075
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/76430
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
  • Loading branch information
Ilya Smirnov authored and dcrowell77 committed Apr 24, 2019
1 parent 5b0de29 commit 387ed5a
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions src/usr/lpc/lpcdd.C
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
/* */
/* OpenPOWER HostBoot Project */
/* */
/* Contributors Listed Below - COPYRIGHT 2014,2018 */
/* Contributors Listed Below - COPYRIGHT 2014,2019 */
/* [+] Google Inc. */
/* [+] International Business Machines Corp. */
/* */
Expand Down Expand Up @@ -200,15 +200,10 @@ errlHndl_t lpcWrite(DeviceFW::OperationType i_opType,
// then we have to use our special side copy of the driver
if( i_target == TARGETING::MASTER_PROCESSOR_CHIP_TARGET_SENTINEL )
{
//First check/clear the LPC bus of errors and commit any errors found
l_err = Singleton<LpcDD>::instance().checkForLpcErrors();
if (!l_err)
{
l_err = Singleton<LpcDD>::instance().writeLPC( l_type,
l_addr,
io_buffer,
io_buflen );
}
l_err = Singleton<LpcDD>::instance().writeLPC(l_type,
l_addr,
io_buffer,
io_buflen);
}
else
{
Expand Down

0 comments on commit 387ed5a

Please sign in to comment.