[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] kvm/powerpc: Add MPC8544DS board support
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] kvm/powerpc: Add MPC8544DS board support |
Date: |
Tue, 17 Feb 2009 17:28:08 +0200 |
On 2/17/09, Liu Yu <address@hidden> wrote:
> This patch add emulation of MPC8544DS board.
> And it can work on All E500 platforms.
>
> Signed-off-by: Liu Yu <address@hidden>
> + * Copyright (C) 2009 Freescale Semiconductor, Inc. All rights reserved.
Again, "All rights reserved" is not what GPL says.
> + ret = kvmppc_read_host_property(node, prop, &cell, sizeof(cell));
> + if (ret < 0) {
> + fprintf(stderr, "couldn't read host %s/%s\n", node, prop);
> + goto out;
> + }
For the Qemu case, the host CPU or system may be different from the
emulated target CPU or system. These should be moved to
target-ppc/kvm_ppc.c.
> + path = qemu_malloc(pathlen);
> + if (path == NULL)
> + goto out;
qemu_malloc will not return NULL anymore, please remove the check.
> + free(path);
s/free/qemu_free/g
> + sprintf(buf, "/cpus/%s", dirp->d_name);
Please use snprintf.
> + /* Register Memory */
> + cpu_register_physical_memory(0, ram_size, 0);
This was the old way. Please use qemu_ram_alloc instead.
- [Qemu-devel] powerpc/kvm: Add MPC8544DS board support, Liu Yu, 2009/02/17
- Re: [Qemu-devel] [PATCH 2/5] kvm/powerpc: Add freescale pci controller's support, Blue Swirl, 2009/02/17
- Re: [Qemu-devel] [PATCH 2/5] kvm/powerpc: Add freescale pci controller's support, M. Warner Losh, 2009/02/17
- Re: [Qemu-devel] [PATCH 2/5] kvm/powerpc: Add freescale pci controller's support, M. Warner Losh, 2009/02/17
- Re: [Qemu-devel] [PATCH 2/5] kvm/powerpc: Add freescale pci controller's support, Jamie Lokier, 2009/02/17
- RE: [Qemu-devel] [PATCH 2/5] kvm/powerpc: Add freescale pci controller's support, Liu Yu-B13201, 2009/02/18
Re: [Qemu-devel] [PATCH 1/5] kvm/powerpc: Enable MPIC for E500 platform., Blue Swirl, 2009/02/17