* [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings
@ 2018-03-21 13:44 Felipe Balbi
2018-03-21 13:51 ` Tom Rini
0 siblings, 1 reply; 4+ messages in thread
From: Felipe Balbi @ 2018-03-21 13:44 UTC (permalink / raw)
To: u-boot
The following warnings are fixed:
arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing interrupt-controller or interrupt-map property in /ocp/gpmc at 50000000
arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing #interrupt-cells in interrupt-parent /ocp/gpmc at 50000000
arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
The fix was basically to copy the missing data from mainline linux.
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
---
arch/arm/dts/am4372.dtsi | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/arch/arm/dts/am4372.dtsi b/arch/arm/dts/am4372.dtsi
index 3ffa8e016e08..5c205f0ff0fa 100644
--- a/arch/arm/dts/am4372.dtsi
+++ b/arch/arm/dts/am4372.dtsi
@@ -764,7 +764,8 @@
reg = <0x48038000 0x2000>,
<0x46000000 0x400000>;
reg-names = "mpu", "dat";
- interrupts = <80>, <81>;
+ interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
interrupt-names = "tx", "rx";
status = "disabled";
dmas = <&edma 8>,
@@ -778,7 +779,8 @@
reg = <0x4803C000 0x2000>,
<0x46400000 0x400000>;
reg-names = "mpu", "dat";
- interrupts = <82>, <83>;
+ interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>;
interrupt-names = "tx", "rx";
status = "disabled";
dmas = <&edma 10>,
@@ -807,6 +809,8 @@
gpmc,num-waitpins = <2>;
#address-cells = <2>;
#size-cells = <1>;
+ interrupt-controller;
+ #interrupt-cells = <2>;
status = "disabled";
};
--
2.16.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings
2018-03-21 13:44 [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings Felipe Balbi
@ 2018-03-21 13:51 ` Tom Rini
2018-03-22 7:57 ` Felipe Balbi
0 siblings, 1 reply; 4+ messages in thread
From: Tom Rini @ 2018-03-21 13:51 UTC (permalink / raw)
To: u-boot
On Wed, Mar 21, 2018 at 03:44:45PM +0200, Felipe Balbi wrote:
> The following warnings are fixed:
>
> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing interrupt-controller or interrupt-map property in /ocp/gpmc at 50000000
> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing #interrupt-cells in interrupt-parent /ocp/gpmc at 50000000
> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
>
> The fix was basically to copy the missing data from mainline linux.
Shouldn't we just re-sync with v4.16-rc? Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180321/2aeb4a8b/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings
2018-03-21 13:51 ` Tom Rini
@ 2018-03-22 7:57 ` Felipe Balbi
2018-03-22 11:15 ` Tom Rini
0 siblings, 1 reply; 4+ messages in thread
From: Felipe Balbi @ 2018-03-22 7:57 UTC (permalink / raw)
To: u-boot
Hi,
Tom Rini <trini@konsulko.com> writes:
> On Wed, Mar 21, 2018 at 03:44:45PM +0200, Felipe Balbi wrote:
>> The following warnings are fixed:
>>
>> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
>> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
>> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing interrupt-controller or interrupt-map property in /ocp/gpmc at 50000000
>> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing #interrupt-cells in interrupt-parent /ocp/gpmc at 50000000
>> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
>> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
>> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
>> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
>> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
>> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
>>
>> The fix was basically to copy the missing data from mainline linux.
>
> Shouldn't we just re-sync with v4.16-rc? Thanks!
sure, if you prefer doing that. But in that case, it seems like
extracting Linux's DTS to its own project (linux-dts.git??) and using
that as submodule would be far better.
--
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180322/b18d4b22/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings
2018-03-22 7:57 ` Felipe Balbi
@ 2018-03-22 11:15 ` Tom Rini
0 siblings, 0 replies; 4+ messages in thread
From: Tom Rini @ 2018-03-22 11:15 UTC (permalink / raw)
To: u-boot
On Thu, Mar 22, 2018 at 09:57:46AM +0200, Felipe Balbi wrote:
>
> Hi,
>
> Tom Rini <trini@konsulko.com> writes:
> > On Wed, Mar 21, 2018 at 03:44:45PM +0200, Felipe Balbi wrote:
> >> The following warnings are fixed:
> >>
> >> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> >> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> >> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing interrupt-controller or interrupt-map property in /ocp/gpmc at 50000000
> >> arch/arm/dts/am43x-epos-evm.dtb: Warning (interrupts_property): Missing #interrupt-cells in interrupt-parent /ocp/gpmc at 50000000
> >> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> >> arch/arm/dts/am437x-gp-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> >> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> >> arch/arm/dts/am437x-idk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> >> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 48038000
> >> arch/arm/dts/am437x-sk-evm.dtb: Warning (interrupts_property): interrupts size is (8), expected multiple of 12 in /ocp/mcasp at 4803C000
> >>
> >> The fix was basically to copy the missing data from mainline linux.
> >
> > Shouldn't we just re-sync with v4.16-rc? Thanks!
>
> sure, if you prefer doing that. But in that case, it seems like
> extracting Linux's DTS to its own project (linux-dts.git??) and using
> that as submodule would be far better.
Except we also have our own files in arch/*/dts/ so we expect people to
just sync the DTS files they use as-needed.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180322/4a3cf1b6/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-22 11:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-21 13:44 [U-Boot] [PATCH] arm: dts: am4372: fix DTC warnings Felipe Balbi
2018-03-21 13:51 ` Tom Rini
2018-03-22 7:57 ` Felipe Balbi
2018-03-22 11:15 ` Tom Rini
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.