[PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-11-25 Thread Ian Campbell
Currently I see:
  DTC arch/powerpc/boot/kilauea.dtb
Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has invalid 
length (12 bytes) (#address-cells == 1, #size-cells == 1)

It appears that unlike the other platforms handled by 3fb7933850fa
powerpc/4xx: Adding PCIe MSI support this platform does not use 
address-cells=2.

Signed-off-by: Ian Campbell ian.campb...@citrix.com
Acked-by: Josh Boyer jwbo...@gmail.com
Cc: Rupjyoti Sarmah rsar...@apm.com
Cc: Tirumala R Marri tma...@apm.com
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Cc: devicet...@vger.kernel.org (open list:OPEN FIRMWARE AND...)
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-ker...@vger.kernel.org
---
Resending, this hasn't been picked up since June
http://patchwork.ozlabs.org/patch/248234/
---
 arch/powerpc/boot/dts/kilauea.dts |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/dts/kilauea.dts 
b/arch/powerpc/boot/dts/kilauea.dts
index 1613d6e..5ba7f01 100644
--- a/arch/powerpc/boot/dts/kilauea.dts
+++ b/arch/powerpc/boot/dts/kilauea.dts
@@ -406,7 +406,7 @@
 
MSI: ppc4xx-msi@C1000 {
compatible = amcc,ppc4xx-msi, ppc4xx-msi;
-   reg =  0x0 0xEF62 0x100;
+   reg = 0xEF62 0x100;
sdr-base = 0x4B0;
msi-data = 0x;
msi-mask = 0x;
-- 
1.7.10.4

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-11-25 Thread Josh Boyer
On Mon, Nov 25, 2013 at 4:40 AM, Ian Campbell ian.campb...@citrix.com wrote:
 Currently I see:
   DTC arch/powerpc/boot/kilauea.dtb
 Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has invalid 
 length (12 bytes) (#address-cells == 1, #size-cells == 1)

 It appears that unlike the other platforms handled by 3fb7933850fa
 powerpc/4xx: Adding PCIe MSI support this platform does not use 
 address-cells=2.

 Signed-off-by: Ian Campbell ian.campb...@citrix.com
 Acked-by: Josh Boyer jwbo...@gmail.com
 Cc: Rupjyoti Sarmah rsar...@apm.com
 Cc: Tirumala R Marri tma...@apm.com
 Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
 Cc: Paul Mackerras pau...@samba.org
 Cc: devicet...@vger.kernel.org (open list:OPEN FIRMWARE AND...)
 Cc: linuxppc-dev@lists.ozlabs.org
 Cc: linux-ker...@vger.kernel.org
 ---
 Resending, this hasn't been picked up since June
 http://patchwork.ozlabs.org/patch/248234/

Ben, please pick this up.

josh

 ---
  arch/powerpc/boot/dts/kilauea.dts |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/powerpc/boot/dts/kilauea.dts 
 b/arch/powerpc/boot/dts/kilauea.dts
 index 1613d6e..5ba7f01 100644
 --- a/arch/powerpc/boot/dts/kilauea.dts
 +++ b/arch/powerpc/boot/dts/kilauea.dts
 @@ -406,7 +406,7 @@

 MSI: ppc4xx-msi@C1000 {
 compatible = amcc,ppc4xx-msi, ppc4xx-msi;
 -   reg =  0x0 0xEF62 0x100;
 +   reg = 0xEF62 0x100;
 sdr-base = 0x4B0;
 msi-data = 0x;
 msi-mask = 0x;
 --
 1.7.10.4

 --
 To unsubscribe from this list: send the line unsubscribe linux-kernel in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 Please read the FAQ at  http://www.tux.org/lkml/
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-09-24 Thread Ian Campbell
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote:
 On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote:
 Currently I see:
   DTC arch/powerpc/boot/kilauea.dtb
 Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has 
 invalid length (12 bytes) (#address-cells == 1, #size-cells == 1)
 
 It appears that unlike the other plarforms handled by 3fb7933850fa
 powerpc/4xx: Adding PCIe MSI support this platform does not use 
 address-cells=2.
 
 Right, it's a 405, not a 440.  I should have caught that in the initial
 review.
 
 Signed-off-by: Ian Campbell ian.campb...@citrix.com
 Cc: Rupjyoti Sarmah rsar...@apm.com
 Cc: Tirumala R Marri tma...@apm.com
 Cc: Josh Boyer jwbo...@linux.vnet.ibm.com
 
 That address should bounce.  It hasn't been active in almost 2 years.
 
 Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
 Cc: Paul Mackerras pau...@samba.org
 Cc: linuxppc-dev@lists.ozlabs.org
 Cc: linux-ker...@vger.kernel.org
 
 Acked-by: Josh Boyer jwbo...@gmail.com

This still doesn't appear to be fixed v3.12-rc2. Ping?

 
 ---
  arch/powerpc/boot/dts/kilauea.dts |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/powerpc/boot/dts/kilauea.dts 
 b/arch/powerpc/boot/dts/kilauea.dts
 index 1613d6e..5ba7f01 100644
 --- a/arch/powerpc/boot/dts/kilauea.dts
 +++ b/arch/powerpc/boot/dts/kilauea.dts
 @@ -406,7 +406,7 @@
  
  MSI: ppc4xx-msi@C1000 {
  compatible = amcc,ppc4xx-msi, ppc4xx-msi;
 -reg =  0x0 0xEF62 0x100;
 +reg = 0xEF62 0x100;
  sdr-base = 0x4B0;
  msi-data = 0x;
  msi-mask = 0x;
 -- 
 1.7.10.4
 
 ___
 Linuxppc-dev mailing list
 Linuxppc-dev@lists.ozlabs.org
 https://lists.ozlabs.org/listinfo/linuxppc-dev


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


[PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-06-03 Thread Ian Campbell
Currently I see:
  DTC arch/powerpc/boot/kilauea.dtb
Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has invalid 
length (12 bytes) (#address-cells == 1, #size-cells == 1)

It appears that unlike the other plarforms handled by 3fb7933850fa
powerpc/4xx: Adding PCIe MSI support this platform does not use 
address-cells=2.

Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Rupjyoti Sarmah rsar...@apm.com
Cc: Tirumala R Marri tma...@apm.com
Cc: Josh Boyer jwbo...@linux.vnet.ibm.com
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-ker...@vger.kernel.org
---
 arch/powerpc/boot/dts/kilauea.dts |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/dts/kilauea.dts 
b/arch/powerpc/boot/dts/kilauea.dts
index 1613d6e..5ba7f01 100644
--- a/arch/powerpc/boot/dts/kilauea.dts
+++ b/arch/powerpc/boot/dts/kilauea.dts
@@ -406,7 +406,7 @@
 
MSI: ppc4xx-msi@C1000 {
compatible = amcc,ppc4xx-msi, ppc4xx-msi;
-   reg =  0x0 0xEF62 0x100;
+   reg = 0xEF62 0x100;
sdr-base = 0x4B0;
msi-data = 0x;
msi-mask = 0x;
-- 
1.7.10.4

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-06-03 Thread Josh Boyer
On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote:
Currently I see:
  DTC arch/powerpc/boot/kilauea.dtb
Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has invalid 
length (12 bytes) (#address-cells == 1, #size-cells == 1)

It appears that unlike the other plarforms handled by 3fb7933850fa
powerpc/4xx: Adding PCIe MSI support this platform does not use 
address-cells=2.

Right, it's a 405, not a 440.  I should have caught that in the initial
review.

Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Rupjyoti Sarmah rsar...@apm.com
Cc: Tirumala R Marri tma...@apm.com
Cc: Josh Boyer jwbo...@linux.vnet.ibm.com

That address should bounce.  It hasn't been active in almost 2 years.

Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-ker...@vger.kernel.org

Acked-by: Josh Boyer jwbo...@gmail.com

---
 arch/powerpc/boot/dts/kilauea.dts |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/dts/kilauea.dts 
b/arch/powerpc/boot/dts/kilauea.dts
index 1613d6e..5ba7f01 100644
--- a/arch/powerpc/boot/dts/kilauea.dts
+++ b/arch/powerpc/boot/dts/kilauea.dts
@@ -406,7 +406,7 @@
 
   MSI: ppc4xx-msi@C1000 {
   compatible = amcc,ppc4xx-msi, ppc4xx-msi;
-  reg =  0x0 0xEF62 0x100;
+  reg = 0xEF62 0x100;
   sdr-base = 0x4B0;
   msi-data = 0x;
   msi-mask = 0x;
-- 
1.7.10.4

___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev


Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-06-03 Thread Ian Campbell
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote:
 On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote:
 Currently I see:
   DTC arch/powerpc/boot/kilauea.dtb
 Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has 
 invalid length (12 bytes) (#address-cells == 1, #size-cells == 1)
 
 It appears that unlike the other plarforms handled by 3fb7933850fa
 powerpc/4xx: Adding PCIe MSI support this platform does not use 
 address-cells=2.
 
 Right, it's a 405, not a 440.  I should have caught that in the initial
 review.
 
 Signed-off-by: Ian Campbell ian.campb...@citrix.com
 Cc: Rupjyoti Sarmah rsar...@apm.com
 Cc: Tirumala R Marri tma...@apm.com
 Cc: Josh Boyer jwbo...@linux.vnet.ibm.com
 
 That address should bounce.  It hasn't been active in almost 2 years.

Funnily enough the previous patch to this file (where I nabbed the
address from) was approximately that long ago ;-)

I've updated my local copy with your Ack with the correct address,
thanks.

I also corrected the plarforms typo which I just spotted...

Ian.

 Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
 Cc: Paul Mackerras pau...@samba.org
 Cc: linuxppc-dev@lists.ozlabs.org
 Cc: linux-ker...@vger.kernel.org
 
 Acked-by: Josh Boyer jwbo...@gmail.com
 
 ---
  arch/powerpc/boot/dts/kilauea.dts |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/powerpc/boot/dts/kilauea.dts 
 b/arch/powerpc/boot/dts/kilauea.dts
 index 1613d6e..5ba7f01 100644
 --- a/arch/powerpc/boot/dts/kilauea.dts
 +++ b/arch/powerpc/boot/dts/kilauea.dts
 @@ -406,7 +406,7 @@
  
  MSI: ppc4xx-msi@C1000 {
  compatible = amcc,ppc4xx-msi, ppc4xx-msi;
 -reg =  0x0 0xEF62 0x100;
 +reg = 0xEF62 0x100;
  sdr-base = 0x4B0;
  msi-data = 0x;
  msi-mask = 0x;
 -- 
 1.7.10.4
 
 ___
 Linuxppc-dev mailing list
 Linuxppc-dev@lists.ozlabs.org
 https://lists.ozlabs.org/listinfo/linuxppc-dev


___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev