]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
drm/i915: Correct the IOSF Dev_FN field for IOSF transfers
authorShobhit Kumar <shobhit.kumar@intel.com>
Thu, 5 Feb 2015 11:40:56 +0000 (17:10 +0530)
committerSasha Levin <sasha.levin@oracle.com>
Sat, 14 Mar 2015 19:37:18 +0000 (15:37 -0400)
commite12d499e9a51646d5ded2c1d108a3f2ff838cc88
tree3fc40f977d154f3f48244ef9a2db1443ab653949
parent5c8bf2b80c3d002b8fb5b76cd6f112da69fc0cc9
drm/i915: Correct the IOSF Dev_FN field for IOSF transfers

commit d180d2bbb66579e3bf449642b8ec2a76f4014fcd upstream.

As per the specififcation, the SB_DevFn is the PCI_DEVFN of the target
device and not the source. So PCI_DEVFN(2,0) is not correct. Further the
port ID should be enough to identify devices unless they are MFD. The
SB_DevFn was intended to remove ambiguity in case of these MFD devices.

For non MFD devices the recommendation for the target device IP was to
ignore these fields, but not all of them followed the recommendation.
Some like CCK ignore these fields and hence PCI_DEVFN(2, 0) works and so
does PCI_DEVFN(0, 0) as it works for DPIO. The issue came to light because
of GPIONC which was not getting programmed correctly with PCI_DEVFN(2, 0).
It turned out that this did not follow the recommendation and expected 0
in this field.

In general the recommendation is to use SB_DevFn as PCI_DEVFN(0, 0) for
all devices except target PCI devices.

Signed-off-by: Shobhit Kumar <shobhit.kumar@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
drivers/gpu/drm/i915/intel_sideband.c