altos: Check for companion init packet validity was busted

[fw/altos] / src / ao_companion.c

diff --git a/src/ao_companion.c b/src/ao_companion.c
index e2e6d05ae81c0f6b468dd044b0857e3af16821cb..f54f5251afb92135065306493ab77c2ba7e8e689 100644 (file)
--- a/src/ao_companion.c
+++ b/src/ao_companion.c
@@ -53,9 +53,8 @@ ao_companion_get_setup(void)
        ao_companion_send_command(AO_COMPANION_SETUP);
        ao_spi_recv(&ao_companion_setup, sizeof (ao_companion_setup));
        COMPANION_DESELECT();
-       if (ao_companion_setup.board_id != ~ao_companion_setup.board_id)
-               return 0;
-       return 1;
+       return (ao_companion_setup.board_id ==
+               ~ao_companion_setup.board_id_inverse);
 }
 
 static void
@@ -72,10 +71,8 @@ ao_companion_get_data(void)
 void
 ao_companion(void)
 {
-       if (!ao_companion_get_setup())
-               ao_exit();
-       ao_companion_running = 1;
-       for (;;) {
+       ao_companion_running = ao_companion_get_setup();
+       while (ao_companion_running) {
                ao_delay(ao_companion_setup.update_period);
                ao_companion_get_data();
        }
@@ -85,6 +82,15 @@ ao_companion(void)
 void
 ao_companion_status(void) __reentrant
 {
+       uint8_t i;
+       printf("Companion running: %d\n", ao_companion_running);
+       printf("device: %d\n", ao_companion_setup.board_id);
+       printf("update period: %d\n", ao_companion_setup.update_period);
+       printf("channels: %d\n", ao_companion_setup.channels);
+       printf("data:");
+       for(i = 0; i < ao_companion_setup.channels; i++)
+               printf(" %5u", ao_companion_data[i]);
+       printf("\n");
 }
 
 __code struct ao_cmds ao_companion_cmds[] = {
@@ -102,6 +108,5 @@ ao_companion_init(void)
        COMPANION_CS_SEL &= ~COMPANION_CS_MASK; /* set CS pins as GPIO */
 
        ao_cmd_register(&ao_companion_cmds[0]);
-
        ao_add_task(&ao_companion_task, ao_companion, "companion");
 }