qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-arm] [PATCH v4 1/8] arm: Uniquely name imx25 I2C buses.


From: Jean-Christophe DUBOIS
Subject: Re: [Qemu-arm] [PATCH v4 1/8] arm: Uniquely name imx25 I2C buses.
Date: Fri, 16 Dec 2016 21:17:03 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1

Le 16/12/2016 à 14:03, Peter Maydell a écrit :
On 15 December 2016 at 05:48, Alastair D'Silva <address@hidden> wrote:
From: Alastair D'Silva <address@hidden>

The imx25 chip provides 3 i2c buses, but they have all been named
"i2c", which makes it difficult to predict which bus a device will
be connected to when specified on the command line.

This patch addresses the issue by naming the buses uniquely:
   i2c-bus.0 i2c-bus.1 i2c-bus.2

Signed-off-by: Alastair D'Silva <address@hidden>
---
  hw/arm/imx25_pdk.c | 2 +-
  hw/i2c/imx_i2c.c   | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/arm/imx25_pdk.c b/hw/arm/imx25_pdk.c
index 025b608..44e741f 100644
--- a/hw/arm/imx25_pdk.c
+++ b/hw/arm/imx25_pdk.c
@@ -139,7 +139,7 @@ static void imx25_pdk_init(MachineState *machine)
           * of simple qtest. See "make check" for details.
           */
          i2c_create_slave((I2CBus *)qdev_get_child_bus(DEVICE(&s->soc.i2c[0]),
-                                                      "i2c"),
+                                                      "i2c-bus.0"),
                           "ds1338", 0x68);
      }
  }
diff --git a/hw/i2c/imx_i2c.c b/hw/i2c/imx_i2c.c
index 37e5a62..6c81b98 100644
--- a/hw/i2c/imx_i2c.c
+++ b/hw/i2c/imx_i2c.c
@@ -310,7 +310,7 @@ static void imx_i2c_realize(DeviceState *dev, Error **errp)
                            IMX_I2C_MEM_SIZE);
      sysbus_init_mmio(SYS_BUS_DEVICE(dev), &s->iomem);
      sysbus_init_irq(SYS_BUS_DEVICE(dev), &s->irq);
-    s->bus = i2c_init_bus(DEVICE(dev), "i2c");
+    s->bus = i2c_init_bus(DEVICE(dev), NULL);
  }

  static void imx_i2c_class_init(ObjectClass *klass, void *data)
This change looks OK to me, but it is an incompatible change to the
command line (albeit only one where you couldn't have reliably
used the command line to start with), so I'm just cc'ing
Jean-Christophe Dubois, who's been active with the iMX code in the
past, for a second opinion.

This change seems fine to me too.

JC


We should flag this up in 2.9's Changelog as well.

thanks
-- PMM





reply via email to

[Prev in Thread] Current Thread [Next in Thread]