sandbox: fix compiler warning

Fix compiler warning about control flow reaching end of non void function.

Signed-off-by Allen Martin <[email protected]>

---
 arch/sandbox/cpu/start.c |    3 +++
 1 file changed, 3 insertions(+)

arch/sandbox/cpu/start.c +3 -1

index 7603bf9..5287fd5 100644
@@ -122,4 +122,7 @@ int main(int argc, char *argv[])
         * never return.
         */
        board_init_f(0);
+
+       /* NOTREACHED - board_init_f() does not return */
+       return 0;
 }
--

1.7.10.4

Quoted 4 lines · Allen Martin

Fix compiler warning about control flow reaching end of non void function.

Signed-off-by Allen Martin <[email protected]>

Should perhaps mention the commit that adding this warning (e05e5de7 I think)

Acked-by Simon Glass <[email protected]>

Quoted 10 lines · Simon Glass

Quoted 4 lines · Allen Martin

Fix compiler warning about control flow reaching end of non void function.

Signed-off-by Allen Martin <[email protected]>

Should perhaps mention the commit that adding this warning (e05e5de7 I think)

Acked-by Simon Glass <[email protected]>

It was this commit:

commit fec79acc864bed049b6beae719ccbf2bbec5403a

Author: Mike Frysinger <[email protected]>
Date:   Wed Oct 26 00:22:14 2011 +0000

sandbox: drop unused return

Signed-off-by Mike Frysinger <[email protected]>

Acked-by Simon Glass <[email protected]>

I'll change the commit messagen and repost and cc Mike as well.

-Allen
--
nvpublic

Hi Allen,

Quoted 24 lines · Allen Martin

Quoted 10 lines · Simon Glass

Quoted 4 lines · Allen Martin

Fix compiler warning about control flow reaching end of non void function.

Signed-off-by Allen Martin <[email protected]>

Should perhaps mention the commit that adding this warning (e05e5de7 I think)

Acked-by Simon Glass <[email protected]>

It was this commit:

commit fec79acc864bed049b6beae719ccbf2bbec5403a

Author: Mike Frysinger <[email protected]>
Date:   Wed Oct 26 00:22:14 2011 +0000

sandbox: drop unused return

Signed-off-by Mike Frysinger <[email protected]>

Acked-by Simon Glass <[email protected]>

Quoted 1 line

I'll change the commit messagen and repost and cc Mike as well.

Actually I think that Mike's commit was correct at the time, and it was Albert's which changed the no-return behaviour of the function signature in common.h?