SUPERH platform development
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 02/10] SH: intc: Add support OF of IRQ
Date: Mon, 17 Dec 2012 08:29:44 +0000	[thread overview]
Message-ID: <2838757.yMqfipzqR7@avalon> (raw)
In-Reply-To: <1355562224-29448-3-git-send-email-horms+renesas@verge.net.au>

Hi Iwamatsu-san,

Thank your for the patch.

On Saturday 15 December 2012 18:03:36 Simon Horman wrote:
> From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
> 
> Add information of device node to struct intc_desc.
> 
> Cc: Magnus Damm <damm@opensource.se>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> ---
> 
> v7
> * Delete "renesas,sh_intcs" and "renesas,sh_intca_irq_pins" as compatible.
>   Update their documentation.
> * Remove of_sh_intc_get_meminfo() and of_sh_intc_get_pint and
>   of_sh_intc_get_intc(). They are not used.
> 
> v2 - v6
> * No change
> ---
>  Documentation/devicetree/bindings/sh/intc.txt |   15 +----
>  drivers/sh/intc/core.c                        |    2 +-
>  drivers/sh/intc/internals.h                   |    3 +-
>  drivers/sh/intc/irqdomain.c                   |    6 +-
>  drivers/sh/intc/of_intc.c                     |   76 ----------------------
>  include/linux/sh_intc.h                       |   29 +---------
>  6 files changed, 9 insertions(+), 122 deletions(-)

Shouldn't this be squashed into patch 01/10 ? 01/10 adds DT bindings support, 
and you already modify them in this patch.

-- 
Regards,

Laurent Pinchart


  reply	other threads:[~2012-12-17  8:29 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-15  9:03 [RFC v7 00/10] ARM: shmobile: DT initialisation of INTC and GIC Simon Horman
2012-12-15  9:03 ` [PATCH 01/10] ARM: shmobile: Add support OF for INTC of shmobile Simon Horman
2012-12-18 10:00   ` Mark Rutland
2013-01-08  8:20     ` Simon Horman
2012-12-15  9:03 ` [PATCH 02/10] SH: intc: Add support OF of IRQ Simon Horman
2012-12-17  8:29   ` Laurent Pinchart [this message]
2012-12-18  0:31     ` Simon Horman
2012-12-15  9:03 ` [PATCH 03/10] ARM: shmobile: Add support OF of INTC for r8a7740 Simon Horman
2012-12-15  9:03 ` [PATCH 04/10] ARM: shmobile: Add support OF of INTC for sh7372 Simon Horman
2012-12-15  9:03 ` [PATCH 05/10] ARM: shmobile: Add DT table " Simon Horman
2012-12-17  8:44   ` Guennadi Liakhovetski
2012-12-18  0:37     ` Simon Horman
2012-12-18  9:34       ` Guennadi Liakhovetski
2012-12-15  9:03 ` [PATCH 06/10] ARM: shmobile: Add DT table of INTC for r8a7740 Simon Horman
2012-12-15  9:03 ` [PATCH 07/10] ARM: shmobile: Include DTSI of r8a7740 to armadillo800eva Simon Horman
2012-12-15  9:03 ` [PATCH 08/10] ARM: shmobile: r8a7740: Use DT initialisation of INTC Simon Horman
2012-12-15  9:03 ` [PATCH 09/10] ARM: shmobile: sh7372: Do not initialise TMU when using DT Simon Horman
2012-12-17  9:58   ` Guennadi Liakhovetski
2012-12-18  0:39     ` Simon Horman
2012-12-15  9:03 ` [PATCH 10/10] ARM: shmobile: sh7372: Use DT initialisation of INTC Simon Horman

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=2838757.yMqfipzqR7@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox