From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Hogan Subject: Re: [PATCH 1/1] mmc:Support of PCI mode for the dw_mmc driver Date: Wed, 30 Nov 2011 15:41:31 +0000 Message-ID: <4ED64EAB.4050405@imgtec.com> References: <1322656585-8363-1-git-send-email-shashidharh@vayavyalabs.com> <4ED63861.7030607@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from multi.imgtec.com ([194.200.65.239]:42351 "EHLO multi.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754360Ab1K3Plf (ORCPT ); Wed, 30 Nov 2011 10:41:35 -0500 In-Reply-To: Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Shashidhar Hiremath Cc: Chris Ball , Shawn Guo , Philip Rakity , "Mark F. Brown" , Wolfram Sang , Will Newton , Jaehoon Chung , Kyungmin Park , linux-mmc@vger.kernel.org Hi, On 11/30/2011 03:27 PM, Shashidhar Hiremath wrote: > Hi James, > Sorry for the minor alignment issues.Will fix them . Thanks > I actually did > compile the code but as separate driver not along with the kernel. In > fact, I have also tested the PCI patch ,but not as separate files but > with the earlier #ifdef method I had sent in initial patch . But the new patch was completely different, you can't just rework the entire patch and expect it to compile and work without at least trying it to make sure, as evidenced by the fact that with mmc_next + your patch, I get: dw_mmc.c: 7 errors, 22 warnings dw_mmc-pci.c: 2 errors dw_mmc-pltfm.c: 6 errors, 1 warning Cheers James > On Wed, Nov 30, 2011 at 7:36 PM, James Hogan wrote: >> *sigh* You haven't actually tried compiling this have you? How do you >> know whether it works or not? You really really should at least make >> sure it compiles (for simple changes) and works without breaking >> anything (for anything non-trivial like adding support for a new bus >> like PCI)!