[oe] [PATCH] Alsa-state: Wildcard removed and all *.state put back in the recipes

Marcin Juszkiewicz marcin at juszkiewicz.com.pl
Sun Mar 22 19:15:33 UTC 2009


> > diff --git a/recipes/alsa/alsa-state.bb

I looked more at that recipe and found few bugs in it:

1. It assumes that 'asound.conf' is same for all devices. This is not
   true for 'at91sam9263ek' which kernel oops with default file.

2. It plays with INITSCRIPT_PARAMS in postinst script. I do not know
   maybe it was required in past but why it is present now? No idea and
   no info in recipe why it is that way.

3. Postinst script restore default ALSA settings which is ugly too - 
   what if user just muted all channels to have device quiet during 
   night and postinst just makes his 1000W 7.1 audio system goes to full
   volume at 02:00 at night?

4. With current version is is broken for a780 device which has few
   states files available.

Regards, 
-- 
JID:      hrw at jabber.org
Website:  http://marcin.juszkiewicz.com.pl/
LinkedIn: http://www.linkedin.com/in/marcinjuszkiewicz






More information about the Openembedded-devel mailing list