Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On 08/30/2016 03:58 AM, Shawn Guo wrote: > On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: >> Add DDR memory controller nodes to enable EDAC driver. >> >> Signed-off-by: York Sun > > The patch subject is too general. I changed it to "arm64: dts: Add DDR > memory

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On 08/30/2016 03:58 AM, Shawn Guo wrote: > On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: >> Add DDR memory controller nodes to enable EDAC driver. >> >> Signed-off-by: York Sun > > The patch subject is too general. I changed it to "arm64: dts: Add DDR > memory controller for

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun The patch subject is too general. I changed it to "arm64: dts: Add DDR memory controller for Layerscape SoCs", and applied patch.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun The patch subject is too general. I changed it to "arm64: dts: Add DDR memory controller for Layerscape SoCs", and applied patch. Shawn

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 02:39:21PM -0700, Olof Johansson wrote: > DT changes need to go through arm-soc. It's how we've been operating > for several years now. Ok ok, we've wasted enough time with this. So you guys pick up this one, I'll take the rest. Thanks. -- Regards/Gruss, Boris.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 02:39:21PM -0700, Olof Johansson wrote: > DT changes need to go through arm-soc. It's how we've been operating > for several years now. Ok ok, we've wasted enough time with this. So you guys pick up this one, I'll take the rest. Thanks. -- Regards/Gruss, Boris.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 1:05 AM, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: >> Is it mandatory to have DTS changes go with driver part altogether? > > Yes, because the EDAC driver needs them to even load properly. > >> Otherwise, I prefer

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 1:05 AM, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: >> Is it mandatory to have DTS changes go with driver part altogether? > > Yes, because the EDAC driver needs them to even load properly. > >> Otherwise, I prefer to have them go

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 03:51:30PM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > > To avoid potential merge conflicts. > > Haven't heard of any so far. And I don't see how adding 1 or 2 DT > entries more per driver is a serious merge conflict.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 03:51:30PM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > > To avoid potential merge conflicts. > > Haven't heard of any so far. And I don't see how adding 1 or 2 DT > entries more per driver is a serious merge conflict.

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > To avoid potential merge conflicts. Haven't heard of any so far. And I don't see how adding 1 or 2 DT entries more per driver is a serious merge conflict. > Unless there are hard dependencies like making it compile, avoiding >

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 04:33:50PM +0800, Shawn Guo wrote: > To avoid potential merge conflicts. Haven't heard of any so far. And I don't see how adding 1 or 2 DT entries more per driver is a serious merge conflict. > Unless there are hard dependencies like making it compile, avoiding >

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 10:05:30AM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: > > Is it mandatory to have DTS changes go with driver part altogether? > > Yes, because the EDAC driver needs them to even load properly. When separate branches get

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 10:05:30AM +0200, Borislav Petkov wrote: > On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: > > Is it mandatory to have DTS changes go with driver part altogether? > > Yes, because the EDAC driver needs them to even load properly. When separate branches get

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: > Is it mandatory to have DTS changes go with driver part altogether? Yes, because the EDAC driver needs them to even load properly. > Otherwise, I prefer to have them go through separate tree. Any particular reason why you prefer that?

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Mon, Aug 29, 2016 at 02:34:48PM +0800, Shawn Guo wrote: > Is it mandatory to have DTS changes go with driver part altogether? Yes, because the EDAC driver needs them to even load properly. > Otherwise, I prefer to have them go through separate tree. Any particular reason why you prefer that?

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Fri, Aug 12, 2016 at 11:13:54AM +0200, Borislav Petkov wrote: > On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > > Add DDR memory controller nodes to enable EDAC driver. > > > > Signed-off-by: York Sun > > > > --- > > Change log > > v4: no change > > v3: no

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Fri, Aug 12, 2016 at 11:13:54AM +0200, Borislav Petkov wrote: > On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > > Add DDR memory controller nodes to enable EDAC driver. > > > > Signed-off-by: York Sun > > > > --- > > Change log > > v4: no change > > v3: no change > > v2: no

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

+Shawn Guo On 08/09/2016 03:00 PM, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun > > --- > Change log > v4: no change > v3: no change > v2: no change > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7 +++ >

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

+Shawn Guo On 08/09/2016 03:00 PM, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun > > --- > Change log > v4: no change > v3: no change > v2: no change > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7 +++ >

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun > > --- > Change log > v4: no change > v3: no change > v2: no change > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7

Re: [Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

On Tue, Aug 09, 2016 at 02:59:39PM -0700, York Sun wrote: > Add DDR memory controller nodes to enable EDAC driver. > > Signed-off-by: York Sun > > --- > Change log > v4: no change > v3: no change > v2: no change > > arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7 +++ >

[Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

Add DDR memory controller nodes to enable EDAC driver. Signed-off-by: York Sun --- Change log v4: no change v3: no change v2: no change arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7 +++ arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 14 ++ 2

[Patch v4 9/9] arm64: Update device tree for Layerscape SoCs

Add DDR memory controller nodes to enable EDAC driver. Signed-off-by: York Sun --- Change log v4: no change v3: no change v2: no change arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi | 7 +++ arch/arm64/boot/dts/freescale/fsl-ls2080a.dtsi | 14 ++ 2 files changed, 21