Re: [stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-09 Thread Sergej Pupykin
At Thu, 9 Dec 2010 09:18:14 -0800,
Jesse Barnes  wrote:
> 
> On Wed, 8 Dec 2010 15:30:26 -0800
> Greg KH  wrote:
> > What kernel version did these options first show up in?  Does any
> > other tools rely on the spaces?
> 
> And it's ugly; can't we fix grub instead?

I think renaming connectors is good idea anyway. It should be typed
before system boot, so names should be short and easy kept in mind.

Such as TV, DIN, VGA, DVI, etc...
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-10 Thread Sergej Pupykin
At Thu, 9 Dec 2010 09:18:14 -0800,
Jesse Barnes  wrote:
> 
> And it's ugly; can't we fix grub instead?

I am searching for bootloader which can pass whitespaces. It looks
like we should patch grub-legacy (0.97), grub (1.98) and lilo...

(I did not try lilo yet, but man page says nothing about passing
spaces in 'append=' option description)
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-13 Thread Sergej Pupykin
At Fri, 10 Dec 2010 19:05:13 +0100,
Peter Stuge  wrote:
> append can contain any arbitrary string, which will be parsed by the
> kernel. I e.g. use:
>
> append="root=/dev/sda2 snd_ac97_codec.power_save=1 usbcore.autosuspend=1 
> hpet=force quiet drm_kms_helper.poll=0"
>
> And grub also supports an append string with spaces, that is probably
> used by every distribution.
>
> The question is, what will the kernel parser do about that space?

It seems lilo also have no support for such parameters.

append="video=9-pin DIN:1024x768...@60e"

does not work (kernel panic: DIN: bad variable name)

As I understand, I should pass somehow to kernel string like:

video="9-pin DIN:1024x768...@60e", but lilo uses doublequotes as
r-value bounds.

in_quote variable in next_arg() is set only for " character.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-13 Thread Sergej Pupykin
At Mon, 13 Dec 2010 13:43:48 +0300,
Sergej Pupykin  wrote:

> video="9-pin DIN:1024x768...@60e", but lilo uses doublequotes as
> r-value bounds.
> 
> in_quote variable in next_arg() is set only for " character.

Sorry, biggest part of this problem is in my linux distribution (in
/init executable from initcpio image)
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-09 Thread Sergej Pupykin
At Thu, 9 Dec 2010 09:18:14 -0800,
Jesse Barnes  wrote:
> 
> On Wed, 8 Dec 2010 15:30:26 -0800
> Greg KH  wrote:
> > What kernel version did these options first show up in?  Does any
> > other tools rely on the spaces?
> 
> And it's ugly; can't we fix grub instead?

I think renaming connectors is good idea anyway. It should be typed
before system boot, so names should be short and easy kept in mind.

Such as TV, DIN, VGA, DVI, etc...


[stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-10 Thread Sergej Pupykin
At Thu, 9 Dec 2010 09:18:14 -0800,
Jesse Barnes  wrote:
> 
> And it's ugly; can't we fix grub instead?

I am searching for bootloader which can pass whitespaces. It looks
like we should patch grub-legacy (0.97), grub (1.98) and lilo...

(I did not try lilo yet, but man page says nothing about passing
spaces in 'append=' option description)


[stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-13 Thread Sergej Pupykin
At Fri, 10 Dec 2010 19:05:13 +0100,
Peter Stuge  wrote:
> append can contain any arbitrary string, which will be parsed by the
> kernel. I e.g. use:
>
> append="root=/dev/sda2 snd_ac97_codec.power_save=1 usbcore.autosuspend=1 
> hpet=force quiet drm_kms_helper.poll=0"
>
> And grub also supports an append string with spaces, that is probably
> used by every distribution.
>
> The question is, what will the kernel parser do about that space?

It seems lilo also have no support for such parameters.

append="video=9-pin DIN:1024x768-24 at 60e"

does not work (kernel panic: DIN: bad variable name)

As I understand, I should pass somehow to kernel string like:

video="9-pin DIN:1024x768-24 at 60e", but lilo uses doublequotes as
r-value bounds.

in_quote variable in next_arg() is set only for " character.


[stable] [PATCH] drm/kms: remove spaces from connector names

2010-12-13 Thread Sergej Pupykin
At Mon, 13 Dec 2010 13:43:48 +0300,
Sergej Pupykin  wrote:

> video="9-pin DIN:1024x768-24 at 60e", but lilo uses doublequotes as
> r-value bounds.
> 
> in_quote variable in next_arg() is set only for " character.

Sorry, biggest part of this problem is in my linux distribution (in
/init executable from initcpio image)