Hi Felipe, > -----Original Message----- > From: Felipe Contreras [mailto:felipe.contre...@gmail.com] > Sent: Tuesday, March 17, 2009 11:17 AM > To: Menon, Nishanth > Cc: linux-omap@vger.kernel.org; Kanigeri, Hari; Hiroshi DOYU; Ameya > Palande; Guzman Lugo, Fernando > Subject: Re: [PATCH] tidspbridge: remove revision history > > >> - *! Revision History: > >> - *! ================ > >> - *! 24-Feb-2003 vp: Code Review Updates. > >> - *! 18-Oct-2002 sb: Ported to Linux platform. > >> - *! 03-Jul-2001 rr: Removed kfuncs.h because of build errors. > >> - *! 07-Dec-1999 ag: Fxn error now causes a WinCE DebugBreak; > >> - *! 30-Aug-1999 ag: Now uses GP_printf for printf and error. > >> - *! > > But we did not have git history 1999-2003. we planning on loosing these > contribs? > > Only if you want dspbridge to be merged which as time passes it's > becoming more clear that you don't. I'm not a kernel developer, but I Errr.. Not my attempt at a flame war ;).. But then, I am not sure if my comment meant anything of the sort :)...
> would challenge you to find a diver that has 1000 lines of revision > history in the source code. <snip> > > >> - *! 30-Aug-1999 ag: Now uses GP_printf for printf and error. > > Not printk? I assume this is not related to Linux then. > > > Are we going to find an issue at some point in time that we'll say: oh > crap! if only we had the revision history log we could solve it! Errr... in the old times of cvs kernel, before we shifted to bitkeeper and later to git, rev history was unfortunately necessary to maintain some sort of acknowledgement of contributions. Just greping linux-omap master branch " grep -Ri "Revision History" drivers/|cut -d ":" -f1|wc -l" shows me 227 files of similar drivers- legacy - agreed. I think bridge is in such a legacy driver. If any new changes are done, it makes no sense to introduce an entry in the revision history - agreed. These driver files have a history and folks have done some work to make it useful, to remove their contributions would be, IMHO, our disregard for what ever they did (good or bad).. ;) But then, that is just my opinion.. Note: There are folks whose contributions are reduced to "vp" "rr" "sp" etc.. I personally have no clue who they are but I guess they would rather be in git log than in anonymous initials if given a choice today.. > > I doubt that, the revision history is useless without the actual > changes. These lines are just introducing noise. > DSPBridge has definitely a long way to go before being merged into mainline kernel. Coding standard is one part of the story. Is this part of a code cleanup effort to prep the code for merge to kernel? I think I have seen tons of discussion on this previously.. If we are planning on prepping this driver for mainline integration that is another discussion and this patch probably is a tiny fragment to that. The bunch of history starts at [1] though.. Regards, Nishanth Menon Ref: [1] http://marc.info/?l=linux-omap&w=4&r=10&s=dspbridge&q=b N�����r��y����b�X��ǧv�^�){.n�+����{��f��{ay� ʇڙ�,j��f���h���z� �w��� ���j:+v���w�j�m��������zZ+�����ݢj"��!�i
