UPSTREAM: dt-bindings: arm, gic-v3: require that reserved cells are always 0
authorWill Deacon <will.deacon@arm.com>
Wed, 3 Feb 2016 18:00:58 +0000 (18:00 +0000)
committerHuang, Tao <huangtao@rock-chips.com>
Fri, 1 Jul 2016 06:20:47 +0000 (14:20 +0800)
commit923b1ab5f523631e59342b1ad5b30aca24e03172
tree8096112d41e4e8d6825684818abec7bc457409e6
parentfcd98e59c2a12ec4a7f28f14deb2862b618da280
UPSTREAM: dt-bindings: arm, gic-v3: require that reserved cells are always 0

The arm,gic-v3 binding was written with good intentions and doesn't
enforce interrupt-cells to be 3, therefore making it easy to extend
the irq description in future if necessary:

  > Cells 4 and beyond are reserved for future use.

Unfortunately, this sentence is immediately followed up with:

  > When the 1st cell has a value of 0 or 1, cells 4 and beyond act as
  > padding, and may be ignored. It is recommended that padding cells
  > have a value of 0.

Consequently, any extensions to the PPI or SPI interrupt specifiers must
be able to work with random crap from legacy DTs, effectively
necessitating a new interrupt type in the first cell. Sigh.

This patch fixes the text so that additional, reserved cells are
required to be zero. This looks like a reasonable thing to require and
is already satisifed by the .dts files in-tree.

Change-Id: Ia5b07ab4243c0a4492b7c4516af95b86974c42a0
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
Acked-by: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Caesar Wang <wxt@rock-chips.com>
(cherry picked from commit 4aff7b854611d91c5fefb1553eb4c328123095ae)
Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.txt