On Fri, Nov 23, 2018 at 11:54 AM Phil Edworthy <phil.edworthy@xxxxxxxxxxx> wrote: > If we assign the result of of_get_child_count() to an unsigned int, > the code will not detect any errors. Therefore assign it to an int > instead. Note that currently of_get_child_count() never returns a negative error code. But it does return int, not unsigned int. > Signed-off-by: Phil Edworthy <phil.edworthy@xxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in sh-pfc-for-v4.21. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds