All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Oliver Kropp <dok@directfb.org>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: linux-fbdev-devel@lists.sourceforge.net,
	directfb-users@directfb.org, directfb-dev@directfb.org
Subject: Re: DirectFB without FBDev
Date: Wed, 28 May 2008 20:33:35 +0200	[thread overview]
Message-ID: <483DA57F.6070602@directfb.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0805281948080.15083@anakin>

Geert Uytterhoeven wrote:
> On Wed, 28 May 2008, Denis Oliver Kropp wrote:
>> Geert Uytterhoeven wrote:
>>> On Wed, 28 May 2008, Denis Oliver Kropp wrote:
>>>> One major bug at the moment is mode switching and pitch values being
>>>> wrong. It's dumb to
>>>> return the pitch of the variable mode settings in the fixed settings
>>>> structure anyhow, but
>>>> if you like to start with the above mentioned mission, that's where it
>>>> could begin.
>>> Do you care to tell us why this is dumb?
>> Ok, the fixed information should not contain information that changes over
>> time,
>> otherwise it is not fixed, at least that's my understanding of "fixed".
> 
> It's `fixed' not in the sense that it can never change, but that it
> purely depends on the info in struct fb_var_screeninfo.

I'd prefer an ioctl which has input and output fields in the accompanied struct,
so that the resulting pitch is returned to the user space with the same system call.

>>> The pitch depends on the video mode and the hardware requirements,
>>> that's why it's in struct fb_fix_screeninfo.
>> It would suffice to encode the pitch requirements independent from the video
>> mode,
>> i.e. just put the pixel or byte alignment into the fixed structure.
> 
> What if the pitch requirements depend on the video mode?

Haven't seen that, usually either pixels or bytes just needs to be a multiple
of something. In these cases you could simply forbid/fail to choose the pitch and
just return it in the variable structure.

> There exists graphics hardware where fb_fix_screeninfo.type depends on
> the video mode.

Sure, type depends on the pixelformat :)

-- 
Best regards,
   Denis Oliver Kropp

.------------------------------------------.
| DirectFB - Hardware accelerated graphics |
| http://www.directfb.org/                 |
"------------------------------------------"

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

  reply	other threads:[~2008-05-28 18:35 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-28 14:59 DirectFB without FBDev Denis Oliver Kropp
2008-05-28 17:04 ` Geert Uytterhoeven
2008-05-28 17:20   ` Denis Oliver Kropp
     [not found]     ` <483D9453.1080700-iGvX3keArt1g9hUCZPvPmw@public.gmane.org>
2008-05-28 17:53       ` [Linux-fbdev-devel] " Brian G. Rhodes
2008-05-28 18:07         ` [directfb-users] " Denis Oliver Kropp
2008-05-28 18:24           ` Alex Deucher
     [not found]             ` <a728f9f90805281124y53df86e7n7ae1918ac97907fc-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-05-28 18:57               ` [Linux-fbdev-devel] " Denis Oliver Kropp
2008-05-28 19:33                 ` [directfb-users] " Alex Deucher
2008-05-28 18:55           ` Brian G. Rhodes
     [not found]             ` <483DAAB7.6080208-3mWahjwryh5BDgjK7y7TUQ@public.gmane.org>
2008-05-28 19:11               ` [Linux-fbdev-devel] " Denis Oliver Kropp
2008-05-28 19:38                 ` [directfb-users] " Brian G. Rhodes
     [not found]           ` <483D9F4E.4000707-iGvX3keArt1g9hUCZPvPmw@public.gmane.org>
2008-05-29 16:54             ` [directfb-dev] [Linux-fbdev-devel] " Sven Luther
2008-05-31  6:08               ` [directfb-dev] [directfb-users] " Denis Oliver Kropp
2008-05-31  6:21                 ` Sven Luther
2008-05-31  6:29                   ` Denis Oliver Kropp
2008-05-31  6:45                     ` Sven Luther
     [not found]                       ` <20080531064501.GA7526-XkLrfMvk983aVgXGlvhGPA@public.gmane.org>
2008-05-31 20:32                         ` George Tsalikis
2008-06-01  7:10                           ` [directfb-dev] " Denis Oliver Kropp
     [not found]                             ` <48424B49.8080202-iGvX3keArt1g9hUCZPvPmw@public.gmane.org>
2008-06-01  9:20                               ` [directfb-dev] " George Tsalikis
2008-06-01  9:38                                 ` Denis Oliver Kropp
2008-05-28 17:55     ` Geert Uytterhoeven
2008-05-28 18:33       ` Denis Oliver Kropp [this message]
     [not found] ` <483D7344.6030503-iGvX3keArt1g9hUCZPvPmw@public.gmane.org>
2008-06-01  9:31   ` [directfb-dev] " Denis Oliver Kropp
     [not found]     ` <48426C76.5080606-iGvX3keArt1g9hUCZPvPmw@public.gmane.org>
2008-06-01 11:57       ` Denis Oliver Kropp

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=483DA57F.6070602@directfb.org \
    --to=dok@directfb.org \
    --cc=directfb-dev@directfb.org \
    --cc=directfb-users@directfb.org \
    --cc=geert@linux-m68k.org \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    /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.