Re: [PATCH] ACPI: return first _ADR match for acpi_get_child

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, November 16, 2012 07:37:34 PM Lekensteyn wrote:
> Thank you for your reply.
> 
> On Friday 16 November 2012 11:25:47 Len Brown wrote:
> > However, this patch can't possibly be the right way to go --
> > as it is just as broken as the code it replaces.
> This is a well-aimed shot in the dark based on the DSDT I see.

Can you post that DSDT by chance?

> It is certainly 
> not a solid proof that is won't break other machines and if it does break. As 
> far as PCI is concerned, it only affects machines with multiple handles that 
> have the same PCI address returned by _ADR.

I wonder if there are any other criteria we can use to choose the "best" handle
in those cases?  It doesn't look like choosing the first on or the last one
is really going to always work.

> This method is also used by USB 
> and others which I am more worried about (still, I think it will not happen, 
> but hey, this is ACPI).
> 
> Nowhere in the ACPI spec is stated what should happen with duplicate _ADR 
> values for such devices. Maybe you can ask for a clarification on this at the 
> ACPI team? (given that the ACPI spec is also worked on by Intel)?
> (the Windows Hardware Certification Requirements [1] is also not helpful on 
> this topic)

Well, added Bob Moore to the CC list, perhaps he knows something about that.

Thanks,
Rafael


-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux