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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux