Re: [Qemu-devel] [PATCH] arm: imx25-pdk: Fix machine name

[Peter Maydell]

On 13 October 2015 at 18:18, Peter Maydell <address@hidden> wrote:
> On 10 October 2015 at 03:56, Peter Crosthwaite
> <address@hidden> wrote:
>> ARM uses dashes instead of underscores for machine names. Fix imx25_pdk
>> which has not seen a release yet (so there is no legacy yet).
>>
>> Cc: Jean-Christophe Dubois <address@hidden>
>> Signed-off-by: Peter Crosthwaite <address@hidden>
>> ---
>>  hw/arm/imx25_pdk.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/arm/imx25_pdk.c b/hw/arm/imx25_pdk.c
>> index 4250114..59a4c11 100644
>> --- a/hw/arm/imx25_pdk.c
>> +++ b/hw/arm/imx25_pdk.c
>> @@ -151,4 +151,4 @@ static void imx25_pdk_machine_init(MachineClass *mc)
>>      mc->init = imx25_pdk_init;
>>  }
>>
>> -DEFINE_MACHINE("imx25_pdk", imx25_pdk_machine_init)
>> +DEFINE_MACHINE("imx25-pdk", imx25_pdk_machine_init)

> Applied to target-arm.next, thanks.

This patch breaks 'make check'. (tests/ds1338-test.c
tries to start a machine "imx25_pdk" still). I have squashed in
this fix:

--- a/tests/ds1338-test.c
+++ b/tests/ds1338-test.c
@@ -61,7 +61,7 @@ int main(int argc, char **argv)

     g_test_init(&argc, &argv, NULL);

-    s = qtest_start("-display none -machine imx25_pdk");
+    s = qtest_start("-display none -machine imx25-pdk");
     i2c = imx_i2c_create(IMX25_I2C_0_BASE);
     addr = DS1338_ADDR;

thanks
-- PMM