From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olof Johansson Subject: Re: [PATCH] ARM: dts: vexpress: Add CCI node to TC2 device-tree Date: Fri, 30 Aug 2013 09:30:17 -0700 Message-ID: References: <1377872760.3655.48.camel@linaro1.home> <20130830161327.GA10045@e102568-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130830161327.GA10045@e102568-lin.cambridge.arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Lorenzo Pieralisi Cc: "Jon Medhurst (Tixy)" , "devicetree@vger.kernel.org" , Pawel Moll , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On Fri, Aug 30, 2013 at 9:13 AM, Lorenzo Pieralisi wrote: > Now that MCPM and CPU idle are heading to mainline, it should get queued asap, > thanks for that, probably as a fix at -rc1, since it is quite late now. This is not a regression, it's a missing piece of a feature. It's not post-rc1 material. It's also an indication that nobody actually tested the feature branch before the pull request was sent. Hrmpf. Shouldn't you be updating some vexpress_defconfig and probably multi_v7_defconfig to at least get some build coverage for this as well, by the way? I really should hold this off for 3.13, but I'll apply it to the late/ branch anyway. -Olof From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Fri, 30 Aug 2013 09:30:17 -0700 Subject: [PATCH] ARM: dts: vexpress: Add CCI node to TC2 device-tree In-Reply-To: <20130830161327.GA10045@e102568-lin.cambridge.arm.com> References: <1377872760.3655.48.camel@linaro1.home> <20130830161327.GA10045@e102568-lin.cambridge.arm.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Aug 30, 2013 at 9:13 AM, Lorenzo Pieralisi wrote: > Now that MCPM and CPU idle are heading to mainline, it should get queued asap, > thanks for that, probably as a fix at -rc1, since it is quite late now. This is not a regression, it's a missing piece of a feature. It's not post-rc1 material. It's also an indication that nobody actually tested the feature branch before the pull request was sent. Hrmpf. Shouldn't you be updating some vexpress_defconfig and probably multi_v7_defconfig to at least get some build coverage for this as well, by the way? I really should hold this off for 3.13, but I'll apply it to the late/ branch anyway. -Olof