[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