From 24db7580a425fff8343c265b3bba85ee0822dc9a Mon Sep 17 00:00:00 2001
From: Adam Bright <brightam1@gmail.com>
Date: Thu, 29 Sep 2016 05:51:00 +0000
Subject: [PATCH] initialize configuration so we aren't unintentionally
 overwriting something on the first configuration change

---
 samples/mcp9808.go        | 13 +++++++++----
 sensor/mcp9808/mcp9808.go |  8 ++++++--
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/samples/mcp9808.go b/samples/mcp9808.go
index aa64526..104c52d 100644
--- a/samples/mcp9808.go
+++ b/samples/mcp9808.go
@@ -15,7 +15,7 @@ func main() {
 	bus := embd.NewI2CBus(1)
 	defer embd.CloseI2C()
 
-	therm := mcp9808.New(bus)
+	therm, _ := mcp9808.New(bus)
 	// set sensor to low power mode when we're done
 	defer therm.SetShutdownMode(true)
 
@@ -38,11 +38,16 @@ func main() {
 	config, _ := therm.Config()
 	fmt.Printf("New Config: %b\n", config)
 
-	if err := therm.SetCriticalTemp(TempFToC(90)); err != nil {
+	if err := therm.SetCriticalTemp(TempFToC(95)); err != nil {
 		panic(err)
 	}
+	critTemp, err := therm.CriticalTemp()
+	if err != nil {
+		fmt.Printf("Error reading critical temp limit: %s\n", err.Error())
+	}
+	fmt.Printf("Critical Temp set to: %fC\n", critTemp)
 
-	if err := therm.SetWindowTempLower(TempFToC(32)); err != nil {
+	if err := therm.SetWindowTempLower(TempFToC(50)); err != nil {
 		panic(err)
 	}
 	lowerTemp, err := therm.WindowTempLower()
@@ -51,7 +56,7 @@ func main() {
 	}
 	fmt.Printf("Lower Temp Limit set to: %fC\n", lowerTemp)
 
-	if err := therm.SetWindowTempUpper(TempFToC(75)); err != nil {
+	if err := therm.SetWindowTempUpper(TempFToC(80)); err != nil {
 		panic(err)
 	}
 	upperTemp, _ := therm.WindowTempUpper()
diff --git a/sensor/mcp9808/mcp9808.go b/sensor/mcp9808/mcp9808.go
index a5882dd..24bc85e 100644
--- a/sensor/mcp9808/mcp9808.go
+++ b/sensor/mcp9808/mcp9808.go
@@ -39,8 +39,12 @@ type MCP9808 struct {
 }
 
 // New returns a handle to a MCP9808 sensor.
-func New(bus embd.I2CBus) *MCP9808 {
-	return &MCP9808{Bus: bus}
+func New(bus embd.I2CBus) (*MCP9808, error) {
+	d := &MCP9808{Bus: bus}
+
+	// initialize the configuration
+	_, err := d.Config()
+	return d, err
 }
 
 // ManufacturerID reads the device manufacturer ID