All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mario Limonciello <mario.limonciello@amd.com>
To: <rafael@kernel.org>, Len Brown <lenb@kernel.org>
Cc: ddadap@nvidia.com, linux-acpi@vger.kernel.org,
	nouveau@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	hdegoede@redhat.com, kai.heng.feng@canonical.com,
	Mario Limonciello <mario.limonciello@amd.com>,
	Dell.Client.Kernel@dell.com
Subject: [Nouveau] [PATCH 1/4] ACPI: OSI: Remove Linux-Dell-Video _OSI string
Date: Tue, 23 Aug 2022 13:51:31 -0500	[thread overview]
Message-ID: <20220823185134.14464-2-mario.limonciello@amd.com> (raw)
In-Reply-To: <20220823185134.14464-1-mario.limonciello@amd.com>

This string was introduced because drivers for NVIDIA hardware
had bugs supporting RTD3 in the past.  Thoes bugs have been fixed
by commit 5775b843a619 ("PCI: Restore config space on runtime resume
despite being unbound"). so vendors shouldn't be using this string
to modify ASL anymore.

Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
---
 drivers/acpi/osi.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/drivers/acpi/osi.c b/drivers/acpi/osi.c
index 9f6853809138..c2f6b2f553d9 100644
--- a/drivers/acpi/osi.c
+++ b/drivers/acpi/osi.c
@@ -44,15 +44,6 @@ osi_setup_entries[OSI_STRING_ENTRIES_MAX] __initdata = {
 	{"Processor Device", true},
 	{"3.0 _SCP Extensions", true},
 	{"Processor Aggregator Device", true},
-	/*
-	 * Linux-Dell-Video is used by BIOS to disable RTD3 for NVidia graphics
-	 * cards as RTD3 is not supported by drivers now.  Systems with NVidia
-	 * cards will hang without RTD3 disabled.
-	 *
-	 * Once NVidia drivers officially support RTD3, this _OSI strings can
-	 * be removed if both new and old graphics cards are supported.
-	 */
-	{"Linux-Dell-Video", true},
 	/*
 	 * Linux-Lenovo-NV-HDMI-Audio is used by BIOS to power on NVidia's HDMI
 	 * audio device which is turned off for power-saving in Windows OS.
-- 
2.34.1


WARNING: multiple messages have this Message-ID (diff)
From: Mario Limonciello <mario.limonciello@amd.com>
To: <rafael@kernel.org>, Len Brown <lenb@kernel.org>
Cc: <nouveau@lists.freedesktop.org>, <hdegoede@redhat.com>,
	<ddadap@nvidia.com>, <Dell.Client.Kernel@dell.com>,
	<kai.heng.feng@canonical.com>, <kherbst@redhat.com>,
	Mario Limonciello <mario.limonciello@amd.com>,
	Lyude Paul <lyude@redhat.com>, <linux-acpi@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: [PATCH 1/4] ACPI: OSI: Remove Linux-Dell-Video _OSI string
Date: Tue, 23 Aug 2022 13:51:31 -0500	[thread overview]
Message-ID: <20220823185134.14464-2-mario.limonciello@amd.com> (raw)
In-Reply-To: <20220823185134.14464-1-mario.limonciello@amd.com>

This string was introduced because drivers for NVIDIA hardware
had bugs supporting RTD3 in the past.  Thoes bugs have been fixed
by commit 5775b843a619 ("PCI: Restore config space on runtime resume
despite being unbound"). so vendors shouldn't be using this string
to modify ASL anymore.

Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
---
 drivers/acpi/osi.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/drivers/acpi/osi.c b/drivers/acpi/osi.c
index 9f6853809138..c2f6b2f553d9 100644
--- a/drivers/acpi/osi.c
+++ b/drivers/acpi/osi.c
@@ -44,15 +44,6 @@ osi_setup_entries[OSI_STRING_ENTRIES_MAX] __initdata = {
 	{"Processor Device", true},
 	{"3.0 _SCP Extensions", true},
 	{"Processor Aggregator Device", true},
-	/*
-	 * Linux-Dell-Video is used by BIOS to disable RTD3 for NVidia graphics
-	 * cards as RTD3 is not supported by drivers now.  Systems with NVidia
-	 * cards will hang without RTD3 disabled.
-	 *
-	 * Once NVidia drivers officially support RTD3, this _OSI strings can
-	 * be removed if both new and old graphics cards are supported.
-	 */
-	{"Linux-Dell-Video", true},
 	/*
 	 * Linux-Lenovo-NV-HDMI-Audio is used by BIOS to power on NVidia's HDMI
 	 * audio device which is turned off for power-saving in Windows OS.
-- 
2.34.1


  reply	other threads:[~2022-08-23 18:51 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-23 18:51 [Nouveau] [PATCH 0/4] Remove Linux specific _OSI strings Mario Limonciello
2022-08-23 18:51 ` Mario Limonciello
2022-08-23 18:51 ` Mario Limonciello [this message]
2022-08-23 18:51   ` [PATCH 1/4] ACPI: OSI: Remove Linux-Dell-Video _OSI string Mario Limonciello
2022-08-24 14:16   ` Kai-Heng Feng
2022-08-24 14:16     ` [Nouveau] " Kai-Heng Feng
2022-08-24 18:28     ` Limonciello, Mario
2022-08-24 18:28       ` [Nouveau] " Limonciello, Mario
2022-08-25 18:20       ` Rafael J. Wysocki
2022-08-25 18:20         ` [Nouveau] " Rafael J. Wysocki
2022-08-23 18:51 ` [Nouveau] [PATCH 2/4] ACPI: OSI: Remove Linux-Lenovo-NV-HDMI-Audio " Mario Limonciello
2022-08-23 18:51   ` Mario Limonciello
2022-08-23 18:51 ` [Nouveau] [PATCH 3/4] ACPI: OSI: Remove Linux-HPI-Hybrid-Graphics " Mario Limonciello
2022-08-23 18:51   ` Mario Limonciello
2022-08-23 18:51 ` [Nouveau] [PATCH 4/4] ACPI: OSI: Update Documentation on custom _OSI strings Mario Limonciello
2022-08-23 18:51   ` Mario Limonciello

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220823185134.14464-2-mario.limonciello@amd.com \
    --to=mario.limonciello@amd.com \
    --cc=Dell.Client.Kernel@dell.com \
    --cc=ddadap@nvidia.com \
    --cc=hdegoede@redhat.com \
    --cc=kai.heng.feng@canonical.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nouveau@lists.freedesktop.org \
    --cc=rafael@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.