linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Antonino A. Daplas" <adaplas@hotpop.com>
To: linux-fbdev-devel@lists.sourceforge.net,
	Komal Shah <komal_shah802003@yahoo.com>
Subject: Re: Fb rotation 2.6.x
Date: Wed, 16 Mar 2005 04:50:48 +0800	[thread overview]
Message-ID: <200503160450.48352.adaplas@hotpop.com> (raw)
In-Reply-To: <20050315085827.56051.qmail@web53603.mail.yahoo.com>

On Tuesday 15 March 2005 16:58, Komal Shah wrote:
> Hi All,
>
> We have added the code for hardware rotation in the
> framebuffer driver for omap242x processor from TI
> [2.6.x kernel].
>
> It has been reported to work on 2.4.x.
>
> But when I had made it changed for 2.6.x follwoing
> behaviour is seen:
>
> 1. When I rotate the framebuffer ( Linux logo screen)
> by 180 degree then it shows only Linux logo image for
> a second only and then it disappears.
>
> 2. Whereas while doing in 90 degree rotation, it shows
> the black square of Linux logo size on the 270 degree
> rotation position.
>
> The reasons we found are as below:
>
> -----------------snip-----------------
> That is because in 2.4, the fb_display's var structure
> is updated by the driver - which we do consistently.
> In 2.6, on the other hand, the fb_info's var structure
> is
> updated by the driver, and all the fb_displays are
> updated by either fb_mem or fb_con - which I think is
> not done correctly. Also the set_par function is
> called so many times - I don't understand why.

Sometimes people use multiple fbdevs mapped to different consoles, so a
set_par() is called for each fb_info mapped for each console.

I guess I can add an "already_initialized" flag so fbcon doesn't do multiple
set_pars().

>
> What is happening is that, first, fb_info's var
> structure is updated, which rotates the display. But
> this var does not get passed to the current
> fb_display's var. Then when the fbcon_switch function
> is called, the current fb_display's var settings are
> affected - which 'unrotates' the display.

What fields do you use in var that is important for rotation? Is it
var->rotate? If that is the case, just add those fields in struct display in
fbcon.h, and update the functions display_to_var() and var_to_display() in
fbcon.c to also include the said fields.

Tony




-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

  reply	other threads:[~2005-03-15 20:50 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-15  8:58 Fb rotation 2.6.x Komal Shah
2005-03-15 20:50 ` Antonino A. Daplas [this message]
2005-03-15 23:47   ` Antonino A. Daplas
2005-03-16 13:59     ` Komal Shah

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=200503160450.48352.adaplas@hotpop.com \
    --to=adaplas@hotpop.com \
    --cc=komal_shah802003@yahoo.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).