[alsa-devel] [PATCH 13/31] ASoC: wm_adsp: Fix BUG_ON() and WARN_ON() usages

Mark Brown broonie at kernel.org
Thu Nov 7 12:51:58 CET 2013


On Thu, Nov 07, 2013 at 12:07:39PM +0100, Takashi Iwai wrote:

> Yes, and think that this is the first step.  At least, we can refuse
> the use of BUG*() in any of merged materials in future.  Leaving
> BUG*() is a bad sign of lazy error handling.

Send patches to remove the definition of BUG then...

> And, actually your vendor-lock assumption doesn't look valid any
> longer.  There are more and more ARM-based systems that can install
> OS later relatively easily.  Most of distros (openSUSE, Ubuntu,

Practically speaking those systems are all in the same class, the
general purpose end user ARM systems don't have audio hardware (things
like the Chromebook are still closed boxes with a developer mode which
isn't quite the same thing).  We certainly don't have anyone actually
working on any of the stuff that would make such environments viable,
the people trying are focusing on servers.

> Fedora, etc) already provide the installation images, and I know many
> users of Chromebooks running such.  (And I know some of them burned
> speakers due to the incorrect UCM setup or usage :-)

Right, there's a whole bunch more stuff to do if you actually want to
handle this stuff in production - the health and safety issues from
excessive volume are even more of a concern for me than the issues with
physical damage to the systems.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20131107/398bae3d/attachment.sig>


More information about the Alsa-devel mailing list