From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bulk0.mail-out.lima-city.de (bulk0.mail-out.lima-city.de [91.216.248.212]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C1BEE46AA79; Wed, 12 Aug 2026 14:48:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.216.248.212 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786546094; cv=none; b=U4OP1Dvv5Vpjz+vC/wItGG7Ul9MFpJiu33RCIartwwRuaTtbGqxQ54AfOdW0cTCV+FYSIcpZxVCABfJPkyQOMhJcZe9UTDyXVy+ll1ClS6dJCq689yjtA24RCFIeflXXDDmZnFKfvtjzLyArIC9RWjGZ+IMPBUViNrvpZrisC5I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786546094; c=relaxed/simple; bh=79vwdvV8p9yudrSia5lQjOpCjUlE3msAERhpr9jn+Uw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ip5+IPl6sCTqbdz65dC9h9MxT+m68ysUPG++HjeT5nvCD2yLwvFckxTP60tdGkqgxxr8Ium9i8/3PK6qTYCpEcEGVtoBtJH451WCgmHMPRAIaVDbX4xj/qQlaNua5hH5hn69hDgQOMkJSvoLTmCzAp3IllL9Kh+M7HI5eoaNDak= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=deq.rocks; spf=none smtp.mailfrom=deq.rocks; dkim=pass (2048-bit key) header.d=deq.rocks header.i=@deq.rocks header.b=aYgG5EMg; arc=none smtp.client-ip=91.216.248.212 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=deq.rocks Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=deq.rocks Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=deq.rocks header.i=@deq.rocks header.b="aYgG5EMg" From: Andre Eikmeyer DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=deq.rocks; s=securedbylima-20251205; t=1786546080; bh=79vwdvV8p9yudrSia5lQjOpCjUlE3msAERhpr9jn+Uw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=aYgG5EMg9jnhxfWzBxM8uWzsJefeApzHspx9voOZPHMjsvNNv1VIvlk6OCURWRiHo Ty1wCCIJsuimQHQB3nAkLRoX3y27ytclMAmr5RBkakzc7vewiEGLKksd0lAUtXoVsd evws3iz0g2XjdzxzfzgHukX4+N69EkkGJyre8TbMbn5vMJDVO5SDlcvyW4OA3/pmpO jUnRX+EtVNkq5JVTzCTSGGZyECwr/+vrLGQif+21mZJePB3TMG4l6fbxvVk873uqBx lgUANunU2LFicHtusxKW4kkvbYb7T05pFjqYkhccxk56k1WhJ32Afj2aEn0PBnilww PJVcYKw9rtpOA== To: platform-driver-x86@vger.kernel.org, amd-gfx@lists.freedesktop.org, linux-sound@vger.kernel.org Cc: Atharva Tiwari , Hans de Goede , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , Alex Deucher , =?UTF-8?q?Christian=20K=C3=B6nig?= , David Airlie , Simona Vetter , Kenneth Feng , Jaroslav Kysela , Takashi Iwai , Takashi Iwai , Harry Wentland , Leo Li , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Andre Eikmeyer Subject: [PATCH v3 1/3] platform/x86: apple-gmux: add MacBookPro15,1 dGPU power sequence Date: Wed, 12 Aug 2026 16:47:48 +0200 Message-ID: <20260812144750.36797-2-dev@deq.rocks> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260812144750.36797-1-dev@deq.rocks> References: <20260812144221.35877-1-dev@deq.rocks> <20260812144750.36797-1-dev@deq.rocks> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Atharva Tiwari The discrete GPU on the MacBookPro15,1 does not return after the legacy GMUX power-on sequence. PCI configuration space remains inaccessible, so runtime PM cannot provide usable hybrid graphics with the integrated GPU as primary. Evaluate the firmware PWG1 and PWG3 link methods around the GMUX transition and wait for PCI configuration space before completing power-on. Keep the sequence limited to the MacBookPro15,1 and retain the existing path for every other model. This was tested on both the 2018 and 2019 MacBookPro15,1 revisions with the integrated GPU as primary. The discrete GPU transitions between DynOff and DynPwr, and external displays work across the transitions. Co-developed-by: Andre Eikmeyer Signed-off-by: Andre Eikmeyer Signed-off-by: Atharva Tiwari --- drivers/platform/x86/apple-gmux.c | 86 ++++++++++++++++++++++++++++--- 1 file changed, 80 insertions(+), 6 deletions(-) diff --git a/drivers/platform/x86/apple-gmux.c b/drivers/platform/x86/apple-gmux.c index 9c728ac..ccb059a 100644 --- a/drivers/platform/x86/apple-gmux.c +++ b/drivers/platform/x86/apple-gmux.c @@ -22,6 +22,7 @@ #include #include #include +#include #include #include @@ -74,6 +75,8 @@ struct apple_gmux_data { enum vga_switcheroo_client_id switch_state_external; enum vga_switcheroo_state power_state; struct completion powerchange_done; + struct pci_dev *discrete_pdev; + bool use_pwg_power_sequence; /* debugfs data */ u8 selected_port; @@ -82,6 +85,34 @@ struct apple_gmux_data { static struct apple_gmux_data *apple_gmux_data; +static int gmux_call_pwg(struct apple_gmux_data *gmux_data, + const char *method) +{ + acpi_handle handle = ACPI_HANDLE(&gmux_data->discrete_pdev->dev); + unsigned long long result; + acpi_status status; + + if (!handle) + return -ENODEV; + + status = acpi_evaluate_integer(handle, (acpi_string)method, NULL, + &result); + if (ACPI_FAILURE(status)) { + dev_err(&gmux_data->discrete_pdev->dev, + "failed to evaluate %s: %s\n", method, + acpi_format_exception(status)); + return -EIO; + } + + if (result) { + dev_err(&gmux_data->discrete_pdev->dev, + "%s failed: %llu\n", method, result); + return -EIO; + } + + return 0; +} + struct apple_gmux_config { u8 (*read8)(struct apple_gmux_data *gmux_data, int port); void (*write8)(struct apple_gmux_data *gmux_data, int port, u8 val); @@ -510,14 +541,49 @@ static int gmux_switch_ddc(enum vga_switcheroo_client_id id) static int gmux_set_discrete_state(struct apple_gmux_data *gmux_data, enum vga_switcheroo_state state) { + int ret; + reinit_completion(&gmux_data->powerchange_done); if (state == VGA_SWITCHEROO_ON) { - gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 1); - gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 3); + if (gmux_data->use_pwg_power_sequence && + gmux_data->discrete_pdev) { + u16 vendor; + int i; + + gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 2); + msleep(100); + gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 3); + + ret = gmux_call_pwg(gmux_data, "PWG1"); + if (ret) + return ret; + + for (i = 0; i < 1000; i++) { + pci_read_config_word(gmux_data->discrete_pdev, + PCI_VENDOR_ID, &vendor); + if (vendor != 0xffff) + break; + usleep_range(1000, 2000); + } + if (vendor == 0xffff) { + dev_err(&gmux_data->discrete_pdev->dev, + "timed out waiting for PCI config space\n"); + return -ETIMEDOUT; + } + + ret = gmux_call_pwg(gmux_data, "PWG3"); + if (ret) + return ret; + } else { + gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 1); + gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 3); + } pr_debug("Discrete card powered up\n"); } else { gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 1); + if (gmux_data->use_pwg_power_sequence) + usleep_range(10000, 11000); gmux_write8(gmux_data, GMUX_PORT_DISCRETE_POWER, 0); pr_debug("Discrete card powered down\n"); } @@ -549,11 +615,16 @@ static enum vga_switcheroo_client_id gmux_get_client_id(struct pci_dev *pdev) */ if (pdev->vendor == PCI_VENDOR_ID_INTEL) return VGA_SWITCHEROO_IGD; - else if (pdev->vendor == PCI_VENDOR_ID_NVIDIA && - pdev->device == 0x0863) + if (pdev->vendor == PCI_VENDOR_ID_NVIDIA && pdev->device == 0x0863) return VGA_SWITCHEROO_IGD; - else - return VGA_SWITCHEROO_DIS; + + if (apple_gmux_data->use_pwg_power_sequence && + apple_gmux_data->discrete_pdev != pdev) { + pci_dev_put(apple_gmux_data->discrete_pdev); + apple_gmux_data->discrete_pdev = pci_dev_get(pdev); + } + + return VGA_SWITCHEROO_DIS; } static const struct vga_switcheroo_handler gmux_handler_no_ddc = { @@ -803,6 +874,8 @@ static int gmux_probe(struct pnp_dev *pnp, const struct pnp_device_id *id) if (!gmux_data) return -ENOMEM; pnp_set_drvdata(pnp, gmux_data); + gmux_data->use_pwg_power_sequence = type == APPLE_GMUX_TYPE_MMIO && + dmi_match(DMI_PRODUCT_NAME, "MacBookPro15,1"); switch (type) { case APPLE_GMUX_TYPE_MMIO: @@ -1009,6 +1082,7 @@ static void gmux_remove(struct pnp_dev *pnp) } else release_region(gmux_data->iostart, gmux_data->iolen); apple_gmux_data = NULL; + pci_dev_put(gmux_data->discrete_pdev); kfree(gmux_data); } -- 2.55.0