On 18 October 2011 15:32, Avi Kivity wrote:
> On 10/18/2011 04:06 PM, Avi Kivity wrote:
>> > I didn't spot this the first time round, but this is wrong.
>> > We shouldn't be g_malloc0()ing the MemoryRegion -- it should
>> > be an element in some suitable device struct.
>> >
>> > I think the right
On 10/18/2011 04:06 PM, Avi Kivity wrote:
> > I didn't spot this the first time round, but this is wrong.
> > We shouldn't be g_malloc0()ing the MemoryRegion -- it should
> > be an element in some suitable device struct.
> >
> > I think the right thing to do here is probably first to do the
> > (fa
On 10/18/2011 01:44 PM, Peter Maydell wrote:
> 2011/10/17 Benoît Canet :
> > -static void icp_control_init(uint32_t base)
> > +static void icp_control_init(target_phys_addr_t base)
> > {
> > -int iomemtype;
> > +MemoryRegion *io;
> >
> > -iomemtype = cpu_register_io_memory(icp_control_
2011/10/17 Benoît Canet :
> -static void icp_control_init(uint32_t base)
> +static void icp_control_init(target_phys_addr_t base)
> {
> - int iomemtype;
> + MemoryRegion *io;
>
> - iomemtype = cpu_register_io_memory(icp_control_readfn,
> - icp_control
2011/10/13 Benoît Canet :
> ---
> hw/integratorcp.c | 35 ---
> 1 files changed, 16 insertions(+), 19 deletions(-)
Again, OK except for the trailing whitespace.
-- PMM
