On Thu, Aug 24, 2017 at 11:29:42AM +0800, jeffy wrote: > and i was trying to do something like: > if (dlc->dai_name && strcmp(dai->name, dlc->dai_name) > && (!dai->driver->name || strcmp(dai->driver->name, dlc->dai_name))) > continue;
> which is add an accept case for: dai driver name is valid and matches the > dai name we are looking for... Writing it as one if statement would at least be clearer. I can't remember the patch you proposed at this point but the above looks plausible.
signature.asc
Description: PGP signature