[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation.
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation. |
Date: |
Thu, 10 Mar 2016 17:38:37 +0700 |
On 2 March 2016 at 05:27, Jean-Christophe Dubois <address@hidden> wrote:
> The sabrelite supports one SPI FLASH memory on SPI1
>
> Signed-off-by: Jean-Christophe Dubois <address@hidden>
> ---
>
> +
> + {
> + /* Add the sst25vf016b NOR FLASH memory to first SPI */
> + Object *spi_dev;
> +
> + spi_dev = object_resolve_path_component(OBJECT(&s->soc), "spi1");
> + if (spi_dev) {
> + SSIBus *spi_bus;
> +
> + spi_bus = (SSIBus *)qdev_get_child_bus(DEVICE(spi_dev), "spi");
This looks odd. You should just be able to do
spi_bus = (SSIBus *)qdev_get_child_bus(DEVICE(&s->soc), "spi1");
without using object_resolve_path_component() to try to find an
SPI device object, because your SoC device should have alias properties
which provide access to its SPI subcomponents' SPI buses.
See hw/arm/xlnx-ep108.c for an example of the board code for this and
hw/arm/xlnx-zynqmp.c for the SoC code which calls
object_property_add_alias() to set up the aliases.
> + if (spi_bus) {
> + DeviceState *flash_dev;
> +
> + flash_dev = ssi_create_slave(spi_bus, "sst25vf016b");
> + if (flash_dev) {
> + qemu_irq cs_line = qdev_get_gpio_in_named(flash_dev,
> + SSI_GPIO_CS,
> 0);
> + sysbus_connect_irq(SYS_BUS_DEVICE(spi_dev), 1, cs_line);
> + }
> + }
> + }
> + }
thanks
-- PMM
- Re: [Qemu-devel] [PATCH v3 09/12] FIFO: Add a FIFO32 implementation, (continued)
- [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Jean-Christophe Dubois, 2016/03/01
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation.,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Jean-Christophe DUBOIS, 2016/03/10
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Peter Maydell, 2016/03/10
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Jean-Christophe DUBOIS, 2016/03/15
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Peter Maydell, 2016/03/16
- Re: [Qemu-devel] [PATCH v3 12/12] i.MX: Add sabrelite i.MX6 emulation., Peter Maydell, 2016/03/16
[Qemu-devel] [PATCH v3 11/12] i.MX: Add i.MX6 SOC implementation., Jean-Christophe Dubois, 2016/03/01
Re: [Qemu-devel] [PATCH v3 00/12] Add i.MX6 (Single/Dual/Quad) support, Peter Maydell, 2016/03/16