All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Airlie <airlied@gmail.com>
To: Egbert Eich <eich@suse.com>
Cc: Egbert Eich <eich@suse.de>, Dave Airlie <airlied@redhat.com>,
	Christopher Harvey <charvey@matrox.com>,
	Mathieu Larouche <mathieu.larouche@matrox.com>,
	dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 11/16] drm/mgag200: Initialize data needed to map fbdev memory
Date: Tue, 23 Jul 2013 15:43:19 +1000	[thread overview]
Message-ID: <CAPM=9twDQXpdGXNKraJZJn_cjJ9QxYW2E=Vfy1tw_QDHxLo6Ew@mail.gmail.com> (raw)
In-Reply-To: <1374066449-21714-12-git-send-email-eich@suse.com>

On Wed, Jul 17, 2013 at 11:07 PM, Egbert Eich <eich@suse.com> wrote:
> Due to a missing initialization there was no way to map fbdev memory.
> Thus for example using the Xserver with the fbdev driver failed.
> This fix adds initialization for fix.smem_start and fix.smem_len
> in the fb_info structure, which fixes this problem.

I don't think this is correct, since it'll give userspace a direct phy
addr for the base of the video ram, though I'm not in a position to
say exactly what should be set here, guess I need to get an mgag200
machine up and running.

Dave.

  reply	other threads:[~2013-07-23  5:43 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-17 13:07 [PATCH 00/16] drm/mgag200: Fixes and enhancements for the MGAG200 KMS driver Egbert Eich
2013-07-17 13:07 ` [PATCH 01/16] drm/mgag200: Fix memleaks in error path in mgag200_fb_create() Egbert Eich
2013-07-17 13:07 ` [PATCH 02/16] drm/mgag200: Fix memleak in error path in mgag200_bo_create() Egbert Eich
2013-07-17 13:07 ` [PATCH 03/16] drm/mgag200: Free container instead of member in mga_user_framebuffer_destroy() Egbert Eich
2013-07-17 13:07 ` [PATCH 04/16] drm/mgag200: Don't unreference when handle creation failed Egbert Eich
2013-07-17 13:28   ` Chris Wilson
2013-07-17 15:55     ` Egbert Eich
2013-07-17 13:07 ` [PATCH 05/16] drm/mgag200: Copy fb name string before using it in mgag200_fb_create() Egbert Eich
2013-07-17 13:07 ` [PATCH 06/16] drm/mgag200: Fix logic in mgag200_bo_pin() Egbert Eich
2013-07-23  5:38   ` Dave Airlie
2013-07-23  6:44     ` [PATCH 06/16 v2] drm/mgag200: Fix logic in mgag200_bo_pin() (v2) Egbert Eich
2013-07-17 13:07 ` [PATCH 07/16] drm/mgag200: Make local function mgag200_gem_init_object() static Egbert Eich
2013-07-17 13:07 ` [PATCH 08/16] drm/mgag200: Simplify function mgag200_bo_unpin() Egbert Eich
2013-07-17 13:07 ` [PATCH 09/16] drm/mgag200: Add an crtc_disable callback to the crtc helper funcs Egbert Eich
2013-07-17 13:07 ` [PATCH 10/16] drm/mgag200: Invalidate page tables when pinning a BO Egbert Eich
2013-07-17 15:40   ` [PATCH V2] " Egbert Eich
2013-07-17 13:07 ` [PATCH 11/16] drm/mgag200: Initialize data needed to map fbdev memory Egbert Eich
2013-07-23  5:43   ` Dave Airlie [this message]
2013-07-17 13:07 ` [PATCH 12/16] drm/mgag200: Add sysfs support for connectors Egbert Eich
2013-07-17 13:07 ` [PATCH 13/16] drm/mgag200: Fix framebuffer pitch calculation Egbert Eich
2013-07-17 13:07 ` [PATCH 14/16] drm/mgag200: Fix LUT programming for 16bpp Egbert Eich
2013-07-17 13:07 ` [PATCH 15/16] drm/mgag200: Reject modes when h-parameters are no multiple of 8 Egbert Eich
2013-07-23  5:46   ` Dave Airlie
2013-07-23 15:30     ` Adam Jackson
2013-07-23 18:07       ` Daniel Vetter
2013-07-17 13:07 ` [PATCH 16/16] drm/mgag200: Add doublescan and interlace support Egbert Eich
2013-07-23  7:14 ` [PATCH 00/16] drm/mgag200: Fixes and enhancements for the MGAG200 KMS driver Dave Airlie
2013-07-23  8:09   ` Egbert Eich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAPM=9twDQXpdGXNKraJZJn_cjJ9QxYW2E=Vfy1tw_QDHxLo6Ew@mail.gmail.com' \
    --to=airlied@gmail.com \
    --cc=airlied@redhat.com \
    --cc=charvey@matrox.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=eich@suse.com \
    --cc=eich@suse.de \
    --cc=mathieu.larouche@matrox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.