The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Mark Rutland <mark.rutland@arm.com>
To: "Joe.C" <yingjoe.chen@mediatek.com>
Cc: "arm@kernel.org" <arm@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Jiang Liu <jiang.liu@linux.intel.com>,
	Marc Zyngier <Marc.Zyngier@arm.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"srv_heupstream@mediatek.com" <srv_heupstream@mediatek.com>,
	"yingjoe.chen@gmail.com" <yingjoe.chen@gmail.com>,
	"hc.yen@mediatek.com" <hc.yen@mediatek.com>,
	"eddie.huang@mediatek.com" <eddie.huang@mediatek.com>,
	"nathan.chung@mediatek.com" <nathan.chung@mediatek.com>,
	"yh.chen@mediatek.com" <yh.chen@mediatek.com>,
	Sascha Hauer <kernel@pengutronix.de>,
	Olof Johansson <olof@lixom.net>, Arnd Bergmann <arnd@arndb.de>,
	Pawel Moll <Pawel.Moll@arm.com>,
	Russell King <linux@arm.linux.org.uk>,
	Jason Cooper <jason@lakedaemon.net>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Santosh Shilimkar <santosh.shilimkar@ti.com>,
	Matt Porter <mporter@linaro.org>,
	Marc Carino <marc.ceeeee@gmail.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Sricharan R <r.sricharan@ti.com>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	"grant.likely@linaro.org" <grant.likely@linaro.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 6/7] dt-bindings: add irq domain parent binding
Date: Thu, 9 Oct 2014 15:57:18 +0100	[thread overview]
Message-ID: <20141009145718.GB18384@leverpostej> (raw)
In-Reply-To: <1412864980-20273-7-git-send-email-yingjoe.chen@mediatek.com>

Hi,

On Thu, Oct 09, 2014 at 03:29:39PM +0100, Joe.C wrote:
> From: "Joe.C" <yingjoe.chen@mediatek.com>
> 
> Add "arm,irq-domain-hierarchy" optional property.
> 
> Signed-off-by: Joe.C <yingjoe.chen@mediatek.com>
> ---
>  Documentation/devicetree/bindings/arm/gic.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/gic.txt b/Documentation/devicetree/bindings/arm/gic.txt
> index c7d2fa1..f0bb759 100644
> --- a/Documentation/devicetree/bindings/arm/gic.txt
> +++ b/Documentation/devicetree/bindings/arm/gic.txt
> @@ -68,6 +68,8 @@ Example:
>  		      <0xfff10100 0x100>;
>  	};
>  
> +- arm,irq-domain-hierarchy : When this property exists, gic will be used as
> +  parent for hierarchy irq domain.

IRQ domains are a Linux internal detail, and their description doesn't
belong in the DT. So this property does not look like it should be
described in the DT.

I don't see a description of any relationship between devices here, so
it's not clear to me what this property is intended to imply.

Why do you think you need this?

Thanks,
Mark.

       reply	other threads:[~2014-10-09 14:58 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1412864980-20273-1-git-send-email-yingjoe.chen@mediatek.com>
     [not found] ` <1412864980-20273-7-git-send-email-yingjoe.chen@mediatek.com>
2014-10-09 14:57   ` Mark Rutland [this message]
     [not found] ` <1412864980-20273-4-git-send-email-yingjoe.chen@mediatek.com>
2014-10-09 16:59   ` [PATCH v3 3/7] irqchip: gic: Support hierarchy irq domain Marc Zyngier
2014-10-09 17:22     ` Arnd Bergmann
     [not found]     ` <1413196996.23455.9.camel@mtksdaap41>
2014-10-13 12:10       ` Marc Zyngier
2014-10-13 19:51         ` Arnd Bergmann
2014-10-13  8:56   ` Marc Zyngier
2014-10-13  9:25     ` Marc Zyngier
2014-10-13  9:27     ` Arnd Bergmann
2014-10-13  9:44       ` Marc Zyngier
     [not found] ` <1412864980-20273-2-git-send-email-yingjoe.chen@mediatek.com>
2014-10-13 12:11   ` [PATCH v3 1/7] irqdomain: Fix irq_domain_alloc_irqs return check Marc Zyngier
2014-10-13 14:13     ` Jiang Liu
     [not found] ` <1412864980-20273-5-git-send-email-yingjoe.chen@mediatek.com>
2014-10-09 14:37   ` [PATCH v3 4/7] ARM: mediatek: Add sysirq interrupt polarity support Arnd Bergmann
2014-10-13 13:43   ` Matthias Brugger
2014-10-13 14:14     ` Matthias Brugger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141009145718.GB18384@leverpostej \
    --to=mark.rutland@arm.com \
    --cc=Marc.Zyngier@arm.com \
    --cc=Pawel.Moll@arm.com \
    --cc=arm@kernel.org \
    --cc=arnd@arndb.de \
    --cc=benh@kernel.crashing.org \
    --cc=devicetree@vger.kernel.org \
    --cc=eddie.huang@mediatek.com \
    --cc=f.fainelli@gmail.com \
    --cc=grant.likely@linaro.org \
    --cc=hc.yen@mediatek.com \
    --cc=jason@lakedaemon.net \
    --cc=jiang.liu@linux.intel.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=marc.ceeeee@gmail.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mporter@linaro.org \
    --cc=nathan.chung@mediatek.com \
    --cc=olof@lixom.net \
    --cc=r.sricharan@ti.com \
    --cc=robh+dt@kernel.org \
    --cc=santosh.shilimkar@ti.com \
    --cc=srv_heupstream@mediatek.com \
    --cc=tglx@linutronix.de \
    --cc=yh.chen@mediatek.com \
    --cc=yingjoe.chen@gmail.com \
    --cc=yingjoe.chen@mediatek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox