On Thu, Jan 28, 2016 at 11:38 PM, Will Deacon [off-list ref] wrote:

On Thu, Jan 28, 2016 at 10:42:17PM +0530, Ganapatrao Kulkarni wrote:

quoted

On Thu, Jan 28, 2016 at 8:09 PM, Will Deacon [off-list ref] wrote:

quoted

On Tue, Jan 26, 2016 at 02:36:04PM -0600, Bjorn Helgaas wrote:

quoted

Subject is "arm64/arm, numa, dt: adding ..."  What is the significance
of the "arm" part?  The other patches only mention "arm64".

General comment: the code below has little, if anything, that is
actually arm64-specific.  Maybe this is the first DT-based NUMA
platform?  I don't see other similar code for other arches, so maybe
it's too early to try to generalize it, but we should try to avoid
adding duplicates of this code if/when other arches do show up.

Having it in the core code would allow us to share it with arch/arm/
fairly straightforwardly.

This binding can be used for arm too.
however at this moment it is the need of arm64 platforms.
can we please keep this to arm64 as it's too early to try to
generalize it(as Bjorn suggested)
I prefer to keep it as it is, otherwise ok.
Please suggest.

My suggestions time and time again on the NUMA patches from you have
consistently been around consolidation of existing code, or moving things
that aren't architecture-specific out of the architecture code.

thanks, i shall move this out to drivers/of

Will

thanks
Ganapat