On Tue, Sep 03, 2013 at 08:31:59AM +0800, Zhang Rui wrote:
> acpi_has_method() is a new ACPI API introduced to check
> the existence of an ACPI control method.
>
> It can be used to replace acpi_get_handle() in the case that
> 1. the calling function doesn't need the ACPI handle of the control method.
> and
> 2. the calling function doesn't care the reason why the method is unavailable.
>
> Convert acpi_get_handle() to acpi_has_method()
> in drivers/gpu/drm/i915/intel_acpi.c in this patch.
>
> Signed-off-by: Zhang Rui <rui.zh...@intel.com>
> CC: Daniel Vetter <daniel.vet...@ffwll.ch>
Acked-by: Daniel Vetter <daniel.vet...@ffwll.ch>
> ---
> drivers/gpu/drm/i915/intel_acpi.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_acpi.c
> b/drivers/gpu/drm/i915/intel_acpi.c
> index 57fe1ae..43959ed 100644
> --- a/drivers/gpu/drm/i915/intel_acpi.c
> +++ b/drivers/gpu/drm/i915/intel_acpi.c
> @@ -193,16 +193,14 @@ static void intel_dsm_platform_mux_info(void)
>
> static bool intel_dsm_pci_probe(struct pci_dev *pdev)
> {
> - acpi_handle dhandle, intel_handle;
> - acpi_status status;
> + acpi_handle dhandle;
> int ret;
>
> dhandle = DEVICE_ACPI_HANDLE(&pdev->dev);
> if (!dhandle)
> return false;
>
> - status = acpi_get_handle(dhandle, "_DSM", &intel_handle);
> - if (ACPI_FAILURE(status)) {
> + if (!acpi_has_method(dhandle, "_DSM")) {
> DRM_DEBUG_KMS("no _DSM method for intel device\n");
> return false;
> }
> --
> 1.8.1.2
>
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
