On Fri, Feb 24, 2017 at 9:23 AM, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> wrote: > There's a some duplication for what's essentially copies of > two loops, so factor it. The upcoming AST2500 POST code adds > more of them. Also cleanup return types for the test functions, > most of them return a boolean, some return a u32. > > Signed-off-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> Acked-by: Joel Stanley <joel@xxxxxxxxx> > -- > > v2. - Keep the split between the "test" and "test2" functions > as they have a different exit condition in the loop and > a different return type. > - Fix the return types accross the call chain > --- > drivers/gpu/drm/ast/ast_post.c | 82 ++++++++++++++++-------------------------- > 1 file changed, 31 insertions(+), 51 deletions(-) _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel