[Intel-gfx] [PATCH i-g-t 1/9] lib: Change IOSF SB devfn to 0.0

ville.syrjala at linux.intel.com ville.syrjala at linux.intel.com
Mon Mar 23 06:14:51 PDT 2015


From: Ville Syrjälä <ville.syrjala at linux.intel.com>

The devfn field in the IOSF SB messages is only relevant when there's a
real PCI bus or some other multifunction device behind a single port.
Otherwise the port number itself is enough to uniquely identify the
target. In fact some IOSF SB targets reject the access when devfn is not
0.0 (eg. GPIO_NC on BYT).

Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
---
 lib/intel_iosf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/intel_iosf.c b/lib/intel_iosf.c
index 27134a0..01577cd 100644
--- a/lib/intel_iosf.c
+++ b/lib/intel_iosf.c
@@ -28,7 +28,7 @@ static int vlv_sideband_rw(uint32_t port, uint8_t opcode, uint32_t addr,
 
 	bar = 0;
 	be = 0xf;
-	devfn = 16;
+	devfn = 0;
 
 	cmd = (devfn << IOSF_DEVFN_SHIFT) | (opcode << IOSF_OPCODE_SHIFT) |
 		(port << IOSF_PORT_SHIFT) | (be << IOSF_BYTE_ENABLES_SHIFT) |
-- 
2.0.5



More information about the Intel-gfx mailing list