From: Michael Ellerman <michael@ellerman.id.au>
To: Geoff Levand <geoffrey.levand@am.sony.com>
Cc: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>,
linuxppc-dev@ozlabs.org, linux-fbdev-devel@lists.sourceforge.net,
cbe-oss-dev@ozlabs.org, "Antonino A. Daplas" <adaplas@gmail.com>
Subject: Re: [Cbe-oss-dev] [patch 0/8] PS3 AV Settings Driver patches for 2.6.24
Date: Fri, 28 Sep 2007 18:40:33 +1000 [thread overview]
Message-ID: <1190968833.15970.3.camel@concordia> (raw)
In-Reply-To: <46FBE6FD.3030605@am.sony.com>
[-- Attachment #1.1: Type: text/plain, Size: 2055 bytes --]
On Thu, 2007-09-27 at 10:23 -0700, Geoff Levand wrote:
> Michael Ellerman wrote:
> > On Wed, 2007-09-26 at 18:33 +0200, Geert Uytterhoeven wrote:
> >> Hi,
> >>
> >> Here are some new patches for the PS3 AV Settings Driver (ps3av), which is
> >> used in close collaboration with the PS3 Virtual Frame Buffer Device Driver
> >> (ps3fb):
> >> [1] ps3av: eliminate unneeded temporary variables
> >> [2] ps3av: eliminate PS3AV_DEBUG
> >> [3] ps3av: use PS3 video mode ids in autodetect code
> >> [4] ps3av: treat DVI-D like HDMI in autodetect
> >> [5] ps3av: add autodetection for VESA modes
> >> [6] ps3av: add quirk database for broken monitors
> >> [7] ps3av: remove unused ps3av_set_mode()
> >> [8] ps3av: don't distinguish between `boot' and `non-boot' autodetection
> >>
> >> Please review, and queue for 2.6.24 if they're ok. Thanks!
> >>
> >> Question: As several DVI-D displays advertise they support 1080i modes while
> >> they actually don't (cfr. the quirk database), perhaps I should drop 1080i
> >> modes completely from the ps3av_preferred_modes[] table? Usually 720p looks
> >> better than 1080i anyway. What do you think?
> >
> > Definitely. If the autodetection fails Linux is basically unusable on
> > PS3 unless you hack the kernel sources and build your own kboot and
> > otheros.bld - not entirely trivial for novice users. So it's pretty
> > important that it works 100%.
>
> It is not that bad. If a bootloader is configued to use autodetection
> so that it is shown at best video mode, then it should have advertised
> key sequences to default to known video modes, with at least a 480p 'safe'
> mode.
Oh OK, that is a good idea. What's the "safe mode" key sequence for the
otheros.bld you provide on kernel.org?
cheers
--
Michael Ellerman
OzLabs, IBM Australia Development Lab
wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)
We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
[-- Attachment #2: Type: text/plain, Size: 146 bytes --]
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
WARNING: multiple messages have this Message-ID (diff)
From: Michael Ellerman <michael@ellerman.id.au>
To: Geoff Levand <geoffrey.levand@am.sony.com>
Cc: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>,
linuxppc-dev@ozlabs.org, linux-fbdev-devel@lists.sourceforge.net,
cbe-oss-dev@ozlabs.org, "Antonino A. Daplas" <adaplas@gmail.com>
Subject: Re: [Cbe-oss-dev] [patch 0/8] PS3 AV Settings Driver patches for 2.6.24
Date: Fri, 28 Sep 2007 18:40:33 +1000 [thread overview]
Message-ID: <1190968833.15970.3.camel@concordia> (raw)
In-Reply-To: <46FBE6FD.3030605@am.sony.com>
[-- Attachment #1: Type: text/plain, Size: 2055 bytes --]
On Thu, 2007-09-27 at 10:23 -0700, Geoff Levand wrote:
> Michael Ellerman wrote:
> > On Wed, 2007-09-26 at 18:33 +0200, Geert Uytterhoeven wrote:
> >> Hi,
> >>
> >> Here are some new patches for the PS3 AV Settings Driver (ps3av), which is
> >> used in close collaboration with the PS3 Virtual Frame Buffer Device Driver
> >> (ps3fb):
> >> [1] ps3av: eliminate unneeded temporary variables
> >> [2] ps3av: eliminate PS3AV_DEBUG
> >> [3] ps3av: use PS3 video mode ids in autodetect code
> >> [4] ps3av: treat DVI-D like HDMI in autodetect
> >> [5] ps3av: add autodetection for VESA modes
> >> [6] ps3av: add quirk database for broken monitors
> >> [7] ps3av: remove unused ps3av_set_mode()
> >> [8] ps3av: don't distinguish between `boot' and `non-boot' autodetection
> >>
> >> Please review, and queue for 2.6.24 if they're ok. Thanks!
> >>
> >> Question: As several DVI-D displays advertise they support 1080i modes while
> >> they actually don't (cfr. the quirk database), perhaps I should drop 1080i
> >> modes completely from the ps3av_preferred_modes[] table? Usually 720p looks
> >> better than 1080i anyway. What do you think?
> >
> > Definitely. If the autodetection fails Linux is basically unusable on
> > PS3 unless you hack the kernel sources and build your own kboot and
> > otheros.bld - not entirely trivial for novice users. So it's pretty
> > important that it works 100%.
>
> It is not that bad. If a bootloader is configued to use autodetection
> so that it is shown at best video mode, then it should have advertised
> key sequences to default to known video modes, with at least a 480p 'safe'
> mode.
Oh OK, that is a good idea. What's the "safe mode" key sequence for the
otheros.bld you provide on kernel.org?
cheers
--
Michael Ellerman
OzLabs, IBM Australia Development Lab
wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)
We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2007-09-28 8:40 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-26 16:33 [patch 0/8] PS3 AV Settings Driver patches for 2.6.24 Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 1/8] ps3av: eliminate unneeded temporary variables Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 2/8] ps3av: eliminate PS3AV_DEBUG Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 3/8] ps3av: use PS3 video mode ids in autodetect code Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 4/8] ps3av: treat DVI-D like HDMI in autodetect Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 5/8] ps3av: add autodetection for VESA modes Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 6/8] ps3av: add quirk database for broken monitors Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 7/8] ps3av: remove unused ps3av_set_mode() Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-26 16:33 ` [patch 8/8] ps3av: dont distinguish between `boot and `non-boot autodetection Geert Uytterhoeven
2007-09-26 16:33 ` Geert Uytterhoeven
2007-09-27 2:31 ` [Cbe-oss-dev] [patch 0/8] PS3 AV Settings Driver patches for 2.6.24 Michael Ellerman
2007-09-27 2:31 ` Michael Ellerman
2007-09-27 11:53 ` Geert Uytterhoeven
2007-09-27 11:53 ` Geert Uytterhoeven
2007-09-28 8:44 ` Michael Ellerman
2007-09-28 8:44 ` Michael Ellerman
2007-09-27 17:23 ` Geoff Levand
2007-09-27 17:23 ` Geoff Levand
2007-09-28 8:40 ` Michael Ellerman [this message]
2007-09-28 8:40 ` Michael Ellerman
2007-09-28 19:29 ` Geoff Levand
2007-09-28 19:29 ` Geoff Levand
2007-09-29 8:00 ` Geert Uytterhoeven
2007-09-29 8:00 ` Geert Uytterhoeven
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=1190968833.15970.3.camel@concordia \
--to=michael@ellerman.id.au \
--cc=Geert.Uytterhoeven@sonycom.com \
--cc=adaplas@gmail.com \
--cc=cbe-oss-dev@ozlabs.org \
--cc=geoffrey.levand@am.sony.com \
--cc=linux-fbdev-devel@lists.sourceforge.net \
--cc=linuxppc-dev@ozlabs.org \
/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.