From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754919AbaAIIQB (ORCPT ); Thu, 9 Jan 2014 03:16:01 -0500 Received: from mail-pa0-f54.google.com ([209.85.220.54]:59445 "EHLO mail-pa0-f54.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751066AbaAIIPy (ORCPT ); Thu, 9 Jan 2014 03:15:54 -0500 Date: Thu, 9 Jan 2014 00:13:52 -0800 From: Olof Johansson To: Stephen Boyd Cc: Rob Clark , arm@kernel.org, Linux Kernel Mailing List , linux-arm-msm@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" , David Brown , Dave Airlie , "dri-devel@lists.freedesktop.org" Subject: Re: [PATCH v2 6/7] drm/msm: Drop unnecessary mach include Message-ID: <20140109081352.GI8285@quad.lixom.net> References: <1388438133-431-1-git-send-email-sboyd@codeaurora.org> <1388438133-431-7-git-send-email-sboyd@codeaurora.org> <20131231051558.GB14405@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20131231051558.GB14405@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 30, 2013 at 09:15:58PM -0800, Stephen Boyd wrote: > On 12/30, Rob Clark wrote: > > On Mon, Dec 30, 2013 at 8:12 PM, Rob Clark wrote: > > > On Mon, Dec 30, 2013 at 4:15 PM, Stephen Boyd wrote: > > >> This file doesn't use the clk_reset() API that is exposed in > > >> mach-msm's mach/clk.h file. Remove the include so that this > > >> driver can be compiled as part of the multi-platform kernel. > > > > > > Thanks! > > > > > > Signed-off-by: Rob Clark > > I think you mean Acked-by? Signed-off-by usually means you're > sending the patch along. I've held off applying this, but I have applied the others -- please merge this through the DRM tree since it will need to go on top of changes there. That also means that enabling the MSM DTM driver in a multiplatform kernel will cause build breakages until the equivalent patch has been picked up. Since it's not enabled by any defconfigs I'm not too worried, but it will show up on randconfigs. -Olof