Once the indentation is fixed:
Reviewed-by: Zach Reizner
On Wed, Apr 1, 2015 at 8:00 PM kbuild test robot
wrote:
>
> Signed-off-by: Fengguang Wu
> ---
> vgem_drv.c |6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/vgem/vgem_drv.c
ion?
On Fri Nov 21 2014 at 10:45:08 AM Zachary Reizner wrote:
> ajax: The consumer of this will be software renderers. We're working on
> one right now that will be using dma-bufs from userspace.
> Daniel: Thanks for your suggestions. I'll work on it and submit a v2.
> On Fri Nov 21 201
ajax: The consumer of this will be software renderers. We're working on one
right now that will be using dma-bufs from userspace.
Daniel: Thanks for your suggestions. I'll work on it and submit a v2.
On Fri Nov 21 2014 at 6:02:45 AM Adam Jackson wrote:
> On Thu, 2014-11-20 at 16:26 -0800, Zach
>
> revision bump would be needed when adding a new interface (not present
> in real hardware) to allow for larger pitches, so you could use 32bpp
> with the default (1024x768) resolution.
>
I did not change the pitch mechanism. The hardware is limited to less than
a 4KB pitch because of the
ajax,
What do you mean by a pci revision id bump? Do you want it in qemu or the
kernel? Why is a revision bump needed when none of the behavior of the
cirrus device has changed?
I agree that reading the BAR size from the device would be an enhancement.
I'm working on the patch to include that now.
Real cirrus did have 32bpp according to both the reference manual and
qemu's emulation of the hardware. The cirrus DRM driver even had the
modesetting code for 32bpp but the driver prevented the creation of 32bpp
framebuffers.
Best,
Zach
On Mon, Oct 27, 2014, 08:30 Gerd Hoffmann wrote:
> On
