On Fri, Apr 20, 2012 at 1:23 PM, Ville Syrj?l? <ville.syrjala at linux.intel.com> wrote: > On Fri, Apr 20, 2012 at 12:43:03PM +0100, Dave Airlie wrote: >> On Thu, Apr 5, 2012 at 7:35 PM, ?<ville.syrjala at linux.intel.com> wrote: >> > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> >> > >> > 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 at > linux.intel.com>
Thanks I'll take a look at that, > > 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. I'm probably not qualified, if you and jbarnes and say one other non-Intel person agree, then send the patch with R-b and I'll apply it. Dave.
