Re: [PATCH 08/21] hw/alpha/dp264: Create the RAM in the board

2019-10-21 Thread Philippe Mathieu-Daudé
On 10/21/19 7:27 PM, Richard Henderson wrote: On 10/20/19 3:56 PM, Philippe Mathieu-Daudé wrote: @@ -73,8 +74,16 @@ static void clipper_init(MachineState *machine) cpus[0]->env.trap_arg1 = 0; cpus[0]->env.trap_arg2 = smp_cpus; +/* + * Main memory region, 0x00...

Re: [PATCH 08/21] hw/alpha/dp264: Create the RAM in the board

2019-10-21 Thread Richard Henderson
On 10/20/19 3:56 PM, Philippe Mathieu-Daudé wrote: > @@ -73,8 +74,16 @@ static void clipper_init(MachineState *machine) > cpus[0]->env.trap_arg1 = 0; > cpus[0]->env.trap_arg2 = smp_cpus; > > +/* > + * Main memory region, 0x00... Real hardware supports 32GB, > + * bu

Re: [PATCH 08/21] hw/alpha/dp264: Create the RAM in the board

2019-10-21 Thread Philippe Mathieu-Daudé
On 10/21/19 12:56 AM, Philippe Mathieu-Daudé wrote: The SDRAM is incorrectly created in the Tyohoon northbridge ASIC. Move its creation in the board code, this will later allow the board to have the QOM ownership of the RAM. Signed-off-by: Philippe Mathieu-Daudé --- checkpatch complains: ERR

[PATCH 08/21] hw/alpha/dp264: Create the RAM in the board

2019-10-20 Thread Philippe Mathieu-Daudé
The SDRAM is incorrectly created in the Tyohoon northbridge ASIC. Move its creation in the board code, this will later allow the board to have the QOM ownership of the RAM. Signed-off-by: Philippe Mathieu-Daudé --- checkpatch complains: ERROR: spaces required around that '*' (ctx:WxV) #10: FIL