[PATCH 4/4] drm: Renesas SH Mobile DRM driver

[Laurent Pinchart]

Hi Lars-Peter,

On Wednesday 27 June 2012 22:06:54 Lars-Peter Clausen wrote:
> On 06/27/2012 02:40 PM, Laurent Pinchart wrote:
> > Hi Lars-Peter,
> > 
> > [...]
> > 
> > Do you plan to post a v3 of your "DRM: Add DRM kms/fb cma helper" patch in
> > the near future ?
> 
> Hi Laurent,
> 
> I don't think there have been any changes since v2 except for adjusting to
> the renamed functions in Sascha's patch. I'll send a v3 tomorrow.

OK, thanks.

As Ville pointed out, we need to validate the offsets and make sure the frame 
buffer fits in the GEM object(s). I don't think there's code in your patch to 
do so, have you thought about where/how to implement that ?

> I think it makes most sense if you carry these two patches (Sascha's and
> mine) as part of your "Renesas SH Mobile DRM driver" series, so reviewers
> can see it in context.

OK I'll do that.

-- 
Regards,

Laurent Pinchart