From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/4] drm: Move drm_format_num_planes() to drm_crtc.c Date: Fri, 20 Apr 2012 15:23:41 +0300 Message-ID: <20120420122341.GA13065@intel.com> References: <1333650918-16919-1-git-send-email-ville.syrjala@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id 8A8299E7BE for ; Fri, 20 Apr 2012 05:23:46 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Dave Airlie Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Fri, Apr 20, 2012 at 12:43:03PM +0100, Dave Airlie wrote: > On Thu, Apr 5, 2012 at 7:35 PM, wrote: > > From: Ville Syrj=E4l=E4 > > > > There will be a need for this function in drm_crtc.c later. This > > avoids making drm_crtc.c depend on drm_crtc_helper.c. > = > Hi Ville, > = > I've applied these 4 patches, but I might have lost some others for > you, let me know if there is some other stuff I should be reviewing > for -next. IIRC only one patch fell through the cracks: Subject: [PATCH] drm: Unify and fix idr error handling Message-Id: <1331834311-30757-1-git-send-email-ville.syrjala@linux.intel.co= m> BTW you never gave any statement wrt. removing NV12M and YUV420M from drm_fourcc.h. I can send a patch if you agree, and if not, well then someone has to actually change the code to treat them exactly the same as NV12 and YUV420. -- = Ville Syrj=E4l=E4 Intel OTC