On Tue, 2007-06-05 at 10:25 +0200, Matthias Saou wrote:
Richard Hughes wrote :
> On Fri, 2007-06-01 at 10:52 +0200, Matthias Saou wrote:
> > I just need to get S3 working again, as I wanted to start fresh and
> > removed all the kernel boot options I had in case they weren't useful
> > anymore...
>
> If you are talking about s3 (suspend), see
>
http://people.freedesktop.org/~hughsient/quirk/
That page was incredibly helpful. I actually understood what all those
different "quirks" did! :-)
Good :-)
I'll continue to add content as required. Soon multimedia keyboard will
be quirks in hal-info and the same site will be used to make keyboards
just work in the future.
For my Dell "Latitude X1", which is not yet in the F7 HAL
files, I had
to enable the --quirk-vbe-post quirk, as otherwise the screen never
turned on.
Cool.
With F7, I switched to the "intel" modesetting driver (also
removed
855resolution), added the "power_management.quirk.vbe_post" key to true
in a string="Latitude X1" section of the Dell HAL file, and it all just
works now! Awesome! :-)
Cool.
I had a last question, though : The Dell models seem to be
identified
with system.hardware.product contains="Foo". This seems like a pretty
bad idea, since it would be contains="X1" in this case, and will/would
match a future Dell "Latitude X10" laptop... It would seem wiser to use
string="Latitude X1", even though it would be partly redundant with the
earlier prefix="Latitude". Or maybe there is a suffix="Bar"?
There is such a thing:
<!ELEMENT match (match|merge|prepend|append|addset|remove|spawn)*
<!ATTLIST match
key CDATA #REQUIRED
string CDATA #IMPLIED
int CDATA #IMPLIED
bool (false|true) #IMPLIED
exists (false|true) #IMPLIED
empty (false|true) #IMPLIED
is_ascii (false|true) #IMPLIED
is_absolute_path (false|true) #IMPLIED
contains CDATA #IMPLIED
contains_ncase CDATA #IMPLIED
contains_not CDATA #IMPLIED
prefix CDATA #IMPLIED
prefix_ncase CDATA #IMPLIED
suffix CDATA #IMPLIED
suffix_ncase CDATA #IMPLIED
compare_lt CDATA #IMPLIED
compare_le CDATA #IMPLIED
compare_gt CDATA #IMPLIED
compare_ge CDATA #IMPLIED
compare_ne CDATA #IMPLIED
sibling_contains CDATA #IMPLIED
You can either use suffix or just use string for the whole text.
I would be very appreciative if you could generate a patch and sent it
to the mailing list. I'm glad things are now working.
Richard.