Re: [PATCH] drm: dumb scanout create/mmap for intel/radeon (v3)

[Chris Wilson <chris@chris-wilson.co.uk>]

On Fri,  7 Jan 2011 09:16:51 +1000, Dave Airlie <airlied@gmail.com> wrote:
> +int
> +i915_gem_dumb_create(struct drm_file *file,
> +		     struct drm_device *dev,
> +		     struct drm_mode_create_dumb *args)
> +{
> +	/* have to work out size/pitch and return them */
> +	args->pitch = ALIGN(args->width & ((args->bpp + 1) / 8), 64);
                                        ^ oops
I think you meant args->width * ((args->bpp + 7) /8)

> +	args->size = args->pitch * args->height;
> +	return i915_gem_create(file, dev,
> +			       args->size, &args->handle);
> +}

The only issue with the dumb libkms fb is the transition to X. Is the
nature of the dumb fb such that we would be prohibited from blitting to
the X fb, or can we use platform specific knowledge to know how we can
abuse it?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre