[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] nand: Don't use qdev_create() in nand_init()
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH] nand: Don't use qdev_create() in nand_init() |
Date: |
Wed, 5 Feb 2014 15:56:23 +0100 |
On Wed, 5 Feb 2014 15:30:53 +0100
Andreas Färber <address@hidden> wrote:
> Commit 7426aa72c36c908a7d0eae3e38568bb0a70de479 (nand: Don't inherit
> from Sysbus) changed the parent type of TYPE_NAND but continued to use
> qdev_create(), which handled a NULL BusState as SysBus.
>
> Use object_new() instead, and reuse the TYPE_NAND define while at it.
>
> Reported-by: Markus Armbruster <address@hidden>
> Cc: Peter Crosthwaite <address@hidden>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
> hw/block/nand.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/block/nand.c b/hw/block/nand.c
> index a871ce0..6d7c804 100644
> --- a/hw/block/nand.c
> +++ b/hw/block/nand.c
> @@ -632,7 +632,7 @@ DeviceState *nand_init(BlockDriverState *bdrv, int
> manf_id, int chip_id)
> if (nand_flash_ids[chip_id].size == 0) {
> hw_error("%s: Unsupported NAND chip ID.\n", __FUNCTION__);
> }
> - dev = qdev_create(NULL, "nand");
> + dev = DEVICE(object_new(TYPE_NAND));
> qdev_prop_set_uint8(dev, "manufacturer_id", manf_id);
> qdev_prop_set_uint8(dev, "chip_id", chip_id);
> if (bdrv) {
> --
> 1.8.4.5
>
>
Reviewed-By: Igor Mammedov <address@hidden>
--
Regards,
Igor