[PATCH 1/2] comm: msgb: don't set backlight on error

Wolfram Sang wolfram at the-dreams.de
Mon May 2 14:27:03 CEST 2011


On 02/05/11 14:02, Sylvain Munaut wrote:

>> I disagree.  We desperately need board_panic, (with much more than just
>> backlight blinking in the future), to be able to solve the various mysterious
>> layer1/firmware crashes that we're seeing more or less all the time.
> 
> Yes, it definitely needed.
> 
> And calling osmo_panic might be the right thing, and then have the
> board register their osmo_panic handler in its init (the buzzer would
> be a good choice :)

Okay, understood that it is needed. Still, the discussion also showed that
I seem to not have the proper knowledge of what is desired instead by
those people hacking at layer 1 (what I don't do) :) Thus, may I suggest
that I just remove the (static value) backlight setting in msgb.c to
remove the calypso-dependency and leave a proper board_panic for someone
else with a seperate patchset? To me, Harald's post indicates that it is a
seperate topic?

Regards,

   Wolfram



More information about the baseband-devel mailing list