Hi,

On May 21 2018 20:58, Yisheng Xie wrote:

match_string() returns the index of an array for a matching string, which can be used intead of open coded variant.

Cc: Clemens Ladisch clemens@ladisch.de Cc: Jaroslav Kysela perex@perex.cz Cc: Takashi Iwai tiwai@suse.com Cc: alsa-devel@alsa-project.org Signed-off-by: Yisheng Xie xieyisheng1@huawei.com

---

sound/firewire/oxfw/oxfw.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-)

Reviewed-by: Takashi Sakamoto o-takashi@sakamocchi.jp

For my information, use_match_string() helper was firstly introduced to v4.6 kernel by a commit 56b060814e2d ('lib/string: introduce match_string() helper').

diff --git a/sound/firewire/oxfw/oxfw.c b/sound/firewire/oxfw/oxfw.c index 413ab63..1e5b2c8 100644 --- a/sound/firewire/oxfw/oxfw.c +++ b/sound/firewire/oxfw/oxfw.c @@ -49,7 +49,6 @@ static bool detect_loud_models(struct fw_unit *unit) "Tapco LINK.firewire 4x6", "U.420"}; char model[32];

• unsigned int i; int err;

  err = fw_csr_string(unit->directory, CSR_MODEL,

@@ -57,12 +56,7 @@ static bool detect_loud_models(struct fw_unit *unit) if (err < 0) return false;

• for (i = 0; i < ARRAY_SIZE(models); i++) {

• if (strcmp(models[i], model) == 0)
  

• 	break;
  

• }
• return (i < ARRAY_SIZE(models));

• return match_string(models, ARRAY_SIZE(models), model) >= 0; }

  static int name_card(struct snd_oxfw *oxfw)

Thanks

Takashi Sakamoto