All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: "Heiko Stübner" <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>
Cc: Kukjin Kim <kgene.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v4 0/5] move s3c24xx-irq to drivers/irqchip and add dt support
Date: Fri, 22 Mar 2013 21:04:54 +0000	[thread overview]
Message-ID: <201303222104.54225.arnd@arndb.de> (raw)
In-Reply-To: <201303221843.37668.heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>

On Friday 22 March 2013, Heiko Stübner wrote:
> As suggested by Rob Herring move back to get the parent releationship from dt
> like in the first versions, but set the handler from the interrupt property
> rather than from a specific list.
> 
> This version also implements (hopefully correctly) an idea from him and
> Arnd Bergmann to have the parent relationship not described in the controller
> node but the device nodes instead.
> 
> Therefore the main controller continues to use a two-cell descriptor to set
> hwirq and trigger type but the sub-controller switches to a three-cell
> descriptor where the third bit describes the hwirq of its parent irq in the
> main controller.
> 
> As a result a serial node would then look like:
>         serial@50000000 {
>                 compatible = "samsung,s3c2410-uart";
>                 reg = <0x50000000 0x4000>;
>                 interrupt-parent = <&subintc>;
>                 interrupts = <0 4 28>, <1 4 28>;
>         };
> 
> Tested on a s3c2416-based board.
> 
> As it depends on changes already pending for 3.10 it should probably go thru
> the samsung tree.

Hi Heiko,

I should probably read the original discussion thread again. I already commented
on the binding here, but I may have missed something important that led to
it being the way it is now.

	Arnd
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/5] move s3c24xx-irq to drivers/irqchip and add dt support
Date: Fri, 22 Mar 2013 21:04:54 +0000	[thread overview]
Message-ID: <201303222104.54225.arnd@arndb.de> (raw)
In-Reply-To: <201303221843.37668.heiko@sntech.de>

On Friday 22 March 2013, Heiko St?bner wrote:
> As suggested by Rob Herring move back to get the parent releationship from dt
> like in the first versions, but set the handler from the interrupt property
> rather than from a specific list.
> 
> This version also implements (hopefully correctly) an idea from him and
> Arnd Bergmann to have the parent relationship not described in the controller
> node but the device nodes instead.
> 
> Therefore the main controller continues to use a two-cell descriptor to set
> hwirq and trigger type but the sub-controller switches to a three-cell
> descriptor where the third bit describes the hwirq of its parent irq in the
> main controller.
> 
> As a result a serial node would then look like:
>         serial at 50000000 {
>                 compatible = "samsung,s3c2410-uart";
>                 reg = <0x50000000 0x4000>;
>                 interrupt-parent = <&subintc>;
>                 interrupts = <0 4 28>, <1 4 28>;
>         };
> 
> Tested on a s3c2416-based board.
> 
> As it depends on changes already pending for 3.10 it should probably go thru
> the samsung tree.

Hi Heiko,

I should probably read the original discussion thread again. I already commented
on the binding here, but I may have missed something important that led to
it being the way it is now.

	Arnd

  parent reply	other threads:[~2013-03-22 21:04 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-22 17:43 [PATCH v4 0/5] move s3c24xx-irq to drivers/irqchip and add dt support Heiko Stübner
2013-03-22 17:43 ` Heiko Stübner
2013-03-22 17:44 ` [PATCH v4 1/5] ARM: S3C24XX: move irq driver to drivers/irqchip Heiko Stübner
2013-03-22 17:44   ` Heiko Stübner
2013-03-22 17:44 ` [PATCH v4 2/5] irqchip: s3c24xx: fix comments on some camera interrupts Heiko Stübner
2013-03-22 17:44   ` Heiko Stübner
2013-03-22 17:45 ` [PATCH v4 3/5] irqchip: s3c24xx: fix irqlist of second s3c2416 controller Heiko Stübner
2013-03-22 17:45   ` Heiko Stübner
2013-03-22 17:46 ` [PATCH v4 4/5] irqchip: s3c24xx: add irq_set_type callback for basic interrupt types Heiko Stübner
2013-03-22 17:46   ` Heiko Stübner
2013-03-22 17:46 ` [PATCH v4 5/5] irqchip: s3c24xx: add devicetree support Heiko Stübner
2013-03-22 17:46   ` Heiko Stübner
2013-03-22 20:55   ` Arnd Bergmann
2013-03-22 20:55     ` Arnd Bergmann
2013-03-22 22:15     ` Heiko Stübner
2013-03-22 22:15       ` Heiko Stübner
2013-03-22 22:42       ` Arnd Bergmann
2013-03-22 22:42         ` Arnd Bergmann
     [not found] ` <201303221843.37668.heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>
2013-03-22 21:04   ` Arnd Bergmann [this message]
2013-03-22 21:04     ` [PATCH v4 0/5] move s3c24xx-irq to drivers/irqchip and add dt support Arnd Bergmann
2013-03-22 21:21     ` Heiko Stübner
2013-03-22 21:21       ` Heiko Stübner

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=201303222104.54225.arnd@arndb.de \
    --to=arnd-r2ngtmty4d4@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org \
    --cc=kgene.kim-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.