* [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees
@ 2013-06-06 5:41 Matt Porter
2013-06-06 5:41 ` [PATCH 1/2] ARM: dts: bcm281xx: use #include for device tree files Matt Porter
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Matt Porter @ 2013-06-06 5:41 UTC (permalink / raw)
To: linux-arm-kernel
This series follows the same approach as taken on Tegra and
OMAP DT files to use the C preprocessor in order to improve
readability. Since bcm281xx does not yet have gpio and pinctrl
support, this series just enables the C preprocessor and removes
the magic constants from existing irq properties. Any new
additions can come in using human readable definitions.
The resulting dtb was diff-tested to validate it versus the non
preprocessed dts version.
Matt Porter (2):
ARM: dts: bcm281xx: use #include for device tree files
ARM: dts: bcm281xx: use existing defines for irqs
arch/arm/boot/dts/bcm11351-brt.dts | 2 +-
arch/arm/boot/dts/bcm11351.dtsi | 9 ++++++---
2 files changed, 7 insertions(+), 4 deletions(-)
--
1.7.9.5
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 1/2] ARM: dts: bcm281xx: use #include for device tree files
2013-06-06 5:41 [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Matt Porter
@ 2013-06-06 5:41 ` Matt Porter
2013-06-06 5:41 ` [PATCH 2/2] ARM: dts: bcm281xx: use existing defines for irqs Matt Porter
2013-06-11 7:03 ` [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Olof Johansson
2 siblings, 0 replies; 6+ messages in thread
From: Matt Porter @ 2013-06-06 5:41 UTC (permalink / raw)
To: linux-arm-kernel
Replace /include/ by #include for bcm281xx device tree
files, enabling use of the C preprocessor.
Signed-off-by: Matt Porter <matt.porter@linaro.org>
---
arch/arm/boot/dts/bcm11351-brt.dts | 2 +-
arch/arm/boot/dts/bcm11351.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm11351-brt.dts b/arch/arm/boot/dts/bcm11351-brt.dts
index 248067c..3392f64 100644
--- a/arch/arm/boot/dts/bcm11351-brt.dts
+++ b/arch/arm/boot/dts/bcm11351-brt.dts
@@ -13,7 +13,7 @@
/dts-v1/;
-/include/ "bcm11351.dtsi"
+#include "bcm11351.dtsi"
/ {
model = "BCM11351 BRT board";
diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi
index 41b2c6c..c08810e 100644
--- a/arch/arm/boot/dts/bcm11351.dtsi
+++ b/arch/arm/boot/dts/bcm11351.dtsi
@@ -11,7 +11,7 @@
* GNU General Public License for more details.
*/
-/include/ "skeleton.dtsi"
+#include "skeleton.dtsi"
/ {
model = "BCM11351 SoC";
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] ARM: dts: bcm281xx: use existing defines for irqs
2013-06-06 5:41 [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Matt Porter
2013-06-06 5:41 ` [PATCH 1/2] ARM: dts: bcm281xx: use #include for device tree files Matt Porter
@ 2013-06-06 5:41 ` Matt Porter
2013-06-11 7:03 ` [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Olof Johansson
2 siblings, 0 replies; 6+ messages in thread
From: Matt Porter @ 2013-06-06 5:41 UTC (permalink / raw)
To: linux-arm-kernel
Use the standard interrupt-controller and ARM GIC constants to
improve the readability of bcm281xx DT irq properties.
Signed-off-by: Matt Porter <matt.porter@linaro.org>
---
arch/arm/boot/dts/bcm11351.dtsi | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/bcm11351.dtsi b/arch/arm/boot/dts/bcm11351.dtsi
index c08810e..824bebe 100644
--- a/arch/arm/boot/dts/bcm11351.dtsi
+++ b/arch/arm/boot/dts/bcm11351.dtsi
@@ -11,6 +11,9 @@
* GNU General Public License for more details.
*/
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+
#include "skeleton.dtsi"
/ {
@@ -41,7 +44,7 @@
status = "disabled";
reg = <0x3e000000 0x1000>;
clock-frequency = <13000000>;
- interrupts = <0x0 67 0x4>;
+ interrupts = <GIC_SPI 67 IRQ_TYPE_LEVEL_HIGH>;
reg-shift = <2>;
reg-io-width = <4>;
};
@@ -56,7 +59,7 @@
timer at 35006000 {
compatible = "bcm,kona-timer";
reg = <0x35006000 0x1000>;
- interrupts = <0x0 7 0x4>;
+ interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
clock-frequency = <32768>;
};
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees
2013-06-06 5:41 [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Matt Porter
2013-06-06 5:41 ` [PATCH 1/2] ARM: dts: bcm281xx: use #include for device tree files Matt Porter
2013-06-06 5:41 ` [PATCH 2/2] ARM: dts: bcm281xx: use existing defines for irqs Matt Porter
@ 2013-06-11 7:03 ` Olof Johansson
2013-06-11 16:56 ` Christian Daudt
2 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2013-06-11 7:03 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 06, 2013 at 01:41:33AM -0400, Matt Porter wrote:
> This series follows the same approach as taken on Tegra and
> OMAP DT files to use the C preprocessor in order to improve
> readability. Since bcm281xx does not yet have gpio and pinctrl
> support, this series just enables the C preprocessor and removes
> the magic constants from existing irq properties. Any new
> additions can come in using human readable definitions.
>
> The resulting dtb was diff-tested to validate it versus the non
> preprocessed dts version.
>
> Matt Porter (2):
> ARM: dts: bcm281xx: use #include for device tree files
> ARM: dts: bcm281xx: use existing defines for irqs
Acked-by: Olof Johansson <olof@lixom.net>
Not sure what else you'll have for 3.11 devicetree-wise, Christian -- your
choice if you want to send a pull request or ack these and we'll apply them
directly.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees
2013-06-11 7:03 ` [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Olof Johansson
@ 2013-06-11 16:56 ` Christian Daudt
2013-06-11 23:40 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Christian Daudt @ 2013-06-11 16:56 UTC (permalink / raw)
To: linux-arm-kernel
On 13-06-11 12:03 AM, Olof Johansson wrote:
> On Thu, Jun 06, 2013 at 01:41:33AM -0400, Matt Porter wrote:
>> This series follows the same approach as taken on Tegra and
>> OMAP DT files to use the C preprocessor in order to improve
>> readability. Since bcm281xx does not yet have gpio and pinctrl
>> support, this series just enables the C preprocessor and removes
>> the magic constants from existing irq properties. Any new
>> additions can come in using human readable definitions.
>>
>> The resulting dtb was diff-tested to validate it versus the non
>> preprocessed dts version.
>>
>> Matt Porter (2):
>> ARM: dts: bcm281xx: use #include for device tree files
>> ARM: dts: bcm281xx: use existing defines for irqs
> Acked-by: Olof Johansson <olof@lixom.net>
Acked-by: Christian Daudt <csd@broadcom.com>
>
> Not sure what else you'll have for 3.11 devicetree-wise, Christian -- your
> choice if you want to send a pull request or ack these and we'll apply them
> directly.
>
I can add them to my tree. I already have a pending dt commit to push.
thanks,
csd
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees
2013-06-11 16:56 ` Christian Daudt
@ 2013-06-11 23:40 ` Olof Johansson
0 siblings, 0 replies; 6+ messages in thread
From: Olof Johansson @ 2013-06-11 23:40 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jun 11, 2013 at 09:56:45AM -0700, Christian Daudt wrote:
> On 13-06-11 12:03 AM, Olof Johansson wrote:
> >On Thu, Jun 06, 2013 at 01:41:33AM -0400, Matt Porter wrote:
> >>This series follows the same approach as taken on Tegra and
> >>OMAP DT files to use the C preprocessor in order to improve
> >>readability. Since bcm281xx does not yet have gpio and pinctrl
> >>support, this series just enables the C preprocessor and removes
> >>the magic constants from existing irq properties. Any new
> >>additions can come in using human readable definitions.
> >>
> >>The resulting dtb was diff-tested to validate it versus the non
> >>preprocessed dts version.
> >>
> >>Matt Porter (2):
> >> ARM: dts: bcm281xx: use #include for device tree files
> >> ARM: dts: bcm281xx: use existing defines for irqs
> >Acked-by: Olof Johansson <olof@lixom.net>
> Acked-by: Christian Daudt <csd@broadcom.com>
> >
> >Not sure what else you'll have for 3.11 devicetree-wise, Christian -- your
> >choice if you want to send a pull request or ack these and we'll apply them
> >directly.
> >
> I can add them to my tree. I already have a pending dt commit to push.
Ok, great.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-06-11 23:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-06 5:41 [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Matt Porter
2013-06-06 5:41 ` [PATCH 1/2] ARM: dts: bcm281xx: use #include for device tree files Matt Porter
2013-06-06 5:41 ` [PATCH 2/2] ARM: dts: bcm281xx: use existing defines for irqs Matt Porter
2013-06-11 7:03 ` [PATCH 0/2] ARM dts: bcm281xx: use preprocessor for device trees Olof Johansson
2013-06-11 16:56 ` Christian Daudt
2013-06-11 23:40 ` Olof Johansson
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).