[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.0 6/7] tmp105-test: add a second sensor an
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH for-2.0 6/7] tmp105-test: add a second sensor and test that one |
Date: |
Mon, 31 Mar 2014 23:01:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
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 <address@hidden>
> ---
> 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);
>
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH for-2.0 0/7] SMBus and tmp105 fixes, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 1/7] smbus: allow returning an error from reads, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 2/7] smbus: return -1 if nothing found at the given address, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 3/7] pm_smbus: correctly report unclaimed cycles, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 4/7] tmp105: read temperature in milli-celsius, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 5/7] tmp105-test: wrap simple building blocks for testing, Paolo Bonzini, 2014/03/31
- [Qemu-devel] [PATCH for-2.0 6/7] tmp105-test: add a second sensor and test that one, Paolo Bonzini, 2014/03/31
- Re: [Qemu-devel] [PATCH for-2.0 6/7] tmp105-test: add a second sensor and test that one,
Andreas Färber <=
- [Qemu-devel] [PATCH for-2.0 7/7] tmp105-test: test QOM property and precision, Paolo Bonzini, 2014/03/31
- Re: [Qemu-devel] [PATCH for-2.0 0/7] SMBus and tmp105 fixes, Andreas Färber, 2014/03/31