On Mon, May 21, 2018 at 2:57 PM, Yisheng Xie <[email protected]> wrote: > match_string() returns the index of an array for a matching string, > which can be used intead of open coded variant. > > Cc: Michael Turquette <[email protected]> > Cc: Stephen Boyd <[email protected]> > Cc: Eric Anholt <[email protected]> > Cc: Stefan Wahren <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Yisheng Xie <[email protected]>
> - size_t i, j; > - int ret; > + int i, ret; I do not see any need to change type for i. > + ret = match_string(cprman_parent_names, > + ARRAY_SIZE(cprman_parent_names), > + parents[i]); > + if (ret >= 0) > + parents[i] = cprman->real_parent_names[ret]; -- With Best Regards, Andy Shevchenko

