Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756279Ab1CWLx5 (ORCPT ); Wed, 23 Mar 2011 07:53:57 -0400 Received: from ihemail4.lucent.com ([135.245.0.39]:56519 "EHLO ihemail4.lucent.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753162Ab1CWLx4 (ORCPT ); Wed, 23 Mar 2011 07:53:56 -0400 X-Greylist: delayed 789 seconds by postgrey-1.27 at vger.kernel.org; Wed, 23 Mar 2011 07:53:56 EDT Date: Wed, 23 Mar 2011 06:40:32 -0500 (CDT) From: Ilija Hadzic X-X-Sender: ihadzic@umail To: Dave Airlie cc: =?ISO-8859-1?Q?Michel_D=E4nzer?= , torvalds@linux-foundation.org, linux-kernel@vger.kernel.org, DRI mailing list Subject: Re: [git pull] drm fixes In-Reply-To: Message-ID: References: <1300864998.3522.71.camel@thor.local> <1300868532.3522.81.camel@thor.local> MIME-Version: 1.0 Content-Type: MULTIPART/MIXED; BOUNDARY="-559023410-959030623-1300880432=:12612" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2147 Lines: 62 This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. ---559023410-959030623-1300880432=:12612 Content-Type: TEXT/PLAIN; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE On Wed, 23 Mar 2011, Dave Airlie wrote: > 2011/3/23 Michel D=E4nzer : >> On Mit, 2011-03-23 at 18:16 +1000, Dave Airlie wrote: >>> 2011/3/23 Michel D=E4nzer : >>>> On Mit, 2011-03-23 at 04:18 +0000, Dave Airlie wrote: >>>>> >>>>> One radeon, 2 core fixes, and an interface update to allow for > 2 cr= tcs >>>>> in vblank. >>>> >>>> [...] >>>> >>>>> Ilija Hadzic (1): >>>>> =A0 =A0 =A0 drm/kernel: vblank wait on crtc > 1 >>>> >>>> This patch was still being debated yesterday, are you deliberately >>>> pushing it regardless? Once it hits mainline, it'll be pretty much set >>>> in stone. >>> >>> From what I can see it was the userspace patches being debated, this >>> one seemed fine and the interface looked okay to me. >> >> The author ignored my suggestions to make the patch smaller and simpler, >> more maintainable and more future-proof all at once. > > It was already small and I'm not sure merging the flags made it more > maintainable. Its always > being a slightly painful ioctl, and hopefully any future changes add a > new ioctl esp if we want 64-bit values. > > The only comment I really thought was necessary was changing the CAP > name, but since that isn't > part of the ABI (just the number) we can quickly fix it with a follow-up. > > Dave. All of the issues debated yesterday, except one, boil down to renaming a=20 handful on #defines without changing the values nor interface nor behavior= =20 of the kernel. For those that I agreed, I'll follow up with a small patch= =20 shortly. -- Ilija ---559023410-959030623-1300880432=:12612-- -- 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/