lkml.org 
[lkml]   [2008]   [Nov]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH 1/2] acpiphp: Identify more removable slots
On Thu, Nov 13, 2008 at 08:27:39PM +0000, Matthew Garrett wrote:
> @@ -94,11 +95,14 @@ static int is_ejectable(acpi_handle handle)
> }
>
> status = acpi_get_handle(handle, "_EJ0", &tmp);
> - if (ACPI_FAILURE(status)) {
> - return 0;
> - }
> + if (ACPI_SUCCESS(status))
> + ret = 1;
> +
> + status = acpi_get_handle(handle, "_RMV", &tmp);
> + if (ACPI_SUCCESS(status))
> + ret = 1;
>
> - return 1;
> + return ret;
> }

Why not:

status = acpi_get_handle(handle, "_EJ0", &tmp);
if (ACPI_SUCCESS(status))
return 1;

status = acpi_get_handle(handle, "_RMV", &tmp);
if (ACPI_SUCCESS(status))
return 1;

return 0;

>
>
> @@ -182,17 +186,26 @@ register_slot(acpi_handle handle, u32 lvl, void *context, void **rv)
> struct acpiphp_func *newfunc;
> acpi_handle tmp;
> acpi_status status = AE_OK;
> - unsigned long long adr, sun;
> - int device, function, retval;
> + unsigned long long adr, sun, removable;
> + int device, function, retval, valid = 0;
>
> status = acpi_evaluate_integer(handle, "_ADR", NULL, &adr);
>
> if (ACPI_FAILURE(status))
> return AE_OK;
>
> - status = acpi_get_handle(handle, "_EJ0", &tmp);
> + status = acpi_get_handle(handle, "_RMV", &tmp);
> + if (ACPI_SUCCESS(status)) {
> + acpi_evaluate_integer(handle, "_RMV", NULL, &removable);
> + if (removable)
> + valid = 1;
> + }
>
> - if (ACPI_FAILURE(status) && !(is_dock_device(handle)))
> + status = acpi_get_handle(handle, "_EJ0", &tmp);
> + if (ACPI_SUCCESS(status) || is_dock_device(handle))
> + valid = 1;
> +
> + if (!valid)
> return AE_OK;

This looks like almost the same code as above. New helper function?

> device = (adr >> 16) & 0xffff;
>
> --
> Matthew Garrett | mjg59@srcf.ucam.org
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html

--
Matthew Wilcox Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."


\
 
 \ /
  Last update: 2008-11-13 21:51    [W:0.044 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site