Re: [PATCH 1/5] ASoC: dapm: If one widget fails, do not force allsubsequent widgets to fail too

From: Mark Brown
Date: Tue Jul 31 2012 - 10:56:10 EST


On Tue, Jul 31, 2012 at 03:45:40PM +0100, Lee Jones wrote:
> If a list of widgets is provided and one of them fails to be added as
> a control, the present semantics fail all subsequent widgets. A better
> solution would be to only fail that widget, but pursue in attempting
> to add the rest of the list.

To reiterate, this is in *no* way urgent or even a bug fix.

> dev_err(dapm->dev,
> "ASoC: Failed to create DAPM control %s\n",
> widget->name);
> - ret = -ENOMEM;
> - break;

Indeed, removing the error return is a regression.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/