On Wed, 2010-09-22 at 08:06 -0700, Greg KH wrote:
On Tue, Sep 21, 2010 at 11:20:22PM -0400, Andy Walls wrote:
+Where: /sys/devices/<...>/drm/cardN/cardN-C-M/polled
For N a decimal system graphics adapter number: 0, 1, 2, ...For C a connector type name (including spaces) from the set:Spaces? Really? Yeah, I know it will work just fine, but why go out of your way to make it hard for people?
Not my fault. It was like that when I got here.
$ ls -d /sys/devices/pci0000:00/0000:00:01.0/0000:01:05.0/drm//card0/card0* /sys/devices/pci0000:00/0000:00:01.0/0000:01:05.0/drm//card0/card0-DVI-D-1 /sys/devices/pci0000:00/0000:00:01.0/0000:01:05.0/drm//card0/card0-HDMI Type A-1 /sys/devices/pci0000:00/0000:00:01.0/0000:01:05.0/drm//card0/card0-VGA-1
I only added one entry, "polled", in those existing directories.
BTW, Jon Smirl, IBM Corp, and you have Copyright credit in the drm_sysfs.c file that creates those directories with spaces. ;)
I'll also gripe that PCI's colons are pretty annoying on the command line too, since they have to be escaped as well.
HDMI Type AHDMI Type BTVEmbedded DisplayPortYou could always just use a '_' instead of a space for those that need it.
A trivial patch here would do that:
http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=blob;f=drive...
but I have no idea what that might break.
Regards, Andy