> -----Original Message----- > From: Felipe Balbi [mailto:[email protected]] > Sent: Tuesday, July 21, 2009 10:55 AM > To: Shilimkar, Santosh > Cc: [email protected]; Krishnamoorthy, Balaji T; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; > Nayak, Rajendra > Subject: Re: [PATCH 1/4] ARM: OMAP: Rename twl4030* driver > files to enable re-use > > hi, > > On Tue, Jul 21, 2009 at 10:01:06AM +0530, Shilimkar, Santosh wrote: > > > So IMO, renaming the files is unnecessary. > > Well tps* are just catalog names of the twl4* family. So > you can keep adding whatever names there as long it's a same IC. > > > > Perhaps you look in hurry to comment about this. Please > look at the intention of these patches. TWL6030 has PM IC > similar to TWL4030 with bit different interrupt management. > Audio IC is a separate one as compared to combined in case of TWL4030. > > > > If it allows code re-use by renaming files, it should be > good. Isn't it ? > > What I'm trying to say is you can re-use the code even > without renaming > those files, just add twl6030-irq and use some sort of > runtime check to > use one or other irq mechanism.
We initially tried this but code is becoming ugly and not at all readable. In fact this was a short cut and would have saved us some time :) By renaming, we thought it can become a generic for further versions of TWL IC's and adding support to those would be cleaner. Let's get feedback from more people on the list Regards, Santosh-- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
