Messages in this thread | | | Date | Thu, 24 Aug 2017 11:18:26 +0100 | From | Mark Brown <> | Subject | Re: [PATCH] ASoC: Add a sanity check before using dai driver name |
| |
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. [unhandled content-type:application/pgp-signature] | |