Fwd: dell Inspiron 6400 sleep quirks not recognized

[Fulko Hew]

I submitted this email to the hal(a)lists.freedesktop.org group yesterday,
but I figure I'd send it to the Fedora list too...


-----------------------------
I decided to play with sleep and suspend on my Dell Inspiron 6400...
Most things work, but during recovery from sleep, the screen was
still dark/blank.  So following the instructions on the quirk site, I tried:


lshal | grep system.hardware

and I get:

[root@localhost log]# lshal | grep system.hardware
  system.hardware.primary_video.product = 10146  (0x27a2)  (int)
  system.hardware.primary_video.vendor = 32902  (0x8086)  (int)
  system.hardware.product = 'MM061'  (string)
  system.hardware.serial = '28W6RB1'  (string)
  system.hardware.uuid = '44454C4C-3800-1057-8036-B2C04F524231'  (string)
  system.hardware.vendor = 'Dell Inc.'  (string)

So when looking in

/usr/share/hal/fdi/information/10freedesktop/20-video-quirk-pm-dell.fdi

I see that the section for
        <match key="system.hardware.product" contains="MM061">

is contained within a section:

      <match key="system.hardware.product" prefix="Inspiron">

But as you can see my reported system.hardware.product line doesn't
contain "Inspiron', let alone as a prefix!

so I created and tested a section, duplicating the existing quirks
except without the prefix="Inspiron", and the quirks are now
recognized when I do:

lshal | grep quirk

and when I test 'sleep' on the laptop it now works!  :-)
(... well iwl3945 doesn't wake up, but everything else works now...)

So... it seems that some Dell Inspiron 6400's may be reported as
'Inspiron MM061' and others are called simply 'MM061'

Given my minimal knowledge of the syntax of the quirks file
and HAL software, I decided to post this message rather
than simply submit a git change.

... Is there a better way to handle this issue given the syntax
of the quirks file without having to resort to two 'replicated'
sections, or should the MM061 quirk simply be moved from
within its prefix='Inspiron' qualifier?

Comments?
TIA
Fulko