diff mbox

[for-2.0,6/7] tmp105-test: add a second sensor and test that one

Message ID 1396283195-6819-7-git-send-email-pbonzini@redhat.com
State New
Headers show

Commit Message

Paolo Bonzini March 31, 2014, 4:26 p.m. UTC
This will make it easier to reach the device under test via QOM.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 tests/tmp105-test.c | 17 +++++++++--------
 5 files changed, 13 insertions(+), 12 deletions(-)

Comments

Andreas Färber March 31, 2014, 9:01 p.m. UTC | #1
Am 31.03.2014 18:26, schrieb Paolo Bonzini:
> This will make it easier to reach the device under test via QOM.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  tests/tmp105-test.c | 17 +++++++++--------
>  5 files changed, 13 insertions(+), 12 deletions(-)
> 
> diff --git a/tests/tmp105-test.c b/tests/tmp105-test.c
> index 20a1894..4e640b4 100644
> --- a/tests/tmp105-test.c
> +++ b/tests/tmp105-test.c
> @@ -15,10 +15,10 @@
>  
>  #define OMAP2_I2C_1_BASE 0x48070000
>  
> -#define N8X0_ADDR 0x48
> +#define TMP105_TEST_ID   "tmp105-test"
> +#define TMP105_TEST_ADDR 0x49
>  
>  static I2CAdapter *i2c;
> -static uint8_t addr;
>  
>  static uint16_t tmp105_get16(I2CAdapter *i2c, uint8_t addr, uint8_t reg)
>  {
> @@ -61,14 +61,14 @@ static void send_and_receive(void)
>  {
>      uint16_t value;
>  
> -    value = tmp105_get16(i2c, addr, TMP105_REG_TEMPERATURE);
> +    value = tmp105_get16(i2c, TMP105_TEST_ADDR, TMP105_REG_TEMPERATURE);
>      g_assert_cmpuint(value, ==, 0);
>  
>      /* reset */
> -    tmp105_set8(i2c, addr, TMP105_REG_CONFIG, 0);
> +    tmp105_set8(i2c, TMP105_TEST_ADDR, TMP105_REG_CONFIG, 0);
>  
> -    tmp105_set16(i2c, addr, TMP105_REG_T_LOW, 0x1234);
> -    tmp105_set16(i2c, addr, TMP105_REG_T_HIGH, 0x4231);
> +    tmp105_set16(i2c, TMP105_TEST_ADDR, TMP105_REG_T_LOW, 0x1234);
> +    tmp105_set16(i2c, TMP105_TEST_ADDR, TMP105_REG_T_HIGH, 0x4231);
>  }
>  
>  int main(int argc, char **argv)
> @@ -78,9 +78,10 @@ int main(int argc, char **argv)
>  
>      g_test_init(&argc, &argv, NULL);
>  
> -    s = qtest_start("-machine n800");
> +    s = qtest_start("-machine n800 "
> +                    "-device tmp105,bus=i2c-bus.0,id=" TMP105_TEST_ID
> +                    ",address=0x49");

Probably we could do some clever stringify() tricks here to reuse your
TMP105_TEST_ADDR constant, but I'll leave it as is for now.

Andreas

>      i2c = omap_i2c_create(OMAP2_I2C_1_BASE);
> -    addr = N8X0_ADDR;
>  
>      qtest_add_func("/tmp105/tx-rx", send_and_receive);
>  
>
diff mbox

Patch

diff --git a/tests/tmp105-test.c b/tests/tmp105-test.c
index 20a1894..4e640b4 100644
--- a/tests/tmp105-test.c
+++ b/tests/tmp105-test.c
@@ -15,10 +15,10 @@ 
 
 #define OMAP2_I2C_1_BASE 0x48070000
 
-#define N8X0_ADDR 0x48
+#define TMP105_TEST_ID   "tmp105-test"
+#define TMP105_TEST_ADDR 0x49
 
 static I2CAdapter *i2c;
-static uint8_t addr;
 
 static uint16_t tmp105_get16(I2CAdapter *i2c, uint8_t addr, uint8_t reg)
 {
@@ -61,14 +61,14 @@  static void send_and_receive(void)
 {
     uint16_t value;
 
-    value = tmp105_get16(i2c, addr, TMP105_REG_TEMPERATURE);
+    value = tmp105_get16(i2c, TMP105_TEST_ADDR, TMP105_REG_TEMPERATURE);
     g_assert_cmpuint(value, ==, 0);
 
     /* reset */
-    tmp105_set8(i2c, addr, TMP105_REG_CONFIG, 0);
+    tmp105_set8(i2c, TMP105_TEST_ADDR, TMP105_REG_CONFIG, 0);
 
-    tmp105_set16(i2c, addr, TMP105_REG_T_LOW, 0x1234);
-    tmp105_set16(i2c, addr, TMP105_REG_T_HIGH, 0x4231);
+    tmp105_set16(i2c, TMP105_TEST_ADDR, TMP105_REG_T_LOW, 0x1234);
+    tmp105_set16(i2c, TMP105_TEST_ADDR, TMP105_REG_T_HIGH, 0x4231);
 }
 
 int main(int argc, char **argv)
@@ -78,9 +78,10 @@  int main(int argc, char **argv)
 
     g_test_init(&argc, &argv, NULL);
 
-    s = qtest_start("-machine n800");
+    s = qtest_start("-machine n800 "
+                    "-device tmp105,bus=i2c-bus.0,id=" TMP105_TEST_ID
+                    ",address=0x49");
     i2c = omap_i2c_create(OMAP2_I2C_1_BASE);
-    addr = N8X0_ADDR;
 
     qtest_add_func("/tmp105/tx-rx", send_and_receive);