Path: csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod From: "Rafael J. Wysocki" Newsgroups: linux.kernel Subject: Re: [PATCH] ACPI / property: Fix subnode lookup scope for data-only subnodes Date: Thu, 22 Oct 2015 01:20:02 +0200 Message-ID: References: X-Original-To: Mika Westerberg User-Agent: KMail/4.11.5 (Linux/4.1.0-rc5+; KDE/4.11.5; x86_64; ; ) MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 29 Organization: linux.* mail to news gateway X-Original-Cc: ACPI Devel Maling List , Linux Kernel Mailing List , Darren Hart X-Original-Date: Thu, 22 Oct 2015 01:44:58 +0200 X-Original-Message-ID: <3357380.Nnsq3b9RG3@vostro.rjw.lan> X-Original-References: <3607867.RKSpENdj9h@vostro.rjw.lan> <20151021132036.GC1526@lahna.fi.intel.com> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1253340 On Wednesday, October 21, 2015 04:20:36 PM Mika Westerberg wrote: > On Wed, Oct 21, 2015 at 01:51:24AM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > The correct scope for looking up the objects to generate data packages for > > data-only subnodes pointed to by another data-only subnode is the scope > > of the parent of that subnode and not the scope containing the _DSD object > > at the top of the hierarchy (the latter works only if all of the objects > > returning data-only subnode packages in a given hierarchy are in the same > > scope). > > > > Fix the code to work as expected. > > > > Fixes: 445b0eb058f5 (ACPI / property: Add support for data-only subnodes) > > Signed-off-by: Rafael J. Wysocki > > Reviewed-by: Mika Westerberg > > All my tests still pass so you can also add > > Tested-by: Mika Westerberg Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/