From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [80.91.229.2] (helo=ciao.gmane.org) by linuxtogo.org with esmtp (Exim 4.69) (envelope-from ) id 1L2WHc-0007hm-Jf for openembedded-devel@openembedded.org; Tue, 18 Nov 2008 20:30:48 +0100 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1L2WFE-0005IH-Dm for openembedded-devel@openembedded.org; Tue, 18 Nov 2008 19:28:16 +0000 Received: from s55917625.adsl.wanadoo.nl ([85.145.118.37]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 18 Nov 2008 19:28:16 +0000 Received: from k.kooi by s55917625.adsl.wanadoo.nl with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Tue, 18 Nov 2008 19:28:16 +0000 X-Injected-Via-Gmane: http://gmane.org/ To: openembedded-devel@openembedded.org From: Koen Kooi Date: Tue, 18 Nov 2008 20:28:05 +0100 Message-ID: References: <20081118155949.GA2279@datenfreihafen.org> <20081118184433.GA12762@dodger.lab.datenfreihafen.org> Mime-Version: 1.0 X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: s55917625.adsl.wanadoo.nl User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b2pre) Gecko/20081114 Shredder/3.0b1pre In-Reply-To: <20081118184433.GA12762@dodger.lab.datenfreihafen.org> Sender: news Subject: Re: E-Ten Glofiish M800 support X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: openembedded-devel@lists.openembedded.org List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Nov 2008 19:30:49 -0000 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 18-11-08 19:44, Stefan Schmidt wrote: > Hello. > > On Tue, 2008-11-18 at 17:57, Koen Kooi wrote: >> On 18-11-08 16:59, Stefan Schmidt wrote: >>> I just pushed a branch with basic machine and kernel support for the M800 from >>> E-Ten. Normally Winodws Mobile based device. Linux support is ongoing. >>> >>> http://git.openembedded.net/?p=openembedded.git;a=commitdiff;h=0ea0875a3e5456f701df08e0974821a4e8852f55 >>> >>> As it is just machine, kernel recipe and sane-srcrev it should not break >>> anything. Still I would like to ask for review before pushing it in. >>> >>> How does folks feel about it? >> Only some small comments: >> >> conf/machine/eten-m800.conf: >> IMAGE_FSTYPES ?= "tar.gz" >> >> change that to += so people don't shoot themselves in the foot too easily > > Good point. Changed. > >> linux-eten_2.6.24+git.bb: >> git://git.openmoko.org/git/kernel.git;protocol=git;branch=stable \ >> >> Would it make sense to merge this into the openmoko kernel recipe since >> it's building from the same tree? I'm trying to cut down the number of >> linux-foo recipes whenever possible, but it a seperate recipe is easier, >> go for it. > > I thought about this. My point about not doing it is that it could easily be > that I switch the git repo soon. We just sat up a git tree with WIP stuff. Not > sure which one I like to have in OE in the long run. I would go for keeping it > and think about how it turns out. Do whatever makes you happy :) > PS: Any people prefer to have patches inlined for easier review, or are all fine > with a link to webgit here? Having the patches inline would make it slightly easier to review, but people would need to use git send-email properly to avoid e.g .spurious nesting. For small patchsets like this I don't find pasting from gitweb too much work, but I can see it getting tedious when >5 points need addressing. Another option would be to use something like reviewboard or gerrit to make pointing out dubious hunks and fixing them easier, but atm we sadly don't get enough stuff to review to install such infrastructure. regards, Koen