From mboxrd@z Thu Jan 1 00:00:00 1970 From: christian.glindkamp@taskit.de (Christian Glindkamp) Date: Fri, 10 Dec 2010 09:44:00 +0100 Subject: [PATCH] at91: Refactor Stamp9G20 and PControl G20 board file In-Reply-To: <1291909193.6251.32.camel@homepc> References: <4CFFE5A9.4040407@bluewatersys.com> <1291889759-12281-1-git-send-email-christian.glindkamp@taskit.de> <1291909193.6251.32.camel@homepc> Message-ID: <20101210084400.GA24295@taskit.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 2010-12-09 18:39, Igor Plyatov wrote: > Dear Christian, > > > As PControl G20 is a carrier board for the Stamp9G20 SoM, some code can > > be shared. Therefore board-stamp9g20.c is refactored to allow reusing the > > SoM initialization and board-pcontrol-g20.c is modified to use it. > > > > Signed-off-by: Christian Glindkamp > > --- > > > > How about this approach? Compile tested for PControl G20 and run time tested > > for Stamp9G20 EVB and PortuxG20. > > > > Just a side note: PortuxG20 is not a carrier board for the Stamp9G20. It just > > shares so much with the evaluation board, that it makes sense to put them both > > into the same file. And there is no intention to put other boards into this > > file. > > The idea from this patch is clean. > Yours patch does not apply to the kernel from > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git, > but I apply its content by hands and kernels compiled successfully with > stamp9g20_defconfig and pcontrol_g20_defconfig. I used exactly this repository. Revision 6313e3c21743cc88bb5bd8aa72948ee1e83937b6 to be precise. Maybe your mailer corrupted the patch. > Please run ./scripts/checkpatch.pl with you patch and correct all errors > and warnings reported. I only get some "line over 80 characters" warnings, which can imho be ignored. > > If you will correct this patch, to be applicable, then I will vote to > include it into mainline. > > Which kernel repository and branch should I use for the AT91 ARM? > I just want to know - whereto send my next patch to support gms machine? > > Best regards! > -- > Igor Plyatov >