annotate tempctrl.c @ 81:781071a115be

Squeeze down the output so it will fit in 80 colums.
author Daniel O'Connor <darius@dons.net.au>
date Mon, 11 May 2009 09:55:51 +0930
parents c8cd6dd74ca1
children 93388163e037
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
1 /*
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
2 * Temperature control logic
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
3 *
73
56165caf744b Missed some USB related crud + copyright updates.
darius@Inchoate
parents: 71
diff changeset
4 * Copyright (c) 2009
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
5 * Daniel O'Connor <darius@dons.net.au>. All rights reserved.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
6 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
7 * Redistribution and use in source and binary forms, with or without
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
8 * modification, are permitted provided that the following conditions
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
9 * are met:
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
10 * 1. Redistributions of source code must retain the above copyright
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
11 * notice, this list of conditions and the following disclaimer.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
12 * 2. Redistributions in binary form must reproduce the above copyright
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
13 * notice, this list of conditions and the following disclaimer in the
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
14 * documentation and/or other materials provided with the distribution.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
15 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
16 * THIS SOFTWARE IS PROVIDED BY AUTHOR AND CONTRIBUTORS ``AS IS'' AND
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
17 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
18 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
19 * ARE DISCLAIMED. IN NO EVENT SHALL AUTHOR OR CONTRIBUTORS BE LIABLE
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
20 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
21 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
22 * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
23 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
24 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
25 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
26 * SUCH DAMAGE.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
27 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
28
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
29 #include <stdio.h>
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
30 #include <stdint.h>
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
31 #include <stdlib.h>
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
32 #include <string.h>
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
33 #include <avr/interrupt.h>
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
34 #include <avr/pgmspace.h>
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
35 #include <avr/eeprom.h>
46
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
36 #include <avr/wdt.h>
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
37 #include <util/crc16.h>
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
38
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
39 #include "cons.h"
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
40 #include "1wire.h"
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
41 #include "tempctrl.h"
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
42 #include "ds1307.h"
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
43
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
44 typedef struct {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
45 int32_t sec;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
46 int32_t usec;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
47 } time_t;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
48
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
49 /* Holds all the settings needed */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
50 typedef struct {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
51 uint8_t fermenter_ROM[8];
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
52 uint8_t fridge_ROM[8];
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
53 uint8_t ambient_ROM[8];
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
54 int16_t target_temp;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
55 uint16_t hysteresis;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
56 /* How much to under/overshoot on heating/cooling */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
57 int16_t minheatovershoot;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
58 int16_t mincoolovershoot;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
59
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
60 /* Minimum time the cooler can be on/off */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
61 int16_t mincoolontime;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
62 int16_t mincoolofftime;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
63
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
64 /* Minimum time the heater can be on/off */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
65 int16_t minheatontime;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
66 int16_t minheatofftime;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
67
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
68 #define TC_MODE_AUTO 'a' /* Automatic control */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
69 #define TC_MODE_HEAT 'h' /* Force heating */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
70 #define TC_MODE_COOL 'c' /* Force cooling */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
71 #define TC_MODE_IDLE 'i' /* Force idle */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
72 #define TC_MODE_NOTHING 'n' /* Do nothing (like idle but log nothing) */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
73 char mode;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
74
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
75 /* Bit patterns for various modes */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
76 uint8_t coolbits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
77 uint8_t heatbits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
78 uint8_t idlebits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
79
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
80 /* Check/stale times */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
81 int16_t check_interval;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
82 int16_t stale_factor;
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
83
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
84 /* Beep if stale */
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
85 int8_t dobeep;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
86 } __attribute__((packed)) settings_t;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
87
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
88 /* Current settings in RAM */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
89 static settings_t settings;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
90
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
91 /* Our map of EEPROM */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
92 struct {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
93 settings_t settings;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
94 uint16_t crc;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
95 } ee_area __attribute__((section(".eeprom")));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
96
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
97 /* Defaults that are shoved into EEPROM if it isn't inited */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
98 const PROGMEM settings_t default_settings = {
66
5c2a1c68e386 Change to new ROM ID (EEPROM appears to be fried now - ARGH)
darius@Inchoate
parents: 65
diff changeset
99 .fermenter_ROM = { 0x10, 0x8b, 0x7a, 0x53, 0x01, 0x08, 0x00, 0xb4 },
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
100 .fridge_ROM = { 0x10, 0xa6, 0x2a, 0xc4, 0x00, 0x08, 0x00, 0x11 },
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
101 .ambient_ROM = { 0x10, 0x97, 0x1b, 0xfe, 0x00, 0x08, 0x00, 0xd1 },
77
ca5fc59ec4f0 Change default temperature to 18C - better for Ale yeast.
darius@Inchoate
parents: 73
diff changeset
102 .target_temp = 1800,
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
103 .hysteresis = 100,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
104 .minheatovershoot = 50,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
105 .mincoolovershoot = -50,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
106 .mincoolontime = 300,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
107 .mincoolofftime = 600,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
108 .minheatontime = 60,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
109 .minheatofftime = 60,
67
52aa7716148c Default to auto (EEPROM is fried so this is the only way..)
darius@Inchoate
parents: 66
diff changeset
110 .mode = TC_MODE_AUTO,
46
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
111 .coolbits = _BV(6),
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
112 .heatbits = _BV(7),
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
113 .idlebits = 0x00,
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
114 .check_interval = 10,
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
115 .stale_factor = 3,
59
c72cf25881fe Default to not beeping (saves eardrums during testing)
darius@Inchoate
parents: 55
diff changeset
116 .dobeep = 0
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
117 };
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
118
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
119 /* Local variable declarations */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
120 volatile static time_t now;
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
121 volatile static uint8_t beeping = 0;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
122 volatile static uint8_t lasttoggle = 0;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
123
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
124 /* Local function prototypes */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
125 static void tempctrl_load_or_init_settings(void);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
126 static void tempctrl_default_settings(void);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
127 static void tempctrl_write_settings(void);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
128 static void setstate(char state);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
129 static const PROGMEM char*state2long(char s);
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
130 static void printtemp(char *name, int tmp, char *trailer);
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
131
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
132 /*
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
133 * tempctrl_init
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
134 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
135 * Setup timer, should be called with interrupts disabled.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
136 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
137 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
138 void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
139 tempctrl_init(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
140 /* Setup timer */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
141 /* 16Mhz / 1024 = 15625 Hz / 125 = 125 Hz = IRQ every 8 ms */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
142
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
143 /* CTC mode, no output on pin, Divide clock by 1024 */
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
144 TCCR2 = _BV(WGM21)| _BV(CS22) | _BV(CS21) | _BV(CS20);
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
145
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
146 /* Compare with ... */
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
147 OCR2 = 125;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
148
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
149 /* Enable interrupt for match on A */
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
150 TIMSK = _BV(OCIE2);
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
151
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
152 now.sec = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
153 now.usec = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
154
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
155 tempctrl_load_or_init_settings();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
156 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
157
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
158 /*
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
159 * Timer 2 Compare IRQ
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
160 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
161 * Update time counter
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
162 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
163
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
164 ISR(TIMER2_COMP_vect) {
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
165 now.usec += 8000; /* 1000000 * 1 / F_CPU / 1024 / 125 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
166 while (now.usec > 1000000) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
167 now.usec -= 1000000;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
168 now.sec++;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
169 }
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
170
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
171 if (beeping) {
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
172 lasttoggle++;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
173 // 63 * 8ms = ~0.5s
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
174 if (lasttoggle > 63) {
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
175 DDRB ^= _BV(PB3);
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
176 lasttoggle = 0;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
177 }
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
178 } else {
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
179 DDRB &= ~_BV(PB3);
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
180 lasttoggle = 0;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
181 }
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
182 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
183
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
184 /*
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
185 * tempctrl_update
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
186 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
187 * Should be called in a normal context, could run things that take a long time.
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
188 * (ie 1wire bus stuff)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
189 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
190 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
191 void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
192 tempctrl_update(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
193 /* State variables */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
194 static int32_t checktime = 0; // Time of next check
46
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
195 static int32_t lastdata = INT32_MIN; // Last time we got data
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
196
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
197 static int16_t fermenter_temp = 0; // Fermenter temperature
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
198 static int16_t fridge_temp = 0; // Fridge temperature
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
199 static int16_t ambient_temp = 0; // Ambient temperature
46
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
200 static int32_t lastheaton = INT32_MIN; // Last time the heater was on
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
201 static int32_t lastheatoff = INT32_MIN;// Last time the heater was off
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
202 static int32_t lastcoolon = INT32_MIN; // Last time the cooler was on
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
203 static int32_t lastcooloff = INT32_MIN;// Last time the cooler was off
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
204 static char currstate = 'i'; // Current state
46
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
205 /* We init to times to INT32_MIN so that things function properly when
fb272cb82bcb Add watchdog reset to timer IRQ.
darius@Inchoate
parents: 44
diff changeset
206 * now < settings.minheat/cool/on/offtime */
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
207
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
208 /* Temporary variables */
62
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
209 int32_t t, tempt;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
210 int16_t diff;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
211 char nextstate;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
212 int forced;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
213 int stale;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
214
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
215 t = gettod();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
216 /* Time to check temperatures? */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
217 if (t < checktime)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
218 return;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
219
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
220 checktime = t + settings.check_interval;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
221
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
222 /* Don't do any logging, just force idle and leave */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
223 if (settings.mode == TC_MODE_NOTHING) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
224 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
225 goto setstate;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
226 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
227
80
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
228 /* Update our temperatures
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
229 * Can take a while (800ms each!) so reset the WDT
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
230 */
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
231 wdt_reset();
62
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
232 tempt = OWGetTemp(settings.fermenter_ROM);
80
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
233 wdt_reset();
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
234 fridge_temp = OWGetTemp(settings.fridge_ROM);
80
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
235 wdt_reset();
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
236 ambient_temp = OWGetTemp(settings.ambient_ROM);
80
c8cd6dd74ca1 Don't call wdt_reset() in the IRQ routine - it makes the WDT useless.
Daniel O'Connor <darius@dons.net.au>
parents: 79
diff changeset
237 wdt_reset();
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
238
62
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
239 /* We only care about this one, only update the value we decide on
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
240 * only if it is valid
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
241 */
68
ac45a96e8034 Check the right value for staleness(!)
darius@Inchoate
parents: 67
diff changeset
242 if (tempt > OW_TEMP_BADVAL) {
62
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
243 fermenter_temp = tempt;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
244 lastdata = t;
62
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
245 }
e955aa7047ed Don't update the temperature we make decisions if it is invalid.
darius@Inchoate
parents: 60
diff changeset
246
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
247 /* Check for stale data */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
248 if (lastdata + (settings.check_interval * settings.stale_factor) < t)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
249 stale = 1;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
250 else
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
251 stale = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
252
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
253 /* Default to remaining as we are */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
254 nextstate = '-';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
255
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
256 /* Temperature diff, -ve => too cold, +ve => too warm */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
257 diff = fermenter_temp - settings.target_temp;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
258
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
259 switch (currstate) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
260 case 'i':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
261 /* If we're idle then only heat or cool if the temperate difference is out of the
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
262 * hysteresis band
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
263 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
264 if (abs(diff) > settings.hysteresis) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
265 if (diff < 0 && settings.minheatofftime + lastheatoff < t)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
266 nextstate = 'h';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
267 else if (diff > 0 && settings.mincoolofftime + lastcooloff < t)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
268 nextstate = 'c';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
269 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
270 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
271
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
272 case 'c':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
273 /* Work out if we should go idle (based on min on time & overshoot) */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
274 if (diff + settings.mincoolovershoot < 0 &&
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
275 settings.mincoolontime + lastcoolon < t)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
276 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
277 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
278
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
279 case 'h':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
280 if (diff - settings.minheatovershoot > 0 &&
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
281 settings.minheatontime + lastheaton < t)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
282 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
283 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
284
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
285 default:
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
286 printf_P(PSTR("\r\nUnknown state %c, going to idle\n"), currstate);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
287 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
288 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
289 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
290
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
291 /* Override if we have stale data */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
292 if (stale)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
293 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
294
55
6b1057409d9a Skip beep related stuff #ifdef WITHUSB.
darius@Inchoate
parents: 54
diff changeset
295 /* Handle beeping */
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
296 if (settings.dobeep && stale)
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
297 beeping = 1;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
298 else
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
299 beeping = 0;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
300
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
301 /* Handle state forcing */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
302 if (settings.mode != TC_MODE_AUTO)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
303 forced = 1;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
304 else
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
305 forced = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
306
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
307 if (settings.mode == TC_MODE_IDLE)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
308 nextstate = 'i';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
309 else if (settings.mode == TC_MODE_HEAT)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
310 nextstate = 'h';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
311 else if (settings.mode == TC_MODE_COOL)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
312 nextstate = 'c';
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
313
42
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
314 // Keep track of when we last turned things on or off
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
315 switch (nextstate) {
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
316 case 'c':
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
317 if (currstate == 'h')
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
318 lastheatoff = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
319 lastcoolon = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
320 break;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
321
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
322 case 'h':
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
323 if (currstate == 'c')
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
324 lastcooloff = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
325 lastheaton = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
326 break;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
327
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
328 default:
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
329 if (currstate == 'c')
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
330 lastcooloff = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
331 if (currstate == 'h')
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
332 lastheatoff = t;
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
333 }
97ae82023d5b Actually keep track of when the heater & cooler go on/off..
darius@inchoate.localdomain
parents: 41
diff changeset
334
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
335 if (nextstate != '-')
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
336 currstate = nextstate;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
337
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
338 #if 0
81
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
339 printf_P(PSTR("T: %10ld, "), t);
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
340 #else
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
341 ds1307_printtime(PSTR(""), PSTR(": "));
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
342 #endif
81
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
343 printtemp(PSTR("Tr"), settings.target_temp, PSTR(", "));
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
344 printtemp(PSTR("Fm"), tempt, PSTR(", ")); // Use actual value from sensor
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
345 printtemp(PSTR("Fr"), fridge_temp, PSTR(", "));
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
346 printtemp(PSTR("Am"), ambient_temp, PSTR(", "));
781071a115be Squeeze down the output so it will fit in 80 colums.
Daniel O'Connor <darius@dons.net.au>
parents: 80
diff changeset
347 printf_P(PSTR("St: %S, Fl: %S%S\r\n"), state2long(currstate),
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
348 forced ? PSTR("F") : PSTR(""),
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
349 stale ? PSTR("S") : PSTR(""));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
350
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
351 setstate:
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
352 setstate(currstate);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
353 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
354
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
355 /* Return 'time of day' (really uptime) */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
356 int32_t
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
357 gettod(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
358 int32_t t;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
359
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
360 cli();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
361 t = now.sec;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
362 sei();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
363
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
364 return(t);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
365 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
366
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
367 /*
78
006a99b406dd Modify comment to match reality.
darius@Inchoate
parents: 77
diff changeset
368 * Print out temperature (or short error code) with specified trailer
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
369 */
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
370 static void
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
371 printtemp(char *name, int tmp, char *trailer) {
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
372 if (tmp > OW_TEMP_BADVAL)
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
373 printf_P(PSTR("%S: %d.%02d%S"), name, GETWHOLE(tmp), GETFRAC(tmp), trailer);
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
374 else
79
a1edf53d904e Use new routine to print short error code rather than a cryptic number.
darius@Inchoate
parents: 78
diff changeset
375 printf_P(PSTR("%S: %S%S"), name, OWTempStatusStr(tmp, 1), trailer);
50
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
376 }
a13e0ccc1d2d Rejig how temperatures are logged. Print out the final line at the end so it
darius@Inchoate
parents: 49
diff changeset
377
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
378 /* Read the settings from EEPROM
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
379 * If the CRC fails then reload from flash
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
380 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
381 static void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
382 tempctrl_load_or_init_settings(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
383 uint8_t *dptr;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
384 uint16_t crc, strcrc;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
385 int i;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
386
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
387 crc = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
388 eeprom_busy_wait();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
389 eeprom_read_block(&settings, &ee_area.settings, sizeof(settings_t));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
390 strcrc = eeprom_read_word(&ee_area.crc);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
391
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
392 dptr = (uint8_t *)&settings;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
393
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
394 for (i = 0; i < sizeof(settings_t); i++)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
395 crc = _crc16_update(crc, dptr[i]);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
396
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
397 /* All OK? */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
398 if (crc == strcrc)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
399 return;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
400
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
401 printf_P(PSTR("CRC mismatch got 0x%04x vs 0x%04x, setting defaults\r\n"), crc, strcrc);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
402 tempctrl_default_settings();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
403 tempctrl_write_settings();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
404 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
405
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
406 /* Load in the defaults from flash */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
407 static void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
408 tempctrl_default_settings(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
409 memcpy_P(&settings, &default_settings, sizeof(settings_t));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
410 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
411
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
412 /* Write the current settings out to EEPROM */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
413 static void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
414 tempctrl_write_settings(void) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
415 uint16_t crc;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
416 uint8_t *dptr;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
417 int i;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
418
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
419 eeprom_busy_wait();
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
420 eeprom_write_block(&settings, &ee_area.settings, sizeof(settings_t));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
421
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
422 dptr = (uint8_t *)&settings;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
423 crc = 0;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
424 for (i = 0; i < sizeof(settings_t); i++)
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
425 crc = _crc16_update(crc, dptr[i]);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
426
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
427 eeprom_write_word(&ee_area.crc, crc);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
428 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
429
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
430 /* Set the relays to match the desired state */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
431 static void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
432 setstate(char state) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
433 switch (state) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
434 case 'c':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
435 PORTC = settings.coolbits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
436 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
437
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
438 case 'h':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
439 PORTC = settings.heatbits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
440 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
441
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
442 default:
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
443 printf_P(PSTR("Unknown state %c, setting idle\r\n"), state);
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
444 /* fallthrough */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
445
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
446 case 'i':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
447 PORTC = settings.idlebits;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
448 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
449 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
450 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
451
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
452 /* Handle user command
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
453 *
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
454 */
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
455 void
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
456 tempctrl_cmd(char *buf) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
457 char cmd[6];
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
458 int16_t data;
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
459 uint8_t ROM[8];
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
460
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
461 if (sscanf_P(buf, PSTR("tc %5s"), cmd, &data) == 0) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
462 printf_P(PSTR("Unable to parse tc subcommand\r\n"));
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
463 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
464 }
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
465
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
466 if (!strcasecmp_P(cmd, PSTR("help"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
467 printf_P(PSTR(
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
468 "tc help This help\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
469 "tc save Save settings to EEPROM\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
470 "tc load Load or default settings from EEPROM\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
471 "tc dflt Load defaults from flash\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
472 "tc list List current settings\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
473 "tc mode [achin] Change control mode, must be one of\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
474 " a Auto\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
475 " c Always cool\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
476 " h Always heat\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
477 " i Always idle\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
478 " n Like idle but don't log anything\r\n"
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
479 "tc beep [01] Enable/disable beeping when data is stale\r\n"
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
480 "tc X Y Set X to Y where X is one of\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
481 " targ Target temperature\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
482 " hys Hysteresis range\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
483 " mhov Minimum heat overshoot\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
484 " mcov Minimum cool overshoot\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
485 " mcon Minimum cool on time\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
486 " mcoff Minimum cool off time\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
487 " mhin Minimum heat on time\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
488 " mhoff Minimum heat off time\r\n"
65
1ef17cd8af7a Tab to space conversion in usage text.
darius@Inchoate
parents: 62
diff changeset
489 "tc A B Set temperature sensor ID\r\n"
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
490 " Where A is ferm, frg or amb\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
491 " and B is of the form xx:xx:xx:xx:xx:xx:xx:xx\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
492 "\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
493 " Times are in seconds\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
494 " Temperatures are in hundredths of degrees Celcius\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
495 ));
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
496 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
497 }
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
498
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
499 if (!strcasecmp_P(cmd, PSTR("save"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
500 tempctrl_write_settings();
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
501 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
502 }
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
503 if (!strcasecmp_P(cmd, PSTR("load"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
504 tempctrl_load_or_init_settings();
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
505 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
506 }
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
507 if (!strcasecmp_P(cmd, PSTR("dflt"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
508 tempctrl_default_settings();
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
509 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
510 }
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
511 if (!strcasecmp_P(cmd, PSTR("list"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
512 printf_P(PSTR("Fermenter ROM ID %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
513 "Fridge ROM ID %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
514 "Ambient ROM ID %02x:%02x:%02x:%02x:%02x:%02x:%02x:%02x\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
515 "Mode - %c, Target - %d, Hystersis - %d\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
516 "Min heat overshoot - %d, Min cool overshoot - %d\r\n"
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
517 "Min cool on time - %d, Min cool off time - %d\r\n"
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
518 "Min heat on time - %d, Min heat off time - %d\r\n"
55
6b1057409d9a Skip beep related stuff #ifdef WITHUSB.
darius@Inchoate
parents: 54
diff changeset
519 "Beep on stale - %S\r\n"
6b1057409d9a Skip beep related stuff #ifdef WITHUSB.
darius@Inchoate
parents: 54
diff changeset
520 ),
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
521 settings.fermenter_ROM[0], settings.fermenter_ROM[1], settings.fermenter_ROM[2], settings.fermenter_ROM[3],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
522 settings.fermenter_ROM[4], settings.fermenter_ROM[5], settings.fermenter_ROM[6], settings.fermenter_ROM[7],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
523 settings.fridge_ROM[0], settings.fridge_ROM[1], settings.fridge_ROM[2], settings.fridge_ROM[3],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
524 settings.fridge_ROM[4], settings.fridge_ROM[5], settings.fridge_ROM[6], settings.fridge_ROM[7],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
525 settings.ambient_ROM[0], settings.ambient_ROM[1], settings.ambient_ROM[2], settings.ambient_ROM[3],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
526 settings.ambient_ROM[4], settings.ambient_ROM[5], settings.ambient_ROM[6], settings.ambient_ROM[7],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
527 settings.mode, settings.target_temp, settings.hysteresis,
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
528 settings.minheatovershoot, settings.mincoolovershoot,
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
529 settings.mincoolontime, settings.minheatontime,
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
530 settings.minheatontime, settings.minheatofftime,
55
6b1057409d9a Skip beep related stuff #ifdef WITHUSB.
darius@Inchoate
parents: 54
diff changeset
531 settings.dobeep ? PSTR("yes") : PSTR("no")
6b1057409d9a Skip beep related stuff #ifdef WITHUSB.
darius@Inchoate
parents: 54
diff changeset
532 );
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
533 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
534 }
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
535 if (!strcasecmp_P(cmd, PSTR("mode"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
536 switch (buf[8]) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
537 case TC_MODE_AUTO:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
538 case TC_MODE_HEAT:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
539 case TC_MODE_COOL:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
540 case TC_MODE_IDLE:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
541 case TC_MODE_NOTHING:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
542 settings.mode = buf[8];
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
543 break;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
544
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
545 default:
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
546 printf_P(PSTR("Unknown mode character '%c'\r\n"), buf[8]);
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
547 break;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
548 }
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
549 return;
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
550 }
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
551 if (!strcasecmp_P(cmd, PSTR("beep"))) {
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
552 if (buf[8] == '1' || buf[8] == 'y')
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
553 settings.dobeep = 1;
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
554 else if (buf[8] == '0' || buf[8] == 'n')
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
555 settings.dobeep = 0;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
556 else
60
50fca9562310 Add support for reading/writing to a DS1307 over TWI/IIC.
darius@Inchoate
parents: 59
diff changeset
557 printf_P(PSTR("Expected a y/1 or n/0\r\n"));
54
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
558 return;
58f1ec46bff6 Add a beeper when the data is stale (only when USB is disabled as they share
darius@Inchoate
parents: 51
diff changeset
559 }
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
560 if (!strcasecmp_P(cmd, PSTR("ferm")) ||
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
561 !strcasecmp_P(cmd, PSTR("frg")) ||
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
562 !strcasecmp_P(cmd, PSTR("amb"))) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
563
69
3dcf933f2640 Write the sub command into the right variable (oops). Now you can set the
darius@Inchoate
parents: 68
diff changeset
564 if (sscanf_P(buf, PSTR("tc %5s %hhx:%hhx:%hhx:%hhx:%hhx:%hhx:%hhx:%hhx"), cmd,
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
565 &ROM[0], &ROM[1], &ROM[2], &ROM[3],
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
566 &ROM[4], &ROM[5], &ROM[6], &ROM[7]) != 9) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
567 printf_P(PSTR("Unable to parse ROM ID\r\n"));
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
568 } else {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
569 if (!strcasecmp_P(cmd, PSTR("ferm")))
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
570 memcpy(settings.fermenter_ROM, ROM, sizeof(ROM));
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
571 if (!strcasecmp_P(cmd, PSTR("frg")))
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
572 memcpy(settings.fridge_ROM, ROM, sizeof(ROM));
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
573 if (!strcasecmp_P(cmd, PSTR("amb")))
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
574 memcpy(settings.ambient_ROM, ROM, sizeof(ROM));
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
575 }
70
ea5862987368 Fix up parsing logic.
darius@Inchoate
parents: 69
diff changeset
576 return;
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
577 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
578
70
ea5862987368 Fix up parsing logic.
darius@Inchoate
parents: 69
diff changeset
579 /* Handle setting the multitude of variables
ea5862987368 Fix up parsing logic.
darius@Inchoate
parents: 69
diff changeset
580 * It's last to simplify things */
51
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
581 if (sscanf_P(buf, PSTR("tc %5s %d"), cmd, &data) != 2) {
cb184206344d Rejig command parsing and assume the compiler isn't dumb (eg it can reuse
darius@Inchoate
parents: 50
diff changeset
582 printf_P(PSTR("Unable to parse tc subcommand & value\r\n"));
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
583 return;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
584 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
585
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
586 if (!strcasecmp_P(cmd, PSTR("targ"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
587 settings.target_temp = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
588 } else if (!strcasecmp_P(cmd, PSTR("hys"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
589 settings.hysteresis = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
590 } else if (!strcasecmp_P(cmd, PSTR("mhov"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
591 settings.minheatovershoot = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
592 } else if (!strcasecmp_P(cmd, PSTR("mcov"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
593 settings.mincoolovershoot = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
594 } else if (!strcasecmp_P(cmd, PSTR("mcon"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
595 settings.mincoolontime = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
596 } else if (!strcasecmp_P(cmd, PSTR("mcoff"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
597 settings.mincoolofftime = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
598 } else if (!strcasecmp_P(cmd, PSTR("mhon"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
599 settings.minheatontime = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
600 } else if (!strcasecmp_P(cmd, PSTR("mhoff"))) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
601 settings.minheatofftime = data;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
602 } else {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
603 printf_P(PSTR("Unknown setting\r\n"));
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
604 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
605 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
606
71
553c061fda7c Keep the newer GCC happy.
darius@Inchoate
parents: 70
diff changeset
607 static const char*
41
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
608 state2long(char s) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
609 switch (s) {
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
610 case 'i':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
611 return PSTR("idle");
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
612 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
613
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
614 case 'c':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
615 return PSTR("cool");
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
616 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
617
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
618 case 'h':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
619 return PSTR("heat");
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
620 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
621
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
622 case '-':
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
623 return PSTR("-");
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
624 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
625
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
626 default:
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
627 return PSTR("unknown");
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
628 break;
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
629 }
5898fba6593c Add temperature control.
darius@inchoate.localdomain
parents:
diff changeset
630 }