From mboxrd@z Thu Jan 1 00:00:00 1970 From: sudeep.holla@arm.com (Sudeep Holla) Date: Wed, 30 Dec 2015 15:34:09 +0000 Subject: [PATCH] arm64: Add DT support for Juno r2 board In-Reply-To: References: <1450872611-15326-1-git-send-email-sudeep.holla@arm.com> <20151229183521.GL12450@rob-hp-laptop> <5683BE7A.9020404@arm.com> Message-ID: <5683F971.9000300@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 30/12/15 15:09, Rob Herring wrote: > On Wed, Dec 30, 2015 at 5:22 AM, Sudeep Holla wrote: >> Hi Rob, >> >> Thanks for the review. >> >> On 29/12/15 18:35, Rob Herring wrote: >>> >>> On Wed, Dec 23, 2015 at 12:10:11PM +0000, Sudeep Holla wrote: >>>> >>>> Juno r2 is identical to Juno r1 with Cortex A57 cores replaced by >>>> Cortex A72 cores. >>> >>> >>> That's a bigger change than I'd expect... >>> >> >> I am not sure if I understood what you meant by that. Do you want me to >> elaborate with more details ? > > No, just my commentary that I'd expect r1 -> r2 to be a board or Si > spin rather than a whole new core. > Ah OK, yes it's new Si and has other minor updates like latest revisions of Cortex-A53 and other IPs. I just mentioned only the DT visible changes in the commit log. >>>> Acked-by: Liviu Dudau >>>> Signed-off-by: Sudeep Holla >>>> --- >>>> Documentation/devicetree/bindings/arm/arm-boards | 1 + >>>> Documentation/devicetree/bindings/arm/cpus.txt | 1 + >>> >>> >>> This should probably be a separate patch, but I already have a patch in >>> my tree for 4.5 adding arm,cortex-a72. So just drop this hunk. >>> >> >> OK, will post DT binding separately. > > No need, it is already in my tree. > Sorry for not being clear earlier. I was referring the "arm,juno-r2" compatible addition. I assume you have no objection if it's part of the same patch. >>>> + pcie-controller at 40000000 { >>>> + compatible = "arm,juno-r1-pcie", "plda,xpressrich3-axi", >>>> "pci-host-ecam-generic"; >>> >>> >>> Is this common with r1? >>> >> >> AFAIK it is. Liviu can you confirm ? > > Then perhaps you should move this to a common spot. > True, will wait for Liviu's response. -- Regards, Sudeep