mei: gsc_proxy: match component when GSC is on different bus
Subsystems (3)
PATCH v2 1/1 mei: gsc_proxy: match component when GSC is on different bus
From: Alexander Usyskin <[email protected]>
On Arrow Lake S systems, MEI is no longer strictly connected to bus 0, while graphics remain exclusively on bus 0. Adapt the component matching logic to accommodate this change:
Original behavior: Required both MEI and graphics to be on the same bus 0.
New behavior: Only enforces graphics to be on bus 0 (integrated), allowing MEI to reside on any bus.
This ensures compatibility with Arrow Lake S and maintains functionality for the legacy systems.
Fixes 1dd924f6885b ("mei: gsc_proxy: add gsc proxy driver")
Cc <[email protected]> # v6.3+
Signed-off-by Alexander Usyskin <[email protected]>
Signed-off-by Tomas Winkler <[email protected]>
---
V2: Add reference to fixed commit
Requires 'mei: me: add arrow lake point S DID'
https://lore.kernel.org/lkml/[email protected]/
drivers/misc/mei/gsc_proxy/mei_gsc_proxy.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)
drivers/misc/mei/gsc_proxy/mei_gsc_proxy.c +5 -3
@@ -96,7 +96,8 @@ static const struct component_master_ops mei_component_master_ops = {* * The function checks if the device is pci device and * Intel VGA adapter, the subcomponent is SW Proxy -* and the parent of MEI PCI and the parent of VGA are the same PCH device. +* and the VGA is on the bus 0 reserved for built-in devices +* to reject discrete GFX. * * @dev: master device * @subcomponent: subcomponent to match (I915_COMPONENT_SWPROXY)
@@ -123,7 +124,8 @@ static int mei_gsc_proxy_component_match(struct device *dev, int subcomponent,if (subcomponent !=I915_COMPONENT_GSC_PROXY )return 0 ; -return component_compare_dev (dev->parent, ((struct device *)data)->parent); +/* Only built-in GFX */ +return (pdev->bus->number ==0 ); } static int mei_gsc_proxy_probe(struct mei_cl_device *cldev,
@@ -146,7 +148,7 @@ static int mei_gsc_proxy_probe(struct mei_cl_device *cldev, }component_match_add_typed (&cldev->dev, &master_match, - mei_gsc_proxy_component_match, cldev->dev.parent); + mei_gsc_proxy_component_match,NULL );if (IS_ERR_OR_NULL (master_match)) { ret = -ENOMEM ; goto err_exit;
Signature · 1 line
2.43.0