Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755367Ab2KMRkf (ORCPT ); Tue, 13 Nov 2012 12:40:35 -0500 Received: from avon.wwwdotorg.org ([70.85.31.133]:57158 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753781Ab2KMRkd (ORCPT ); Tue, 13 Nov 2012 12:40:33 -0500 Message-ID: <50A2860E.1080108@wwwdotorg.org> Date: Tue, 13 Nov 2012 10:40:30 -0700 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120912 Thunderbird/15.0.1 MIME-Version: 1.0 To: Thierry Reding CC: Mark Zhang , "devicetree-discuss@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Rob Clark , "linux-tegra@vger.kernel.org" , Dave Airlie Subject: Re: [PATCH v2 1/2] drm: Add NVIDIA Tegra20 support References: <1352757358-14001-1-git-send-email-thierry.reding@avionic-design.de> <1352757358-14001-2-git-send-email-thierry.reding@avionic-design.de> <50A1F3A3.40905@nvidia.com> <20121113074822.GA8409@avionic-0098.mockup.avionic-design.de> <50A20994.9000305@nvidia.com> <20121113093725.GB8122@avionic-0098.mockup.avionic-design.de> In-Reply-To: <20121113093725.GB8122@avionic-0098.mockup.avionic-design.de> X-Enigmail-Version: 1.4.4 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2452 Lines: 54 On 11/13/2012 02:37 AM, Thierry Reding wrote: > On Tue, Nov 13, 2012 at 04:49:24PM +0800, Mark Zhang wrote: >> On 11/13/2012 03:48 PM, Thierry Reding wrote: >>> * PGP Signed by an unknown key >>> >>> On Tue, Nov 13, 2012 at 03:15:47PM +0800, Mark Zhang wrote: >>>> On 11/13/2012 05:55 AM, Thierry Reding wrote: >>>>> This commit adds a KMS driver for the Tegra20 SoC. This >>>>> includes basic support for host1x and the two display >>>>> controllers found on the Tegra20 SoC. Each display >>>>> controller can drive a separate RGB/LVDS output. >>>>> diff --git a/drivers/gpu/drm/tegra/Kconfig >>>>> b/drivers/gpu/drm/tegra/Kconfig new file mode 100644 index >>>>> 0000000..be1daf7 --- /dev/null +++ >>>>> b/drivers/gpu/drm/tegra/Kconfig @@ -0,0 +1,23 @@ +config >>>>> DRM_TEGRA + tristate "NVIDIA Tegra DRM" + >>>>> depends on DRM && OF && ARCH_TEGRA + select >>>>> DRM_KMS_HELPER + select DRM_GEM_CMA_HELPER + >>>>> select DRM_KMS_CMA_HELPER >>>> >>>> Just for curious, according to my testing, why the >>>> "CONFIG_CMA" is not enabled while DRM_GEM_CMA_HELPER & >>>> DRM_KMS_CMA_HELPER are enabled here? >>> >>> The reason is that CMA doesn't actually provide any API for >>> drivers to use and in fact unless you use very large buffers >>> you could indeed run this code on top of a non-CMA kernel and >>> it will likely even work. >>> >> >> Okay. But I think it's better to turn on CMA defaultly. During >> my testing, it's hard to allocate more 2MB without CMA... > > CMA is enabled by default in one of the Tegra default > configuration patches in my tegra/next branch. I will submit that > patch to Stephen when the 3.8 cycle starts, so that it'll be > automatically enabled along with the DRM driver. > > But I don't think it makes sense to couple it to the DRM_TEGRA > symbol as it isn't strictly required. OK, I guess that approach makes sense; most people will just use the defconfig and hence get a useful kernel, while flexibility will not be lost if someone really wants. Note that I have less than 1 week left to apply patches for 3.8. I hope that if tegradrm makes it into the drm tree for 3.8, so will the defconfig and other enablement patches to activate it. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/