Re: [PATCH 03/10] ACPI / DSD: Add documentation on hierarchical data extension references

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

 



On Tue, Jul 17, 2018 at 12:21:44PM +0200, Rafael J. Wysocki wrote:
> On Tue, Jul 17, 2018 at 12:07 PM, Sakari Ailus
> <sakari.ailus@xxxxxxxxxxxxxxx> wrote:
> > Hi Rafael,
> >
> > Thanks for the review.
> >
> > On Mon, Jul 09, 2018 at 11:52:39AM +0200, Rafael J. Wysocki wrote:
> >> On Wednesday, June 27, 2018 11:37:20 AM CEST Sakari Ailus wrote:
> >> > Add generic documentation on how to refer to hierarchical data nodes. This
> >> > brings ACPI to feature parity with Device tree in terms of being able to
> >> > refer to any node in the tree.
> >> >
> >> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> >> > ---
> >> >  Documentation/acpi/dsd/data-node-references.txt | 18 ++++++++++++++++++
> >> >  1 file changed, 18 insertions(+)
> >> >  create mode 100644 Documentation/acpi/dsd/data-node-references.txt
> >> >
> >> > diff --git a/Documentation/acpi/dsd/data-node-references.txt b/Documentation/acpi/dsd/data-node-references.txt
> >> > new file mode 100644
> >> > index 0000000000000..2f60eda811163
> >> > --- /dev/null
> >> > +++ b/Documentation/acpi/dsd/data-node-references.txt
> >> > @@ -0,0 +1,18 @@
> >>
> >> It would be good to have some authorship/copyright info here.
> >
> > Added.
> >
> >>
> >> > +Referencing hierarchical data nodes
> >> > +-----------------------------------
> >> > +
> >> > +ACPI in general allows referring to device objects in the tree only.
> >> > +Hierarchical data extension nodes may not be referred to directly, hence this
> >> > +document defines a scheme to implement such references.
> >> > +
> >> > +The references consist of the device node followed by the first package entries
> >> > +of the hierarchical data node extensions from the first child node of the device
> >> > +node until the referred data extension node.
> >>
> >> The above is not particularly clear IMO.
> >>
> >> First off, it is not clear what "device node" means in there.  I guess you mean
> >> a pathname or similar.  Also the "first package entries" phrase is not clear at
> >> all.  What package do you mean in the first place?
> >
> > Right.
> >
> > The device node here means the ACPI node that corresponds to struct device
> > in Linux. Perhaps "device object" would be more appropriate in ACPI
> > context?
> 
> I think so.
> 
> > Reading the hierarchical data extension spec, it would seem these would
> > better be referred to as "key" and "target" than first or second package
> > list entries. How about that?
> 
> That would be better, yes.  In any case, the exact meaning of all the
> terms used should be clear.
> 
> >>
> >> > +
> >> > +The first package list entry in the hierarchical data node shall consist of the
> >> > +name of the node, "@" character and the number of the node. If the node has no
> >> > +numerical value, then the number shall be omitted. The first package list
> >> > +entries of the hierarchical data nodes shall be unique directly under the parent
> >> > +of the node, i.e. there may not be two hierarchical data extensions that have
> >> > +the same first package entry under any node (device or hierarchical data
> >> > +extension).
> >> >
> >>
> >> It would be good to give an example here.
> >
> > The graph data structure documentation is added in later patches and a
> > reference is added to this file in the last patch of the set.
> 
> I'm not sure how this is related to the example thing?

It demonstrates the use of the referencing mechanism as an example would
do. Anyway, reading the documentation again, I agree a more simple example
would be beneficial here. I'll add that for v2.

-- 
Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx
--
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