(2010/07/31 7:36), Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <rjw@xxxxxxx> > > All of the remaining callers of acpi_pci_osc_control_set() either > use acpi_pci_root_osc_query() right before calling it, like > pcie_port_acpi_setup(), or ask for control of one feature only, > like acpi_get_hp_hw_control_from_firmware(). Thus there is no > reason to preserve the _OSC control bits returned by an _OSC query > and the osc_control_qry and osc_queried fields of struct > acpi_pci_root are not necessary any more. Remove them and modify the > code that uses them accordingly. > > Signed-off-by: Rafael J. Wysocki <rjw@xxxxxxx> > --- > drivers/acpi/pci_root.c | 15 --------------- > include/acpi/acpi_bus.h | 3 --- > 2 files changed, 18 deletions(-) > > Index: linux-2.6/drivers/acpi/pci_root.c > =================================================================== > --- linux-2.6.orig/drivers/acpi/pci_root.c > +++ linux-2.6/drivers/acpi/pci_root.c > @@ -247,10 +247,8 @@ static acpi_status acpi_pci_query_osc(st > status = acpi_pci_run_osc(root->device->handle, capbuf); > if (ACPI_SUCCESS(status)) { > root->osc_support_set = support; > - root->osc_control_qry = capbuf[OSC_CONTROL_TYPE]; > if (control) > *control = capbuf[OSC_CONTROL_TYPE]; > - root->osc_queried = 1; > } > return status; > } > @@ -433,19 +431,6 @@ acpi_status acpi_pci_osc_control_set(acp > if ((root->osc_control_set & control_req) == control_req) > goto out; > > - /* Need to query controls first before requesting them */ > - if (!root->osc_queried) { > - status = acpi_pci_query_osc(root, root->osc_support_set, NULL); > - if (ACPI_FAILURE(status)) > - goto out; > - } > - if ((root->osc_control_qry & control_req) != control_req) { > - printk(KERN_DEBUG > - "Firmware did not grant requested _OSC control\n"); > - status = AE_SUPPORT; > - goto out; > - } > - > capbuf[OSC_QUERY_TYPE] = 0; > capbuf[OSC_SUPPORT_TYPE] = root->osc_support_set; > capbuf[OSC_CONTROL_TYPE] = root->osc_control_set | control_req; > Index: linux-2.6/include/acpi/acpi_bus.h > =================================================================== > --- linux-2.6.orig/include/acpi/acpi_bus.h > +++ linux-2.6/include/acpi/acpi_bus.h > @@ -377,9 +377,6 @@ struct acpi_pci_root { > > u32 osc_support_set; /* _OSC state of support bits */ > u32 osc_control_set; /* _OSC state of control bits */ > - u32 osc_control_qry; /* the latest _OSC query result */ > - > - u32 osc_queried:1; /* has _OSC control been queried? */ > }; > > /* helper */ > Reviewed-by: Hidetoshi Seto <seto.hidetoshi@xxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html