All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.