Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-27 Thread Chunyan Zhang
On 24 February 2017 at 23:07, Rob Herring wrote: > On Fri, Feb 24, 2017 at 1:57 AM, Chunyan Zhang > wrote: >> On 四, 2月 23, 2017 at 06:00:20下午 -0600, Rob Herring wrote: >>> On Tue, Feb 21, 2017 at 12:55 AM, Chunyan Zhang >>> wrote: >>> > From: Orson Zhai >>> > >>> > SC9860G is a 8 cores of A53

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-24 Thread Rob Herring
On Fri, Feb 24, 2017 at 1:57 AM, Chunyan Zhang wrote: > On 四, 2月 23, 2017 at 06:00:20下午 -0600, Rob Herring wrote: >> On Tue, Feb 21, 2017 at 12:55 AM, Chunyan Zhang >> wrote: >> > From: Orson Zhai >> > >> > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. >> > >> > Accor

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-24 Thread Chunyan Zhang
On 四, 2月 23, 2017 at 06:00:20下午 -0600, Rob Herring wrote: > On Tue, Feb 21, 2017 at 12:55 AM, Chunyan Zhang > wrote: > > From: Orson Zhai > > > > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. > > > > According to regular hierarchy of sprd dts, whale2.dtsi contains SoC

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-23 Thread Rob Herring
On Tue, Feb 21, 2017 at 12:55 AM, Chunyan Zhang wrote: > From: Orson Zhai > > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. > > According to regular hierarchy of sprd dts, whale2.dtsi contains SoC > peripherals IP nodes, sc9860.dtsi contains stuff related to ARM core st

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-23 Thread Sudeep Holla
Hi Chunyan, On 23/02/17 06:20, Chunyan Zhang wrote: [...] >>> I noticed Juno use the 2), would you suggest that way? >> >> It is better to describe the HW component themselves rather than where >> they are in the topology - the address of the component will make sure >> the names are unique. So

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-22 Thread Chunyan Zhang
[...] >>> > + >>> > + soc { >>> > + soc_funnel: funnel@10001000 { >>> >>> There is no need for a label ("soc_funnel) before the device name if that >>> device is not referenced elsewhere in the DTS. The same comment applies to >>> most >>> of the component listed below. >>> >> >> OK,

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-22 Thread Mathieu Poirier
On 21 February 2017 at 20:46, Chunyan Zhang wrote: > Hello Mathieu, > > On 二, 2月 21, 2017 at 09:27:44上午 -0700, Mathieu Poirier wrote: >> On Tue, Feb 21, 2017 at 02:55:02PM +0800, Chunyan Zhang wrote: >> > From: Orson Zhai >> >> Hello Chunyan, >> >> > >> > SC9860G is a 8 cores of A53 SoC with 4G

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-21 Thread Chunyan Zhang
Hello Mathieu, On 二, 2月 21, 2017 at 09:27:44上午 -0700, Mathieu Poirier wrote: > On Tue, Feb 21, 2017 at 02:55:02PM +0800, Chunyan Zhang wrote: > > From: Orson Zhai > > Hello Chunyan, > > > > > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. > > > > According to regula

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-21 Thread Mathieu Poirier
On Tue, Feb 21, 2017 at 02:55:02PM +0800, Chunyan Zhang wrote: > From: Orson Zhai Hello Chunyan, > > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. > > According to regular hierarchy of sprd dts, whale2.dtsi contains SoC > peripherals IP nodes, sc9860.dtsi contains st

Re: [PATCH V2 1/3] arm64: dts: Add basic DT to support Spreadtrum's SP9860G

2017-02-21 Thread Sudeep Holla
On 21/02/17 06:55, Chunyan Zhang wrote: > From: Orson Zhai > > SC9860G is a 8 cores of A53 SoC with 4G LTE support SoC from Spreadtrum. > > According to regular hierarchy of sprd dts, whale2.dtsi contains SoC > peripherals IP nodes, sc9860.dtsi contains stuff related to ARM core stuff > and sp