Hans, Ok. I will rework the patch and send you the same.
Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 new phone: 301-407-9583 Old Phone : 301-515-3736 (will be deprecated) email: m-kariche...@ti.com >-----Original Message----- >From: Hans Verkuil [mailto:hverk...@xs4all.nl] >Sent: Monday, August 17, 2009 2:47 PM >To: Karicheri, Muralidharan >Cc: linux-me...@vger.kernel.org; davinci-linux-open- >sou...@linux.davincidsp.com; khil...@deeprootsystems.com >Subject: Re: [PATCH v1 - 1/5] DaVinci - restructuring code to support vpif >capture driver > >On Monday 17 August 2009 16:52:20 Karicheri, Muralidharan wrote: >> Hans, >> >> They are applied against davinci tree (also mentioned in the patch). >General procedure what I follow is to create platform code against davinci >tree and v4l patches against v4l-dvb linux-next tree. The architecture part >of linux-next is not up to date. >> >> Davinci tree is at >> >> git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-davinci.git > >I must have missed the mention of this tree. > >I have a problem, though, as the current v4l-dvb repository doesn't compile >against the linux-davinci git tree. And the only way I can get it to >compile >is to apply all five patches first. > >However, the whole tree should still compile after each patch is applied. >And >that goes wrong with your second patch where the Kconfig and Makefile are >modified when the new sources aren't even added yet! > >What I would like to see is a patch series that starts with one patch that >makes the current v4l-dvb tree compile again, then the arch patch is added, >then a series of v4l-dvb patches in such an order that everything compiles >after each step. > >Merging this is already complicated enough without breaking compilation in >this way. > >Regards, > > Hans > >-- >Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
