| Submitter | Simon Horman |
|---|---|
| Date | Nov. 19, 2012, 5:13 a.m. |
| Message ID | <1353302007-31045-1-git-send-email-horms@verge.net.au> |
| Download | mbox |
| Permalink | /patch/199926/ |
| State | New |
| Headers | show |
Pull-request
git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git devel/ofComments
Hi Simon, Thank you for the patch. On Monday 19 November 2012 14:13:21 Simon Horman wrote: > From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > Cc: Magnus Damm <damm@opensource.se> > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > Signed-off-by: Simon Horman <horms@verge.net.au> I came up today with pretty much the same patch :-) > --- > arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts index a7505a9..a8e7bd9 > 100644 > --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > @@ -9,7 +9,7 @@ > */ > > /dts-v1/; > -/include/ "skeleton.dtsi" > +/include/ "r8a7740.dtsi" > > / { > model = "armadillo 800 eva"; Shouldn't the compatible property be modified to the following values ? compatible = "renesas,armadillo800eva", "renesas,r8a7740";
On Tue, Nov 20, 2012 at 01:45:20PM +0100, Laurent Pinchart wrote: > Hi Simon, > > Thank you for the patch. > > On Monday 19 November 2012 14:13:21 Simon Horman wrote: > > From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > Cc: Magnus Damm <damm@opensource.se> > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > Signed-off-by: Simon Horman <horms@verge.net.au> > > I came up today with pretty much the same patch :-) > > > --- > > arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts index a7505a9..a8e7bd9 > > 100644 > > --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > @@ -9,7 +9,7 @@ > > */ > > > > /dts-v1/; > > -/include/ "skeleton.dtsi" > > +/include/ "r8a7740.dtsi" > > > > / { > > model = "armadillo 800 eva"; > > Shouldn't the compatible property be modified to the following values ? > > compatible = "renesas,armadillo800eva", "renesas,r8a7740"; To be honest, I'm not sure. But as this is a dts file for a board "renesas,r8a7740" seems strange to me.
Hi Simon, On Wednesday 21 November 2012 09:14:57 Simon Horman wrote: > On Tue, Nov 20, 2012 at 01:45:20PM +0100, Laurent Pinchart wrote: > > Hi Simon, > > > > Thank you for the patch. > > > > On Monday 19 November 2012 14:13:21 Simon Horman wrote: > > > From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > > > Cc: Magnus Damm <damm@opensource.se> > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > Signed-off-by: Simon Horman <horms@verge.net.au> > > > > I came up today with pretty much the same patch :-) > > > > > --- > > > > > > arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts index a7505a9..a8e7bd9 > > > 100644 > > > --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > @@ -9,7 +9,7 @@ > > > > > > */ > > > > > > /dts-v1/; > > > > > > -/include/ "skeleton.dtsi" > > > +/include/ "r8a7740.dtsi" > > > > > > / { > > > > > > model = "armadillo 800 eva"; > > > > Shouldn't the compatible property be modified to the following values ? > > > > compatible = "renesas,armadillo800eva", "renesas,r8a7740"; > > To be honest, I'm not sure. > But as this is a dts file for a board "renesas,r8a7740" seems strange to me. If I understand DT correctly, the goal is to get rid of the board file at some point and only keep the generic r8a7740 board code. Shouldn't we then match on "renesas,r8a7740" ?
On Wed, Nov 21, 2012 at 02:03:02AM +0100, Laurent Pinchart wrote: > Hi Simon, > > On Wednesday 21 November 2012 09:14:57 Simon Horman wrote: > > On Tue, Nov 20, 2012 at 01:45:20PM +0100, Laurent Pinchart wrote: > > > Hi Simon, > > > > > > Thank you for the patch. > > > > > > On Monday 19 November 2012 14:13:21 Simon Horman wrote: > > > > From: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > > > > > Cc: Magnus Damm <damm@opensource.se> > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > Signed-off-by: Simon Horman <horms@verge.net.au> > > > > > > I came up today with pretty much the same patch :-) > > > > > > > --- > > > > > > > > arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > > b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts index a7505a9..a8e7bd9 > > > > 100644 > > > > --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > > +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts > > > > @@ -9,7 +9,7 @@ > > > > > > > > */ > > > > > > > > /dts-v1/; > > > > > > > > -/include/ "skeleton.dtsi" > > > > +/include/ "r8a7740.dtsi" > > > > > > > > / { > > > > > > > > model = "armadillo 800 eva"; > > > > > > Shouldn't the compatible property be modified to the following values ? > > > > > > compatible = "renesas,armadillo800eva", "renesas,r8a7740"; > > > > To be honest, I'm not sure. > > But as this is a dts file for a board "renesas,r8a7740" seems strange to me. > > If I understand DT correctly, the goal is to get rid of the board file at > some point and only keep the generic r8a7740 board code. Shouldn't we > then match on "renesas,r8a7740" ? Yes, I think that makes sense.
Hi, this is an updated series that is still very much work-in-progress. So far contributions bave been made by Magnus Damm, Iwamatsu-san and myself. The aims of this series are: * Adding DT support to INTC * Allowing relevant ARM shmobile SoCs to use DT to initialise INTC * Allowing relevant ARM shmobile boards to use DT to initialise INTC * Allowing the sh73a0 SoC use DT to initialise GIC * Allowing the sh73a0 SoC use DT to initialise GIC, which in turn allows the kzm9g board to use DT to initialise GIC Oustanding work includes: * Work has been done on the sh7372 to consolidate the C code, minimise #ifdefs and make use of a common compatibility string. This work should be reflected in the r8a7740 and sh73a0 SoCs which this series already modifies. * Evaluate why the following, as described by Magnus Damm, doesn't work. "I tried using the INTC DT interrupt controller code with external interrupt pins on the sh7372 mackerel board with DT, but for some reason the following network device refuses to probe due to some interrupt issue. lan9220@14000000 { compatible = "smsc,lan9220", "smsc,lan9115"; reg = <0x14000000 0x2000000>; phy-mode = "mii"; interrupt-parent = <&intca_irq_pins_lo>; interrupts = <0x2c0>; reg-io-width = <4>; smsc,irq-push-pull; }; The above snippet is nicked and adjusted from my earlier prototype patches: [PATCH] sh: INTC IRQ domain and DT support prototype [PATCH] ARM: mach-shmobile: sh7372 DT IRQ prototype I wonder, is there any outstanding INTC work related to IRQ domains?" Diffstat and git information is provided to aid review. Please consider making any changes on top of the branch mentioned below or as modified versions of patches in this series. ---------------------------------------------------------------- The following changes since commit ddffeb8c4d0331609ef2581d84de4d763607bd37: Linux 3.7-rc1 (2012-10-14 14:41:04 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git devel/of for you to fetch changes up to 2a19ffff9917a62df50ceee565e55237bb175bc3: ARM: shmobile: sh73a0: Use DT for GIC (2012-11-19 13:56:34 +0900) ---------------------------------------------------------------- Nobuhiro Iwamatsu (12): ARM: mach-shmobile: Add support OF for INTC of shmobile SH: intc: Add support OF of IRQ ARM: mach-shmobile: Add support OF of INTC for r8a7740 ARM: mach-shmobile: Add support OF of INTC for sh73a0 ARM: mach-shmobile: Add support OF of INTC for sh7372 ARM: mach-shmobile: Add DT table of INTC for sh73a0 ARM: mach-shmobile: Add DT table of INTC for sh7372 ARM: mach-shmobile: Add DT table of INTC for r8a7740 ARM: shmobile: Include DTSI of r8a7740 to armadillo800eva ARM: shmobile: Include DTSI of sh73a0 to kzm9g board ARM: shmobile: r8a7740: Use DT initialisation of INTC ARM: shmobile: sh7372: Use DT initialisation of INTC Simon Horman (3): ARM: shmobile: kzm9g: Use DT initialisation of INTC ARM: shmobile: Add DT table of GIC for sh73a0 ARM: shmobile: sh73a0: Use DT for GIC Documentation/devicetree/bindings/sh/intc.txt | 163 ++++ arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 2 +- arch/arm/boot/dts/r8a7740.dtsi | 965 ++++++++++++++++++++- arch/arm/boot/dts/sh7372.dtsi | 1138 +++++++++++++++++++++++++ arch/arm/boot/dts/sh73a0-kzm9g.dts | 2 +- arch/arm/boot/dts/sh73a0.dtsi | 819 ++++++++++++++++++ arch/arm/mach-shmobile/board-kzm9g.c | 2 +- arch/arm/mach-shmobile/include/mach/common.h | 3 + arch/arm/mach-shmobile/intc-r8a7740.c | 136 ++- arch/arm/mach-shmobile/intc-sh7372.c | 112 ++- arch/arm/mach-shmobile/intc-sh73a0.c | 248 +++++- arch/arm/mach-shmobile/setup-r8a7740.c | 2 +- arch/arm/mach-shmobile/setup-sh7372.c | 2 +- drivers/sh/intc/Makefile | 1 + drivers/sh/intc/core.c | 2 +- drivers/sh/intc/internals.h | 3 +- drivers/sh/intc/irqdomain.c | 6 +- drivers/sh/intc/of_intc.c | 646 ++++++++++++++ include/linux/sh_intc.h | 84 ++ 19 files changed, 4252 insertions(+), 84 deletions(-) create mode 100644 Documentation/devicetree/bindings/sh/intc.txt create mode 100644 arch/arm/boot/dts/sh73a0.dtsi create mode 100644 drivers/sh/intc/of_intc.c