Attachment 151870 Details for Bug 162859 – A patch to test

[patch] A patch to test

ec.diff (text/plain), 1.95 KB, created by Jung-uk Kim on 2015-01-20 00:42:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jung-uk Kim

Created: 2015-01-20 00:42:14 UTC

Size: 1.95 KB

patch

obsolete

>Index: sys/dev/acpica/acpi_ec.c
>===================================================================
>--- sys/dev/acpica/acpi_ec.c	(ë¦¬ë¹ì  277392)
>+++ sys/dev/acpica/acpi_ec.c	(ìì ì¬ë³¸)
>@@ -620,7 +620,7 @@ EcGpeQueryHandler(void *Context)
>     struct acpi_ec_softc	*sc = (struct acpi_ec_softc *)Context;
>     UINT8			Data;
>     ACPI_STATUS			Status;
>-    int				retry, sci_enqueued;
>+    int				retry;
>     char			qxx[5];
> 
>     ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
>@@ -641,7 +641,6 @@ EcGpeQueryHandler(void *Context)
>      * that may arise from running the query from causing another query
>      * to be queued, we clear the pending flag only after running it.
>      */
>-    sci_enqueued = sc->ec_sci_pend;
>     for (retry = 0; retry < 2; retry++) {
> 	Status = EcCommand(sc, EC_COMMAND_QUERY);
> 	if (ACPI_SUCCESS(Status))
>@@ -682,13 +681,11 @@ EcGpeQueryHandler(void *Context)
> 	    qxx, AcpiFormatException(Status));
>     }
> 
>-    /* Reenable runtime GPE if its execution was deferred. */
>-    if (sci_enqueued) {
>-	Status = AcpiFinishGpe(sc->ec_gpehandle, sc->ec_gpebit);
>-	if (ACPI_FAILURE(Status))
>-	    device_printf(sc->ec_dev, "reenabling runtime GPE failed: %s\n",
>-		AcpiFormatException(Status));
>-    }
>+    /* Reenable runtime GPE. */
>+    Status = AcpiFinishGpe(sc->ec_gpehandle, sc->ec_gpebit);
>+    if (ACPI_FAILURE(Status))
>+	device_printf(sc->ec_dev, "reenabling runtime GPE failed: %s\n",
>+	    AcpiFormatException(Status));
> }
> 
> /*
>@@ -722,13 +719,12 @@ EcGpeHandler(ACPI_HANDLE GpeDevice, UINT32 GpeNumb
>     if ((EcStatus & EC_EVENT_SCI) && !sc->ec_sci_pend) {
> 	CTR0(KTR_ACPI, "ec gpe queueing query handler");
> 	Status = AcpiOsExecute(OSL_GPE_HANDLER, EcGpeQueryHandler, Context);
>-	if (ACPI_SUCCESS(Status)) {
>+	if (ACPI_SUCCESS(Status))
> 	    sc->ec_sci_pend = TRUE;
>-	    return (0);
>-	} else
>+	else
> 	    printf("EcGpeHandler: queuing GPE query handler failed\n");
>     }
>-    return (ACPI_REENABLE_GPE);
>+    return (0);
> }
> 
> static ACPI_STATUS

Actions: View | Diff

Attachments on bug 162859: 120111 | 149774 | 151870 | 151884