[PATCH 4/5] PalmGSM: generalisation of Treo680 code to PalmGSM
Eric Miao
eric.y.miao at gmail.com
Tue Sep 1 05:39:47 EDT 2009
Marek Vasut wrote:
> Dne Út 1. září 2009 02:07:16 Pavel Machek napsal(a):
>> Ahoj! / Hi!
> Ahoj! / Hi! / Guten Tag!
>>>>>>> arch/arm/mach-pxa/Kconfig | 5 +
>>>>>>> arch/arm/mach-pxa/Makefile | 2 +-
>>>>>>> arch/arm/mach-pxa/include/mach/palmgsm.h | 60 ++++
>>>>>>> arch/arm/mach-pxa/include/mach/treo680.h | 49 ---
>>>>>>> arch/arm/mach-pxa/palmgsm.c | 559
>>>>>>> ++++++++++++++++++++++++++++++ arch/arm/mach-pxa/treo680.c
>>>>>>> | 528 ----------------------------
>>>>>> I don't actually care too much about the naming - as long as 685
>>>>>> can stay within treo680.c, I don't see any problem with that. The
>>>>>> real question is: is palmgsm.c the proper name for all these
>>>>>> devices?
>>>>> Well, actually your completely right. PalmGSM is not fitting for all
>>>>> devices to come, more fitting would be something like Palm
>>>>> smartphones because there are CDMA devices to come too. On the other
>>>>> hand, this name is the best I found so far. If you don't mind I would
>>>>> even so...
>>>> I believe treo680 was better name then. Don't change it if you don't
>>>> need to...
>>> But I need to have different generic name - treo680 will be occupied by
>>> Treo680 specific stuff, which makes more sense, doesn't it?
>> And rename generic code to treo.c? Then you could have treo.c for
>> generic code, and treo680.c / treo685.c for specific models...
>>
>> palmgsm.c sounds like driver for their GSM chip...
>> Pavel
>
> Yeah, or treo-common.c in case you'd want to split it into core file + model
> files. But treo.c/palm(smart)phone.c (?) seems more ok. I dont like the idea of
> splitting the code into more files (one big, and many small).
>
Well, treo680.c can just include all the specific stuffs in a single
file as long as possibility is concerned. And #ifdef .. #endif can
be used to include the code or not. See ezx.c for a good example.
> Cheers!
More information about the linux-arm-kernel
mailing list