All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: G 3 <programmingkidx@gmail.com>
Cc: The OpenBIOS Mailinglist <openbios@openbios.org>,
	qemu-devel qemu-devel <qemu-devel@nongnu.org>,
	"list@suse.de:PowerPC list:PowerPC" <qemu-ppc@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH] Add resolutions via the command-line
Date: Thu, 22 Sep 2016 10:05:36 +1000	[thread overview]
Message-ID: <1474502736.2857.157.camel@kernel.crashing.org> (raw)
In-Reply-To: <D7361F27-063D-4BEF-956A-9DDB94F3EB50@gmail.com>

On Wed, 2016-09-21 at 19:18 -0400, G 3 wrote:
> 
> That can be done, but I was hoping to be able to do this via a  
> command-line switch.

But you still do that !

Your switch puts stuff in /options which OpenBIOS then picks up to
construct the final list that it puts in the driver node.

> That would make things so much easier on the user. Altering  
> properties in OpenBIOS can be challenging using forth. OpenBIOS does
> support using  
> startup
> scripts so maybe someone could make a script that adds user  
> resolutions to the
> QEMU,VGA node from the options node.
> 
> I'm hoping you had a look at version three of my patch. If there
> are  
> any other issues
> with it, I would like to know before making version 4.

I'm really not fan of the whole ASCII parsing business. In any case,
I'm a bit too busy right now, I'll try to get back onto this next week.

  reply	other threads:[~2016-09-22  0:05 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-18  3:31 [Qemu-devel] [PATCH] Add resolutions via the command-line G 3
2016-09-19  6:24 ` Benjamin Herrenschmidt
2016-09-19 12:44   ` G 3
2016-09-19 22:44     ` Benjamin Herrenschmidt
2016-09-19 23:56       ` Alfonso Gamboa
2016-09-20  3:57         ` G 3
2016-09-20  4:01       ` G 3
2016-09-21  0:18         ` Benjamin Herrenschmidt
2016-09-21  0:35           ` G 3
2016-09-21  0:41             ` Benjamin Herrenschmidt
2016-09-21  2:54               ` G 3
2016-09-21  3:41                 ` [Qemu-devel] [OpenBIOS] " Alfonso Gamboa
2016-09-21  9:01                 ` [Qemu-devel] " Benjamin Herrenschmidt
2016-09-21 18:26                   ` G 3
2016-09-21 21:53                     ` Benjamin Herrenschmidt
2016-09-21 23:18                       ` G 3
2016-09-22  0:05                         ` Benjamin Herrenschmidt [this message]
2016-09-23 14:09                       ` [Qemu-devel] [OpenBIOS] " Mark Cave-Ayland

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=1474502736.2857.157.camel@kernel.crashing.org \
    --to=benh@kernel.crashing.org \
    --cc=openbios@openbios.org \
    --cc=programmingkidx@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.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.