Subrata Banik uploaded patch set #3 to this change.

View Change

soc/intel/common/block/sata: Fix SATA detection issue between Ports 3-7

This patch ensures SATA PCI config space offset 0x92 (SPD)
is written correctly based on MMIO offset 0x0c (GHC_PI)

PCH-H platform has more than 3 SATA port hence observed this issue when
SATA device is connected at Port > 3.

Without this CL :
GHC_PI = 0x0F (Port 0-3 enable)
Value written into SPD = 0xFF (Assume SPD read returns 0xFF)
Results = All SATA Ports are disabled

With this CL :
GHC_PI = 0x0F (Port 0-3 enable)
Value written into SPD = 0xF0
Results = Port 0-3 are enable and 4-7 are disabled.

TEST=Able to detect SATA device connect at Port 4 on CML-H.

Change-Id: Ied3832b26ba1fdd4c30fafe8149689a01d302c3e
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
---
M src/soc/intel/common/block/sata/sata.c
1 file changed, 2 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/41674/3

To view, visit change 41674. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ied3832b26ba1fdd4c30fafe8149689a01d302c3e
Gerrit-Change-Number: 41674
Gerrit-PatchSet: 3
Gerrit-Owner: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra@intel.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan@google.com>
Gerrit-Reviewer: Michael Niewöhner
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Wonkyu Kim <wonkyu.kim@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Andrey Petrov <andrey.petrov@gmail.com>
Gerrit-CC: Lance Zhao <lance.zhao@gmail.com>
Gerrit-CC: Maxim Polyakov <max.senia.poliak@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset