* [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string
@ 2013-11-20 0:07 Magnus Damm
2013-11-21 4:59 ` Simon Horman
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Magnus Damm @ 2013-11-20 0:07 UTC (permalink / raw)
To: linux-arm-kernel
From: Magnus Damm <damm@opensource.se>
Add "renesas,irqc-r8a7790" to the compatible string for IRQC
in case of r8a7790. This makes the IRQC follow the same style
as the other devices and also makes it more future proof.
Signed-off-by: Magnus Damm <damm@opensource.se>
---
arch/arm/boot/dts/r8a7790.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- 0003/arch/arm/boot/dts/r8a7790.dtsi
+++ work/arch/arm/boot/dts/r8a7790.dtsi 2013-11-20 08:20:30.000000000 +0900
@@ -171,7 +171,7 @@
};
irqc0: interrupt-controller at e61c0000 {
- compatible = "renesas,irqc";
+ compatible = "renesas,irqc-r8a7790", "renesas,irqc";
#interrupt-cells = <2>;
interrupt-controller;
reg = <0 0xe61c0000 0 0x200>;
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string
2013-11-20 0:07 [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string Magnus Damm
@ 2013-11-21 4:59 ` Simon Horman
2013-11-27 10:11 ` Laurent Pinchart
2013-11-28 6:36 ` Simon Horman
2 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2013-11-21 4:59 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Nov 20, 2013 at 09:07:40AM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Add "renesas,irqc-r8a7790" to the compatible string for IRQC
> in case of r8a7790. This makes the IRQC follow the same style
> as the other devices and also makes it more future proof.
>
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
>
> arch/arm/boot/dts/r8a7790.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks, I will queue this up.
Assuming all goes will I plan to push this and other updates
today or tomorrow.
>
> --- 0003/arch/arm/boot/dts/r8a7790.dtsi
> +++ work/arch/arm/boot/dts/r8a7790.dtsi 2013-11-20 08:20:30.000000000 +0900
> @@ -171,7 +171,7 @@
> };
>
> irqc0: interrupt-controller at e61c0000 {
> - compatible = "renesas,irqc";
> + compatible = "renesas,irqc-r8a7790", "renesas,irqc";
> #interrupt-cells = <2>;
> interrupt-controller;
> reg = <0 0xe61c0000 0 0x200>;
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string
2013-11-20 0:07 [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string Magnus Damm
2013-11-21 4:59 ` Simon Horman
@ 2013-11-27 10:11 ` Laurent Pinchart
2013-11-28 6:30 ` Simon Horman
2013-11-28 6:36 ` Simon Horman
2 siblings, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2013-11-27 10:11 UTC (permalink / raw)
To: linux-arm-kernel
Hi Magnus,
Thank you for the patch.
On Wednesday 20 November 2013 09:07:40 Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Add "renesas,irqc-r8a7790" to the compatible string for IRQC
> in case of r8a7790. This makes the IRQC follow the same style
> as the other devices and also makes it more future proof.
You should update the device tree bindings documentation as well. As there's
no such documentation of the irqc, you should create it ;-)
> Signed-off-by: Magnus Damm <damm@opensource.se>
> ---
>
> arch/arm/boot/dts/r8a7790.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- 0003/arch/arm/boot/dts/r8a7790.dtsi
> +++ work/arch/arm/boot/dts/r8a7790.dtsi 2013-11-20 08:20:30.000000000 +0900
> @@ -171,7 +171,7 @@
> };
>
> irqc0: interrupt-controller at e61c0000 {
> - compatible = "renesas,irqc";
> + compatible = "renesas,irqc-r8a7790", "renesas,irqc";
> #interrupt-cells = <2>;
> interrupt-controller;
> reg = <0 0xe61c0000 0 0x200>;
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string
2013-11-27 10:11 ` Laurent Pinchart
@ 2013-11-28 6:30 ` Simon Horman
0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2013-11-28 6:30 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Nov 27, 2013 at 11:11:47AM +0100, Laurent Pinchart wrote:
> Hi Magnus,
>
> Thank you for the patch.
>
> On Wednesday 20 November 2013 09:07:40 Magnus Damm wrote:
> > From: Magnus Damm <damm@opensource.se>
> >
> > Add "renesas,irqc-r8a7790" to the compatible string for IRQC
> > in case of r8a7790. This makes the IRQC follow the same style
> > as the other devices and also makes it more future proof.
>
> You should update the device tree bindings documentation as well. As there's
> no such documentation of the irqc, you should create it ;-)
I will submit a patch for this.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string
2013-11-20 0:07 [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string Magnus Damm
2013-11-21 4:59 ` Simon Horman
2013-11-27 10:11 ` Laurent Pinchart
@ 2013-11-28 6:36 ` Simon Horman
2 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2013-11-28 6:36 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Nov 20, 2013 at 09:07:40AM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
>
> Add "renesas,irqc-r8a7790" to the compatible string for IRQC
> in case of r8a7790. This makes the IRQC follow the same style
> as the other devices and also makes it more future proof.
>
> Signed-off-by: Magnus Damm <damm@opensource.se>
Should I make a similar patch for the r8a73a4?
> ---
>
> arch/arm/boot/dts/r8a7790.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- 0003/arch/arm/boot/dts/r8a7790.dtsi
> +++ work/arch/arm/boot/dts/r8a7790.dtsi 2013-11-20 08:20:30.000000000 +0900
> @@ -171,7 +171,7 @@
> };
>
> irqc0: interrupt-controller at e61c0000 {
> - compatible = "renesas,irqc";
> + compatible = "renesas,irqc-r8a7790", "renesas,irqc";
> #interrupt-cells = <2>;
> interrupt-controller;
> reg = <0 0xe61c0000 0 0x200>;
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-28 6:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-20 0:07 [PATCH] ARM: shmobile: Use r8a7790 suffix for IRQC compat string Magnus Damm
2013-11-21 4:59 ` Simon Horman
2013-11-27 10:11 ` Laurent Pinchart
2013-11-28 6:30 ` Simon Horman
2013-11-28 6:36 ` Simon Horman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).