[PATCH 2/2] [ARM] pxa: make use of 'struct machine_class'

Eric Miao eric.miao at canonical.com
Sun Jun 20 21:06:33 EDT 2010


On Mon, Jun 21, 2010 at 12:08 AM, Antonio Ospite
<ospite at studenti.unina.it> wrote:
> On Sun, 20 Jun 2010 22:00:50 +0800
> Eric Miao <eric.miao at canonical.com> wrote:
>
>> From: Eric Miao <eric.y.miao at gmail.com>
>>
>> Signed-off-by: Eric Miao <eric.miao at canonical.com>
>> ---
>>  arch/arm/mach-pxa/ezx.c      |   60 ++++-------------------------------------
>
> The ezx part looks OK. A comment about magician below.
>
> [...]
>> diff --git a/arch/arm/mach-pxa/magician.c b/arch/arm/mach-pxa/magician.c
>> index e81dd0c..6a86f22 100644
>> --- a/arch/arm/mach-pxa/magician.c
>> +++ b/arch/arm/mach-pxa/magician.c
>> @@ -764,11 +764,6 @@ static void __init magician_init(void)
>>
>>
>>  MACHINE_START(MAGICIAN, "HTC Magician")
>> -     .phys_io = 0x40000000,
>> -     .io_pg_offst = (io_p2v(0x40000000) >> 18) & 0xfffc,
>> -     .boot_params = 0xa0000100,
>> -     .map_io = pxa_map_io,
>> -     .init_irq = pxa27x_init_irq,
>> -     .init_machine = magician_init,
>> -     .timer = &pxa_timer,
>> +     .class          = &machine_class_pxa27x,
>> +     .init_machine   = magician_init,
>>  MACHINE_END
>
> Magician can use PXA27X_MACHINE() as well, can't it?

Yes, it can. Missed that :-)



More information about the linux-arm-kernel mailing list