On Fri, Jun 10, 2011 at 02:28:55PM +0300, Jarkko Nikula wrote: > > + if (mcbsp_data->active) > + return 0; > +
Shouldn't we be returning -EBUSY or something here so the caller knows we didn't do what it asked (after checking to make sure it's not just trying to set the existing configuration)? Otherwise it might get surprised, assume the new configuration and just move the bug elsewhere. _______________________________________________ stable mailing list [email protected] http://linux.kernel.org/mailman/listinfo/stable
