Subject: Re: [PATCH] powerpc: update interrupt info in booting-without-of.txt

From: Segher Boessenkool <[email-masked]>
Date: Wed, 28 Feb 2007 00:23:30 +0100

> +Devices that generate interrupts have an 'interrupt parent' in
> +the interrupt tree.  An interrupt parent can be explicitly
> +defined through the interrupt-parent property. For nodes with
> +no interrupt-parent, the interrupt parent is assumed to be
> +the node's  _device tree_ parent.

Or its parent, or its grandparent, etc.

> +The encoding of an interrupt specifier is determined by the
> +domain (or nexus)

Domain.  An interrupt nexus is to the interrupt tree
what a node is to the device tree.

> +The PCI bus binding specifies a #interrupt-cell value of 1 to encode
> +which interrupt line (INTA,INTB,INTC,INTD) is used.

s/line/pin/ to agree with PCI terminology.

> +The interrupt-parent property is specified to define an explicit
> +link between a device node and its interrupt parent in
> +the interrupt tree.  The value of interrupt-parent is the
> +phandle to the parent node.

phandle _of_ the parent node please.

> +If the interrupt-parent property is not defined for a node, it's
> +interrupt parent is assumed to be it's device tree_ parent.

Same thing again.  You missed an underscore, too.

> +OpenPIC interrupt controllers required 2 cells to encode

s/required/require/

> +ISA PIC interrupt controllers required 2 cells to encode

same thing

Looks good otherwise, thanks.

Segher