* [PATCH] Hide _OSI warning if osi_linux is enabled
@ 2007-11-06 19:11 Alex Williamson
2008-01-17 1:05 ` Len Brown
0 siblings, 1 reply; 2+ messages in thread
From: Alex Williamson @ 2007-11-06 19:11 UTC (permalink / raw)
To: Len Brown; +Cc: linux-acpi
Seems like we don't need to print a big ugly warning when osi_linux
is already properly enabled. Thanks,
Alex
Signed-off-by: Alex Williamson <alex.williamson@hp.com>
--
diff -r c7f1be4e5832 drivers/acpi/osl.c
--- a/drivers/acpi/osl.c Thu Nov 01 12:09:33 2007 -0700
+++ b/drivers/acpi/osl.c Tue Nov 06 12:08:38 2007 -0700
@@ -1167,14 +1167,15 @@ acpi_os_validate_interface (char *interf
if (!strncmp(osi_additional_string, interface, OSI_STRING_LENGTH_MAX))
return AE_OK;
if (!strcmp("Linux", interface)) {
+ if (osi_linux)
+ return AE_OK;
+
printk(KERN_WARNING PREFIX
"System BIOS is requesting _OSI(Linux)\n");
printk(KERN_WARNING PREFIX
"If \"acpi_osi=Linux\" works better,\n"
"Please send dmidecode "
"to linux-acpi@vger.kernel.org\n");
- if(osi_linux)
- return AE_OK;
}
return AE_SUPPORT;
}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Hide _OSI warning if osi_linux is enabled
2007-11-06 19:11 [PATCH] Hide _OSI warning if osi_linux is enabled Alex Williamson
@ 2008-01-17 1:05 ` Len Brown
0 siblings, 0 replies; 2+ messages in thread
From: Len Brown @ 2008-01-17 1:05 UTC (permalink / raw)
To: Alex Williamson; +Cc: linux-acpi
On Tuesday 06 November 2007 14:11, Alex Williamson wrote:
>
> Seems like we don't need to print a big ugly warning when osi_linux
> is already properly enabled. Thanks,
True if "properly enabled" were not an oxymoron:-)
i need to hack on this code a bit so that it doesn't
complain when DMI or the user enabled it, but otherwise does.
thanks,
-Len
> Alex
>
> Signed-off-by: Alex Williamson <alex.williamson@hp.com>
> --
>
> diff -r c7f1be4e5832 drivers/acpi/osl.c
> --- a/drivers/acpi/osl.c Thu Nov 01 12:09:33 2007 -0700
> +++ b/drivers/acpi/osl.c Tue Nov 06 12:08:38 2007 -0700
> @@ -1167,14 +1167,15 @@ acpi_os_validate_interface (char *interf
> if (!strncmp(osi_additional_string, interface, OSI_STRING_LENGTH_MAX))
> return AE_OK;
> if (!strcmp("Linux", interface)) {
> + if (osi_linux)
> + return AE_OK;
> +
> printk(KERN_WARNING PREFIX
> "System BIOS is requesting _OSI(Linux)\n");
> printk(KERN_WARNING PREFIX
> "If \"acpi_osi=Linux\" works better,\n"
> "Please send dmidecode "
> "to linux-acpi@vger.kernel.org\n");
> - if(osi_linux)
> - return AE_OK;
> }
> return AE_SUPPORT;
> }
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-01-17 1:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-11-06 19:11 [PATCH] Hide _OSI warning if osi_linux is enabled Alex Williamson
2008-01-17 1:05 ` Len Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).